diff options
author | Jiang Biao <benbjiang@tencent.com> | 2019-04-24 08:58:57 +0800 |
---|---|---|
committer | Sasha Levin <sashal@kernel.org> | 2024-03-26 18:22:39 -0400 |
commit | 1974c13019b97140149690cd622a9c6732c32124 (patch) | |
tree | 8b9b8b2a64dd5298540c0bb1c577880877d30752 /fs/quota | |
parent | d9e4ab12b60a49204435102f8120c0d3e62583f1 (diff) | |
download | linux-stable-1974c13019b97140149690cd622a9c6732c32124.tar.gz linux-stable-1974c13019b97140149690cd622a9c6732c32124.tar.bz2 linux-stable-1974c13019b97140149690cd622a9c6732c32124.zip |
fs/quota: erase unused but set variable warning
[ Upstream commit 78bc3334a69ff289dbc973a9db7c52a2d7757e5b ]
Local variable *reserved* of remove_dquot_ref() is only used if
define CONFIG_QUOTA_DEBUG, but not ebraced in CONFIG_QUOTA_DEBUG
macro, which leads to unused-but-set-variable warning when compiling.
This patch ebrace it into CONFIG_QUOTA_DEBUG macro like what is done
in add_dquot_ref().
Signed-off-by: Jiang Biao <benbjiang@tencent.com>
Signed-off-by: Jan Kara <jack@suse.cz>
Stable-dep-of: 179b8c97ebf6 ("quota: Fix rcu annotations of inode dquot pointers")
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'fs/quota')
-rw-r--r-- | fs/quota/dquot.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/quota/dquot.c b/fs/quota/dquot.c index 683727c5758c..b55d91d3d87c 100644 --- a/fs/quota/dquot.c +++ b/fs/quota/dquot.c @@ -1129,7 +1129,9 @@ static void remove_dquot_ref(struct super_block *sb, int type, struct list_head *tofree_head) { struct inode *inode; +#ifdef CONFIG_QUOTA_DEBUG int reserved = 0; +#endif spin_lock(&sb->s_inode_list_lock); list_for_each_entry(inode, &sb->s_inodes, i_sb_list) { @@ -1141,8 +1143,10 @@ static void remove_dquot_ref(struct super_block *sb, int type, */ spin_lock(&dq_data_lock); if (!IS_NOQUOTA(inode)) { +#ifdef CONFIG_QUOTA_DEBUG if (unlikely(inode_get_rsv_space(inode) > 0)) reserved = 1; +#endif remove_inode_dquot_ref(inode, type, tofree_head); } spin_unlock(&dq_data_lock); |