Re: [PATCH 0/8] memblock: introduce memsize showing reserved memory

From: Mike Rapoport <rppt@kernel.org>
To: Jaewon Kim <jaewon31.kim@samsung.com>
Cc: "vbabka@suse.cz" <vbabka@suse.cz>,
	"akpm@linux-foundation.org" <akpm@linux-foundation.org>,
	"linux-mm@kvack.org" <linux-mm@kvack.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	YongTaek Lee <ytk.lee@samsung.com>,
	"jaewon31.kim@gmail.com" <jaewon31.kim@gmail.com>
Subject: Re: [PATCH 0/8] memblock: introduce memsize showing reserved memory
Date: Sun, 27 Mar 2022 10:40:08 +0300	[thread overview]
Message-ID: <YkAU2JlcX7nlvbwp@kernel.org> (raw)
In-Reply-To: <20220325083846epcms1p372559472ceb511cc45d39c110563063a@epcms1p3>

Hi,

On Fri, Mar 25, 2022 at 05:38:46PM +0900, Jaewon Kim wrote:

> >--------- Original Message ---------
> >Sender : Mike Rapoport <rppt@kernel.org>
> >Date : 2022-03-25 16:46 (GMT+9)
> >Title : Re: [PATCH 0/8] memblock: introduce memsize showing reserved memory
> > 
> >Hi,
> > 
> >On Thu, Mar 24, 2022 at 04:01:50PM +0900, Jaewon Kim wrote:
> >> Some of memory regions can be reserved for a specific purpose. They are
> >> usually defined through reserved-memory in device tree. If only size
> >> without address is specified in device tree, the address of the region
> >> will be determined at boot time.
> >> 
> >> We may find the address of the memory regions through booting log, but
> >> it does not show all. And it could be hard to catch the very beginning
> >> log. The memblock_dump_all shows all memblock status but it does not
> >> show region name and its information is difficult to summarize.
> >> 
> >> This patch introduce a debugfs node, memblock/memsize, to see reserved
> >> memory easily.
> >> 
> >> Here's an example
> >> 
> >> $ cat debugfs/memblock/memsize
> >> 0x0f9000000-0x0fb000000 0x02000000 (   32768 KB )   map reusable linux,cma
> >> 0x0b1900000-0x0b1b00000 0x00200000 (    2048 KB ) nomap unusable test1
> >> 0x0b0200000-0x0b0400000 0x00200000 (    2048 KB )   map unusable test2
> >>  (snipped)
> >> 
> >> Reserved    :  746924 KB
> >>  .kernel    :  137027 KB
> >>   .text     :   28158 KB
> >>   .rwdata   :    3238 KB
> >>   .rodata   :   13468 KB
> >>   .bss      :   12570 KB
> >>   .etc      :   79593 KB
> >>  .unusable  :  609897 KB
> >> System      : 3447380 KB
> >>  .common    : 3152468 KB
> >>  .reusable  :  294912 KB
> >> Total       : 4194304 KB (  4096.00 MB )
> > 
> >Most of this information information is already available at various
> >places, like the existing memblock debugfs, /proc/iomem and DT sysfs.
> > 
> >I don't see why we need yet another debugfs file to expose it.
> 
> Hi.
> Thank you for your reply.
> 
> Especially I'd like to create a node showing all reserved memory status, their 
> total size is same as the physical memory size. This was very useful when I 
> compare reserved memory and kernel init time memory between different chipsets,
> or between different sw release versions.

I'm still not following. The reserved region sizes are available in the
existing memblock debugfs.
Why the names are important? What is the value of having names for *some*
of the reserved regions?

> Thank you
> Jaewon Kim

-- 
Sincerely yours,
Mike.