diff mbox series

[3/5] dm vdo block-map: remove extra vdo arg from initialize_block_map_zone

Message ID 8feda8993a1720e4b8dd577ce875bf466e8ca020.1706322049.git.msakai@redhat.com (mailing list archive)
State Accepted, archived
Delegated to: Mike Snitzer
Headers show
Series dm vdo block-map: various small improvements | expand

Commit Message

Matthew Sakai Jan. 27, 2024, 2:35 a.m. UTC 
From: Mike Snitzer <snitzer@kernel.org>

The block_map is passed to initialize_block_map_zone, but the
block_map's vdo member is already initialized with the same vdo
instance, so just use it.

Signed-off-by: Mike Snitzer <snitzer@kernel.org>
Signed-off-by: Matthew Sakai <msakai@redhat.com>
---
 drivers/md/dm-vdo/block-map.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
diff mbox series

Patch

diff --git a/drivers/md/dm-vdo/block-map.c b/drivers/md/dm-vdo/block-map.c
index eb838e6ae3e6..eda42383a2c1 100644
--- a/drivers/md/dm-vdo/block-map.c
+++ b/drivers/md/dm-vdo/block-map.c
@@ -2719,12 +2719,12 @@  void vdo_traverse_forest(struct block_map *map, vdo_entry_callback_fn callback,
  */
 static int __must_check initialize_block_map_zone(struct block_map *map,
 						  zone_count_t zone_number,
-						  struct vdo *vdo,
 						  page_count_t cache_size,
 						  block_count_t maximum_age)
 {
 	int result;
 	block_count_t i;
+	struct vdo *vdo = map->vdo;
 	struct block_map_zone *zone = &map->zones[zone_number];
 
 	BUILD_BUG_ON(sizeof(struct page_descriptor) != sizeof(u64));
@@ -2898,8 +2898,7 @@  int vdo_decode_block_map(struct block_map_state_2_0 state, block_count_t logical
 
 	map->zone_count = vdo->thread_config.logical_zone_count;
 	for (zone = 0; zone < map->zone_count; zone++) {
-		result = initialize_block_map_zone(map, zone, vdo, cache_size,
-						   maximum_age);
+		result = initialize_block_map_zone(map, zone, cache_size, maximum_age);
 		if (result != VDO_SUCCESS) {
 			vdo_free_block_map(map);
 			return result;