2024-01-18 05:50:29

by Zhiguo Niu

[permalink] [raw]
Subject: [PATCH] f2fs: compress: remove some redundant codes in f2fs_cache_compressed_page

Just remove some redundant codes, no logic change.

Signed-off-by: Zhiguo Niu <[email protected]>
---
fs/f2fs/compress.c | 4 ----
1 file changed, 4 deletions(-)

diff --git a/fs/f2fs/compress.c b/fs/f2fs/compress.c
index ff26b49..624212d 100644
--- a/fs/f2fs/compress.c
+++ b/fs/f2fs/compress.c
@@ -1889,12 +1889,8 @@ void f2fs_cache_compressed_page(struct f2fs_sb_info *sbi, struct page *page,

set_page_private_data(cpage, ino);

- if (!f2fs_is_valid_blkaddr(sbi, blkaddr, DATA_GENERIC_ENHANCE_READ))
- goto out;
-
memcpy(page_address(cpage), page_address(page), PAGE_SIZE);
SetPageUptodate(cpage);
-out:
f2fs_put_page(cpage, 1);
}

--
1.9.1



2024-01-22 02:29:57

by Chao Yu

[permalink] [raw]
Subject: Re: [PATCH] f2fs: compress: remove some redundant codes in f2fs_cache_compressed_page

On 2024/1/18 13:48, Zhiguo Niu wrote:
> Just remove some redundant codes, no logic change.
>
> Signed-off-by: Zhiguo Niu <[email protected]>

Reviewed-by: Chao Yu <[email protected]>

Thanks,

2024-01-30 20:56:16

by patchwork-bot+f2fs

[permalink] [raw]
Subject: Re: [f2fs-dev] [PATCH] f2fs: compress: remove some redundant codes in f2fs_cache_compressed_page

Hello:

This patch was applied to jaegeuk/f2fs.git (dev)
by Jaegeuk Kim <[email protected]>:

On Thu, 18 Jan 2024 13:48:31 +0800 you wrote:
> Just remove some redundant codes, no logic change.
>
> Signed-off-by: Zhiguo Niu <[email protected]>
> ---
> fs/f2fs/compress.c | 4 ----
> 1 file changed, 4 deletions(-)

Here is the summary with links:
- [f2fs-dev] f2fs: compress: remove some redundant codes in f2fs_cache_compressed_page
https://git.kernel.org/jaegeuk/f2fs/c/16c326c7a519

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html