Re: [PATCH v4 0/3] mm: swap: mTHP swap allocator base on swap cluster order
From: Chris Li
Date: Fri Jul 26 2024 - 01:52:18 EST
On Wed, Jul 17, 2024 at 10:54 PM Huang, Ying <ying.huang@xxxxxxxxx> wrote:
> > - HDD swap allocation does not need to consider clusters any more.
>
> It appears that my comments in the following emails are ignored?
>
Sorry for missing some email catching up.
> https://lore.kernel.org/linux-mm/87bk3pzr5p.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/
Will reply to that. BTW, V4 already reverted to the previous SSD
behavior and allocated a new cluster before nonfull cluster.
> https://lore.kernel.org/linux-mm/874j9hzqr3.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/
Already replied to the renaming in another email.
Chris
>
> > changes in v3:
> > - Using V1 as base.
> > - Rename "next" to "list" for the list field, suggested by Ying.
> > - Update comment for the locking rules for cluster fields and list,
> > suggested by Ying.
> > - Allocate from the nonfull list before attempting free list, suggested
> > by Kairui.
> > - Link to v2: https://lore.kernel.org/r/20240614-swap-allocator-v2-0-2a513b4a7f2f@xxxxxxxxxx
> >
> > Changes in v2:
> > - Abandoned.
> > - Link to v1: https://lore.kernel.org/r/20240524-swap-allocator-v1-0-47861b423b26@xxxxxxxxxx
> >
> > ---
> > Chris Li (3):
> > mm: swap: swap cluster switch to double link list
> > mm: swap: mTHP allocate swap entries from nonfull list
> > RFC: mm: swap: seperate SSD allocation from scan_swap_map_slots()
> >
> > include/linux/swap.h | 30 ++--
> > mm/swapfile.c | 490 +++++++++++++++++++++++----------------------------
> > 2 files changed, 238 insertions(+), 282 deletions(-)
> > ---
> > base-commit: ff3a648ecb9409aff1448cf4f6aa41d78c69a3bc
> > change-id: 20240523-swap-allocator-1534c480ece4
> >
>
> --
> Best Regards,
> Huang, Ying