[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 4/9] parallels: Move check of fragmentation to a separate functio
From: |
Alexander Ivanov |
Subject: |
[PATCH 4/9] parallels: Move check of fragmentation to a separate function |
Date: |
Mon, 8 Aug 2022 14:07:29 +0200 |
Signed-off-by: Alexander Ivanov <alexander.ivanov@virtuozzo.com>
---
block/parallels.c | 38 +++++++++++++++++++++++++-------------
1 file changed, 25 insertions(+), 13 deletions(-)
diff --git a/block/parallels.c b/block/parallels.c
index 6d4cfb738b..0edbb812dd 100644
--- a/block/parallels.c
+++ b/block/parallels.c
@@ -530,12 +530,34 @@ static int parallels_check_leak(BlockDriverState *bs,
return 0;
}
+static void parallels_check_fragmentation(BlockDriverState *bs,
+ BdrvCheckResult *res,
+ BdrvCheckMode fix)
+{
+ BDRVParallelsState *s = bs->opaque;
+ uint32_t i;
+ int64_t off, prev_off;
+
+ prev_off = 0;
+ for (i = 0; i < s->bat_size; i++) {
+ off = bat2sect(s, i) << BDRV_SECTOR_BITS;
+ if (off == 0) {
+ prev_off = 0;
+ continue;
+ }
+ if (prev_off != 0 && (prev_off + s->cluster_size) != off) {
+ res->bfi.fragmented_clusters++;
+ }
+ prev_off = off;
+ }
+
+}
+
static int coroutine_fn parallels_co_check(BlockDriverState *bs,
BdrvCheckResult *res,
BdrvCheckMode fix)
{
BDRVParallelsState *s = bs->opaque;
- int64_t prev_off;
int ret;
uint32_t i;
@@ -553,23 +575,13 @@ static int coroutine_fn
parallels_co_check(BlockDriverState *bs,
goto out;
}
+ parallels_check_fragmentation(bs, res, fix);
+
res->bfi.total_clusters = s->bat_size;
res->bfi.compressed_clusters = 0; /* compression is not supported */
- prev_off = 0;
for (i = 0; i < s->bat_size; i++) {
- int64_t off = bat2sect(s, i) << BDRV_SECTOR_BITS;
- if (off == 0) {
- prev_off = 0;
- continue;
- }
-
res->bfi.allocated_clusters++;
-
- if (prev_off != 0 && (prev_off + s->cluster_size) != off) {
- res->bfi.fragmented_clusters++;
- }
- prev_off = off;
}
ret = 0;
--
2.34.1
- [PATCH 0/9] parallels: Refactor the code of images checks and fix a bug, Alexander Ivanov, 2022/08/08
- [PATCH 1/9] parallels: Move check of unclean image to a separate function, Alexander Ivanov, 2022/08/08
- [PATCH 3/9] parallels: Move check of leaks to a separate function, Alexander Ivanov, 2022/08/08
- [PATCH 2/9] parallels: Move check of cluster outside image to a separate function, Alexander Ivanov, 2022/08/08
- [PATCH 4/9] parallels: Move check of fragmentation to a separate function,
Alexander Ivanov <=
- [PATCH 5/9] parallels: Move statistic collection to a separate function, Alexander Ivanov, 2022/08/08
- [PATCH 6/9] parallels: Replace qemu_co_mutex_lock by WITH_QEMU_LOCK_GUARD, Alexander Ivanov, 2022/08/08
- [PATCH 7/9] parallels: Out of image offset in BAT leads to image inflation, Alexander Ivanov, 2022/08/08
- [PATCH 8/9] parallels: Replace bdrv_co_pwrite_sync by bdrv_co_flush for BAT flushing, Alexander Ivanov, 2022/08/08
- [PATCH 9/9] parallels: Merge parallels_check_fragmentation to parallels_collect_statistics, Alexander Ivanov, 2022/08/08