From: Max Reitz <mreitz@redhat.com>
To: Fam Zheng <famz@redhat.com>, qemu-devel@nongnu.org
Cc: Kevin Wolf <kwolf@redhat.com>,
Markus Armbruster <armbru@redhat.com>,
Stefan Hajnoczi <stefanha@redhat.com>
Subject: Re: [Qemu-devel] [PATCH 3/4] blockdev: Convert bdrv_find to blk_by_name
Date: Mon, 02 Mar 2015 10:48:43 -0500 [thread overview]
Message-ID: <54F4865B.20105@redhat.com> (raw)
In-Reply-To: <1425296209-1476-4-git-send-email-famz@redhat.com>
On 2015-03-02 at 06:36, Fam Zheng wrote:
> Signed-off-by: Fam Zheng <famz@redhat.com>
> ---
> blockdev.c | 92 ++++++++++++++++++++++++++++++++++++++++----------------------
> 1 file changed, 59 insertions(+), 33 deletions(-)
I think I touched the same problem in my "BB and media" series, and I
think I replaced the bdrv_get_aio_context() calls by
blk_get_aio_context() (because it looked nicer to me to call blk_bs()
only after the AIO context has been acquired).
However, at least before that series it is not possible to change the
BDS attached to a BB while the BB is already in use in another AIO
context, so:
Reviewed-by: Max Reitz <mreitz@redhat.com>
next prev parent reply other threads:[~2015-03-02 15:48 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-02 11:36 [Qemu-devel] [PATCH 0/4] block: Convert bdrv_find to blk_by_name and drop it Fam Zheng
2015-03-02 11:36 ` [Qemu-devel] [PATCH 1/4] monitor: Convert bdrv_find to blk_by_name Fam Zheng
2015-03-02 15:30 ` Max Reitz
2015-03-02 11:36 ` [Qemu-devel] [PATCH 2/4] migration: " Fam Zheng
2015-03-02 15:32 ` Max Reitz
2015-03-02 11:36 ` [Qemu-devel] [PATCH 3/4] blockdev: " Fam Zheng
2015-03-02 15:48 ` Max Reitz [this message]
2015-03-02 11:36 ` [Qemu-devel] [PATCH 4/4] block: Drop bdrv_find Fam Zheng
2015-03-02 15:51 ` Max Reitz
2015-03-03 20:47 ` [Qemu-devel] [PATCH 0/4] block: Convert bdrv_find to blk_by_name and drop it Max Reitz
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=54F4865B.20105@redhat.com \
--to=mreitz@redhat.com \
--cc=armbru@redhat.com \
--cc=famz@redhat.com \
--cc=kwolf@redhat.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.