Message ID | 1617260610-29770-1-git-send-email-zhangshaokun@hisilicon.com |
---|---|
State | Awaiting Upstream |
Headers | show |
Series | ext4: Delete redundant uptodate check for buffer | expand |
On 21/04/01 03:03PM, Shaokun Zhang wrote: > From: Yang Guo <guoyang2@huawei.com> > > 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" <tytso@mit.edu> > Cc: Andreas Dilger <adilger.kernel@dilger.ca> > Signed-off-by: Yang Guo <guoyang2@huawei.com> > Signed-off-by: Shaokun Zhang <zhangshaokun@hisilicon.com> > --- > fs/ext4/inode.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) Thanks for the patch. Changes looks good and trivial. Feel free to add - Reviewed-by: Ritesh Harjani <ritesh.list@gmail.com>
On Thu, Apr 01, 2021 at 04:50:01PM +0530, Ritesh Harjani wrote: > On 21/04/01 03:03PM, Shaokun Zhang wrote: > > From: Yang Guo <guoyang2@huawei.com> > > > > 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" <tytso@mit.edu> > > Cc: Andreas Dilger <adilger.kernel@dilger.ca> > > Signed-off-by: Yang Guo <guoyang2@huawei.com> > > Signed-off-by: Shaokun Zhang <zhangshaokun@hisilicon.com> > > --- > > fs/ext4/inode.c | 6 ++---- > > 1 file changed, 2 insertions(+), 4 deletions(-) > > Thanks for the patch. Changes looks good and trivial. > > Feel free to add - > Reviewed-by: Ritesh Harjani <ritesh.list@gmail.com> Thanks, applied. - Ted
diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c index 0948a43f1b3d..32fa3ad38797 100644 --- a/fs/ext4/inode.c +++ b/fs/ext4/inode.c @@ -1066,8 +1066,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; } @@ -1092,8 +1091,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) &&