This patchset should be applied on the top of *[PATCH v8 00/11]
parallels: Refactor the code of images checks and fix a bug*
On 12.01.2023 16:01, Alexander Ivanov wrote:
> Fix incorrect data end calculation in parallels_open().
>
> Split image leak handling to separate check and fix helpers.
>
> Add checking and repairing duplicate offsets in BAT
>
> Replace fprintf() by qemu_log().
>
> Image repairing in parallels_open().
>
> v2:
> 2: Moved outsude parallels_check_leak() 2 helpers:
> parallels_get_leak_size() and parallels_fix_leak().
>
> 3: Used highest_offset() helper in parallels_check_leak(). Fixed a typo.
> Added comments. Replaced g_malloc() call by qemu_memalign(). Replaced
> bdrv_pread() call by bdrv_co_pread(). Got rid of keeping bytes and
> sectors in the same variable. Added setting the bitmap of the used
> clusters for a new allocated cluster if it isn't out of the bitmap.
> Moved the leak fix to the end of all the checks. Removed a dependence
> on image format for the duplicate check.
>
> 4 (old): Merged this patch to the previous.
> 4 (former 5): Fixed formatting.
> 5 (former 6): Fixed comments. Added O_INACTIVE check in the condition.
> Replaced inuse detection by header_unclean checking.
> Replaced playing with corutines by bdrv_check() usage.
>
> Alexander Ivanov (5):
> parallels: Incorrect data end calculation in parallels_open()
> parallels: Split image leak handling to separate check and fix helpers
> parallels: Add checking and repairing duplicate offsets in BAT
> parallels: Replace fprintf by qemu_log in check
> parallels: Image repairing in parallels_open()
>
> block/parallels.c | 321 +++++++++++++++++++++++++++++++++++-----------
> 1 file changed, 247 insertions(+), 74 deletions(-)
>