diff options
author | Jiang Biao <benbjiang@tencent.com> | 2019-04-23 20:58:57 -0400 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2019-04-25 11:04:43 -0400 |
commit | 78bc3334a69ff289dbc973a9db7c52a2d7757e5b (patch) | |
tree | 60347d04fe03d77f5093d0abd82e4c6c5f24c752 | |
parent | 3ef177ec9d5dc225bf30927a2d222d8bb1fbee0f (diff) |
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>
-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 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, | |||
1050 | struct list_head *tofree_head) | 1050 | struct list_head *tofree_head) |
1051 | { | 1051 | { |
1052 | struct inode *inode; | 1052 | struct inode *inode; |
1053 | #ifdef CONFIG_QUOTA_DEBUG | ||
1053 | int reserved = 0; | 1054 | int reserved = 0; |
1055 | #endif | ||
1054 | 1056 | ||
1055 | spin_lock(&sb->s_inode_list_lock); | 1057 | spin_lock(&sb->s_inode_list_lock); |
1056 | list_for_each_entry(inode, &sb->s_inodes, i_sb_list) { | 1058 | 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, | |||
1062 | */ | 1064 | */ |
1063 | spin_lock(&dq_data_lock); | 1065 | spin_lock(&dq_data_lock); |
1064 | if (!IS_NOQUOTA(inode)) { | 1066 | if (!IS_NOQUOTA(inode)) { |
1067 | #ifdef CONFIG_QUOTA_DEBUG | ||
1065 | if (unlikely(inode_get_rsv_space(inode) > 0)) | 1068 | if (unlikely(inode_get_rsv_space(inode) > 0)) |
1066 | reserved = 1; | 1069 | reserved = 1; |
1070 | #endif | ||
1067 | remove_inode_dquot_ref(inode, type, tofree_head); | 1071 | remove_inode_dquot_ref(inode, type, tofree_head); |
1068 | } | 1072 | } |
1069 | spin_unlock(&dq_data_lock); | 1073 | spin_unlock(&dq_data_lock); |