Re: [PATCH v11 2/3] mm: add a field to store names for private anonymous memory

[Suren Baghdasaryan]

On Mon, Nov 15, 2021 at 9:19 PM Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> wrote:
>
> On Tue, 19 Oct 2021 14:58:36 -0700 Suren Baghdasaryan <surenb@xxxxxxxxxx> wrote:
>
> > As Andrew suggested, I checked the image sizes with allnoconfig builds:
> >
> > unpatched Linus' ToT
> >    text    data     bss     dec     hex filename
> > 1324759      32   73928 1398719 1557bf vmlinux
> >
> > After the first patch is applied (madvise refactoring)
> >    text    data     bss     dec     hex filename
> > 1322346      32   73928 1396306 154e52 vmlinux
> > >>> 2413 bytes decrease vs ToT <<<
> >
> > After all patches applied with CONFIG_ANON_VMA_NAME=n
> >    text    data     bss     dec     hex filename
> > 1322337      32   73928 1396297 154e49 vmlinux
> > >>> 2422 bytes decrease vs ToT <<<
> >
> > After all patches applied with CONFIG_ANON_VMA_NAME=y
> >    text    data     bss     dec     hex filename
> > 1325228      32   73928 1399188 155994 vmlinux
> > >>> 469 bytes increase vs ToT <<<
>
> Nice.  Presumably there are memory savings from no longer duplicating
> the vma names?

The third patch does have this effect.

>
> I fudged up a [0/n] changelog (please don't forget this) and merged it
> all for testing.

Thanks!