Re: [Qemu-devel] [PATCH v4 01/17] block: stop relying on io_flush() in bdrv_drain_all()

2013-06-27 Thread Paolo Bonzini
Il 14/06/2013 13:43, Stefan Hajnoczi ha scritto: If a block driver has no file descriptors to monitor but there are still active requests, it can return 1 from .io_flush(). This is used to spin during synchronous I/O. Stop relying on .io_flush() and instead check

Re: [Qemu-devel] [PATCH v4 01/17] block: stop relying on io_flush() in bdrv_drain_all()

2013-06-27 Thread Stefan Hajnoczi
On Thu, Jun 27, 2013 at 03:13:19PM +0200, Paolo Bonzini wrote: Il 14/06/2013 13:43, Stefan Hajnoczi ha scritto: If a block driver has no file descriptors to monitor but there are still active requests, it can return 1 from .io_flush(). This is used to spin during synchronous I/O. Stop

[Qemu-devel] [PATCH v4 01/17] block: stop relying on io_flush() in bdrv_drain_all()

2013-06-14 Thread Stefan Hajnoczi
If a block driver has no file descriptors to monitor but there are still active requests, it can return 1 from .io_flush(). This is used to spin during synchronous I/O. Stop relying on .io_flush() and instead check QLIST_EMPTY(bs-tracked_requests) to decide whether there are active requests.