diff mbox

docs/specs/qcow2.txt: fix incorrect bits of offsets in entries

Message ID 1388382585-30853-1-git-send-email-hutao@cn.fujitsu.com
State New
Headers show

Commit Message

Hu Tao Dec. 30, 2013, 5:49 a.m. UTC
Fix incorrect bits of offsets in refcount table entry, l1 table
entry and l2 table entry. See REFT_OFFSET_MASK, L1E_OFFSET_MASK
and L2E_OFFSET_MASK.

Fix a typo while at the file.

Signed-off-by: Hu Tao <hutao@cn.fujitsu.com>
---
 docs/specs/qcow2.txt | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

Comments

Stefan Weil Dec. 30, 2013, 7:49 a.m. UTC | #1
Am 30.12.2013 06:49, schrieb Hu Tao:
> Fix incorrect bits of offsets in refcount table entry, l1 table
> entry and l2 table entry. See REFT_OFFSET_MASK, L1E_OFFSET_MASK
> and L2E_OFFSET_MASK.
> 
> Fix a typo while at the file.
> 
> Signed-off-by: Hu Tao <hutao@cn.fujitsu.com>
> ---
>  docs/specs/qcow2.txt | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/docs/specs/qcow2.txt b/docs/specs/qcow2.txt
> index f19536a..408f93f 100644
> --- a/docs/specs/qcow2.txt
> +++ b/docs/specs/qcow2.txt
> @@ -192,9 +192,9 @@ as follows:
>  
>  Refcount table entry:
>  
> -    Bit  0 -  8:    Reserved (set to 0)
> +    Bit  0 -  7:    Reserved (set to 0)
>  
> -         9 - 63:    Bits 9-63 of the offset into the image file at which the
> +         8 - 63:    Bits 9-63 of the offset into the image file at which the


What about the comment? Does it need a change, too?


>                      refcount block starts. Must be aligned to a cluster
>                      boundary.
>  
> @@ -218,7 +218,7 @@ The L1 table has a variable size (stored in the header) and may use multiple
>  clusters, however it must be contiguous in the image file. L2 tables are
>  exactly one cluster in size.
>  
> -Given a offset into the virtual disk, the offset into the image file can be
> +Given an offset into the virtual disk, the offset into the image file can be
>  obtained as follows:
>  
>      l2_entries = (cluster_size / sizeof(uint64_t))
> @@ -233,9 +233,9 @@ obtained as follows:
>  
>  L1 table entry:
>  
> -    Bit  0 -  8:    Reserved (set to 0)
> +    Bit  0 -  7:    Reserved (set to 0)
>  
> -         9 - 55:    Bits 9-55 of the offset into the image file at which the L2
> +         8 - 55:    Bits 9-55 of the offset into the image file at which the L2


Change comment, too?


>                      table starts. Must be aligned to a cluster boundary. If the
>                      offset is 0, the L2 table and all clusters described by this
>                      L2 table are unallocated.
> @@ -268,9 +268,9 @@ Standard Cluster Descriptor:
>  
>                      With version 2, this is always 0.
>  
> -         1 -  8:    Reserved (set to 0)
> +         1 -  7:    Reserved (set to 0)
>  
> -         9 - 55:    Bits 9-55 of host cluster offset. Must be aligned to a
> +         8 - 55:    Bits 9-55 of host cluster offset. Must be aligned to a


Comment?

>                      cluster boundary. If the offset is 0, the cluster is
>                      unallocated.
>  
>
Hu Tao Dec. 30, 2013, 7:53 a.m. UTC | #2
On Mon, Dec 30, 2013 at 08:49:58AM +0100, Stefan Weil wrote:
> Am 30.12.2013 06:49, schrieb Hu Tao:
> > Fix incorrect bits of offsets in refcount table entry, l1 table
> > entry and l2 table entry. See REFT_OFFSET_MASK, L1E_OFFSET_MASK
> > and L2E_OFFSET_MASK.
> > 
> > Fix a typo while at the file.
> > 
> > Signed-off-by: Hu Tao <hutao@cn.fujitsu.com>
> > ---
> >  docs/specs/qcow2.txt | 14 +++++++-------
> >  1 file changed, 7 insertions(+), 7 deletions(-)
> > 
> > diff --git a/docs/specs/qcow2.txt b/docs/specs/qcow2.txt
> > index f19536a..408f93f 100644
> > --- a/docs/specs/qcow2.txt
> > +++ b/docs/specs/qcow2.txt
> > @@ -192,9 +192,9 @@ as follows:
> >  
> >  Refcount table entry:
> >  
> > -    Bit  0 -  8:    Reserved (set to 0)
> > +    Bit  0 -  7:    Reserved (set to 0)
> >  
> > -         9 - 63:    Bits 9-63 of the offset into the image file at which the
> > +         8 - 63:    Bits 9-63 of the offset into the image file at which the
> 
> 
> What about the comment? Does it need a change, too?

Absolutely. Thanks.
diff mbox

Patch

diff --git a/docs/specs/qcow2.txt b/docs/specs/qcow2.txt
index f19536a..408f93f 100644
--- a/docs/specs/qcow2.txt
+++ b/docs/specs/qcow2.txt
@@ -192,9 +192,9 @@  as follows:
 
 Refcount table entry:
 
-    Bit  0 -  8:    Reserved (set to 0)
+    Bit  0 -  7:    Reserved (set to 0)
 
-         9 - 63:    Bits 9-63 of the offset into the image file at which the
+         8 - 63:    Bits 9-63 of the offset into the image file at which the
                     refcount block starts. Must be aligned to a cluster
                     boundary.
 
@@ -218,7 +218,7 @@  The L1 table has a variable size (stored in the header) and may use multiple
 clusters, however it must be contiguous in the image file. L2 tables are
 exactly one cluster in size.
 
-Given a offset into the virtual disk, the offset into the image file can be
+Given an offset into the virtual disk, the offset into the image file can be
 obtained as follows:
 
     l2_entries = (cluster_size / sizeof(uint64_t))
@@ -233,9 +233,9 @@  obtained as follows:
 
 L1 table entry:
 
-    Bit  0 -  8:    Reserved (set to 0)
+    Bit  0 -  7:    Reserved (set to 0)
 
-         9 - 55:    Bits 9-55 of the offset into the image file at which the L2
+         8 - 55:    Bits 9-55 of the offset into the image file at which the L2
                     table starts. Must be aligned to a cluster boundary. If the
                     offset is 0, the L2 table and all clusters described by this
                     L2 table are unallocated.
@@ -268,9 +268,9 @@  Standard Cluster Descriptor:
 
                     With version 2, this is always 0.
 
-         1 -  8:    Reserved (set to 0)
+         1 -  7:    Reserved (set to 0)
 
-         9 - 55:    Bits 9-55 of host cluster offset. Must be aligned to a
+         8 - 55:    Bits 9-55 of host cluster offset. Must be aligned to a
                     cluster boundary. If the offset is 0, the cluster is
                     unallocated.