On 18.08.25 10:27, Pratyush Brahma wrote:
On Mon, Aug 18, 2025 at 12:29 PM David Hildenbrand <da...@redhat.com> wrote:
On 18.08.25 08:41, pratyush.bra...@oss.qualcomm.com wrote:
From: Pratyush Brahma <pratyush.bra...@oss.qualcomm.com>
The function `memory_add_physaddr_to_nid` seems a misnomer.
It does not to "add" a physical address to a NID mapping,
but rather it gets the NID associated with a given physical address.
You probably misunderstood what the function is used for: memory hotplug
aka "memory_add".
Thanks for your feedback. I get the part about memory hotplug here but
using memory_add still seems a little odd as it doesn't truly reflect
what this api is doing.
However, I agree that my current suggestion
may not be the perfect choice for the name, so I'm open to suggestions.
Perhaps, something like "memory_add_get_nid_by_phys" may work here?
I don't think this name is really any better and worth the churn :(
--
Cheers
David / dhildenb
