[PATCH 0/5] Removing the lru_cache_add() wrapper

Vishal Moola (Oracle) posted 5 patches 3 years, 5 months ago
fs/fuse/dev.c           | 55 +++++++++++++++++++++--------------------
include/linux/pagemap.h |  2 +-
include/linux/swap.h    |  1 -
mm/filemap.c            | 52 +++++++++++++++++++-------------------
mm/folio-compat.c       |  6 -----
mm/khugepaged.c         | 11 ++++++---
mm/truncate.c           |  2 +-
mm/userfaultfd.c        |  6 +++--
mm/workingset.c         |  2 +-
9 files changed, 67 insertions(+), 70 deletions(-)
[PATCH 0/5] Removing the lru_cache_add() wrapper
Posted by Vishal Moola (Oracle) 3 years, 5 months ago
This patchset replaces all calls of lru_cache_add() with the folio
equivalent: folio_add_lru(). This is allows us to get rid of the wrapper 
The series passes xfstests and the userfaultfd selftests.

I haven't been able to hit the fuse_try_move_page() function, so I
haven't been able to test those changes. Since replace_page_cache_page()
is only called by fuse_try_move_page() I haven't been able to test that
as well. Testing and review of both would be beneficial.

Vishal Moola (Oracle) (5):
  filemap: Convert replace_page_cache_page() to
    replace_page_cache_folio()
  fuse: Convert fuse_try_move_page() to use folios
  userfualtfd: Replace lru_cache functions with folio_add functions
  khugepage: Replace lru_cache_add() with folio_add_lru()
  folio-compat: Remove lru_cache_add()

 fs/fuse/dev.c           | 55 +++++++++++++++++++++--------------------
 include/linux/pagemap.h |  2 +-
 include/linux/swap.h    |  1 -
 mm/filemap.c            | 52 +++++++++++++++++++-------------------
 mm/folio-compat.c       |  6 -----
 mm/khugepaged.c         | 11 ++++++---
 mm/truncate.c           |  2 +-
 mm/userfaultfd.c        |  6 +++--
 mm/workingset.c         |  2 +-
 9 files changed, 67 insertions(+), 70 deletions(-)

-- 
2.38.1
Re: [PATCH 0/5] Removing the lru_cache_add() wrapper
Posted by Vishal Moola 3 years, 4 months ago
On Tue, Nov 1, 2022 at 10:53 AM Vishal Moola (Oracle)
<vishal.moola@gmail.com> wrote:
>
> This patchset replaces all calls of lru_cache_add() with the folio
> equivalent: folio_add_lru(). This is allows us to get rid of the wrapper
> The series passes xfstests and the userfaultfd selftests.

All of these patches have been reviewed. Andrew, is there anything
you'd like to see changed, or will you take these?