Re: [PATCH 05/19] iommu/pages: Replace iommu_free_pages() with iommu_free_page()

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

 



On Wed, Feb 05, 2025 at 03:55:58PM +0000, Robin Murphy wrote:
> On 2025-02-04 6:34 pm, Jason Gunthorpe wrote:
> > Now that we have a folio under the allocation iommu_free_page() can know
> > the order of the original allocation and do the correct thing to free it.
> > 
> > Callers no longer need to pass in the allocation order to the free
> > function. Just use iommu_free_page() everywhere.
> 
> Nit: we end up left with the _alloc_pages/_free_page combo which to me looks
> slightly odd and uncomfortable - any chance we could go the other way and
> align on _free_pages for consistency?

Yes, I added an additional patch to change the remaining free_page to
pages

Jason




[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux