On Tue, Jun 12, 2018 at 1:00 AM, Eric Blake <ebl...@redhat.com> wrote: > On 06/11/2018 03:25 PM, Ville Skyttä wrote: >> >> Signed-off-by: Ville Skyttä <ville.sky...@iki.fi> >> --- > [...] > >> +++ b/docs/interop/qcow2.txt >> @@ -326,7 +326,7 @@ in the image file. >> It contains pointers to the second level structures which are called >> refcount >> blocks and are exactly one cluster in size. >> -Given a offset into the image file, the refcount of its cluster can be >> obtained >> +Given an offset into the image file, the refcount of its cluster can be >> obtained >> as follows: > > > This line is now long; it might be better if you reflow 'obtained', more > like... > >> refcount_block_entries = (cluster_size * 8 / refcount_bits) >> @@ -365,7 +365,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: > > > ...this line. > > With or without the reflow, > Reviewed-by: Eric Blake <ebl...@redhat.com>
Reflowing, new patch follows in a separate mail.
