From: "Benoît Canet" <benoit.canet@nodalink.com>
To: "Benoît Canet" <benoit.canet@nodalink.com>
Cc: kwolf@redhat.com, jcody@redhat.com, famz@redhat.com,
qemu-devel@nongnu.org, stefanha@redhat.com
Subject: Re: [Qemu-devel] [PATCH v2 0/2] Recursive op blockers
Date: Mon, 29 Sep 2014 16:08:46 +0000 [thread overview]
Message-ID: <20140929160846.GD26702@nodalink.com> (raw)
In-Reply-To: <1411412452-8655-1-git-send-email-benoit.canet@nodalink.com>
On Mon, Sep 22, 2014 at 09:00:50PM +0200, Benoît Canet wrote:
> in v2:
> Added a base parameters [Kevin]
> Remove multiple blockers [Kevin]
>
> Addressed Kevin's comments except BLOCK_OP_TYPE_MIRROR_REPLACE relaxing
> and "You need to unblock all of the BDSes that are going to be removed.
> Aren't you unblocking more than that here?".
>
>
> pass ./check -qcow2
>
> Benoît Canet (2):
> block: Rename BLOCK_OP_TYPE_REPLACE to BLOCK_OP_TYPE_MIRROR_REPLACE
> block: Make op blockers recursive
>
> block-migration.c | 4 +-
> block.c | 123 ++++++++++++++++++++++++++++++++++++++++++----
> block/blkverify.c | 21 ++++++++
> block/commit.c | 2 +
> block/mirror.c | 26 +++++++---
> block/quorum.c | 29 +++++++++++
> block/stream.c | 2 +
> block/vmdk.c | 32 ++++++++++++
> blockjob.c | 6 +--
> include/block/block.h | 14 ++++--
> include/block/block_int.h | 10 ++++
> 11 files changed, 242 insertions(+), 27 deletions(-)
>
> --
> 2.1.0
>
Hello people,
Gentle ping,
I would really like these patches to get rev-by and merged for the next release
since they actually fix QEMU.
Under the new rule of code review I choose Markus or Jeff to do the initial
revision.
I could nag Kevin into reviewing it but it's not the process anymore.
Best regards
Benoît
next prev parent reply other threads:[~2014-09-29 16:09 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-22 19:00 [Qemu-devel] [PATCH v2 0/2] Recursive op blockers Benoît Canet
2014-09-22 19:00 ` [Qemu-devel] [PATCH v2 1/2] block: Rename BLOCK_OP_TYPE_REPLACE to BLOCK_OP_TYPE_MIRROR_REPLACE Benoît Canet
2014-09-22 19:00 ` [Qemu-devel] [PATCH v2 2/2] block: Make op blockers recursive Benoît Canet
2014-10-01 4:38 ` Jeff Cody
2014-10-01 4:59 ` Benoît Canet
2014-10-01 9:29 ` Benoît Canet
2014-10-01 15:19 ` Jeff Cody
2014-10-01 15:42 ` Benoît Canet
2014-09-29 16:08 ` Benoît Canet [this message]
-- strict thread matches above, loose matches on Subject: below --
2014-09-22 12:40 [Qemu-devel] [PATCH v2 0/2] Recursive op blockers Benoît Canet
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=20140929160846.GD26702@nodalink.com \
--to=benoit.canet@nodalink.com \
--cc=famz@redhat.com \
--cc=jcody@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.