Unverified Commit bc986b1d authored by Josef Bacik's avatar Josef Bacik Committed by Christian Brauner
Browse files

fs: stop accessing ->i_count directly in f2fs and gfs2 



Instead of accessing ->i_count directly in these file systems, use the
appropriate __iget and iput helpers.

Signed-off-by: default avatarJosef Bacik <josef@toxicpanda.com>
Link: https://lore.kernel.org/b8e6eb8a3e690ce082828d3580415bf70dfa93aa.1755806649.git.josef@toxicpanda.com


Signed-off-by: default avatarChristian Brauner <brauner@kernel.org>
parent f0883b9c

fs/f2fs/super.c

+2 −2
Original line number Diff line number Diff line
@@ -1750,7 +1750,7 @@ static int f2fs_drop_inode(struct inode *inode) 
	if ((!inode_unhashed(inode) && inode->i_state & I_SYNC)) {

		if (!inode->i_nlink && !is_bad_inode(inode)) {

			/* to avoid evict_inode call simultaneously */

			atomic_inc(&inode->i_count);

			__iget(inode);

			spin_unlock(&inode->i_lock);



			/* should remain fi->extent_tree for writepage */
@@ -1769,7 +1769,7 @@ static int f2fs_drop_inode(struct inode *inode) 
			sb_end_intwrite(inode->i_sb);



			spin_lock(&inode->i_lock);

			atomic_dec(&inode->i_count);

			iput(inode);

		}

		trace_f2fs_drop_inode(inode, 0);

		return 0;

fs/gfs2/ops_fstype.c

+1 −1
Original line number Diff line number Diff line
@@ -1754,7 +1754,7 @@ static void gfs2_evict_inodes(struct super_block *sb) 
			spin_unlock(&inode->i_lock);

			continue;

		}

		atomic_inc(&inode->i_count);

		__iget(inode);

		spin_unlock(&inode->i_lock);

		spin_unlock(&sb->s_inode_list_lock);