]> git.proxmox.com Git - mirror_zfs.git/commitdiff
Silence compile-time warning on unused variable
authorTomohiro Kusumi <kusumi.tomohiro@gmail.com>
Tue, 1 May 2018 18:15:54 +0000 (11:15 -0700)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Tue, 1 May 2018 18:15:54 +0000 (11:15 -0700)
ASSERT3U() could be NOP which then leads to having unused pointer *spa.

metaslab.c: In function 'metaslab_condense':
metaslab.c:2075:9: warning: unused variable 'spa' [-Wunused-variable]
  spa_t *spa = msp->ms_group->mg_vd->vdev_spa;

Reviewed-by: Giuseppe Di Natale <dinatale2@llnl.gov>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: George Melikov <mail@gmelikov.ru>
Signed-off-by: Tomohiro Kusumi <kusumi.tomohiro@osnexus.com>
Closes #7489

module/zfs/metaslab.c

index 1dbe1e98c1aa13c6840fd955b481967bd645b367..9f0065b532d20de31741f77c9d2a65822b956354 100644 (file)
@@ -2072,12 +2072,11 @@ metaslab_should_condense(metaslab_t *msp)
 static void
 metaslab_condense(metaslab_t *msp, uint64_t txg, dmu_tx_t *tx)
 {
-       spa_t *spa = msp->ms_group->mg_vd->vdev_spa;
        range_tree_t *condense_tree;
        space_map_t *sm = msp->ms_sm;
 
        ASSERT(MUTEX_HELD(&msp->ms_lock));
-       ASSERT3U(spa_sync_pass(spa), ==, 1);
+       ASSERT3U(spa_sync_pass(msp->ms_group->mg_vd->vdev_spa), ==, 1);
        ASSERT(msp->ms_loaded);