From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43884) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W69dn-0003Ug-Ot for qemu-devel@nongnu.org; Wed, 22 Jan 2014 21:03:41 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1W69dh-00015o-P3 for qemu-devel@nongnu.org; Wed, 22 Jan 2014 21:03:35 -0500 Received: from mx1.redhat.com ([209.132.183.28]:3699) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W69dh-00014q-GZ for qemu-devel@nongnu.org; Wed, 22 Jan 2014 21:03:29 -0500 Received: from int-mx02.intmail.prod.int.phx2.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s0N23QOZ025850 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Wed, 22 Jan 2014 21:03:27 -0500 From: Fam Zheng Date: Thu, 23 Jan 2014 10:03:26 +0800 Message-Id: <1390442606-31320-1-git-send-email-famz@redhat.com> Subject: [Qemu-devel] [PATCH v2] qapi: Add "backing" to BlockStats List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: kwolf@redhat.com, stefanha@redhat.com Currently there is no way to query BlockStats of the backing chain. This adds "backing" field into BlockStats to make it possible. The comment of "parent" is reworded. Signed-off-by: Fam Zheng --- v2: Only add "backing", drop "file". Signed-off-by: Fam Zheng --- block/qapi.c | 5 +++++ qapi-schema.json | 10 ++++++---- 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/block/qapi.c b/block/qapi.c index a32cb79..98b1b83 100644 --- a/block/qapi.c +++ b/block/qapi.c @@ -318,6 +318,11 @@ BlockStats *bdrv_query_stats(const BlockDriverState *bs) s->parent = bdrv_query_stats(bs->file); } + if (bs->backing_hd) { + s->has_backing = true; + s->backing = bdrv_query_stats(bs->backing_hd); + } + return s; } diff --git a/qapi-schema.json b/qapi-schema.json index f27c48a..0e534d0 100644 --- a/qapi-schema.json +++ b/qapi-schema.json @@ -1022,15 +1022,17 @@ # # @stats: A @BlockDeviceStats for the device. # -# @parent: #optional This may point to the backing block device if this is a -# a virtual block device. If it's a backing block, this will point -# to the backing file is one is present. +# @parent: #optional This describes the file block device if it has one. +# +# @backing: #optional This describes the backing block device if it has one. +# (Since 2.0) # # Since: 0.14.0 ## { 'type': 'BlockStats', 'data': {'*device': 'str', 'stats': 'BlockDeviceStats', - '*parent': 'BlockStats'} } + '*parent': 'BlockStats', + '*backing': 'BlockStats'} } ## # @query-blockstats: -- 1.8.5.3