From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:58467)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <stefanha@gmail.com>) id 1WsCGs-0005eT-8f
	for qemu-devel@nongnu.org; Wed, 04 Jun 2014 10:34:39 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <stefanha@gmail.com>) id 1WsCGj-0004Z5-4m
	for qemu-devel@nongnu.org; Wed, 04 Jun 2014 10:34:30 -0400
Received: from mail-wi0-x233.google.com ([2a00:1450:400c:c05::233]:35081)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <stefanha@gmail.com>) id 1WsCGi-0004Xt-UK
	for qemu-devel@nongnu.org; Wed, 04 Jun 2014 10:34:21 -0400
Received: by mail-wi0-f179.google.com with SMTP id bs8so1608345wib.0
	for <qemu-devel@nongnu.org>; Wed, 04 Jun 2014 07:34:19 -0700 (PDT)
Date: Wed, 4 Jun 2014 16:34:14 +0200
From: Stefan Hajnoczi <stefanha@gmail.com>
Message-ID: <20140604143414.GJ11073@stefanha-thinkpad.redhat.com>
References: <1401537111-10221-1-git-send-email-benoit.canet@irqsave.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <1401537111-10221-1-git-send-email-benoit.canet@irqsave.net>
Subject: Re: [Qemu-devel] [PATCH v1 00/72] Extract qapi commons and block
 definitions
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: =?iso-8859-1?Q?Beno=EEt?= Canet <benoit.canet@irqsave.net>
Cc: kwolf@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com, lcapitulino@redhat.com

On Sat, May 31, 2014 at 01:50:39PM +0200, Benoît Canet wrote:
> This hudge chainsawing series sit on top of my quorum maintenance series.
> 
> It extract commons definition and block definition into separate files.
> 
> -qapi/common.json contains some definition required by all qapi modules
> -qapi/block-core.json contains core qapi block definition usable without the emulation
> code.
> -qapi/block.json is a superset of the previous usable with emulation
> 
> Transaction where left appart because they relies on internal snapshot and hence
> cannot be included in block-core.json for now.

I'm fine with this, it's mostly a mechanical change.

But please squash it down into just a few patches:

1. Extract block-core.json and include it.
2. Extract block.json and include it.

I also suggest you drop the quorum maintenance command dependency
because rebasing this series will be no fun.  This way you minimize
churn.  We can merge this qapi series very quickly, while the quorum
maintenance series may require more discussion.

Stefan