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

Commit 60ed9a0f authored by Gu Zheng's avatar Gu Zheng Committed by Jaegeuk Kim
Browse files

f2fs: use list_for_each rather than list_for_each_safe, in remove_orphan_inode() 



As we remove the target single node, so list_for_each is enought, in order to
clean up, we use list_for_each_entry instead.

Signed-off-by: default avatarGu Zheng <guz.fnst@cn.fujitsu.com>
Signed-off-by: default avatarJaegeuk Kim <jaegeuk.kim@samsung.com>
parent 2d219c51

fs/f2fs/checkpoint.c

+2 −3
Original line number Original line Diff line number Diff line
@@ -237,13 +237,12 @@ void add_orphan_inode(struct f2fs_sb_info *sbi, nid_t ino) 




void remove_orphan_inode(struct f2fs_sb_info *sbi, nid_t ino)

void remove_orphan_inode(struct f2fs_sb_info *sbi, nid_t ino)

{

{

	struct list_head *this, *next, *head;

	struct list_head *head;

	struct orphan_inode_entry *orphan;

	struct orphan_inode_entry *orphan;





	mutex_lock(&sbi->orphan_inode_mutex);

	mutex_lock(&sbi->orphan_inode_mutex);

	head = &sbi->orphan_inode_list;

	head = &sbi->orphan_inode_list;

	list_for_each_safe(this, next, head) {

	list_for_each_entry(orphan, head, list) {

		orphan = list_entry(this, struct orphan_inode_entry, list);

		if (orphan->ino == ino) {

		if (orphan->ino == ino) {

			list_del(&orphan->list);

			list_del(&orphan->list);

			kmem_cache_free(orphan_entry_slab, orphan);

			kmem_cache_free(orphan_entry_slab, orphan);