dm space map disk: remove redundant calls to sm_disk_get_nr_free()
authorJoe Thornber <ejt@redhat.com>
Tue, 9 Mar 2021 14:56:59 +0000 (14:56 +0000)
committerMike Snitzer <snitzer@redhat.com>
Mon, 19 Apr 2021 16:36:46 +0000 (12:36 -0400)
Both sm_disk_new_block and sm_disk_commit are needlessly calling
sm_disk_get_nr_free(). Looks like old queries used for some
debugging.

Signed-off-by: Joe Thornber <ejt@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
drivers/md/persistent-data/dm-space-map-disk.c

index bf4c5e2ccb6ffc8d8e0fa5ba049429bdb6dc18d0..61f56909e00bebd7862976a07b71b268e17fc6b0 100644 (file)
@@ -187,13 +187,8 @@ static int sm_disk_new_block(struct dm_space_map *sm, dm_block_t *b)
 static int sm_disk_commit(struct dm_space_map *sm)
 {
        int r;
-       dm_block_t nr_free;
        struct sm_disk *smd = container_of(sm, struct sm_disk, sm);
 
-       r = sm_disk_get_nr_free(sm, &nr_free);
-       if (r)
-               return r;
-
        r = sm_ll_commit(&smd->ll);
        if (r)
                return r;
@@ -202,10 +197,6 @@ static int sm_disk_commit(struct dm_space_map *sm)
        smd->begin = 0;
        smd->nr_allocated_this_transaction = 0;
 
-       r = sm_disk_get_nr_free(sm, &nr_free);
-       if (r)
-               return r;
-
        return 0;
 }