From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34982) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a1Ufq-0004FR-S7 for qemu-devel@nongnu.org; Wed, 25 Nov 2015 02:39:32 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1a1Ufp-0006a1-UG for qemu-devel@nongnu.org; Wed, 25 Nov 2015 02:39:30 -0500 From: Fam Zheng Date: Wed, 25 Nov 2015 15:38:59 +0800 Message-Id: <1448437153-27090-1-git-send-email-famz@redhat.com> Subject: [Qemu-devel] [PATCH v2 00/14] qemu-img map: Allow driver to return file of the allocated block List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Kevin Wolf , qemu-block@nongnu.org, Jeff Cody , Peter Lieven , Markus Armbruster , Stefan Hajnoczi , Paolo Bonzini v2: Add Eric's rev-by in patches 2, 4, 5. 01: Refering -> referring in commit message. [Eric] Recurse to "file" for sensible "zero" flag. [Paolo] 12: New. Make MapEntry a QAPI struct. [Paolo, Markus] I stumbled upon this when looking at external bitmap formats. Current "qemu-img map" command only displays filename if the data is allocated in bs (bs->file) itself, or in the backing chain. Otherwise, it displays an unfriendly error message: $ qemu-img create -f vmdk -o subformat=monolithicFlat /tmp/test.vmdk 1G $ qemu-img map /tmp/test.vmdk Offset Length Mapped to File qemu-img: File contains external, encrypted or compressed clusters. This can be improved. This series extends the .bdrv_co_get_block_status callback, to let block driver return the BDS of file; then updates all driver to implement it; and lastly, it changes qemu-img to use this information in "map" command: $ qemu-img map /tmp/test.vmdk Offset Length Mapped to File 0 0x40000000 0 /tmp/test-flat.vmdk $ qemu-img map --output json /tmp/test.vmdk [{"length": 1073741824, "start": 0, "zero": false, "offset": 0, "depth": 0, "file": "/tmp/test-flat.vmdk", "data": true} ] Fam Zheng (14): block: Add "file" output parameter to block status query functions qcow: Assign bs->file->bs to file in qcow_co_get_block_status qcow2: Assign bs->file->bs to file in qcow2_co_get_block_status raw: Assign bs to file in raw_co_get_block_status iscsi: Assign bs to file in iscsi_co_get_block_status parallels: Assign bs->file->bs to file in parallels_co_get_block_status qed: Assign bs->file->bs to file in bdrv_qed_co_get_block_status sheepdog: Assign bs to file in sd_co_get_block_status vdi: Assign bs->file->bs to file in vdi_co_get_block_status vpc: Assign bs->file->bs to file in vpc_co_get_block_status vmdk: Return extent's file in bdrv_get_block_status qemu-img: Make MapEntry a QAPI struct qemu-img: Use QAPI visitor to generate JSON iotests: Add "qemu-img map" test for VMDK extents block/io.c | 42 +++++++++++++------- block/iscsi.c | 9 ++++- block/mirror.c | 3 +- block/parallels.c | 3 +- block/qcow.c | 3 +- block/qcow2.c | 3 +- block/qed.c | 6 ++- block/raw-posix.c | 4 +- block/raw_bsd.c | 4 +- block/sheepdog.c | 5 ++- block/vdi.c | 3 +- block/vmdk.c | 13 +++---- block/vpc.c | 4 +- block/vvfat.c | 2 +- include/block/block.h | 6 ++- include/block/block_int.h | 3 +- qapi/block-core.json | 28 ++++++++++++++ qemu-img.c | 80 +++++++++++++++++++++----------------- tests/qemu-iotests/059 | 10 +++++ tests/qemu-iotests/059.out | 38 ++++++++++++++++++ tests/qemu-iotests/122.out | 96 ++++++++++++++++++++++++++-------------------- 21 files changed, 251 insertions(+), 114 deletions(-) -- 2.4.3