mbox series

[0/3] Some more filemap folio conversions

Message ID 20230116193941.2148487-1-willy@infradead.org (mailing list archive)
Headers show
Series Some more filemap folio conversions | expand

Message

Matthew Wilcox Jan. 16, 2023, 7:39 p.m. UTC 
Three more places which could easily be converted to folios.  The third
one fixes a minor bug in readahead_expand(), but it's only a performance
bug and there's few users of readahead_expand(), so I don't think it's
worth backporting.

Matthew Wilcox (Oracle) (3):
  filemap: Convert filemap_map_pmd() to take a folio
  filemap: Convert filemap_range_has_page() to use a folio
  readahead: Convert readahead_expand() to use a folio

 mm/filemap.c   | 28 +++++++++++++++-------------
 mm/readahead.c | 39 ++++++++++++++++++++++-----------------
 2 files changed, 37 insertions(+), 30 deletions(-)

Comments

William Kucharski Jan. 16, 2023, 11 p.m. UTC | #1 
These all look good to me.

Reviewed-by: William Kucharski <william.kucharski@oracle.com>


> On Jan 16, 2023, at 12:39 PM, Matthew Wilcox (Oracle) <willy@infradead.org> wrote:
> 
> Three more places which could easily be converted to folios.  The third
> one fixes a minor bug in readahead_expand(), but it's only a performance
> bug and there's few users of readahead_expand(), so I don't think it's
> worth backporting.
> 
> Matthew Wilcox (Oracle) (3):
>  filemap: Convert filemap_map_pmd() to take a folio
>  filemap: Convert filemap_range_has_page() to use a folio
>  readahead: Convert readahead_expand() to use a folio
> 
> mm/filemap.c   | 28 +++++++++++++++-------------
> mm/readahead.c | 39 ++++++++++++++++++++++-----------------
> 2 files changed, 37 insertions(+), 30 deletions(-)
> 
> -- 
> 2.35.1
> 
>