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

Commit 0ef31c7b authored by Jaegeuk Kim's avatar Jaegeuk Kim Committed by Jaegeuk Kim
Browse files

f2fs: keep dirty inodes selectively for checkpoint 



commit 7c45729a4d6d1c90879e6c5c2df325c2f6db7191 upstream.

This is to avoid no free segment bug during checkpoint caused by a number of
dirty inodes.

The case was reported by Chao like this.
1. mount with lazytime option
2. fill 4k file until disk is full
3. sync filesystem
4. read all files in the image
5. umount

In this case, we actually don't need to flush dirty inode to inode page during
checkpoint.

Reviewed-by: default avatarChao Yu <yuchao0@huawei.com>
Signed-off-by: default avatarJaegeuk Kim <jaegeuk@kernel.org>
parent dafac77e

fs/f2fs/acl.c

+1 −1
Original line number Diff line number Diff line
@@ -387,7 +387,7 @@ int f2fs_init_acl(struct inode *inode, struct inode *dir, struct page *ipage, 
	if (error)

		return error;



	f2fs_mark_inode_dirty_sync(inode);

	f2fs_mark_inode_dirty_sync(inode, true);



	if (default_acl) {

		error = __f2fs_set_acl(inode, ACL_TYPE_DEFAULT, default_acl,

fs/f2fs/dir.c

+3 −3
Original line number Diff line number Diff line
@@ -313,7 +313,7 @@ void f2fs_set_link(struct inode *dir, struct f2fs_dir_entry *de, 
	set_page_dirty(page);



	dir->i_mtime = dir->i_ctime = current_time(dir);

	f2fs_mark_inode_dirty_sync(dir);

	f2fs_mark_inode_dirty_sync(dir, false);

	f2fs_put_page(page, 1);

}
@@ -466,7 +466,7 @@ void update_parent_metadata(struct inode *dir, struct inode *inode, 
		clear_inode_flag(inode, FI_NEW_INODE);

	}

	dir->i_mtime = dir->i_ctime = current_time(dir);

	f2fs_mark_inode_dirty_sync(dir);

	f2fs_mark_inode_dirty_sync(dir, false);



	if (F2FS_I(dir)->i_current_depth != current_depth)

		f2fs_i_depth_write(dir, current_depth);
@@ -731,7 +731,7 @@ void f2fs_delete_entry(struct f2fs_dir_entry *dentry, struct page *page, 
	set_page_dirty(page);



	dir->i_ctime = dir->i_mtime = current_time(dir);

	f2fs_mark_inode_dirty_sync(dir);

	f2fs_mark_inode_dirty_sync(dir, false);



	if (inode)

		f2fs_drop_nlink(dir, inode);

fs/f2fs/extent_cache.c

+1 −1
Original line number Diff line number Diff line
@@ -172,7 +172,7 @@ static void __drop_largest_extent(struct inode *inode, 


	if (fofs < largest->fofs + largest->len && fofs + len > largest->fofs) {

		largest->len = 0;

		f2fs_mark_inode_dirty_sync(inode);

		f2fs_mark_inode_dirty_sync(inode, true);

	}

}

fs/f2fs/f2fs.h

+13 −13
Original line number Diff line number Diff line
@@ -541,13 +541,13 @@ static inline bool __is_front_mergeable(struct extent_info *cur, 
	return __is_extent_mergeable(cur, front);

}



extern void f2fs_mark_inode_dirty_sync(struct inode *);

extern void f2fs_mark_inode_dirty_sync(struct inode *, bool);

static inline void __try_update_largest_extent(struct inode *inode,

			struct extent_tree *et, struct extent_node *en)

{

	if (en->ei.len > et->largest.len) {

		et->largest = en->ei;

		f2fs_mark_inode_dirty_sync(inode);

		f2fs_mark_inode_dirty_sync(inode, true);

	}

}
@@ -1680,7 +1680,7 @@ static inline void __mark_inode_dirty_flag(struct inode *inode, 
			return;

	case FI_DATA_EXIST:

	case FI_INLINE_DOTS:

		f2fs_mark_inode_dirty_sync(inode);

		f2fs_mark_inode_dirty_sync(inode, true);

	}

}
@@ -1707,7 +1707,7 @@ static inline void set_acl_inode(struct inode *inode, umode_t mode) 
{

	F2FS_I(inode)->i_acl_mode = mode;

	set_inode_flag(inode, FI_ACL_MODE);

	f2fs_mark_inode_dirty_sync(inode);

	f2fs_mark_inode_dirty_sync(inode, false);

}



static inline void f2fs_i_links_write(struct inode *inode, bool inc)
@@ -1716,7 +1716,7 @@ static inline void f2fs_i_links_write(struct inode *inode, bool inc) 
		inc_nlink(inode);

	else

		drop_nlink(inode);

	f2fs_mark_inode_dirty_sync(inode);

	f2fs_mark_inode_dirty_sync(inode, true);

}



static inline void f2fs_i_blocks_write(struct inode *inode,
@@ -1727,7 +1727,7 @@ static inline void f2fs_i_blocks_write(struct inode *inode, 


	inode->i_blocks = add ? inode->i_blocks + diff :

				inode->i_blocks - diff;

	f2fs_mark_inode_dirty_sync(inode);

	f2fs_mark_inode_dirty_sync(inode, true);

	if (clean || recover)

		set_inode_flag(inode, FI_AUTO_RECOVER);

}
@@ -1741,7 +1741,7 @@ static inline void f2fs_i_size_write(struct inode *inode, loff_t i_size) 
		return;



	i_size_write(inode, i_size);

	f2fs_mark_inode_dirty_sync(inode);

	f2fs_mark_inode_dirty_sync(inode, true);

	if (clean || recover)

		set_inode_flag(inode, FI_AUTO_RECOVER);

}
@@ -1756,19 +1756,19 @@ static inline bool f2fs_skip_inode_update(struct inode *inode) 
static inline void f2fs_i_depth_write(struct inode *inode, unsigned int depth)

{

	F2FS_I(inode)->i_current_depth = depth;

	f2fs_mark_inode_dirty_sync(inode);

	f2fs_mark_inode_dirty_sync(inode, true);

}



static inline void f2fs_i_xnid_write(struct inode *inode, nid_t xnid)

{

	F2FS_I(inode)->i_xattr_nid = xnid;

	f2fs_mark_inode_dirty_sync(inode);

	f2fs_mark_inode_dirty_sync(inode, true);

}



static inline void f2fs_i_pino_write(struct inode *inode, nid_t pino)

{

	F2FS_I(inode)->i_pino = pino;

	f2fs_mark_inode_dirty_sync(inode);

	f2fs_mark_inode_dirty_sync(inode, true);

}



static inline void get_inline_info(struct inode *inode, struct f2fs_inode *ri)
@@ -1896,13 +1896,13 @@ static inline int is_file(struct inode *inode, int type) 
static inline void set_file(struct inode *inode, int type)

{

	F2FS_I(inode)->i_advise |= type;

	f2fs_mark_inode_dirty_sync(inode);

	f2fs_mark_inode_dirty_sync(inode, true);

}



static inline void clear_file(struct inode *inode, int type)

{

	F2FS_I(inode)->i_advise &= ~type;

	f2fs_mark_inode_dirty_sync(inode);

	f2fs_mark_inode_dirty_sync(inode, true);

}



static inline int f2fs_readonly(struct super_block *sb)
@@ -2054,7 +2054,7 @@ static inline int f2fs_add_link(struct dentry *dentry, struct inode *inode) 
/*

 * super.c

 */

int f2fs_inode_dirtied(struct inode *);

int f2fs_inode_dirtied(struct inode *, bool);

void f2fs_inode_synced(struct inode *);

int f2fs_commit_super(struct f2fs_sb_info *, bool);

int f2fs_sync_fs(struct super_block *, int);

fs/f2fs/file.c

+5 −4
Original line number Diff line number Diff line
@@ -265,7 +265,7 @@ sync_nodes: 
	}



	if (need_inode_block_update(sbi, ino)) {

		f2fs_mark_inode_dirty_sync(inode);

		f2fs_mark_inode_dirty_sync(inode, true);

		f2fs_write_inode(inode, NULL);

		goto sync_nodes;

	}
@@ -633,7 +633,7 @@ int f2fs_truncate(struct inode *inode) 
		return err;



	inode->i_mtime = inode->i_ctime = current_time(inode);

	f2fs_mark_inode_dirty_sync(inode);

	f2fs_mark_inode_dirty_sync(inode, false);

	return 0;

}
@@ -722,7 +722,8 @@ int f2fs_setattr(struct dentry *dentry, struct iattr *attr) 
		}

	}



	f2fs_mark_inode_dirty_sync(inode);

	/* update attributes only */

	f2fs_mark_inode_dirty_sync(inode, false);



	/* inode change will produce dirty node pages flushed by checkpoint */

	f2fs_balance_fs(F2FS_I_SB(inode), true);
@@ -1403,7 +1404,7 @@ static long f2fs_fallocate(struct file *file, int mode, 


	if (!ret) {

		inode->i_mtime = inode->i_ctime = current_time(inode);

		f2fs_mark_inode_dirty_sync(inode);

		f2fs_mark_inode_dirty_sync(inode, false);

		f2fs_update_time(F2FS_I_SB(inode), REQ_TIME);

	}