Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 1974c130 authored by Jiang Biao's avatar Jiang Biao Committed by Sasha Levin
Browse files

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: default avatarJiang Biao <benbjiang@tencent.com>
Signed-off-by: default avatarJan Kara <jack@suse.cz>
Stable-dep-of: 179b8c97ebf6 ("quota: Fix rcu annotations of inode dquot pointers")
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent d9e4ab12

fs/quota/dquot.c

+4 −0
Original line number Diff line number Diff line
@@ -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);