[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v8 04/11] parallels: create parallels_set_bat_entry_helper() to a
From: |
Alexander Ivanov |
Subject: |
[PATCH v8 04/11] parallels: create parallels_set_bat_entry_helper() to assign BAT value |
Date: |
Sun, 15 Jan 2023 16:58:14 +0100 |
This helper will be reused in next patches during parallels_co_check
rework to simplify its code.
Signed-off-by: Alexander Ivanov <alexander.ivanov@virtuozzo.com>
Reviewed-by: Denis V. Lunev <den@openvz.org>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
---
block/parallels.c | 11 ++++++++---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/block/parallels.c b/block/parallels.c
index 9fe0f33ba9..2144ecff7d 100644
--- a/block/parallels.c
+++ b/block/parallels.c
@@ -165,6 +165,13 @@ static int64_t block_status(BDRVParallelsState *s, int64_t
sector_num,
return start_off;
}
+static void parallels_set_bat_entry(BDRVParallelsState *s,
+ uint32_t index, uint32_t offset)
+{
+ s->bat_bitmap[index] = cpu_to_le32(offset);
+ bitmap_set(s->bat_dirty_bmap, bat_entry_off(index) / s->bat_dirty_block,
1);
+}
+
static coroutine_fn int64_t allocate_clusters(BlockDriverState *bs,
int64_t sector_num,
int nb_sectors, int *pnum)
@@ -251,10 +258,8 @@ static coroutine_fn int64_t
allocate_clusters(BlockDriverState *bs,
}
for (i = 0; i < to_allocate; i++) {
- s->bat_bitmap[idx + i] = cpu_to_le32(s->data_end / s->off_multiplier);
+ parallels_set_bat_entry(s, idx + i, s->data_end / s->off_multiplier);
s->data_end += s->tracks;
- bitmap_set(s->bat_dirty_bmap,
- bat_entry_off(idx + i) / s->bat_dirty_block, 1);
}
return bat2sect(s, idx) + sector_num % s->tracks;
--
2.34.1
- [PATCH v8 00/11] parallels: Refactor the code of images checks and fix a bug, Alexander Ivanov, 2023/01/15
- [PATCH v8 02/11] parallels: Fix high_off calculation in parallels_co_check(), Alexander Ivanov, 2023/01/15
- [PATCH v8 06/11] parallels: Move check of unclean image to a separate function, Alexander Ivanov, 2023/01/15
- [PATCH v8 10/11] parallels: Replace qemu_co_mutex_lock by WITH_QEMU_LOCK_GUARD, Alexander Ivanov, 2023/01/15
- [PATCH v8 08/11] parallels: Move check of leaks to a separate function, Alexander Ivanov, 2023/01/15
- [PATCH v8 05/11] parallels: Use generic infrastructure for BAT writing in parallels_co_check(), Alexander Ivanov, 2023/01/15
- [PATCH v8 01/11] parallels: Out of image offset in BAT leads to image inflation, Alexander Ivanov, 2023/01/15
- [PATCH v8 03/11] parallels: Fix data_end after out-of-image check, Alexander Ivanov, 2023/01/15
- [PATCH v8 04/11] parallels: create parallels_set_bat_entry_helper() to assign BAT value,
Alexander Ivanov <=
- [PATCH v8 09/11] parallels: Move statistic collection to a separate function, Alexander Ivanov, 2023/01/15
- [PATCH v8 07/11] parallels: Move check of cluster outside image to a separate function, Alexander Ivanov, 2023/01/15
- [PATCH v8 11/11] parallels: Incorrect condition in out-of-image check, Alexander Ivanov, 2023/01/15