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

Reply via email to