[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v9 00/16] qemu-img map: Allow driver to return f
From: |
Max Reitz |
Subject: |
Re: [Qemu-devel] [PATCH v9 00/16] qemu-img map: Allow driver to return file of the allocated block |
Date: |
Wed, 27 Jan 2016 19:44:55 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 |
On 26.01.2016 04:58, Fam Zheng wrote:
> v9: 01: Add comment for the new parameter, and remove the superfluous reset of
> 'file'. [Kevin]
> 04: bs => bs->file->bs. [Kevin]
> 11: Split the bugfix. [Kevin]
> 15: Don't corrupt 059.out. [Max]
>
> v8: Fix patch 15. [Max]
> Add Max's rev-by in patch 1.
>
> v7: Rebase, update patch 1 for two new bdrv_get_block_status_above() callers
> in
> qemu-img.c. [Max]
> Add Max's rev-by in patch 12.
>
> Original cover letter
> ---------------------
>
> 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 (16):
> 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: Fix calculation of block status's offset
> vmdk: Return extent's file in bdrv_get_block_status
> block: Use returned *file in bdrv_co_get_block_status
> qemu-img: In "map", use the returned "file" from bdrv_get_block_status
> qemu-img: Make MapEntry a QAPI struct
> iotests: Add "qemu-img map" test for VMDK extents
>
> block/io.c | 44 ++++++++++++++++--------
> 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 | 12 ++++---
> block/vpc.c | 4 ++-
> block/vvfat.c | 2 +-
> include/block/block.h | 11 +++---
> include/block/block_int.h | 3 +-
> qapi/block-core.json | 27 +++++++++++++++
> qemu-img.c | 84
> ++++++++++++++++++++++++++++------------------
> tests/qemu-iotests/059 | 10 ++++++
> tests/qemu-iotests/059.out | 25 ++++++++++++++
> 20 files changed, 195 insertions(+), 70 deletions(-)
Thanks, applied to my block tree:
https://github.com/XanClic/qemu/commits/block
Max
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v9 10/16] vpc: Assign bs->file->bs to file in vpc_co_get_block_status, (continued)
- [Qemu-devel] [PATCH v9 10/16] vpc: Assign bs->file->bs to file in vpc_co_get_block_status, Fam Zheng, 2016/01/25
- [Qemu-devel] [PATCH v9 11/16] vmdk: Fix calculation of block status's offset, Fam Zheng, 2016/01/25
- [Qemu-devel] [PATCH v9 12/16] vmdk: Return extent's file in bdrv_get_block_status, Fam Zheng, 2016/01/25
- [Qemu-devel] [PATCH v9 13/16] block: Use returned *file in bdrv_co_get_block_status, Fam Zheng, 2016/01/25
- [Qemu-devel] [PATCH v9 14/16] qemu-img: In "map", use the returned "file" from bdrv_get_block_status, Fam Zheng, 2016/01/25
- [Qemu-devel] [PATCH v9 15/16] qemu-img: Make MapEntry a QAPI struct, Fam Zheng, 2016/01/25
- [Qemu-devel] [PATCH v9 16/16] iotests: Add "qemu-img map" test for VMDK extents, Fam Zheng, 2016/01/25
- Re: [Qemu-devel] [PATCH v9 00/16] qemu-img map: Allow driver to return file of the allocated block,
Max Reitz <=