From: Lee Schermerhorn <Lee.Schermerhorn@hp.com>
To: Paul Jackson <pj@sgi.com>
Cc: rientjes@google.com, akpm@linux-foundation.org, ak@suse.de,
clameter@sgi.com, linux-kernel@vger.kernel.org
Subject: Re: [patch 3/3] cpusets: add memory_spread_user option
Date: Fri, 26 Oct 2007 16:39:09 -0400 [thread overview]
Message-ID: <1193431149.5032.60.camel@localhost> (raw)
In-Reply-To: <20071026105431.77d56253.pj@sgi.com>
On Fri, 2007-10-26 at 10:54 -0700, Paul Jackson wrote:
> > Will it handle the case of MPOL_INTERLEAVE policy on a shm segment that
> > is mapped by tasks in different, possibly disjoint, cpusets. Local
> > allocation does, and my patch does. That was one of the primary
> > goals--to address an issue that Christoph has with shared policies.
> > cpusets really muck these up!
>
> It probably won't handle that. I don't get along too well with shmem.
Not surprising :). shmem doesn't get along too well with cpusets.
>
> Can you to an anti-shmem bigot how MPOL_INTERLEAVE should work with
^ explain ?
> shmem segments mapped in diverse ways by different tasks in different
> cpusets? What would be the key attribute(s) of a proper solution?
> Maybe if we keep it simple enough, I can avoid mucking it up too much
> this time around.
Personally, I'm of the opinion "if it hurts when you do that, don't do
that". I have uses for shared memory and mempolicies on the same, but
they don't involve sharing shmem [nor mapped files] between cpusets nor
dynamically changing cpusets. So, my approach would be to document the
issues clearly [another reason I'd like to see cpuset man pages] and
make sure that folks can't accidentally trip over them. But, I suppose
all the documentation in the world won't stop some people from hurting
themselves. As my grandmother used to tell me, "children and fools
shouldn't play with sharp tools." [Then she'd always ask me, "Which one
are you?" I guess time has answered that question...]
Lee
next prev parent reply other threads:[~2007-10-26 20:41 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-10-26 2:14 [patch 1/3] cpusets: extract mmarray loading from update_nodemask David Rientjes
2007-10-26 2:14 ` [patch 2/3] mempolicy: mpol_rebind_policy cleanup David Rientjes
2007-10-26 2:14 ` [patch 3/3] cpusets: add memory_spread_user option David Rientjes
2007-10-26 6:04 ` Paul Jackson
2007-10-26 9:23 ` David Rientjes
2007-10-26 9:56 ` Paul Jackson
2007-10-26 17:18 ` Paul Jackson
2007-10-26 17:39 ` Christoph Lameter
2007-10-26 17:43 ` Paul Jackson
2007-10-26 17:43 ` Lee Schermerhorn
2007-10-26 17:54 ` Paul Jackson
2007-10-26 18:00 ` Christoph Lameter
2007-10-26 20:39 ` Lee Schermerhorn [this message]
2007-10-26 20:41 ` David Rientjes
2007-10-26 2:46 ` [patch 2/3] mempolicy: mpol_rebind_policy cleanup Paul Jackson
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=1193431149.5032.60.camel@localhost \
--to=lee.schermerhorn@hp.com \
--cc=ak@suse.de \
--cc=akpm@linux-foundation.org \
--cc=clameter@sgi.com \
--cc=linux-kernel@vger.kernel.org \
--cc=pj@sgi.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 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.