From 915f93100d643d293179ecbf708aeaa79e6b789b Mon Sep 17 00:00:00 2001 From: Yang Guo Date: Tue, 13 Jun 2023 20:45:12 +0800 Subject: [PATCH] ext4: delete redundant uptodate check for buffer mainline inclusion from mainline-v5.13-rc1 commit 3cd461712cb450ea6aa8a8200f35f24eb296bfb2 category: bugfix bugzilla: https://gitee.com/openeuler/kernel/issues/I7D878 CVE: NA Reference: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=3cd461712cb450ea6aa8a8200f35f24eb296bfb2 --------------------------- The buffer uptodate state has been checked in function set_buffer_uptodate, there is no need use buffer_uptodate before calling set_buffer_uptodate and delete it. Cc: "Theodore Ts'o" Cc: Andreas Dilger Signed-off-by: Yang Guo Signed-off-by: Shaokun Zhang Reviewed-by: Ritesh Harjani Link: https://lore.kernel.org/r/1617260610-29770-1-git-send-email-zhangshaokun@hisilicon.com Signed-off-by: Theodore Ts'o Signed-off-by: Zheng Zengkai (cherry picked from commit 47b62d723ea118619e7eb35888996024a8031c9e) --- fs/ext4/inode.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c index 9379a062dba4..3c97cf3e9f65 100644 --- a/fs/ext4/inode.c +++ b/fs/ext4/inode.c @@ -1067,8 +1067,7 @@ static int ext4_block_write_begin(struct page *page, loff_t pos, unsigned len, block_end = block_start + blocksize; if (block_end <= from || block_start >= to) { if (PageUptodate(page)) { - if (!buffer_uptodate(bh)) - set_buffer_uptodate(bh); + set_buffer_uptodate(bh); } continue; } @@ -1093,8 +1092,7 @@ static int ext4_block_write_begin(struct page *page, loff_t pos, unsigned len, } } if (PageUptodate(page)) { - if (!buffer_uptodate(bh)) - set_buffer_uptodate(bh); + set_buffer_uptodate(bh); continue; } if (!buffer_uptodate(bh) && !buffer_delay(bh) && -- Gitee