From: Tomohiro Kusumi <kusumi.tomohiro@gmail.com>
Date: Tue, 1 May 2018 18:15:54 +0000 (-0700)
Subject: Silence compile-time warning on unused variable
X-Git-Tag: zfs-0.8.0-rc1~191
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=d6133fc500ea8b4fdc1a51944cb17ea5e70f3137;p=zfs

Silence compile-time warning on unused variable

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
---

diff --git a/module/zfs/metaslab.c b/module/zfs/metaslab.c
index 1dbe1e98c..9f0065b53 100644
--- a/module/zfs/metaslab.c
+++ b/module/zfs/metaslab.c
@@ -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);