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