summaryrefslogtreecommitdiffstats
path: root/fs/quota
diff options
context:
space:
mode:
authorJiang Biao <benbjiang@tencent.com>2019-04-24 08:58:57 +0800
committerJan Kara <jack@suse.cz>2019-04-25 17:04:43 +0200
commit78bc3334a69ff289dbc973a9db7c52a2d7757e5b (patch)
tree60347d04fe03d77f5093d0abd82e4c6c5f24c752 /fs/quota
parent3ef177ec9d5dc225bf30927a2d222d8bb1fbee0f (diff)
downloadlinux-78bc3334a69ff289dbc973a9db7c52a2d7757e5b.tar.bz2
fs/quota: erase unused but set variable warning
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>
Diffstat (limited to 'fs/quota')
-rw-r--r--fs/quota/dquot.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/quota/dquot.c b/fs/quota/dquot.c
index d2f2972b33db..67e534dbfc0e 100644
--- a/fs/quota/dquot.c
+++ b/fs/quota/dquot.c
@@ -1050,7 +1050,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) {
@@ -1062,8 +1064,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);