qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] Add delay option to blkdebug


From: Kevin Wolf
Subject: Re: [Qemu-devel] [PATCH] Add delay option to blkdebug
Date: Mon, 1 Jul 2013 12:23:53 +0200
User-agent: Mutt/1.5.21 (2010-09-15)

Am 29.06.2013 um 20:02 hat Alex Bligh geschrieben:
> Add a delay option to blkdebug, allowing operations to be delayed by
> a specifiable number of microseconds. Example configuration:
> 
> [inject-error]
> event = "read_aio"
> delay = "200000"
> 
> Signed-off-by: Alex Bligh <address@hidden>

"inject-error" doesn't really describe this well. Shouldn't we rather
introduce a new section "[delay]" or something like that?

I remember that I once tried something similar, but never submitted it.
I think the reason was that using timers inside requests doesn't really
work. It works as long as everything is indeed asynchronous, but
bdrv_drain_all() or a loop waiting for a single request that tries to
make progress with qemu_aio_wait() will simply hang because timers
aren't processed in these nested event loops.

Kevin



reply via email to

[Prev in Thread] Current Thread [Next in Thread]