From: Lee Schermerhorn <lee.schermerhorn@hp.com>
To: linux-mm@kvack.org, linux-numa@vger.kernel.org
Cc: akpm@linux-foundation.org, Mel Gorman <mel@csn.ul.ie>,
Nishanth Aravamudan <nacc@us.ibm.com>,
David Rientjes <rientjes@google.com>, Adam Litke <agl@us.ibm.com>,
Andy Whitcroft <apw@canonical.com>,
eric.whitney@hp.com
Subject: [PATCH 0/3] hugetlb: V2 constrain allocation/free based on task mempolicy
Date: Wed, 08 Jul 2009 15:24:30 -0400 [thread overview]
Message-ID: <20090708192430.20687.30157.sendpatchset@lts-notebook> (raw)
PATCH 0/3 hugetlb: constrain allocation/free based on task mempolicy
Against: 25jun09 mmotm atop the "hugetlb: balance freeing..."
series
This is V2 of a series of patches to constrain the allocation and
freeing of persistent huge pages using the task NUMA mempolicy of
the task modifying "nr_hugepages". This series is based on Mel
Gorman's suggestion to use task mempolicy.
V2 addresses review comments from Mel Gorman and Andrew Morton.
See the patch description of patch 2/3.
I have some concerns about a subtle change in behavior [see patch
2/3 and the updated documentation] and the fact that
this mechanism ignores some of the semantics of the mempolicy
mode [again, see the doc]. However, this method seems to work
fairly well. And, IMO, the resulting code doesn't look all that
bad.
A couple of limitations in this version:
1) I haven't implemented a boot time parameter to constrain the
boot time allocation of huge pages. This can be added if
anyone feels strongly that it is required.
2) I have not implemented a per node nr_overcommit_hugepages as
David Rientjes and I discussed earlier. Again, this can be
added and specific nodes can be addressed using the mempolicy
as this series does for allocation and free. However, after
some experience with the libhugetlbfs test suite, specifically
attempting to run the test suite constrained by mempolicy and
a cpuset, I'm thinking that per node overcommit limits might
not be such a good idea. This would require an application
[or the library] to sum the per node limits over the allowed
nodes and possibly compare to global limits to determine the
available resources. Per cpuset limits might work better.
This are requires more investigation, but this patch series
doesn't seem to make things worse than they already are in
this regard.
next reply other threads:[~2009-07-08 19:24 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-08 19:24 Lee Schermerhorn [this message]
2009-07-08 19:24 ` [PATCH 1/3] hugetlb: add nodemask arg to huge page alloc, free and surplus adjust fcns Lee Schermerhorn
2009-07-09 8:33 ` Mel Gorman
2009-07-09 19:24 ` Lee Schermerhorn
2009-07-08 19:24 ` [PATCH 2/3] hugetlb: derive huge pages nodes allowed from task mempolicy Lee Schermerhorn
2009-07-09 13:30 ` Mel Gorman
2009-07-09 13:38 ` Mel Gorman
2009-07-08 19:24 ` [PATCH 3/3] hugetlb: update hugetlb documentation for mempolicy based management Lee Schermerhorn
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=20090708192430.20687.30157.sendpatchset@lts-notebook \
--to=lee.schermerhorn@hp.com \
--cc=agl@us.ibm.com \
--cc=akpm@linux-foundation.org \
--cc=apw@canonical.com \
--cc=eric.whitney@hp.com \
--cc=linux-mm@kvack.org \
--cc=linux-numa@vger.kernel.org \
--cc=mel@csn.ul.ie \
--cc=nacc@us.ibm.com \
--cc=rientjes@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).