Re: [Qemu-devel] [PATCH RFC 0/3] iothread: release iothread around aio_poll

2015-04-22 Thread Stefan Hajnoczi
On Tue, Apr 21, 2015 at 05:54:59PM +0200, Paolo Bonzini wrote: > > > On 21/04/2015 17:40, Stefan Hajnoczi wrote: > >> > > >> > Stefan, can you put this on track for 2.4 or do you need a repost? > > This series causes qemu-iotests -qcow2 091 to fail: > > > > 9f83aea22314d928bb272153ff37d2d7f5adbf

Re: [Qemu-devel] [PATCH RFC 0/3] iothread: release iothread around aio_poll

2015-04-21 Thread Paolo Bonzini
On 21/04/2015 17:40, Stefan Hajnoczi wrote: >> > >> > Stefan, can you put this on track for 2.4 or do you need a repost? > This series causes qemu-iotests -qcow2 091 to fail: > > 9f83aea22314d928bb272153ff37d2d7f5adbf06 is the first bad commit > commit 9f83aea22314d928bb272153ff37d2d7f5adbf06 >

Re: [Qemu-devel] [PATCH RFC 0/3] iothread: release iothread around aio_poll

2015-04-21 Thread Stefan Hajnoczi
On Tue, Mar 31, 2015 at 11:35 AM, Paolo Bonzini wrote: > On 20/02/2015 17:26, Paolo Bonzini wrote: >> Right now, iothreads are relying on a "contention callback" to release >> the AioContext (e.g. for block device operations or to do bdrv_drain_all). >> This is necessary because aio_poll needs to

Re: [Qemu-devel] [PATCH RFC 0/3] iothread: release iothread around aio_poll

2015-03-31 Thread Stefan Hajnoczi
On Tue, Mar 31, 2015 at 12:35:16PM +0200, Paolo Bonzini wrote: > > > On 20/02/2015 17:26, Paolo Bonzini wrote: > > Right now, iothreads are relying on a "contention callback" to release > > the AioContext (e.g. for block device operations or to do bdrv_drain_all). > > This is necessary because ai

Re: [Qemu-devel] [PATCH RFC 0/3] iothread: release iothread around aio_poll

2015-03-31 Thread Stefan Hajnoczi
On Fri, Feb 20, 2015 at 05:26:49PM +0100, Paolo Bonzini wrote: > Right now, iothreads are relying on a "contention callback" to release > the AioContext (e.g. for block device operations or to do bdrv_drain_all). > This is necessary because aio_poll needs to be called within an > aio_context_acquir

Re: [Qemu-devel] [PATCH RFC 0/3] iothread: release iothread around aio_poll

2015-03-31 Thread Paolo Bonzini
On 20/02/2015 17:26, Paolo Bonzini wrote: > Right now, iothreads are relying on a "contention callback" to release > the AioContext (e.g. for block device operations or to do bdrv_drain_all). > This is necessary because aio_poll needs to be called within an > aio_context_acquire. > > This series

[Qemu-devel] [PATCH RFC 0/3] iothread: release iothread around aio_poll

2015-02-20 Thread Paolo Bonzini
Right now, iothreads are relying on a "contention callback" to release the AioContext (e.g. for block device operations or to do bdrv_drain_all). This is necessary because aio_poll needs to be called within an aio_context_acquire. This series drops this requirement for aio_poll, with two effects: