mbox series

[f2fs-dev,00/11] f2fs folio patches

Message ID 20241128045828.527559-1-willy@infradead.org (mailing list archive)
Headers show
Series f2fs folio patches | expand

Message

Matthew Wilcox Nov. 28, 2024, 4:58 a.m. UTC
This round of f2fs patches accomplishes two things:

 - Removal of all references to page->index in f2fs.  I'm looking to
   finish this conversion in the January merge window.
 - Removal of all calls to page_file_mapping() and folio_file_mapping()

I have only compile-tested these patches.

Matthew Wilcox (Oracle) (11):
  f2fs: Use a folio in f2fs_all_cluster_page_ready()
  f2fs: Use a folio in f2fs_compress_write_end()
  f2fs: Use a folio in f2fs_truncate_partial_cluster()
  f2fs: Use a folio in f2fs_write_compressed_pages()
  f2fs: Convert submit tracepoints to take a folio
  f2fs: Add F2FS_F_SB()
  f2fs: Convert f2fs_finish_read_bio() to use folios
  f2fs: Use a folio more in f2fs_submit_page_bio()
  f2fs: Use a data folio in f2fs_submit_page_bio()
  f2fs: Convert __read_io_type() to take a folio
  f2fs: Remove calls to folio_file_mapping()

 fs/f2fs/compress.c          | 28 ++++++++++++--------
 fs/f2fs/data.c              | 53 +++++++++++++++----------------------
 fs/f2fs/f2fs.h              |  7 ++++-
 fs/f2fs/inline.c            |  2 +-
 include/trace/events/f2fs.h | 39 +++++++++++++--------------
 5 files changed, 65 insertions(+), 64 deletions(-)

Comments

Matthew Wilcox Dec. 16, 2024, 1:59 p.m. UTC | #1
On Thu, Nov 28, 2024 at 04:58:15AM +0000, Matthew Wilcox (Oracle) wrote:
> This round of f2fs patches accomplishes two things:

Ping.  It's been two weeks and these patches aren't in linux-next yet.

>  - Removal of all references to page->index in f2fs.  I'm looking to
>    finish this conversion in the January merge window.
>  - Removal of all calls to page_file_mapping() and folio_file_mapping()
> 
> I have only compile-tested these patches.
> 
> Matthew Wilcox (Oracle) (11):
>   f2fs: Use a folio in f2fs_all_cluster_page_ready()
>   f2fs: Use a folio in f2fs_compress_write_end()
>   f2fs: Use a folio in f2fs_truncate_partial_cluster()
>   f2fs: Use a folio in f2fs_write_compressed_pages()
>   f2fs: Convert submit tracepoints to take a folio
>   f2fs: Add F2FS_F_SB()
>   f2fs: Convert f2fs_finish_read_bio() to use folios
>   f2fs: Use a folio more in f2fs_submit_page_bio()
>   f2fs: Use a data folio in f2fs_submit_page_bio()
>   f2fs: Convert __read_io_type() to take a folio
>   f2fs: Remove calls to folio_file_mapping()
> 
>  fs/f2fs/compress.c          | 28 ++++++++++++--------
>  fs/f2fs/data.c              | 53 +++++++++++++++----------------------
>  fs/f2fs/f2fs.h              |  7 ++++-
>  fs/f2fs/inline.c            |  2 +-
>  include/trace/events/f2fs.h | 39 +++++++++++++--------------
>  5 files changed, 65 insertions(+), 64 deletions(-)
> 
> -- 
> 2.45.2
>
Chao Yu Dec. 16, 2024, 2:39 p.m. UTC | #2
On 2024/11/28 12:58, Matthew Wilcox (Oracle) wrote:
> This round of f2fs patches accomplishes two things:
> 
>   - Removal of all references to page->index in f2fs.  I'm looking to
>     finish this conversion in the January merge window.
>   - Removal of all calls to page_file_mapping() and folio_file_mapping()
> 
> I have only compile-tested these patches.
> 
> Matthew Wilcox (Oracle) (11):
>    f2fs: Use a folio in f2fs_all_cluster_page_ready()
>    f2fs: Use a folio in f2fs_compress_write_end()
>    f2fs: Use a folio in f2fs_truncate_partial_cluster()
>    f2fs: Use a folio in f2fs_write_compressed_pages()
>    f2fs: Convert submit tracepoints to take a folio
>    f2fs: Add F2FS_F_SB()
>    f2fs: Convert f2fs_finish_read_bio() to use folios
>    f2fs: Use a folio more in f2fs_submit_page_bio()
>    f2fs: Use a data folio in f2fs_submit_page_bio()
>    f2fs: Convert __read_io_type() to take a folio
>    f2fs: Remove calls to folio_file_mapping()

For all patches in this serial, feel free to add:

Reviewed-by: Chao Yu <chao@kernel.org>

Thanks,

> 
>   fs/f2fs/compress.c          | 28 ++++++++++++--------
>   fs/f2fs/data.c              | 53 +++++++++++++++----------------------
>   fs/f2fs/f2fs.h              |  7 ++++-
>   fs/f2fs/inline.c            |  2 +-
>   include/trace/events/f2fs.h | 39 +++++++++++++--------------
>   5 files changed, 65 insertions(+), 64 deletions(-)
>
Jaegeuk Kim Dec. 16, 2024, 4:11 p.m. UTC | #3
On 12/16, Matthew Wilcox wrote:
> On Thu, Nov 28, 2024 at 04:58:15AM +0000, Matthew Wilcox (Oracle) wrote:
> > This round of f2fs patches accomplishes two things:
> 
> Ping.  It's been two weeks and these patches aren't in linux-next yet.

I was testing the series in dev-test.

https://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git/log/?h=dev-test

> 
> >  - Removal of all references to page->index in f2fs.  I'm looking to
> >    finish this conversion in the January merge window.
> >  - Removal of all calls to page_file_mapping() and folio_file_mapping()
> > 
> > I have only compile-tested these patches.
> > 
> > Matthew Wilcox (Oracle) (11):
> >   f2fs: Use a folio in f2fs_all_cluster_page_ready()
> >   f2fs: Use a folio in f2fs_compress_write_end()
> >   f2fs: Use a folio in f2fs_truncate_partial_cluster()
> >   f2fs: Use a folio in f2fs_write_compressed_pages()
> >   f2fs: Convert submit tracepoints to take a folio
> >   f2fs: Add F2FS_F_SB()
> >   f2fs: Convert f2fs_finish_read_bio() to use folios
> >   f2fs: Use a folio more in f2fs_submit_page_bio()
> >   f2fs: Use a data folio in f2fs_submit_page_bio()
> >   f2fs: Convert __read_io_type() to take a folio
> >   f2fs: Remove calls to folio_file_mapping()
> > 
> >  fs/f2fs/compress.c          | 28 ++++++++++++--------
> >  fs/f2fs/data.c              | 53 +++++++++++++++----------------------
> >  fs/f2fs/f2fs.h              |  7 ++++-
> >  fs/f2fs/inline.c            |  2 +-
> >  include/trace/events/f2fs.h | 39 +++++++++++++--------------
> >  5 files changed, 65 insertions(+), 64 deletions(-)
> > 
> > -- 
> > 2.45.2
> >
patchwork-bot+f2fs--- via Linux-f2fs-devel Dec. 16, 2024, 4:40 p.m. UTC | #4
Hello:

This series was applied to jaegeuk/f2fs.git (dev)
by Jaegeuk Kim <jaegeuk@kernel.org>:

On Thu, 28 Nov 2024 04:58:15 +0000 you wrote:
> This round of f2fs patches accomplishes two things:
> 
>  - Removal of all references to page->index in f2fs.  I'm looking to
>    finish this conversion in the January merge window.
>  - Removal of all calls to page_file_mapping() and folio_file_mapping()
> 
> I have only compile-tested these patches.
> 
> [...]

Here is the summary with links:
  - [f2fs-dev,01/11] f2fs: Use a folio in f2fs_all_cluster_page_ready()
    https://git.kernel.org/jaegeuk/f2fs/c/a909c1795353
  - [f2fs-dev,02/11] f2fs: Use a folio in f2fs_compress_write_end()
    https://git.kernel.org/jaegeuk/f2fs/c/ff6c82a934f7
  - [f2fs-dev,03/11] f2fs: Use a folio in f2fs_truncate_partial_cluster()
    https://git.kernel.org/jaegeuk/f2fs/c/1cda5bc0b2fe
  - [f2fs-dev,04/11] f2fs: Use a folio in f2fs_write_compressed_pages()
    https://git.kernel.org/jaegeuk/f2fs/c/ac866908d7a9
  - [f2fs-dev,05/11] f2fs: Convert submit tracepoints to take a folio
    https://git.kernel.org/jaegeuk/f2fs/c/87e2a15bc008
  - [f2fs-dev,06/11] f2fs: Add F2FS_F_SB()
    https://git.kernel.org/jaegeuk/f2fs/c/1cf746007005
  - [f2fs-dev,07/11] f2fs: Convert f2fs_finish_read_bio() to use folios
    https://git.kernel.org/jaegeuk/f2fs/c/e0821645dd2d
  - [f2fs-dev,08/11] f2fs: Use a folio more in f2fs_submit_page_bio()
    https://git.kernel.org/jaegeuk/f2fs/c/0765b3f989a7
  - [f2fs-dev,09/11] f2fs: Use a data folio in f2fs_submit_page_bio()
    https://git.kernel.org/jaegeuk/f2fs/c/f58d8645824b
  - [f2fs-dev,10/11] f2fs: Convert __read_io_type() to take a folio
    https://git.kernel.org/jaegeuk/f2fs/c/19bbd306ddfd
  - [f2fs-dev,11/11] f2fs: Remove calls to folio_file_mapping()
    https://git.kernel.org/jaegeuk/f2fs/c/c910a64bc4e2

You are awesome, thank you!