[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 03/12] block/vmdk: Change extent info type
From: |
Kevin Wolf |
Subject: |
Re: [PATCH v2 03/12] block/vmdk: Change extent info type |
Date: |
Thu, 19 Jan 2023 16:20:16 +0100 |
Am 20.06.2022 um 18:26 hat Hanna Reitz geschrieben:
> VMDK's implementation of .bdrv_get_specific_info() returns information
> about its extent files, ostensibly in the form of ImageInfo objects.
> However, it does not get this information through
> bdrv_query_image_info(), but fills only a select few fields with custom
> information that does not always match the fields' purposes.
>
> For example, @format, which is supposed to be a block driver name, is
> filled with the extent type, e.g. SPARSE or FLAT.
>
> In ImageInfo, @compressed shows whether the data that can be seen in the
> image is stored in compressed form or not. For example, a compressed
> qcow2 image will store compressed data in its data file, but when
> accessing the qcow2 node, you will see normal data. This is not how
> VMDK uses the @compressed field for its extent files: Instead, it
> signifies whether accessing the extent file will yield compressed data
> (which the VMDK driver then (de-)compresses).
Actually, VMDK was the only user of the field in ImageInfo. qcow2
doesn't set the field at all because it would have to parse all L2
tables in order to set it.
So I suppose @compressed can now be removed from ImageInfo?
Kevin
- Re: [PATCH v2 03/12] block/vmdk: Change extent info type,
Kevin Wolf <=