Re: [Qemu-devel] [PATCH v6 2/3] linux-aio: handling -EAGAIN for !s-io_q.plugged case

2014-11-28 Thread Kevin Wolf
Am 28.11.2014 um 03:27 hat Ming Lei geschrieben: Hi Kevin, On Wed, Nov 26, 2014 at 7:27 PM, Kevin Wolf kw...@redhat.com wrote: Am 25.11.2014 um 08:23 hat Ming Lei geschrieben: Previously -EAGAIN is simply ignored for !s-io_q.plugged case, and sometimes it is easy to cause -EIO to VM,

Re: [Qemu-devel] [PATCH v6 2/3] linux-aio: handling -EAGAIN for !s-io_q.plugged case

2014-11-27 Thread Ming Lei
Hi Kevin, On Wed, Nov 26, 2014 at 7:27 PM, Kevin Wolf kw...@redhat.com wrote: Am 25.11.2014 um 08:23 hat Ming Lei geschrieben: Previously -EAGAIN is simply ignored for !s-io_q.plugged case, and sometimes it is easy to cause -EIO to VM, such as NVME device. This patch handles -EAGAIN by io

Re: [Qemu-devel] [PATCH v6 2/3] linux-aio: handling -EAGAIN for !s-io_q.plugged case

2014-11-26 Thread Kevin Wolf
Am 25.11.2014 um 08:23 hat Ming Lei geschrieben: Previously -EAGAIN is simply ignored for !s-io_q.plugged case, and sometimes it is easy to cause -EIO to VM, such as NVME device. This patch handles -EAGAIN by io queue for !s-io_q.plugged case, and it will be retried in following aio

Re: [Qemu-devel] [PATCH v6 2/3] linux-aio: handling -EAGAIN for !s-io_q.plugged case

2014-11-25 Thread Stefan Hajnoczi
On Tue, Nov 25, 2014 at 03:23:12PM +0800, Ming Lei wrote: Previously -EAGAIN is simply ignored for !s-io_q.plugged case, and sometimes it is easy to cause -EIO to VM, such as NVME device. This patch handles -EAGAIN by io queue for !s-io_q.plugged case, and it will be retried in following aio

[Qemu-devel] [PATCH v6 2/3] linux-aio: handling -EAGAIN for !s-io_q.plugged case

2014-11-24 Thread Ming Lei
Previously -EAGAIN is simply ignored for !s-io_q.plugged case, and sometimes it is easy to cause -EIO to VM, such as NVME device. This patch handles -EAGAIN by io queue for !s-io_q.plugged case, and it will be retried in following aio completion cb. Reviewed-by: Paolo Bonzini pbonz...@redhat.com