public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] cpuset: mempolicy migration typo fix
@ 2006-10-20  7:41 Paul Jackson
  0 siblings, 0 replies; only message in thread
From: Paul Jackson @ 2006-10-20  7:41 UTC (permalink / raw)
  To: akpm; +Cc: Paul Jackson, ak, linux-kernel

From: Paul Jackson <pj@sgi.com>

Mistyped an ifdef CONFIG_CPUSETS - fixed.

I doubt that anyone ever noticed.  The impact of this typo was
that if someone:
 1) was using MPOL_BIND to force off node allocations
 2) while using cpusets to constrain memory placement
 3) when that cpuset was migrating that jobs memory
 4) while the tasks in that job were actively forking
then there was a rare chance that future allocations using
that MPOL_BIND policy would be node local, not off node.

Signed-off-by: Paul Jackson <pj@sgi.com>

---

It builds under my cross tool setup, it runs my usual
cpuset tests and it runs my not particularly elaborate
cpuset memory migration test.

 include/linux/mempolicy.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- 2.6.19-rc2-mm1.orig/include/linux/mempolicy.h	2006-10-19 23:08:11.396940924 -0700
+++ 2.6.19-rc2-mm1/include/linux/mempolicy.h	2006-10-19 23:26:03.703107004 -0700
@@ -150,7 +150,7 @@ extern void mpol_rebind_mm(struct mm_str
 extern void mpol_fix_fork_child_flag(struct task_struct *p);
 #define set_cpuset_being_rebound(x) (cpuset_being_rebound = (x))
 
-#ifdef CONFIG_CPUSET
+#ifdef CONFIG_CPUSETS
 #define current_cpuset_is_being_rebound() \
 				(cpuset_being_rebound == current->cpuset)
 #else

-- 
                  I won't rest till it's the best ...
                  Programmer, Linux Scalability
                  Paul Jackson <pj@sgi.com> 1.925.600.0401

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2006-10-20  7:41 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-10-20  7:41 [PATCH] cpuset: mempolicy migration typo fix Paul Jackson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox