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

Commit 1d9d5c74 authored by Jaegeuk Kim's avatar Jaegeuk Kim Committed by Jaegeuk Kim
Browse files

Revert "f2fs: introduce f2fs_set_page_dirty_nobuffer" 



Cherry-pick from origin/upstream-f2fs-stable-linux-3.18.y:
commit 1e79b9b2 ("Revert "f2fs: introduce f2fs_set_page_dirty_nobuffer")

This patch reverts copied f2fs_set_page_dirty_nobuffer to use generic function
for stability.

This reverts commit fe76b796fc5194cc3d57265002e3a748566d073f.

Change-Id: Ie3084e047d91cd1d01eb1cc2cb324654ab908121
Reviewed-by: default avatarChao Yu <yuchao0@huawei.com>
Signed-off-by: default avatarJaegeuk Kim <jaegeuk@kernel.org>
parent 777eb8a1

fs/f2fs/checkpoint.c

+1 −1
Original line number Diff line number Diff line
@@ -386,7 +386,7 @@ static int f2fs_set_meta_page_dirty(struct page *page) 
	if (!PageUptodate(page))

		SetPageUptodate(page);

	if (!PageDirty(page)) {

		f2fs_set_page_dirty_nobuffers(page);

		__set_page_dirty_nobuffers(page);

		inc_page_count(F2FS_P_SB(page), F2FS_DIRTY_META);

		SetPagePrivate(page);

		f2fs_trace_pid(page);

fs/f2fs/data.c

+1 −30
Original line number Diff line number Diff line
@@ -20,8 +20,6 @@ 
#include <linux/bio.h>

#include <linux/prefetch.h>

#include <linux/uio.h>

#include <linux/mm.h>

#include <linux/memcontrol.h>

#include <linux/cleancache.h>



#include "f2fs.h"
@@ -2485,33 +2483,6 @@ int f2fs_release_page(struct page *page, gfp_t wait) 
	return 1;

}



/*

 * This was copied from __set_page_dirty_buffers which gives higher performance

 * in very high speed storages. (e.g., pmem)

 */

void f2fs_set_page_dirty_nobuffers(struct page *page)

{

	struct address_space *mapping = page->mapping;

	unsigned long flags;



	if (unlikely(!mapping))

		return;



	spin_lock(&mapping->private_lock);

	SetPageDirty(page);

	spin_unlock(&mapping->private_lock);



	spin_lock_irqsave(&mapping->tree_lock, flags);

	WARN_ON_ONCE(!PageUptodate(page));

	account_page_dirtied(page, mapping);

	radix_tree_tag_set(&mapping->page_tree,

			page_index(page), PAGECACHE_TAG_DIRTY);

	spin_unlock_irqrestore(&mapping->tree_lock, flags);



	__mark_inode_dirty(mapping->host, I_DIRTY_PAGES);

	return;

}



static int f2fs_set_data_page_dirty(struct page *page)

{

	struct address_space *mapping = page->mapping;
@@ -2535,7 +2506,7 @@ static int f2fs_set_data_page_dirty(struct page *page) 
	}



	if (!PageDirty(page)) {

		f2fs_set_page_dirty_nobuffers(page);

		__set_page_dirty_nobuffers(page);

		update_dirty_page(inode, page);

		return 1;

	}

fs/f2fs/f2fs.h

+0 −1
Original line number Diff line number Diff line
@@ -3023,7 +3023,6 @@ int f2fs_fiemap(struct inode *inode, struct fiemap_extent_info *fieinfo, 
			u64 start, u64 len);

bool should_update_inplace(struct inode *inode, struct f2fs_io_info *fio);

bool should_update_outplace(struct inode *inode, struct f2fs_io_info *fio);

void f2fs_set_page_dirty_nobuffers(struct page *page);

int __f2fs_write_data_pages(struct address_space *mapping,

						struct writeback_control *wbc,

						enum iostat_type io_type);

fs/f2fs/node.c

+1 −1
Original line number Diff line number Diff line
@@ -1776,7 +1776,7 @@ static int f2fs_set_node_page_dirty(struct page *page) 
	if (!PageUptodate(page))

		SetPageUptodate(page);

	if (!PageDirty(page)) {

		f2fs_set_page_dirty_nobuffers(page);

		__set_page_dirty_nobuffers(page);

		inc_page_count(F2FS_P_SB(page), F2FS_DIRTY_NODES);

		SetPagePrivate(page);

		f2fs_trace_pid(page);