From 50c40f22e6220312847a902c5196dadff8d178ae Mon Sep 17 00:00:00 2001 From: Yang Guo Date: Wed, 3 Jul 2024 20:34:58 +0800 Subject: [PATCH] ext4: delete redundant uptodate check for buffer mainline inclusion from mainline-v5.13-rc1 commit 3cd461712cb450ea6aa8a8200f35f24eb296bfb2 category: performance 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 f26d9b88d48d..690f7866dc0c 100644 --- a/fs/ext4/inode.c +++ b/fs/ext4/inode.c @@ -1061,8 +1061,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; } @@ -1087,8 +1086,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