Re: [PATCH v1 02/11] mm: migrate_device: use more folio in __migrate_device_pages()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 2024/4/2 2:22, Vishal Moola wrote:
On Thu, Mar 21, 2024 at 11:27:38AM +0800, Kefeng Wang wrote:
if (!newpage) {
@@ -728,14 +729,13 @@ static void __migrate_device_pages(unsigned long *src_pfns,
  			continue;
  		}
- mapping = page_mapping(page);
+		newfolio = page_folio(newpage);

You could save another compound_head() call by passing the folio through
to migrate_vma_insert_page() and make it migrate_vma_insert_folio(),
since its already converted to use folios.

Sure, but let's do it later, we could convert more functions in migrate_device.c to use folios, thanks for your review, do you
mind to help to review other patches, hope that the poison recover
from migrate folio was merged firstly.


+		folio = page_folio(page);
+		mapping = folio_mapping(folio);




[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [NTFS 3]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [NTFS 3]     [Samba]     [Device Mapper]     [CEPH Development]

  Powered by Linux