Unverified Commit 0735dc48 authored by openeuler-ci-bot's avatar openeuler-ci-bot Committed by Gitee
Browse files

!8737 ubifs: Set page uptodate in the correct place 

Merge Pull Request from: @ci-robot 
 
PR sync from: Wang Zhaolong <wangzhaolong1@huawei.com>
https://mailweb.openeuler.org/hyperkitty/list/kernel@openeuler.org/message/DHDI34K2GWCZMZTFVD5OOA6V7DVBZ3YI/ 
 
https://gitee.com/src-openeuler/kernel/issues/I9Q97O 
 
Link:https://gitee.com/openeuler/kernel/pulls/8737

 

Reviewed-by: default avatarLiu YongQiang <liuyongqiang13@huawei.com>
Signed-off-by: default avatarZhang Changzhong <zhangchangzhong@huawei.com>
parents 07b2479c a1db7188

fs/ubifs/file.c

+4 −9
Original line number Diff line number Diff line
@@ -274,9 +274,6 @@ static int write_begin_slow(struct address_space *mapping, 
				return err;

			}

		}



		SetPageUptodate(page);

		ClearPageError(page);

	}



	if (PagePrivate(page))
@@ -475,9 +472,6 @@ static int ubifs_write_begin(struct file *file, struct address_space *mapping, 
				return err;

			}

		}



		SetPageUptodate(page);

		ClearPageError(page);

	}



	err = allocate_budget(c, page, ui, appending);
@@ -487,10 +481,8 @@ static int ubifs_write_begin(struct file *file, struct address_space *mapping, 
		 * If we skipped reading the page because we were going to

		 * write all of it, then it is not up to date.

		 */

		if (skipped_read) {

		if (skipped_read)

			ClearPageChecked(page);

			ClearPageUptodate(page);

		}

		/*

		 * Budgeting failed which means it would have to force

		 * write-back but didn't, because we set the @fast flag in the
@@ -581,6 +573,9 @@ static int ubifs_write_end(struct file *file, struct address_space *mapping, 
		goto out;

	}



	if (len == PAGE_SIZE)

		SetPageUptodate(page);



	if (!PagePrivate(page)) {

		SetPagePrivate(page);

		atomic_long_inc(&c->dirty_pg_cnt);