[0/6] parallels: Add duplication check, repair at open, fix bugs

Message

Alexander Ivanov Sept. 2, 2022, 8:52 a.m. UTC

This patchset is based on
git: https://src.openvz.org/~den/qemu.git parallels

Fix incorrect data end calculation in parallels_open().

Add parallels_handle_leak() and highest_offset() helpers.

Add checking and repairing duplicate offsets in BAT.

Deduplicate highest offset calculation.

Replace fprintf() by qemu_log().

Add check and repair to parallels_open().


Alexander Ivanov (6):
  parallels: Incorrect data end calculation in parallels_open()
  parallels: Create parallels_handle_leak() to truncate excess clusters
  parallels: Add checking and repairing duplicate offsets in BAT
  parallels: Use highest_offset() helper in leak check
  parallels: Replace fprintf by qemu_log
  parallels: Image repairing in parallels_open()

 block/parallels.c | 297 +++++++++++++++++++++++++++++++++++++---------
 1 file changed, 241 insertions(+), 56 deletions(-)

Comments

Denis V. Lunev Sept. 7, 2022, 4:23 p.m. UTC | #1

On 9/2/22 10:52, Alexander Ivanov wrote:
> This patchset is based on
> git: https://src.openvz.org/~den/qemu.git parallels
>
> Fix incorrect data end calculation in parallels_open().
>
> Add parallels_handle_leak() and highest_offset() helpers.
>
> Add checking and repairing duplicate offsets in BAT.
>
> Deduplicate highest offset calculation.
>
> Replace fprintf() by qemu_log().
>
> Add check and repair to parallels_open().
>
>
> Alexander Ivanov (6):
>    parallels: Incorrect data end calculation in parallels_open()
>    parallels: Create parallels_handle_leak() to truncate excess clusters
>    parallels: Add checking and repairing duplicate offsets in BAT
>    parallels: Use highest_offset() helper in leak check
>    parallels: Replace fprintf by qemu_log
>    parallels: Image repairing in parallels_open()
>
>   block/parallels.c | 297 +++++++++++++++++++++++++++++++++++++---------
>   1 file changed, 241 insertions(+), 56 deletions(-)
>
common note - please fix formatting in the patch descriptions.
74 characters on a line, empty line between paragraphs.

Den