Hi Jaegeuk, On 7/9/25 01:02, Matthew Wilcox (Oracle) wrote: > Some more folio conversions for f2fs. Again, I have checked these patches > build, but otherwise they are untested. There are three inline functions > in fscrypt that I change to take a const struct folio pointer instead > of a mutable pointer that I don't think should cause any conflicts. > > Matthew Wilcox (Oracle) (60): > f2fs: Pass a folio to recover_dentry() > f2fs: Pass a folio to recover_inode() > f2fs: Pass a folio to recover_quota_data() > f2fs: Pass a folio to f2fs_recover_inode_page() > f2fs: Pass a folio to sanity_check_extent_cache() > f2fs: Pass a folio to sanity_check_inode() > f2fs: Pass a folio to f2fs_sanity_check_inline_data() > f2fs: Pass a folio to inode_has_blocks() > f2fs: Pass a folio to F2FS_INODE() > f2fs: Pass a folio to ino_of_node() > f2fs: Pass a folio to nid_of_node() > f2fs: Pass a folio to is_recoverable_dnode() > f2fs: Pass a folio to set_dentry_mark() > f2fs: Pass a folio to set_fsync_mark() > f2fs: Pass a folio to set_mark() > f2fs: Pass a folio to f2fs_allocate_data_block() > f2fs: Pass a folio to f2fs_inode_chksum_set() > f2fs: Pass a folio to f2fs_enable_inode_chksum() > f2fs: Pass a folio to f2fs_inode_chksum() > f2fs: Pass a folio to fill_node_footer_blkaddr() > f2fs: Pass a folio to get_nid() > f2fs: Pass a folio to set_cold_node() > f2fs: Pass folios to copy_node_footer() > f2fs: Pass a folio to fill_node_footer() > f2fs: Pass a folio to cpver_of_node() > f2fs: Pass a folio to f2fs_recover_xattr_data() > f2fs: Pass a folio to is_fsync_dnode() > f2fs: Pass a folio to is_dent_dnode() > f2fs: Add fio->folio > f2fs: Use folio_unlock() in f2fs_write_compressed_pages() > f2fs: Pass a folio to is_cold_node() > f2fs: Pass a folio to is_node() > f2fs: Pass a folio to IS_DNODE() > f2fs: Pass a folio to ofs_of_node() > f2fs: Pass a folio to get_dnode_base() > f2fs: Pass a folio to ADDRS_PER_PAGE() > f2fs: Pass a folio to IS_INODE()
All above patches in dev-test branch look good to me, please feel free to add my reviewed-by tag. I've checked below patches as well, but still have one question to confirm. Thanks, > f2fs: Add folio counterparts to page_private_flags functions > f2fs: Use a folio in f2fs_is_cp_guaranteed() > f2fs: Convert set_page_private_data() to folio_set_f2fs_data() > f2fs: Convert get_page_private_data() to folio_get_f2fs_data() > f2fs: Pass a folio to f2fs_compress_write_end_io() > f2fs: Use a folio in f2fs_merge_page_bio() > f2fs: Use a bio in f2fs_submit_page_write() > f2fs: Pass a folio to WB_DATA_TYPE() and f2fs_is_cp_guaranteed() > f2fs: Use a folio iterator in f2fs_handle_step_decompress() > f2fs: Pass a folio to f2fs_end_read_compressed_page() > f2fs: Use a folio iterator in f2fs_verify_bio() > f2fs: Pass a folio to f2fs_is_compressed_page() > f2fs: Convert get_next_nat_page() to get_next_nat_folio() > f2fs: Pass the nat_blk to __update_nat_bits() > f2fs: Pass a folio to F2FS_NODE() > f2fs: Pass a folio to f2fs_cache_compressed_page() > f2fs: Use a folio in f2fs_encrypted_get_link() > f2fs: Use F2FS_F_SB() in f2fs_read_end_io() > f2fs: Remove clear_page_private_all() > f2fs: Remove use of page from f2fs_write_single_data_page() > f2fs: Pass a folio to f2fs_submit_merged_write_cond() > f2fs: Pass a folio to __has_merged_page() > f2fs: Remove F2FS_P_SB() > > fs/f2fs/checkpoint.c | 8 +- > fs/f2fs/compress.c | 44 +++++------ > fs/f2fs/data.c | 138 +++++++++++++++++----------------- > fs/f2fs/dir.c | 4 +- > fs/f2fs/extent_cache.c | 8 +- > fs/f2fs/f2fs.h | 135 +++++++++++++++++---------------- > fs/f2fs/file.c | 28 +++---- > fs/f2fs/gc.c | 18 ++--- > fs/f2fs/inline.c | 20 ++--- > fs/f2fs/inode.c | 61 +++++++-------- > fs/f2fs/namei.c | 12 +-- > fs/f2fs/node.c | 162 ++++++++++++++++++++-------------------- > fs/f2fs/node.h | 75 ++++++++++--------- > fs/f2fs/recovery.c | 78 +++++++++---------- > fs/f2fs/segment.c | 27 ++++--- > include/linux/f2fs_fs.h | 2 +- > include/linux/fscrypt.h | 7 +- > 17 files changed, 414 insertions(+), 413 deletions(-) > _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel