Re: [PATCH v1 13/36] mm/hugetlb: cleanup hugetlb_folio_init_tail_vmemmap()
Lorenzo Stoakes <[email protected]> Fri, 29 Aug 2025 13:02:38 +0100
| Newsgroups | com.zx2c4.lists.wireguard,dev.linux.lists.iommu,dev.linux.lists.virtualization,org.freedesktop.lists.dri-devel,org.freedesktop.lists.intel-gfx,org.infradead.lists.linux-arm-kernel,org.infradead.lists.linux-riscv,org.kernel.vger.io-uring,org.kernel.vger.kvm,org.kernel.vger.linux-crypto,org.kernel.vger.linux-ide,org.kernel.vger.linux-kernel,org.kernel.vger.linux-kselftest,org.kernel.vger.linux-mips,org.kernel.vger.linux-mmc,org.kernel.vger.linux-s390,org.kernel.vger.linux-scsi,org.kernel.vger.netdev,org.kvack.linux-mm |
|---|---|
| Message-ID | <[email protected]> |
On Fri, Aug 29, 2025 at 01:59:19PM +0200, David Hildenbrand wrote: > On 28.08.25 17:37, Lorenzo Stoakes wrote: > > On Thu, Aug 28, 2025 at 12:01:17AM +0200, David Hildenbrand wrote: > > > We can now safely iterate over all pages in a folio, so no need for the > > > pfn_to_page(). > > > > > > Also, as we already force the refcount in __init_single_page() to 1, > > > > Mega huge nit (ignore if you want), but maybe worth saying 'via > > init_page_count()'. > > Will add, thanks! Thanks! > > -- > Cheers > > David / dhildenb > >