qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v3] specs/qcow2: Fix documentation of the compre


From: Eric Blake
Subject: Re: [Qemu-devel] [PATCH v3] specs/qcow2: Fix documentation of the compressed cluster descriptor
Date: Wed, 21 Feb 2018 10:12:08 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0

CC: address@hidden

Incorrect docs make for difficult interoperability.

On 02/21/2018 08:08 AM, Alberto Garcia wrote:
This patch fixes several mistakes in the documentation of the
compressed cluster descriptor:

1) the documentation claims that the cluster descriptor contains the
    number of sectors used to store the compressed data, but what it
    actually contains is the number of sectors *minus one* or, in other
    words, the number of additional sectors after the first one.

2) the width of the fields is incorrectly specified. The number of bits
    used by each field is

       x = 62 - (cluster_bits - 8)   for the offset field
       y = (cluster_bits - 8)        for the size field

    So the offset field's location is [0, x-1], not [0, x] as stated.

3) the size field does not contain the size of the compressed data,
    but rather the number of sectors where that data is stored. The
    compressed data starts at the exact point specified in the offset
    field and ends when there's enough data to produce a cluster of
    decompressed data. Both points can be in the middle of a sector,
    allowing several compressed clusters to be stored next to one
    another, sharing sectors if necessary.

Signed-off-by: Alberto Garcia <address@hidden>
---

v3: Fix the specification of the width of the fields, and update the
     explanation of how the compressed data is stored [Eric].

v2: I realized that the documentation is not completely clear about
     the exact location and size of the compressed data, so I updated
     the patch to clarify this.

---
  docs/interop/qcow2.txt | 16 +++++++++++++---
  1 file changed, 13 insertions(+), 3 deletions(-)

diff --git a/docs/interop/qcow2.txt b/docs/interop/qcow2.txt
index d7fdb1fee3..feb711fb6a 100644
--- a/docs/interop/qcow2.txt
+++ b/docs/interop/qcow2.txt
@@ -426,10 +426,20 @@ Standard Cluster Descriptor:
Compressed Clusters Descriptor (x = 62 - (cluster_bits - 8)): - Bit 0 - x: Host cluster offset. This is usually _not_ aligned to a
-                    cluster boundary!
+    Bit  0 - x-1:   Host cluster offset. This is usually _not_ aligned to a
+                    cluster or sector boundary!
- x+1 - 61: Compressed size of the images in sectors of 512 bytes
+         x - 61:    Number of additional 512-byte sectors used for the
+                    compressed data, beyond the sector containing the offset
+                    in the previous field. Some of these sectors may reside
+                    in the next contiguous host cluster.
+
+                    Note that the compressed data does not necessarily occupy
+                    all of the bytes in the final sector; rather, decompression
+                    stops when it has produced a cluster of data.
+
+                    Another compressed cluster may map to the tail of the final
+                    sector used by this compressed cluster.
If a cluster is unallocated, read requests shall read the data from the backing
  file (except if bit 0 in the Standard Cluster Descriptor is set). If there is


--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org



reply via email to

[Prev in Thread] Current Thread [Next in Thread]