From: David Rientjes <rientjes@google.com>
To: Satoru Moriya <satoru.moriya@hds.com>
Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org,
linux-doc@vger.kernel.org,
Andrew Morton <akpm@linux-foundation.org>,
Mel Gorman <mel@csn.ul.ie>,
KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>,
Randy Dunlap <rdunlap@xenotime.net>,
dle-develop@lists.sourceforge.net,
Seiji Aguchi <seiji.aguchi@hds.com>
Subject: RE: [RFC][PATCH 0/2] Tunable watermark
Date: Thu, 13 Jan 2011 14:20:29 -0800 (PST) [thread overview]
Message-ID: <alpine.DEB.2.00.1101131414110.26770@chino.kir.corp.google.com> (raw)
In-Reply-To: <65795E11DBF1E645A09CEC7EAEE94B9C3B8DF645@USINDEVS02.corp.hds.com>
On Thu, 13 Jan 2011, Satoru Moriya wrote:
> > You didn't mention why it wouldn't be possible to modify
> > setup_per_zone_wmarks() in some way for your configuration so this happens
> > automatically. If you can find a deterministic way to set these
> > watermarks from userspace, you should be able to do it in the kernel as
> > well based on the configuration.
>
> Do you mean that we should introduce a mechanism into kernel that changes
> watermarks dynamically depending on its loads (such as cpu frequency control)
> or we should change the calculation method in setup_per_zone_wmarks()?
>
The watermarks you're exposing through this patchset to userspace for the
first time are meant to be internal to the VM. Userspace is not intended
to manipulate them in an effort to cover-up deficiencies within the memory
manager itself. If you have actual cases where tuning the watermarks from
userspace is helpful, then it logically means:
- the VM is acting incorrectly in response to situations where it
approaches the tunable min watermark (all watermarks are a function of
the min watermark) which shouldn't representative in just a handfull
of cases, and
- you can deterministically do the same calculation within the kernel
itself.
I'm skeptical that any tuning is actually helpful to your workload that
doesn't also indicate a problem internal to the VM itself. I think what
would be more helpful is if you would show how the watermarks currently
don't trigger fast enough (or aggressive enough) and then address the
issue in the kernel itself so everyone can benefit from your work, whether
that's adjusting where the watermarks are based on external factors or
whether the semantics of those watermarks are to slightly change.
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom policy in Canada: sign http://dissolvethecrtc.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2011-01-13 22:20 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-07 22:03 Satoru Moriya
2011-01-07 22:04 ` [RFC][PATCH 1/2] Add explanation about min_free_kbytes to clarify its effect Satoru Moriya
2011-01-07 22:27 ` David Rientjes
2011-01-07 22:07 ` [RFC][PATCH 2/2] Make watermarks tunable separately Satoru Moriya
2011-01-07 22:23 ` [RFC][PATCH 0/2] Tunable watermark David Rientjes
2011-01-07 22:35 ` Ying Han
2011-01-07 22:39 ` David Rientjes
2011-01-13 22:05 ` Satoru Moriya
2011-01-13 22:24 ` David Rientjes
2011-01-13 22:05 ` Satoru Moriya
2011-01-13 22:20 ` David Rientjes [this message]
2011-01-21 0:16 ` Rik van Riel
2011-02-10 18:30 ` Satoru Moriya
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=alpine.DEB.2.00.1101131414110.26770@chino.kir.corp.google.com \
--to=rientjes@google.com \
--cc=akpm@linux-foundation.org \
--cc=dle-develop@lists.sourceforge.net \
--cc=kosaki.motohiro@jp.fujitsu.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=mel@csn.ul.ie \
--cc=rdunlap@xenotime.net \
--cc=satoru.moriya@hds.com \
--cc=seiji.aguchi@hds.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