From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48429) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZqmGY-0003cF-CI for qemu-devel@nongnu.org; Mon, 26 Oct 2015 14:13:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZqmGX-0002Dh-FY for qemu-devel@nongnu.org; Mon, 26 Oct 2015 14:13:06 -0400 From: John Snow Date: Mon, 26 Oct 2015 14:12:57 -0400 Message-Id: <1445883177-11795-1-git-send-email-jsnow@redhat.com> Subject: [Qemu-devel] [PATCH] block: allow best-effort query List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-block@nongnu.org Cc: kwolf@redhat.com, John Snow , qemu-devel@nongnu.org, mreitz@redhat.com For more complex BDS trees that can be created under normal circumstances, we lose the ability to issue query commands because of our inability to re-construct the absolute filename. Instead, omit this field when it is a problem and present as much information as we can. This will change the expected output in iotest 110, where we will now see a json filename and the lack of an absolute filename instead of an error. Signed-off-by: John Snow --- block/qapi.c | 10 ++++++---- tests/qemu-iotests/110.out | 5 ++++- 2 files changed, 10 insertions(+), 5 deletions(-) diff --git a/block/qapi.c b/block/qapi.c index 355ba32..c4101d5 100644 --- a/block/qapi.c +++ b/block/qapi.c @@ -245,13 +245,15 @@ void bdrv_query_image_info(BlockDriverState *bs, info->has_backing_filename = true; bdrv_get_full_backing_filename(bs, backing_filename2, PATH_MAX, &err); if (err) { - error_propagate(errp, err); - qapi_free_ImageInfo(info); + /* Couldn't reconstruct that backing filename, + * So we'll skip this field, but apply a Best Effort to the query */ g_free(backing_filename2); - return; + backing_filename2 = NULL; + error_free(err); } - if (strcmp(backing_filename, backing_filename2) != 0) { + if (backing_filename2 && + strcmp(backing_filename, backing_filename2) != 0) { info->full_backing_filename = g_strdup(backing_filename2); info->has_full_backing_filename = true; diff --git a/tests/qemu-iotests/110.out b/tests/qemu-iotests/110.out index 0270980..425485f 100644 --- a/tests/qemu-iotests/110.out +++ b/tests/qemu-iotests/110.out @@ -11,7 +11,10 @@ backing file: t.IMGFMT.base (actual path: TEST_DIR/t.IMGFMT.base) === Non-reconstructable filename === -qemu-img: Cannot use relative backing file names for 'json:{"driver": "IMGFMT", "file": {"set-state.0.event": "read_aio", "image": {"driver": "file", "filename": "TEST_DIR/t.IMGFMT"}, "driver": "blkdebug", "set-state.0.new_state": 42}}' +image: json:{"driver": "IMGFMT", "file": {"set-state.0.event": "read_aio", "image": {"driver": "file", "filename": "TEST_DIR/t.IMGFMT"}, "driver": "blkdebug", "set-state.0.new_state": 42}} +file format: IMGFMT +virtual size: 64M (67108864 bytes) +backing file: t.IMGFMT.base === Backing name is always relative to the backed image === -- 2.4.3