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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <87ilmgyz34.fsf@meer.lwn.net>
Date:   Thu, 25 Aug 2022 12:09:35 -0600
From:   Jonathan Corbet <corbet@....net>
To:     Yixuan Cao <caoyixuan2019@...il.szu.edu.cn>,
        akpm@...ux-foundation.org
Cc:     yejiajian2018@...il.szu.edu.cn, skhan@...uxfoundation.org,
        akiyks@...il.com, rppt@...nel.org, zhangyinan2019@...il.szu.edu.cn,
        linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org,
        Yixuan Cao <caoyixuan2019@...il.szu.edu.cn>
Subject: Re: [PATCH] Documentation/mm/page_owner.rst: update experimental data

Yixuan Cao <caoyixuan2019@...il.szu.edu.cn> writes:

> I noticed some experimental data need to be updated.
>
> Signed-off-by: Yixuan Cao <caoyixuan2019@...il.szu.edu.cn>
> ---
>  Documentation/mm/page_owner.rst | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/Documentation/mm/page_owner.rst b/Documentation/mm/page_owner.rst
> index f18fd8907049..8e2545bb4e17 100644
> --- a/Documentation/mm/page_owner.rst
> +++ b/Documentation/mm/page_owner.rst
> @@ -41,17 +41,17 @@ size change due to this facility.
>  - 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?

Thanks,

jon