[v4] parallels: Add full dirty bitmap support

[PATCH v4 07/21] parallels: Set data_end value in parallels_check_leak()

Posted by Alexander Ivanov 11 months ago

In parallels_check_leak() we change file size but don't correct data_end
field of BDRVParallelsState structure. Fix it.

Signed-off-by: Alexander Ivanov <alexander.ivanov@virtuozzo.com>
---
 block/parallels.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/block/parallels.c b/block/parallels.c
index 658902ae51..8a6e2ba7ee 100644
--- a/block/parallels.c
+++ b/block/parallels.c
@@ -806,6 +806,7 @@ parallels_check_leak(BlockDriverState *bs, BdrvCheckResult *res,
                 res->check_errors++;
                 return ret;
             }
+            s->data_end = res->image_end_offset >> BDRV_SECTOR_BITS;
             if (explicit) {
                 res->leaks_fixed += count;
             }
-- 
2.40.1

Re: [PATCH v4 07/21] parallels: Set data_end value in parallels_check_leak()

Posted by Denis V. Lunev 10 months, 2 weeks ago

On 12/28/23 11:12, Alexander Ivanov wrote:
> In parallels_check_leak() we change file size but don't correct data_end
> field of BDRVParallelsState structure. Fix it.
>
> Signed-off-by: Alexander Ivanov <alexander.ivanov@virtuozzo.com>
> ---
>   block/parallels.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/block/parallels.c b/block/parallels.c
> index 658902ae51..8a6e2ba7ee 100644
> --- a/block/parallels.c
> +++ b/block/parallels.c
> @@ -806,6 +806,7 @@ parallels_check_leak(BlockDriverState *bs, BdrvCheckResult *res,
>                   res->check_errors++;
>                   return ret;
>               }
> +            s->data_end = res->image_end_offset >> BDRV_SECTOR_BITS;
>               if (explicit) {
>                   res->leaks_fixed += count;
>               }
Reviewed-by: Denis V. Lunev <den@openvz.org>