Re: [PATCH V1 1/9] mm/gup: repin_folio_unhugely

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

 



On 20.09.24 15:28, Jason Gunthorpe wrote:
On Thu, Sep 19, 2024 at 10:11:38AM +0200, David Hildenbrand wrote:

And repin is really misleading. We are simply adding more pins to an already pinned one ...

Jason suggests a better name in the other thread.

I would prefer something that simply adds more pins to an already pinned
folio. Much easier to get.

Yes, but also nobody should ever want to do that operation, it should
always be part of some kind of "splitting" sort of behavior..

I remember patches from Dave Howells that needed that for O_DIRECT handling. Never say never ;)

Adding is much more intuitive than splitting ... just like we add references when splitting a THP, using folio_ref_add().

--
Cheers,

David / dhildenb





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux