]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
mm/migrate: rename PAGE_ migration flags to FOLIO_
authorShivank Garg <shivankg@amd.com>
Tue, 24 Mar 2026 19:07:09 +0000 (19:07 +0000)
committerAndrew Morton <akpm@linux-foundation.org>
Fri, 29 May 2026 04:05:00 +0000 (21:05 -0700)
These flags only track folio-specific state during migration and are not
used for movable_ops pages.  Rename the enum values and the old_page_state
variable to match.

No functional change.

Link: https://lore.kernel.org/20260324190706.964555-4-shivankg@amd.com
Signed-off-by: Shivank Garg <shivankg@amd.com>
Suggested-by: David Hildenbrand <david@kernel.org>
Acked-by: David Hildenbrand (Arm) <david@kernel.org>
Reviewed-by: Zi Yan <ziy@nvidia.com>
Reviewed-by: Baolin Wang <baolin.wang@linux.alibaba.com>
Reviewed-by: Lance Yang <lance.yang@linux.dev>
Reviewed-by: Huang Ying <ying.huang@linux.alibaba.com>
Cc: Alistair Popple <apopple@nvidia.com>
Cc: Byungchul Park <byungchul@sk.com>
Cc: Gregory Price <gourry@gourry.net>
Cc: Joshua Hahn <joshua.hahnjy@gmail.com>
Cc: Matthew Brost <matthew.brost@intel.com>
Cc: Matthew Wilcox (Oracle) <willy@infradead.org>
Cc: Rakie Kim <rakie.kim@sk.com>
Cc: Shivank Garg <shivankg@amd.com>
Cc: Zi Yan <ziy@nvidia.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
mm/migrate.c

index 8a64291ab5b44c401e1e0356bf39588e7b5d7b0d..0c6a0ab6eccef26cbd5d681b81d32355a7256cdc 100644 (file)
@@ -1135,26 +1135,24 @@ static int move_to_new_folio(struct folio *dst, struct folio *src,
  * This is safe because nobody is using it except us.
  */
 enum {
-       PAGE_WAS_MAPPED = BIT(0),
-       PAGE_WAS_MLOCKED = BIT(1),
-       PAGE_OLD_STATES = PAGE_WAS_MAPPED | PAGE_WAS_MLOCKED,
+       FOLIO_WAS_MAPPED = BIT(0),
+       FOLIO_WAS_MLOCKED = BIT(1),
+       FOLIO_OLD_STATES = FOLIO_WAS_MAPPED | FOLIO_WAS_MLOCKED,
 };
 
 static void __migrate_folio_record(struct folio *dst,
-                                  int old_page_state,
-                                  struct anon_vma *anon_vma)
+               int old_folio_state, struct anon_vma *anon_vma)
 {
-       dst->private = (void *)anon_vma + old_page_state;
+       dst->private = (void *)anon_vma + old_folio_state;
 }
 
 static void __migrate_folio_extract(struct folio *dst,
-                                  int *old_page_state,
-                                  struct anon_vma **anon_vmap)
+               int *old_folio_state, struct anon_vma **anon_vmap)
 {
        unsigned long private = (unsigned long)dst->private;
 
-       *anon_vmap = (struct anon_vma *)(private & ~PAGE_OLD_STATES);
-       *old_page_state = private & PAGE_OLD_STATES;
+       *anon_vmap = (struct anon_vma *)(private & ~FOLIO_OLD_STATES);
+       *old_folio_state = private & FOLIO_OLD_STATES;
        dst->private = NULL;
 }
 
