mbox series

[v2,0/5] mm: convert to folio_isolate_movable()

Message ID 20240829145456.2591719-1-wangkefeng.wang@huawei.com (mailing list archive)
Headers show
Series mm: convert to folio_isolate_movable() | expand

Message

Kefeng Wang Aug. 29, 2024, 2:54 p.m. UTC
Convert to the new folio_isolate_movable() to get rid of
isolate_movable_page().

Based on next-20240829.

v2:
- get page reference before isolate_movable_page, per Baolin
- since isolate_movable_page() callers with an elevated refcount
  of folio, directly use folio_get() and check folio reference in
  folio_isolate_movable(), per Vishal

Kefeng Wang (5):
  mm: compaction: get reference before non LRU movable folio isolation
  mm: migrate: add folio_isolate_movable()
  mm: migrate: convert to folio_isolate_movable()
  mm: compaction: convert to folio_isolate_movable()
  mm: migrate: remove isolate_movable_page()

 include/linux/migrate.h |  5 +++--
 mm/compaction.c         | 38 ++++++++++++++++----------------
 mm/migrate.c            | 49 +++++++++++++++++++----------------------
 3 files changed, 45 insertions(+), 47 deletions(-)

Comments

Vishal Moola (Oracle) Aug. 29, 2024, 8:19 p.m. UTC | #1
On Thu, Aug 29, 2024 at 10:54:51PM +0800, Kefeng Wang wrote:
> Convert to the new folio_isolate_movable() to get rid of
> isolate_movable_page().
> 
> Based on next-20240829.

For the whole series:

Reviewed-by: Vishal Moola (Oracle) <vishal.moola@gmail.com>