[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20181123131237.GO8625@dhcp22.suse.cz>
Date: Fri, 23 Nov 2018 14:12:37 +0100
From: Michal Hocko <mhocko@...nel.org>
To: David Hildenbrand <david@...hat.com>
Cc: Oscar Salvador <osalvador@...e.com>, linux-mm@...ck.org,
rppt@...ux.vnet.ibm.com, akpm@...ux-foundation.org,
arunks@...eaurora.org, bhe@...hat.com, dan.j.williams@...el.com,
Pavel.Tatashin@...rosoft.com, Jonathan.Cameron@...wei.com,
jglisse@...hat.com, linux-kernel@...r.kernel.org
Subject: Re: [RFC PATCH 0/4] mm, memory_hotplug: allocate memmap from
hotadded memory
On Fri 23-11-18 13:51:57, David Hildenbrand wrote:
> On 23.11.18 13:42, Michal Hocko wrote:
> > On Fri 23-11-18 12:55:41, Oscar Salvador wrote:
[...]
> >> It is not memory that the system can use.
> >
> > same as bootmem ;)
>
> Fair enough, just saying that it represents a change :)
>
> (but people also already complained if their VM has XGB but they don't
> see actual XGB as total memory e.g. due to the crash kernel size)
I can imagine. I have seen many "where's my memory dude" questions... We
have so many unaccounted usages that it is simply impossible to see the
full picture of where the memory is consumed. The current implementation
would account memmaps in unreclaimable slabs but you still do not know
how much was spent for it...
> >> I also guess that if there is a strong opinion on this, we could create
> >> a counter, something like NR_VMEMMAP_PAGES, and show it under /proc/meminfo.
> >
> > Do we really have to? Isn't the number quite obvious from the size of
> > the hotpluged memory?
>
> At least the size of vmmaps cannot reliably calculated from "MemTotal" .
> But maybe based on something else. (there, it is indeed obvious)
Everybody knows the struct page size obviously :p and the rest is a
simple exercise. But more seriously, I see what you are saying. We do
not have a good counter now and the patch doesn't improve that. But I
guess this is a separate discussion.
--
Michal Hocko
SUSE Labs
Powered by blists - more mailing lists