@@ -1209,7 +1207,7 @@ static int migrate_folio_unmap(new_folio_t get_new_folio,
 {
        struct folio *dst;
        int rc = -EAGAIN;
-       int old_page_state = 0;
+       int old_folio_state = 0;
        struct anon_vma *anon_vma = NULL;
        bool locked = false;
        bool dst_locked = false;
@@ -1253,7 +1251,7 @@ static int migrate_folio_unmap(new_folio_t get_new_folio,
        }
        locked = true;
        if (folio_test_mlocked(src))
-               old_page_state |= PAGE_WAS_MLOCKED;
+               old_folio_state |= FOLIO_WAS_MLOCKED;
 
        if (folio_test_writeback(src)) {
                /*
@@ -1302,7 +1300,7 @@ static int migrate_folio_unmap(new_folio_t get_new_folio,
        dst_locked = true;
 
        if (unlikely(page_has_movable_ops(&src->page))) {
-               __migrate_folio_record(dst, old_page_state, anon_vma);
+               __migrate_folio_record(dst, old_folio_state, anon_vma);
                return 0;
        }
 
@@ -1328,11 +1326,11 @@ static int migrate_folio_unmap(new_folio_t get_new_folio,
                VM_BUG_ON_FOLIO(folio_test_anon(src) &&
                               !folio_test_ksm(src) && !anon_vma, src);
                try_to_migrate(src, mode == MIGRATE_ASYNC ? TTU_BATCH_FLUSH : 0);
-               old_page_state |= PAGE_WAS_MAPPED;
+               old_folio_state |= FOLIO_WAS_MAPPED;
        }
 
        if (!folio_mapped(src)) {
-               __migrate_folio_record(dst, old_page_state, anon_vma);
+               __migrate_folio_record(dst, old_folio_state, anon_vma);
                return 0;
        }
 
@@ -1344,7 +1342,7 @@ out:
        if (rc == -EAGAIN)
                ret = NULL;
 
-       migrate_folio_undo_src(src, old_page_state & PAGE_WAS_MAPPED,
+       migrate_folio_undo_src(src, old_folio_state & FOLIO_WAS_MAPPED,
                               anon_vma, locked, ret);
        migrate_folio_undo_dst(dst, dst_locked, put_new_folio, private);
 
@@ -1358,13 +1356,13 @@ static int migrate_folio_move(free_folio_t put_new_folio, unsigned long private,
                              struct list_head *ret)
 {
        int rc;
-       int old_page_state = 0;
+       int old_folio_state = 0;
        struct anon_vma *anon_vma = NULL;
        bool src_deferred_split = false;
        bool src_partially_mapped = false;
        struct list_head *prev;
 
-       __migrate_folio_extract(dst, &old_page_state, &anon_vma);
+       __migrate_folio_extract(dst, &old_folio_state, &anon_vma);
        prev = dst->lru.prev;
        list_del(&dst->lru);
 
@@ -1404,10 +1402,10 @@ static int migrate_folio_move(free_folio_t put_new_folio, unsigned long private,
         * isolated from the unevictable LRU: but this case is the easiest.
         */
        folio_add_lru(dst);
-       if (old_page_state & PAGE_WAS_MLOCKED)
+       if (old_folio_state & FOLIO_WAS_MLOCKED)
                lru_add_drain();
 
-       if (old_page_state & PAGE_WAS_MAPPED)
+       if (old_folio_state & FOLIO_WAS_MAPPED)
                remove_migration_ptes(src, dst, 0);
 
 out_unlock_both:
@@ -1439,11 +1437,11 @@ out:
         */
        if (rc == -EAGAIN) {
                list_add(&dst->lru, prev);
-               __migrate_folio_record(dst, old_page_state, anon_vma);
+               __migrate_folio_record(dst, old_folio_state, anon_vma);
                return rc;
        }
 
-       migrate_folio_undo_src(src, old_page_state & PAGE_WAS_MAPPED,
+       migrate_folio_undo_src(src, old_folio_state & FOLIO_WAS_MAPPED,
                               anon_vma, true, ret);
        migrate_folio_undo_dst(dst, true, put_new_folio, private);
 
@@ -1777,11 +1775,11 @@ static void migrate_folios_undo(struct list_head *src_folios,
        dst = list_first_entry(dst_folios, struct folio, lru);
        dst2 = list_next_entry(dst, lru);
        list_for_each_entry_safe(folio, folio2, src_folios, lru) {
-               int old_page_state = 0;
+               int old_folio_state = 0;
                struct anon_vma *anon_vma = NULL;
 
-               __migrate_folio_extract(dst, &old_page_state, &anon_vma);
-               migrate_folio_undo_src(folio, old_page_state & PAGE_WAS_MAPPED,
+               __migrate_folio_extract(dst, &old_folio_state, &anon_vma);
+               migrate_folio_undo_src(folio, old_folio_state & FOLIO_WAS_MAPPED,
                                anon_vma, true, ret_folios);
                list_del(&dst->lru);
                migrate_folio_undo_dst(dst, true, put_new_folio, private);