Commit d13732cc authored by Jia Yang's avatar Jia Yang Committed by Jaegeuk Kim
Browse files

f2fs: remove unnecessary read for F2FS_FITS_IN_INODE 



F2FS_FITS_IN_INODE only cares the type of f2fs inode, so there
is no need to read node page of f2fs inode.

Signed-off-by: default avatarJia Yang <jiayang5@huawei.com>
Reviewed-by: default avatarChao Yu <chao@kernel.org>
Signed-off-by: default avatarJaegeuk Kim <jaegeuk@kernel.org>
parent 7f8e249d

fs/f2fs/file.c

+4 −13
Original line number Diff line number Diff line
@@ -812,7 +812,7 @@ int f2fs_getattr(struct user_namespace *mnt_userns, const struct path *path, 
{

	struct inode *inode = d_inode(path->dentry);

	struct f2fs_inode_info *fi = F2FS_I(inode);

	struct f2fs_inode *ri;

	struct f2fs_inode *ri = NULL;

	unsigned int flags;



	if (f2fs_has_extra_attr(inode) &&
@@ -2999,7 +2999,7 @@ static int f2fs_ioc_setproject(struct inode *inode, __u32 projid) 
{

	struct f2fs_inode_info *fi = F2FS_I(inode);

	struct f2fs_sb_info *sbi = F2FS_I_SB(inode);

	struct page *ipage;

	struct f2fs_inode *ri = NULL;

	kprojid_t kprojid;

	int err;
@@ -3023,17 +3023,8 @@ static int f2fs_ioc_setproject(struct inode *inode, __u32 projid) 
	if (IS_NOQUOTA(inode))

		return err;



	ipage = f2fs_get_node_page(sbi, inode->i_ino);

	if (IS_ERR(ipage))

		return PTR_ERR(ipage);



	if (!F2FS_FITS_IN_INODE(F2FS_INODE(ipage), fi->i_extra_isize,

								i_projid)) {

		err = -EOVERFLOW;

		f2fs_put_page(ipage, 1);

		return err;

	}

	f2fs_put_page(ipage, 1);

	if (!F2FS_FITS_IN_INODE(ri, fi->i_extra_isize, i_projid))

		return -EOVERFLOW;



	err = f2fs_dquot_initialize(inode);

	if (err)