mbox series

[v4,00/10] qcow2 check: check some reserved bits and subcluster bitmaps

Message ID 20210914122454.141075-1-vsementsov@virtuozzo.com (mailing list archive)
Headers show
Series qcow2 check: check some reserved bits and subcluster bitmaps | expand

Message

Vladimir Sementsov-Ogievskiy Sept. 14, 2021, 12:24 p.m. UTC 
Hi all!

Here are some good refactorings and new (qemu-img check) checks for
qcow2.

06  qcow2-refcount: check_refcounts_l2(): check l2_bitmap
      don't fix unallocated cluster with allocated subclusters (no strong opinion how to do it correctly)
      drop Eric's r-b
      keep Kirill's t-b (I believe it still applies, my original task didn't include error fixing)
      define l2_bitmap near l2_entry
      add separate assertion and "break;" for QCOW2_CLUSTER_ZERO_PLAIN
others: add r-b: Hanna


Vladimir Sementsov-Ogievskiy (10):
  qcow2-refcount: improve style of check_refcounts_l2()
  qcow2: compressed read: simplify cluster descriptor passing
  qcow2: introduce qcow2_parse_compressed_l2_entry() helper
  qcow2-refcount: introduce fix_l2_entry_by_zero()
  qcow2-refcount: fix_l2_entry_by_zero(): also zero L2 entry bitmap
  qcow2-refcount: check_refcounts_l2(): check l2_bitmap
  qcow2-refcount: check_refcounts_l2(): check reserved bits
  qcow2-refcount: improve style of check_refcounts_l1()
  qcow2-refcount: check_refcounts_l1(): check reserved bits
  qcow2-refcount: check_refblocks(): add separate message for reserved

 block/qcow2.h          |   7 +-
 block/qcow2-cluster.c  |  20 ++-
 block/qcow2-refcount.c | 324 ++++++++++++++++++++++++++---------------
 block/qcow2.c          |  13 +-
 4 files changed, 236 insertions(+), 128 deletions(-)

Comments

Hanna Czenczek Sept. 15, 2021, 10:26 a.m. UTC | #1 
On 14.09.21 14:24, Vladimir Sementsov-Ogievskiy wrote:
> Hi all!
>
> Here are some good refactorings and new (qemu-img check) checks for
> qcow2.

Thanks, applied to my block branch:

https://gitlab.com/hreitz/qemu/-/commits/block/

(Patch 6 is here: 
https://gitlab.com/hreitz/qemu/-/commit/93c40e7dab205047245028e97f7470d89c3a7ef3 
– just to confirm the resolution fits what you had in mind)

Hanna
Vladimir Sementsov-Ogievskiy Sept. 15, 2021, 11:36 a.m. UTC | #2 
15.09.2021 13:26, Hanna Reitz wrote:
> On 14.09.21 14:24, Vladimir Sementsov-Ogievskiy wrote:
>> Hi all!
>>
>> Here are some good refactorings and new (qemu-img check) checks for
>> qcow2.
> 
> Thanks, applied to my block branch:
> 
> https://gitlab.com/hreitz/qemu/-/commits/block/
> 
> (Patch 6 is here: https://gitlab.com/hreitz/qemu/-/commit/93c40e7dab205047245028e97f7470d89c3a7ef3 – just to confirm the resolution fits what you had in mind)
> 

It's OK, thanks!