From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40155) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XlaUO-0006O4-E1 for qemu-devel@nongnu.org; Tue, 04 Nov 2014 04:33:29 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XlaUJ-0003TW-JT for qemu-devel@nongnu.org; Tue, 04 Nov 2014 04:33:24 -0500 Received: from mx1.redhat.com ([209.132.183.28]:51776) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XlaUJ-0003TK-C3 for qemu-devel@nongnu.org; Tue, 04 Nov 2014 04:33:19 -0500 Message-ID: <54589D5A.5010805@redhat.com> Date: Tue, 04 Nov 2014 10:33:14 +0100 From: Max Reitz MIME-Version: 1.0 References: <1414726377-13063-1-git-send-email-famz@redhat.com> In-Reply-To: <1414726377-13063-1-git-send-email-famz@redhat.com> Content-Type: text/plain; charset=iso-8859-15; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 0/4] block: Allow query stats for drive-mirror target List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Fam Zheng , qemu-devel@nongnu.org Cc: Kevin Wolf , Benoit Canet , Stefan Hajnoczi , Markus Armbruster On 2014-10-31 at 04:32, Fam Zheng wrote: > v2: Add Max's and Eric's rev-by's for patch 1~3. > Fix spelling, documentation and indentation on 4. > Thanks, Max and Eric! > > This series adds an optional bool parameter "query-nodes" to query-blockstats. > > By default, if omitted, the behavior is unchanged. > > If set to "true", the command will iterate through all named nodes in BDS graph > and report the statistics in a list, similarly. But the backing chain is not > built. > > This provides a way for libvirt to watch the allocation status > (wr_highest_offset) of target image. Now, libvirt can start drive-mirror job > specifying a node-name parameter. Thus the created target image gets a node > name that can be queried with the new query-blockstats. > > > Fam Zheng (4): > block: Add bdrv_next_node > block: Add bdrv_get_node_name > block: Include "node-name" if present in query-blockstats > qmp: Add optional switch "query-nodes" in query-blockstats > > block.c | 13 +++++++++++++ > block/qapi.c | 25 ++++++++++++++++++------- > hmp.c | 2 +- > include/block/block.h | 2 ++ > qapi/block-core.json | 16 ++++++++++++++-- > qmp-commands.hx | 2 +- > 6 files changed, 49 insertions(+), 11 deletions(-) As I seem to be the supporting submaintainer of the day: Thanks, applied to my block-next tree with the / #$/ in patch 4 removed: https://github.com/XanClic/qemu/commits/block-next