Re: [Qemu-devel] [PATCH v2 05/17] aio: stop using .io_flush()

2013-04-17 Thread Stefan Hajnoczi
On Wed, Apr 17, 2013 at 03:20:02PM +0200, Paolo Bonzini wrote: > Il 17/04/2013 15:07, Stefan Hajnoczi ha scritto: > > > > Due to this change we need to update tests/test-aio.c which asserts > > aio_poll() return values. Note that QEMU doesn't actually rely on these > > return values so only tests

Re: [Qemu-devel] [PATCH v2 05/17] aio: stop using .io_flush()

2013-04-17 Thread Paolo Bonzini
Il 17/04/2013 15:07, Stefan Hajnoczi ha scritto: > > Due to this change we need to update tests/test-aio.c which asserts > aio_poll() return values. Note that QEMU doesn't actually rely on these > return values so only tests/test-aio.c cares. > > Note that ctx->notifier, the EventNotifier fd use

[Qemu-devel] [PATCH v2 05/17] aio: stop using .io_flush()

2013-04-17 Thread Stefan Hajnoczi
Now that aio_poll() users check their termination condition themselves, it is no longer necessary to call .io_flush() handlers. The behavior of aio_poll() changes as follows: 1. .io_flush() is no longer invoked and file descriptors are *always* monitored. Previously returning 0 from .io_flush()