Re: [PATCH v5 1/1] mm: report per-page metadata information

From: David Hildenbrand
Date: Thu Nov 02 2023 - 11:54:10 EST


On 02.11.23 16:50, Pasha Tatashin wrote:
Adding reserved memory to MemTotal is a cleaner approach IMO as well.
But it changes the semantics of MemTotal, which may have compatibility
issues.

I object.

Could you please elaborate what you object (and why): you object that
it will have compatibility issues, or you object to include memblock
reserves into MemTotal?

Sorry, I object to changing the semantics of MemTotal. MemTotal is traditionally the memory managed by the buddy, not all memory in the system. I know people/scripts that are relying on that [although it's been source of confusion a couple of times].

--
Cheers,

David / dhildenb