[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 3/5] parallels: Add checking and repairing duplicate offse
From: |
Mike Maslenkin |
Subject: |
Re: [PATCH v4 3/5] parallels: Add checking and repairing duplicate offsets in BAT |
Date: |
Thu, 27 Apr 2023 00:56:56 +0300 |
On Mon, Apr 24, 2023 at 12:44 PM Alexander Ivanov
<alexander.ivanov@virtuozzo.com> wrote:
>
> Cluster offsets must be unique among all the BAT entries. Find duplicate
> offsets in the BAT and fix it by copying the content of the relevant
> cluster to a newly allocated cluster and set the new cluster offset to the
> duplicated entry.
>
> Add host_cluster_index() helper to deduplicate the code.
>
> Move parallels_fix_leak() call to parallels_co_check() to fix both types
> of leak: real corruption and a leak produced by allocate_clusters()
> during deduplication.
>
> Signed-off-by: Alexander Ivanov <alexander.ivanov@virtuozzo.com>
> ---
> block/parallels.c | 134 ++++++++++++++++++++++++++++++++++++++++++++--
> 1 file changed, 129 insertions(+), 5 deletions(-)
>
> diff --git a/block/parallels.c b/block/parallels.c
> index ec89ed894b..3b992e8173 100644
> --- a/block/parallels.c
> +++ b/block/parallels.c
> @@ -136,6 +136,12 @@ static int cluster_remainder(BDRVParallelsState *s,
> int64_t sector_num,
> return MIN(nb_sectors, ret);
> }
>
> +static uint32_t host_cluster_index(BDRVParallelsState *s, int64_t off)
> +{
> + off -= s->header->data_off << BDRV_SECTOR_BITS;
> + return off / s->cluster_size;
> +}
> +
I guess there should be:
off -= le32_to_cpu(s->header->data_off) << BDRV_SECTOR_BITS
Regards,
Mike.
- [PATCH v4 0/5] parallels: Add duplication check, repair at open, fix bugs, Alexander Ivanov, 2023/04/24
- [PATCH v4 1/5] parallels: Incorrect data end calculation in parallels_open(), Alexander Ivanov, 2023/04/24
- [PATCH v4 2/5] parallels: Split image leak handling to separate check and fix helpers, Alexander Ivanov, 2023/04/24
- [PATCH v4 4/5] parallels: Replace fprintf by qemu_log in check, Alexander Ivanov, 2023/04/24
- [PATCH v4 5/5] parallels: Image repairing in parallels_open(), Alexander Ivanov, 2023/04/24