[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
- Re: [Qemu-devel] [PATCH] Add delay option to blkdebug,
Kevin Wolf <=