Re: [PATCH 06/16] bcache: Suppress more warnings about set-but-not-used variables
From: Coly Li <hidden>
Date: 2018-03-15 16:21:14
On 15/03/2018 11:08 PM, Bart Van Assche wrote:
This patch does not change any functionality. Signed-off-by: Bart Van Assche <redacted>
Hi Bart, This patch looks good to me. A question is, does GCC later than 4.3 supports such annotation like '__maybe_unused' ? Thanks. Coly Li
quoted hunk ↗ jump to hunk
--- drivers/md/bcache/bset.c | 4 ++-- drivers/md/bcache/journal.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-)diff --git a/drivers/md/bcache/bset.c b/drivers/md/bcache/bset.c index e56d3ecdbfcb..579c696a5fe0 100644 --- a/drivers/md/bcache/bset.c +++ b/drivers/md/bcache/bset.c@@ -1072,7 +1072,7 @@ EXPORT_SYMBOL(bch_btree_iter_init); static inline struct bkey *__bch_btree_iter_next(struct btree_iter *iter, btree_iter_cmp_fn *cmp) { - struct btree_iter_set unused; + struct btree_iter_set b __maybe_unused; struct bkey *ret = NULL; if (!btree_iter_end(iter)) {@@ -1087,7 +1087,7 @@ static inline struct bkey *__bch_btree_iter_next(struct btree_iter *iter, } if (iter->data->k == iter->data->end) - heap_pop(iter, unused, cmp); + heap_pop(iter, b, cmp); else heap_sift(iter, 0, cmp); }diff --git a/drivers/md/bcache/journal.c b/drivers/md/bcache/journal.c index 1b736b860739..605368ff13c9 100644 --- a/drivers/md/bcache/journal.c +++ b/drivers/md/bcache/journal.c@@ -493,7 +493,7 @@ static void journal_reclaim(struct cache_set *c) struct cache *ca; uint64_t last_seq; unsigned iter, n = 0; - atomic_t p; + atomic_t p __maybe_unused; atomic_long_inc(&c->reclaim);