All of lore.kernel.org
 help / color / mirror / Atom feed
From: Suren Baghdasaryan <surenb@google.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: Colin Cross <ccross@google.com>,
	Sumit Semwal <sumit.semwal@linaro.org>,
	Michal Hocko <mhocko@suse.com>,
	Dave Hansen <dave.hansen@intel.com>,
	Kees Cook <keescook@chromium.org>,
	Matthew Wilcox <willy@infradead.org>,
	"Kirill A . Shutemov" <kirill.shutemov@linux.intel.com>,
	Vlastimil Babka <vbabka@suse.cz>,
	Johannes Weiner <hannes@cmpxchg.org>,
	Jonathan Corbet <corbet@lwn.net>,
	Al Viro <viro@zeniv.linux.org.uk>,
	Randy Dunlap <rdunlap@infradead.org>,
	Kalesh Singh <kaleshsingh@google.com>,
	Peter Xu <peterx@redhat.com>,
	rppt@kernel.org, Peter Zijlstra <peterz@infradead.org>,
	Catalin Marinas <catalin.marinas@arm.com>,
	vincenzo.frascino@arm.com,
	Chinwen Chang <chinwen.chang@mediatek.com>,
	Axel Rasmussen <axelrasmussen@google.com>,
	Andrea Arcangeli <aarcange@redhat.com>,
	Jann Horn <jannh@google.com>,
	apopple@nvidia.com, John Hubbard <jhubbard@nvidia.com>,
	Yu Zhao <yuzhao@google.com>, Will Deacon <will@kernel.org>,
	fenghua.yu@intel.com, thunder.leizhen@huawei.com,
	Hugh Dickins <hughd@google.com>,
	feng.tang@intel.com, Jason Gunthorpe <jgg@ziepe.ca>,
	Roman Gushchin <guro@fb.com>,
	Thomas Gleixner <tglx@linutronix.de>,
	krisman@collabora.com, Chris Hyser <chris.hyser@oracle.com>,
	Peter Collingbourne <pcc@google.com>,
	"Eric W. Biederman" <ebiederm@xmission.com>,
	Jens Axboe <axboe@kernel.dk>,
	legion@kernel.org, Rolf Eike Beer <eb@emlix.com>,
	Cyrill Gorcunov <gorcunov@gmail.com>, Pavel Machek <pavel@ucw.cz>,
	Muchun Song <songmuchun@bytedance.com>,
	Viresh Kumar <viresh.kumar@linaro.org>,
	Thomas Cedeno <thomascedeno@google.com>,
	sashal@kernel.org, cxfcosmos@gmail.com,
	Rasmus Villemoes <linux@rasmusvillemoes.dk>,
	LKML <linux-kernel@vger.kernel.org>,
	linux-fsdevel@vger.kernel.org, linux-doc@vger.kernel.org,
	linux-mm <linux-mm@kvack.org>,
	kernel-team <kernel-team@android.com>
Subject: Re: [PATCH v11 2/3] mm: add a field to store names for private anonymous memory
Date: Mon, 15 Nov 2021 22:10:21 -0800	[thread overview]
Message-ID: <CAJuCfpEqO8C7eRO1Mr+MULnuKxjjh1zq2j1yoGZhocghpr7V9w@mail.gmail.com> (raw)
In-Reply-To: <20211115211905.faef6f9db3ce4a6fb9ed66a2@linux-foundation.org>

On Mon, Nov 15, 2021 at 9:19 PM Andrew Morton <akpm@linux-foundation.org> wrote:
>
> On Tue, 19 Oct 2021 14:58:36 -0700 Suren Baghdasaryan <surenb@google.com> 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!

  reply	other threads:[~2021-11-16  6:10 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-19 21:55 [PATCH v11 1/3] mm: rearrange madvise code to allow for reuse Suren Baghdasaryan
2021-10-19 21:55 ` [PATCH v11 2/3] mm: add a field to store names for private anonymous memory Suren Baghdasaryan
2021-10-19 21:58   ` Suren Baghdasaryan
2021-11-16  5:19     ` Andrew Morton
2021-11-16  6:10       ` Suren Baghdasaryan [this message]
2021-10-27 18:35   ` Alexey Alexandrov
2021-10-27 20:01     ` Suren Baghdasaryan
2021-10-28 22:08       ` Suren Baghdasaryan
2021-11-15 18:59         ` Suren Baghdasaryan
2021-11-16  9:51           ` Michal Hocko
2021-11-16 16:29             ` Suren Baghdasaryan
2021-10-19 21:55 ` [PATCH v11 3/3] mm: add anonymous vma name refcounting Suren Baghdasaryan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAJuCfpEqO8C7eRO1Mr+MULnuKxjjh1zq2j1yoGZhocghpr7V9w@mail.gmail.com \
    --to=surenb@google.com \
    --cc=aarcange@redhat.com \
    --cc=akpm@linux-foundation.org \
    --cc=apopple@nvidia.com \
    --cc=axboe@kernel.dk \
    --cc=axelrasmussen@google.com \
    --cc=catalin.marinas@arm.com \
    --cc=ccross@google.com \
    --cc=chinwen.chang@mediatek.com \
    --cc=chris.hyser@oracle.com \
    --cc=corbet@lwn.net \
    --cc=cxfcosmos@gmail.com \
    --cc=dave.hansen@intel.com \
    --cc=eb@emlix.com \
    --cc=ebiederm@xmission.com \
    --cc=feng.tang@intel.com \
    --cc=fenghua.yu@intel.com \
    --cc=gorcunov@gmail.com \
    --cc=guro@fb.com \
    --cc=hannes@cmpxchg.org \
    --cc=hughd@google.com \
    --cc=jannh@google.com \
    --cc=jgg@ziepe.ca \
    --cc=jhubbard@nvidia.com \
    --cc=kaleshsingh@google.com \
    --cc=keescook@chromium.org \
    --cc=kernel-team@android.com \
    --cc=kirill.shutemov@linux.intel.com \
    --cc=krisman@collabora.com \
    --cc=legion@kernel.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=linux@rasmusvillemoes.dk \
    --cc=mhocko@suse.com \
    --cc=pavel@ucw.cz \
    --cc=pcc@google.com \
    --cc=peterx@redhat.com \
    --cc=peterz@infradead.org \
    --cc=rdunlap@infradead.org \
    --cc=rppt@kernel.org \
    --cc=sashal@kernel.org \
    --cc=songmuchun@bytedance.com \
    --cc=sumit.semwal@linaro.org \
    --cc=tglx@linutronix.de \
    --cc=thomascedeno@google.com \
    --cc=thunder.leizhen@huawei.com \
    --cc=vbabka@suse.cz \
    --cc=vincenzo.frascino@arm.com \
    --cc=viresh.kumar@linaro.org \
    --cc=viro@zeniv.linux.org.uk \
    --cc=will@kernel.org \
    --cc=willy@infradead.org \
    --cc=yuzhao@google.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.