diff options
author | Paul Jackson <pj@sgi.com> | 2006-01-08 04:02:04 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-01-08 23:13:45 -0500 |
commit | 03a285f58064b8e0af08383e082e383753d9c33e (patch) | |
tree | af458f3357d4e2d01dc04d2d41a7dd7d502c4755 | |
parent | 7edc59628b2f5d6516b4677b3b56f5f056e45cd9 (diff) |
[PATCH] cpuset: skip rcu check if task is in root cpuset
For systems that aren't using cpusets, but have them CONFIG_CPUSET enabled in
their kernel (eventually this may be most distribution kernels), this patch
removes even the minimal rcu_read_lock() from the memory page allocation path.
Actually, it removes that rcu call for any task that is in the root cpuset
(top_cpuset), which on systems not actively using cpusets, is all tasks.
We don't need the rcu check for tasks in the top_cpuset, because the
top_cpuset is statically allocated, so at no risk of being freed out from
underneath us.
Signed-off-by: Paul Jackson <pj@sgi.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | kernel/cpuset.c | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/kernel/cpuset.c b/kernel/cpuset.c index 681a5d58d40d..e04c2da9dadb 100644 --- a/kernel/cpuset.c +++ b/kernel/cpuset.c | |||
@@ -647,10 +647,15 @@ void cpuset_update_task_memory_state() | |||
647 | struct task_struct *tsk = current; | 647 | struct task_struct *tsk = current; |
648 | struct cpuset *cs; | 648 | struct cpuset *cs; |
649 | 649 | ||
650 | rcu_read_lock(); | 650 | if (tsk->cpuset == &top_cpuset) { |
651 | cs = rcu_dereference(tsk->cpuset); | 651 | /* Don't need rcu for top_cpuset. It's never freed. */ |
652 | my_cpusets_mem_gen = cs->mems_generation; | 652 | my_cpusets_mem_gen = top_cpuset.mems_generation; |
653 | rcu_read_unlock(); | 653 | } else { |
654 | rcu_read_lock(); | ||
655 | cs = rcu_dereference(tsk->cpuset); | ||
656 | my_cpusets_mem_gen = cs->mems_generation; | ||
657 | rcu_read_unlock(); | ||
658 | } | ||
654 | 659 | ||
655 | if (my_cpusets_mem_gen != tsk->cpuset_mems_generation) { | 660 | if (my_cpusets_mem_gen != tsk->cpuset_mems_generation) { |
656 | down(&callback_sem); | 661 | down(&callback_sem); |