Received-SPF: pass (google.com: domain of linux-kernel+bounces-184797-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Precedence: bulk
Mime-Version: 1.0
Subject: RE:(2) [RESEND PATCH 00/10] memblock: introduce memsize showing
 reserved memory
Reply-To: jaewon31.kim@samsung.com
Sender: Jaewon Kim <jaewon31.kim@samsung.com>
From: Jaewon Kim <jaewon31.kim@samsung.com>
To: Mike Rapoport <rppt@kernel.org>, 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>,
	"jaewon31.kim@gmail.com" <jaewon31.kim@gmail.com>, "tkjos@google.com"
	<tkjos@google.com>
In-Reply-To: <ZkxN0yQ7Fb0X26hT@kernel.org>
Message-ID: <20240521101753epcms1p50443f6b88adea211dd9bbb417dd57cb1@epcms1p5>
Date: Tue, 21 May 2024 19:17:53 +0900
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset="utf-8"
CMS-TYPE: 101P
DLP-Filter: Pass
References: <ZkxN0yQ7Fb0X26hT@kernel.org>
	<20240521023957.2587005-1-jaewon31.kim@samsung.com>
	<20240521025329epcms1p6ce11064c0f0608a0156d82fda7ef285c@epcms1p6>
	<CGME20240521024009epcas1p10ed9f9b929203183a29f79508e79bb76@epcms1p5>

>On Tue, May 21, 2024 at 11:53:29AM +0900, Jaewon Kim wrote:
>> >--------- Original Message ---------
>> >Sender : =EA=B9=80=EC=9E=AC=EC=9B=90=20<jaewon31.kim=40samsung.com>Syst=
em=20Performance=20Lab.(MX)/=EC=82=BC=EC=84=B1=EC=A0=84=EC=9E=90=0D=0A>>=20=
>Date=20=20=20:=202024-05-21=2011:40=20(GMT+9)=0D=0A>>=20>Title=20=20:=20=
=5BRESEND=20PATCH=2000/10=5D=20memblock:=20introduce=20memsize=20showing=20=
reserved=20memory=0D=0A>>=20>?=0D=0A>>=20>Some=20of=20memory=20regions=20ca=
n=20be=20reserved=20for=20a=20specific=20purpose.=20They=20are=0D=0A>>=20>u=
sually=20defined=20through=20reserved-memory=20in=20device=20tree.=20If=20o=
nly=20size=0D=0A>>=20>without=20address=20is=20specified=20in=20device=20tr=
ee,=20the=20address=20of=20the=20region=0D=0A>>=20>will=20be=20determined=
=20at=20boot=20time.=0D=0A>>=20>=0D=0A>>=20>We=20may=20find=20the=20address=
=20of=20the=20memory=20regions=20through=20booting=20log,=20but=0D=0A>>=20>=
it=20does=20not=20show=20all.=20And=20it=20could=20be=20hard=20to=20catch=
=20the=20very=20beginning=0D=0A>>=20>log.=20The=20memblock_dump_all=20shows=
=20all=20memblock=20status=20but=20it=20does=20not=0D=0A>>=20>show=20region=
=20name=20and=20its=20information=20is=20difficult=20to=20summarize.=0D=0A>=
>=20>=0D=0A>>=20>This=20patch=20introduce=20a=20debugfs=20node,=20memblock/=
memsize,=20to=20see=20reserved=0D=0A>>=20>memory=20easily.=0D=0A>>=20>=0D=
=0A>>=20>Here's=20an=20example=0D=0A>>=20>=0D=0A>>=20>=24=20cat=20debugfs/m=
emblock/memsize=0D=0A>>=20>=0D=0A>>=20>0x0000000000000000-0x000000000000000=
0=200x02000000=20(?=20=2032768=20KB=20)?=20=20map=20reusable=20linux,cma=0D=
=0A>>=20>0x0000000000000000-0x0000000000000000=200x01000000=20(?=20=2016384=
=20KB=20)?=20=20map=20reusable=20vxxxxx=0D=0A>>=20>...=0D=0A>>=20>0x0000000=
000000000-0x0000000000000000=200x004e0000=20(?=20?=204992=20KB=20)=20nomap=
=20unusable=20unknown=0D=0A>>=20>0x0000000000000000-0x0000000000000000=200x=
00400000=20(?=20?=204096=20KB=20)=20nomap=20unusable=20cxxxxx=0D=0A>>=20>0x=
0000000000000000-0x0000000000000000=200x00e00000=20(?=20=2014336=20KB=20)=
=20nomap=20unusable=20gxxxxx=0D=0A>>=20>=0D=0A>>=20>Reserved?=20?=20:=20122=
3856=20KB=0D=0A>>=20>=20.kernel?=20?=20:?=20275208=20KB=0D=0A>>=20>?=20.tex=
t?=20?=20=20:?=20=2016576=20KB=0D=0A>>=20>?=20.rwdata?=20=20:?=20?=201963=
=20KB=0D=0A>>=20>?=20.rodata?=20=20:?=20=2011920=20KB=0D=0A>>=20>?=20.bss?=
=20?=20?=20:?=20?=202450=20KB=0D=0A>>=20>?=20.memmap?=20=20:?=20186368=20KB=
=0D=0A>>=20>?=20.etc?=20?=20?=20:?=20=2055933=20KB=0D=0A>>=20>=20.unusable?=
=20:?=20948648=20KB=0D=0A>>=20>System?=20?=20?=20:=2011359056=20KB=0D=0A>>=
=20>=20.common?=20?=20:=2010306384=20KB=0D=0A>>=20>=20.reusable?=20:=201052=
672=20KB=0D=0A>>=20>Total?=20?=20?=20=20:=2012582912=20KB=20(=2012288.00=20=
MB=20)=0D=0A>>=20>=0D=0A>>=20>Jaewon=20Kim=20(10):=0D=0A>>=20>?=20memblock:=
=20introduce=20memsize=20showing=20reserved=20memory=0D=0A>>=20>?=20membloc=
k:=20detect=20hidden=20memory=20hole=20size=0D=0A>>=20>?=20memblock:=20hand=
le=20overlapped=20reserved=20memory=20region=0D=0A>>=20>?=20memblock:=20tak=
e=20a=20region=20intersecting=20an=20unknown=20region=0D=0A>>=20>?=20memblo=
ck:=20track=20memblock=20changed=20at=20early=20param=0D=0A>>=20>?=20memblo=
ck:=20recognize=20late=20freed=20size=20by=20checking=20PageReserved=0D=0A>=
>=20>?=20memblock:=20track=20kernel=20size=20on=20memsize=0D=0A>>=20>?=20me=
mblock:=20print=20memsize=20summary=20information=0D=0A>>=20>?=20memblock:=
=20print=20kernel=20internal=20size=0D=0A>>=20>?=20memblock:=20support=20me=
msize=20reusable=20to=20consider=20as=20reusable=0D=0A>>=20>=0D=0A>>=20>=20=
drivers/of/fdt.c?=20?=20?=20?=20?=20?=20=20=7C?=2011=20+=0D=0A>>=20>=20driv=
ers/of/of_reserved_mem.c=20=7C?=2012=20+-=0D=0A>>=20>=20include/linux/membl=
ock.h?=20?=20=20=7C?=2029=20++=0D=0A>>=20>=20init/main.c?=20?=20?=20?=20?=
=20?=20?=20?=20?=20=7C?=2013=20+-=0D=0A>>=20>=20kernel/dma/contiguous.c?=20=
?=20?=20=7C?=20=209=20+-=0D=0A>>=20>=20mm/Kconfig?=20?=20?=20?=20?=20?=20?=
=20?=20?=20=20=7C?=2016=20++=0D=0A>>=20>=20mm/memblock.c?=20?=20?=20?=20?=
=20?=20?=20?=20=7C=20502=20++++++++++++++++++++++++++++++++++-=0D=0A>>=20>=
=20mm/mm_init.c?=20?=20?=20?=20?=20?=20?=20?=20=20=7C?=20=206=20+-=0D=0A>>=
=20>=20mm/page_alloc.c?=20?=20?=20?=20?=20?=20?=20=7C?=2010=20+-=0D=0A>>=20=
>=209=20files=20changed,=20597=20insertions(+),=2011=20deletions(-)=0D=0A>>=
=20>=0D=0A>>=20>--=20=0D=0A>>=20>2.25.1=0D=0A>>=20=0D=0A>>=20Hello=20Mike=
=20=0D=0A>>=20=0D=0A>>=20This=20is=20actually=20RESEND=20as=20it=20was=20in=
troduced=202=20years=20ago.=0D=0A>>=20Please=20refer=20to=20https://lore.ke=
rnel.org/linux-mm/YkQB6Ah603yPR3qf=40kernel.org/=23t=0D=0A>>=20=0D=0A>>=20>=
=20But=20you=20never=20provided=20details=20about=20*why*=20you=20want=20th=
is=20information=20exposed.=0D=0A>>=20=0D=0A>>=20For=20your=20question,=20I=
'd=20like=20to=20say=20;=0D=0A>>=20We=20can=20see=20the=20same=20format=20a=
nd=20exact=20information=20between=20different=20version=20of=20kernel=20st=
atus.=0D=0A>>=20=0D=0A>>=201)=20Internally=20we=20can=20check=20if=20the=20=
reserved=20memory=20changes.=0D=0A>>=202)=20Externally=20we=20can=20communi=
cate=20between=20chipset=20vendors=20and=20OEM,=20with=20a=20same=20format.=
=0D=0A>=0D=0A>Why=20the=20existing=20debugfs=20interface=20is=20not=20suffi=
cient?=0D=0A=0D=0Adebugfs/memblock/memory=20&=20debugfs/memblock/reserved=
=20have=20changed=20its=20format=20but=20still=20does=20not=20show=20name,=
=20reusable,=20kernel=20size.=0D=0AIf=20memory=20is=20reserved=20from=20mem=
block,=20and=20did=20not=20freed=20back=20to=20memblock.=20Memblock=20does=
=20not=20know=20even=20after=20the=20memory=20is=20freed=20to=20system.=0D=
=0AI=20think=20a=20simple=20debug=20interface=20is=20needed=20to=20easily=
=20communicate=20with=20others=20or=20compare=20different=20SW=20releases.=
=0D=0A=0D=0A>=20=0D=0A>>=20This=20helps=20us=20to=20communitcate=20well,=20=
to=20easily=20detect=20changes=20or=20just=20to=20see=20differences.=0D=0A>=
>=20=0D=0A>>=20Jaewon=20Kim=0D=0A>>=20=0D=0A>=0D=0A>--=20=0D=0A>Sincerely=
=20yours,=0D=0A>Mike.=0D=0A=0D=0A