Re: [Qemu-devel] [PATCH 6/9] virtio-blk: use aio handler for data plane

2016-04-01 Thread Cornelia Huck
On Fri, 1 Apr 2016 15:19:51 +0200 Paolo Bonzini wrote: > From: "Michael S. Tsirkin" > > In addition to handling IO in vcpu thread and in io thread, dataplane > introduces yet another mode: handling it by aio. > > This reuses the same handler as previous modes, which triggers races as > these

[Qemu-devel] [PATCH 6/9] virtio-blk: use aio handler for data plane

2016-04-01 Thread Paolo Bonzini
From: "Michael S. Tsirkin" In addition to handling IO in vcpu thread and in io thread, dataplane introduces yet another mode: handling it by aio. This reuses the same handler as previous modes, which triggers races as these were not designed to be reentrant. Use a separate handler just for aio,

Re: [Qemu-devel] [PATCH 6/9] virtio-blk: use aio handler for data plane

2016-03-30 Thread Paolo Bonzini
On 30/03/2016 17:25, Cornelia Huck wrote: >> > >> > -/* Some guests kick before setting VIRTIO_CONFIG_S_DRIVER_OK so start >> > - * dataplane here instead of waiting for .set_status(). >> > - */ >> > -if (s->dataplane && !s->dataplane_started) { >> > -virtio_blk_data_plan

Re: [Qemu-devel] [PATCH 6/9] virtio-blk: use aio handler for data plane

2016-03-30 Thread Cornelia Huck
On Wed, 30 Mar 2016 14:48:05 +0200 Paolo Bonzini wrote: > From: "Michael S. Tsirkin" > > In addition to handling IO in vcpu thread and in io thread, dataplane > introduces yet another mode: handling it by aio. > > This reuses the same handler as previous modes, which triggers races as > these

[Qemu-devel] [PATCH 6/9] virtio-blk: use aio handler for data plane

2016-03-30 Thread Paolo Bonzini
From: "Michael S. Tsirkin" In addition to handling IO in vcpu thread and in io thread, dataplane introduces yet another mode: handling it by aio. This reuses the same handler as previous modes, which triggers races as these were not designed to be reentrant. Use a separate handler just for aio,