From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43734) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XYdVT-0007xq-M2 for qemu-devel@nongnu.org; Mon, 29 Sep 2014 12:09:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XYdVM-0003uc-7s for qemu-devel@nongnu.org; Mon, 29 Sep 2014 12:08:59 -0400 Received: from dew.nodalink.com ([95.130.14.197]:39180) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XYdVM-0003tL-1m for qemu-devel@nongnu.org; Mon, 29 Sep 2014 12:08:52 -0400 Date: Mon, 29 Sep 2014 16:08:46 +0000 From: =?iso-8859-1?Q?Beno=EEt?= Canet Message-ID: <20140929160846.GD26702@nodalink.com> References: <1411412452-8655-1-git-send-email-benoit.canet@nodalink.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <1411412452-8655-1-git-send-email-benoit.canet@nodalink.com> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v2 0/2] Recursive op blockers List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?iso-8859-1?Q?Beno=EEt?= Canet Cc: kwolf@redhat.com, jcody@redhat.com, famz@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com On Mon, Sep 22, 2014 at 09:00:50PM +0200, Beno=EEt Canet wrote: > in v2: > Added a base parameters [Kevin] > Remove multiple blockers [Kevin] >=20 > 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?". >=20 >=20 > pass ./check -qcow2 >=20 > Beno=EEt Canet (2): > block: Rename BLOCK_OP_TYPE_REPLACE to BLOCK_OP_TYPE_MIRROR_REPLACE > block: Make op blockers recursive >=20 > 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(-) >=20 > --=20 > 2.1.0 >=20 Hello people, Gentle ping, I would really like these patches to get rev-by and merged for the next r= elease since they actually fix QEMU. Under the new rule of code review I choose Markus or Jeff to do the initi= al revision. I could nag Kevin into reviewing it but it's not the process anymore. Best regards Beno=EEt