linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Brian Geffon <bgeffon@google.com>
To: Minchan Kim <minchan@kernel.org>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	LKML <linux-kernel@vger.kernel.org>,
	 linux-mm <linux-mm@kvack.org>,
	Linux API <linux-api@vger.kernel.org>,
	oleksandr@redhat.com,  Suren Baghdasaryan <surenb@google.com>,
	Tim Murray <timmurray@google.com>,
	 Daniel Colascione <dancol@google.com>,
	Sandeep Patil <sspatil@google.com>,
	Sonny Rao <sonnyrao@google.com>,  Michal Hocko <mhocko@suse.com>,
	Johannes Weiner <hannes@cmpxchg.org>,
	Shakeel Butt <shakeelb@google.com>,
	 John Dias <joaodias@google.com>,
	Joel Fernandes <joel@joelfernandes.org>,
	sj38.park@gmail.com,  alexander.h.duyck@linux.intel.com,
	Jann Horn <jannh@google.com>
Subject: Re: [PATCH v6 0/7] introduce memory hinting API for external process
Date: Wed, 19 Feb 2020 14:51:53 -0800	[thread overview]
Message-ID: <CADyq12wn-0ngnqf7WvAWBk+0jiju1_DXWs6LwiQoQ0dfAjzMRQ@mail.gmail.com> (raw)
In-Reply-To: <20200219223241.GA148976@google.com>

To expand on how ChromeOS benefits from this, we've advanced far
beyond the experimentation phase we've deployed an older version of
this idea that was procfs based on several ChromeOS kernels. These are
now rolled out to ChromeOS stable channel where we've been testing and
the results have been amazing. To elaborate on the setup, Chrome is a
multi process architecture where each tab is a separate process and
sometimes a single tab can even represent multiple processes. The
primary Chrome process has a lot of visibility into the amount of time
a user has been spending interacting with a tab (process) and using
this knowledge these hints provided to the kernel allow it to make
much better swap decisions and amortize the cost of swap over
different memory pressure levels meaning that we were better able to
reclaim memory which allow us to avoid having to discard tabs or even
worse oom.

I'd be happy to expand even more if anyone is interested.

Brian

On Wed, Feb 19, 2020 at 2:32 PM Minchan Kim <minchan@kernel.org> wrote:
>
> Hi Andrew,
>
> On Wed, Feb 19, 2020 at 12:01:23PM -0800, Andrew Morton wrote:
> > On Tue, 18 Feb 2020 17:44:26 -0800 Minchan Kim <minchan@kernel.org> wrote:
> >
> > > Now, we have MADV_PAGEOUT and MADV_COLD as madvise hinting API. With that,
> > > application could give hints to kernel what memory range are preferred to be
> > > reclaimed. However, in some platform(e.g., Android), the information
> > > required to make the hinting decision is not known to the app.
> > > Instead, it is known to a centralized userspace daemon(e.g., ActivityManagerService),
> > > and that daemon must be able to initiate reclaim on its own without any app
> > > involvement.
> > >
> >
> > This patchset doesn't seem to be getting a lot of interest from other
> > potential users?  It seems very specialized.  Are there or will there
> > ever be any users of this apart from one Android daemon?
>
>
> Quote from http://lkml.kernel.org/r/20190531064313.193437-1-minchan@kernel.org
>
> "
> Brian Geffon in ChromeOS team had an experiment with process_madvise(2)
> Quote form him:
> "What I found is that by using process_madvise after a tab has been back
> grounded for more than 45 seconds reduced the average tab switch times by
> 25%! This is a huge result and very obvious validation that process_madvise
> hints works well for the ChromeOS use case."
> "
>
> >
> > Also, it doesn't terribly hard for ActivityManagerService to tell
> > another process "now run madvise with these arguments".  Please explain
> > why this is not practical in ActivityManagerService and also within
> > other potential users of this syscall.
>
> I think that's the almost a same question why ptrace doesn't work so
> I summarizes the part in [2/7]:
>
> * makes target task runnable creates memory layout change window so
>  hiniting a wrong vma
>
> * target task(e.g., background task) could live in little core with
>   cpuset/group limited environment so we couldn't react quick enough,
>   which causes more killing.
>
>
> Thanks.


  reply	other threads:[~2020-02-19 22:52 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-19  1:44 [PATCH v6 0/7] introduce memory hinting API for external process Minchan Kim
2020-02-19  1:44 ` [PATCH v6 1/7] mm: pass task and mm to do_madvise Minchan Kim
2020-02-28 22:15   ` Suren Baghdasaryan
2020-02-19  1:44 ` [PATCH v6 2/7] mm: introduce external memory hinting API Minchan Kim
2020-02-20 19:13   ` kbuild test robot
2020-02-20 21:15     ` Minchan Kim
2020-02-20 21:21       ` Minchan Kim
2020-02-28 22:14         ` Suren Baghdasaryan
2020-03-02 19:18           ` Minchan Kim
2020-02-20 20:48   ` kbuild test robot
2020-02-19  1:44 ` [PATCH v6 3/7] mm: check fatal signal pending of target process Minchan Kim
2020-02-28 22:20   ` Suren Baghdasaryan
2020-02-19  1:44 ` [PATCH v6 4/7] pid: move pidfd_get_pid function to pid.c Minchan Kim
2020-02-28 22:22   ` Suren Baghdasaryan
2020-02-19  1:44 ` [PATCH v6 5/7] mm: support both pid and pidfd for process_madvise Minchan Kim
2020-02-28 22:41   ` Suren Baghdasaryan
2020-03-02 19:23     ` Minchan Kim
2020-03-02 19:38       ` Suren Baghdasaryan
2020-02-19  1:44 ` [PATCH v6 6/7] mm/madvise: employ mmget_still_valid for write lock Minchan Kim
2020-02-28 23:19   ` Suren Baghdasaryan
2020-03-02  7:33     ` Oleksandr Natalenko
2020-03-02 16:32       ` Suren Baghdasaryan
2020-02-19  1:44 ` [PATCH v6 7/7] mm/madvise: allow KSM hints for remote API Minchan Kim
2020-02-19 20:01 ` [PATCH v6 0/7] introduce memory hinting API for external process Andrew Morton
2020-02-19 21:05   ` Suren Baghdasaryan
2020-02-19 22:32   ` Minchan Kim
2020-02-19 22:51     ` Brian Geffon [this message]
2020-02-20  9:16   ` SeongJae Park

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=CADyq12wn-0ngnqf7WvAWBk+0jiju1_DXWs6LwiQoQ0dfAjzMRQ@mail.gmail.com \
    --to=bgeffon@google.com \
    --cc=akpm@linux-foundation.org \
    --cc=alexander.h.duyck@linux.intel.com \
    --cc=dancol@google.com \
    --cc=hannes@cmpxchg.org \
    --cc=jannh@google.com \
    --cc=joaodias@google.com \
    --cc=joel@joelfernandes.org \
    --cc=linux-api@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mhocko@suse.com \
    --cc=minchan@kernel.org \
    --cc=oleksandr@redhat.com \
    --cc=shakeelb@google.com \
    --cc=sj38.park@gmail.com \
    --cc=sonnyrao@google.com \
    --cc=sspatil@google.com \
    --cc=surenb@google.com \
    --cc=timmurray@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).