All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: Ming Lei <ming.lei@canonical.com>,
	qemu-devel@nongnu.org, Stefan Hajnoczi <stefanha@redhat.com>,
	Kevin Wolf <kwolf@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v5 0/3] linux-aio: fix batch submission
Date: Mon, 24 Nov 2014 15:31:32 +0100	[thread overview]
Message-ID: <54734144.1070107@redhat.com> (raw)
In-Reply-To: <1416839379-19677-1-git-send-email-ming.lei@canonical.com>



On 24/11/2014 15:29, Ming Lei wrote:
> The 1st patch fixes batch submission.
> 
> The 2nd one fixes -EAGAIN for non-batch case.
> 
> The 3rd one is a cleanup.
> 
> This patchset is splitted from previous patchset(dataplane: optimization
> and multi virtqueue support), as suggested by Stefan.
> 
> V5:
>     - in case of submission failure, return -EIO for new coming requests
>       until aborting is handled
> 	- in patch2, follow Paolo's suggestion about ioq_enqueue() changes
> 
> V4:
> 	- abort reuqests in BH to abvoid potential "Co-routine re-entered recursively"
> 	- remove 'enqueue' parameter to ioq_submit() to simpify change
> 	- beautify code as suggested by Paolo
> 
> v3:
>     - rebase on QEMU master
> v2:
>     - code style fix and commit log fix as suggested by Benoît Canet
> v1:
>     - rebase on latest QEMU master
> 
>  block/linux-aio.c |  131 +++++++++++++++++++++++++++++++++++++++++++----------
>  1 file changed, 107 insertions(+), 24 deletions(-)
> 
> Thanks,
> Ming Lei
> 
> 

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>

  parent reply	other threads:[~2014-11-24 14:31 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-24 14:29 [Qemu-devel] [PATCH v5 0/3] linux-aio: fix batch submission Ming Lei
2014-11-24 14:29 ` [Qemu-devel] [PATCH v5 1/3] linux-aio: fix submit aio as a batch Ming Lei
2014-11-24 16:40   ` Stefan Hajnoczi
2014-11-24 19:01   ` Stefan Hajnoczi
2014-11-24 14:29 ` [Qemu-devel] [PATCH v5 2/3] linux-aio: handling -EAGAIN for !s->io_q.plugged case Ming Lei
2014-11-24 18:58   ` Stefan Hajnoczi
2014-11-24 14:29 ` [Qemu-devel] [PATCH v5 3/3] linux-aio: remove 'node' from 'struct qemu_laiocb' Ming Lei
2014-11-24 14:31 ` Paolo Bonzini [this message]
2014-11-24 14:50   ` [Qemu-devel] [PATCH v5 0/3] linux-aio: fix batch submission Ming Lei

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=54734144.1070107@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=ming.lei@canonical.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.