Commit 6aaaef5b authored by Kefeng Wang's avatar Kefeng Wang Committed by Andrew Morton

mm: migrate_device: use a newfolio in __migrate_device_pages()

Use a newfolio instead of newpage and convert to more folio api in
__migrate_device_pages().

Link: https://lkml.kernel.org/r/20240524052843.182275-3-wangkefeng.wang@huawei.comSigned-off-by: default avatarKefeng Wang <wangkefeng.wang@huawei.com>
Reviewed-by: default avatarMatthew Wilcox (Oracle) <willy@infradead.org>
Reviewed-by: default avatarVishal Moola (Oracle) <vishal.moola@gmail.com>
Reviewed-by: default avatarMiaohe Lin <linmiaohe@huawei.com>
Cc: Alistair Popple <apopple@nvidia.com>
Cc: Benjamin LaHaise <bcrl@kvack.org>
Cc: David Hildenbrand <david@redhat.com>
Cc: Hugh Dickins <hughd@google.com>
Cc: Jérôme Glisse <jglisse@redhat.com>
Cc: Jiaqi Yan <jiaqiyan@google.com>
Cc: Muchun Song <muchun.song@linux.dev>
Cc: Naoya Horiguchi <nao.horiguchi@gmail.com>
Cc: Tony Luck <tony.luck@intel.com>
Cc: Zi Yan <ziy@nvidia.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent 01878f10
...@@ -692,7 +692,7 @@ static void __migrate_device_pages(unsigned long *src_pfns, ...@@ -692,7 +692,7 @@ static void __migrate_device_pages(unsigned long *src_pfns,
struct page *newpage = migrate_pfn_to_page(dst_pfns[i]); struct page *newpage = migrate_pfn_to_page(dst_pfns[i]);
struct page *page = migrate_pfn_to_page(src_pfns[i]); struct page *page = migrate_pfn_to_page(src_pfns[i]);
struct address_space *mapping; struct address_space *mapping;
struct folio *folio; struct folio *newfolio, *folio;
int r; int r;
if (!newpage) { if (!newpage) {
...@@ -727,11 +727,12 @@ static void __migrate_device_pages(unsigned long *src_pfns, ...@@ -727,11 +727,12 @@ static void __migrate_device_pages(unsigned long *src_pfns,
continue; continue;
} }
newfolio = page_folio(newpage);
folio = page_folio(page); folio = page_folio(page);
mapping = folio_mapping(folio); mapping = folio_mapping(folio);
if (is_device_private_page(newpage) || if (folio_is_device_private(newfolio) ||
is_device_coherent_page(newpage)) { folio_is_device_coherent(newfolio)) {
if (mapping) { if (mapping) {
/* /*
* For now only support anonymous memory migrating to * For now only support anonymous memory migrating to
...@@ -745,7 +746,7 @@ static void __migrate_device_pages(unsigned long *src_pfns, ...@@ -745,7 +746,7 @@ static void __migrate_device_pages(unsigned long *src_pfns,
continue; continue;
} }
} }
} else if (is_zone_device_page(newpage)) { } else if (folio_is_zone_device(newfolio)) {
/* /*
* Other types of ZONE_DEVICE page are not supported. * Other types of ZONE_DEVICE page are not supported.
*/ */
...@@ -754,11 +755,11 @@ static void __migrate_device_pages(unsigned long *src_pfns, ...@@ -754,11 +755,11 @@ static void __migrate_device_pages(unsigned long *src_pfns,
} }
if (migrate && migrate->fault_page == page) if (migrate && migrate->fault_page == page)
r = migrate_folio_extra(mapping, page_folio(newpage), r = migrate_folio_extra(mapping, newfolio, folio,
folio, MIGRATE_SYNC_NO_COPY, 1); MIGRATE_SYNC_NO_COPY, 1);
else else
r = migrate_folio(mapping, page_folio(newpage), r = migrate_folio(mapping, newfolio, folio,
folio, MIGRATE_SYNC_NO_COPY); MIGRATE_SYNC_NO_COPY);
if (r != MIGRATEPAGE_SUCCESS) if (r != MIGRATEPAGE_SUCCESS)
src_pfns[i] &= ~MIGRATE_PFN_MIGRATE; src_pfns[i] &= ~MIGRATE_PFN_MIGRATE;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment