Re: [PATCH] Documentation/mm/page_owner.rst: update experimental data

[Bagas Sanjaya <bagasdotme@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1732 bytes --]

On Thu, Aug 25, 2022 at 12:09:35PM -0600, Jonathan Corbet wrote:
> >  - Without page owner::
> >  
> >     text    data     bss     dec     hex filename
> > -   48392   2333     644   51369    c8a9 mm/page_alloc.o
> > +   58581   3166     652   62399    f3bf mm/page_alloc.o
> >  
> >  - With page owner::
> >  
> >     text    data     bss     dec     hex filename
> > -   48800   2445     644   51889    cab1 mm/page_alloc.o
> > -   6662     108      29    6799    1a8f mm/page_owner.o
> > -   1025       8       8    1041     411 mm/page_ext.o
> > +   59085   3294     652   63031    f637 mm/page_alloc.o
> > +   7464     125      28    7617    1dc1 mm/page_owner.o
> > +   1396      32       8    1436     59c mm/page_ext.o
> >  
> > -Although, roughly, 8 KB code is added in total, page_alloc.o increase by
> > -520 bytes and less than half of it is in hotpath. Building the kernel with
> > +Although, roughly, 9 KB code is added in total, page_alloc.o increase by
> > +632 bytes and less than half of it is in hotpath. Building the kernel with
> >  page owner and turning it on if needed would be great option to debug
> >  kernel memory problem.
> 
> This seems like a fine patch...but I do have to wonder if it really
> makes sense to maintain this information, which will surely be subject
> to frequent change.  Perhaps the whole thing is better replaced with a
> line saying that enabling page owner adds ~9KB?
> 

The size figure above is from size(1) from binutils, for which the
command is run (maybe) against vmlinux.

So I would like to say "Enabling page owner increases vmlinux size by
several kilobytes."

Thanks.

-- 
An old man doll... just what I always wanted! - Clara

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]