diff options
| author | Nikunj A. Dadhania <nikunj@linux.vnet.ibm.com> | 2012-02-17 08:34:30 +0530 |
|---|---|---|
| committer | Ingo Molnar <mingo@elte.hu> | 2012-02-22 12:28:28 +0100 |
| commit | 62f6536a630affe3176deb48554d27ee58b65077 (patch) | |
| tree | cf1365f5c0d0b9aebb5877afa8a7b5da4aa11917 /kernel/sched | |
| parent | 9388dc3047a88bedfd867e9ba3e1980c815ac524 (diff) | |
| download | linux-stable-62f6536a630affe3176deb48554d27ee58b65077.tar.gz linux-stable-62f6536a630affe3176deb48554d27ee58b65077.tar.bz2 linux-stable-62f6536a630affe3176deb48554d27ee58b65077.zip | |
sched: Remove rcu_read_lock/unlock() from select_idle_sibling()
select_idle_sibling() is called from select_task_rq_fair(), which
already has the RCU read lock held.
Signed-off-by: Nikunj A. Dadhania <nikunj@linux.vnet.ibm.com>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/20120217030409.11748.12491.stgit@abhimanyu
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/sched')
| -rw-r--r-- | kernel/sched/fair.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index 4ab60a24ea49..6ce9992926d0 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c @@ -2670,8 +2670,6 @@ static int select_idle_sibling(struct task_struct *p, int target) /* * Otherwise, iterate the domains and find an elegible idle cpu. */ - rcu_read_lock(); - sd = rcu_dereference(per_cpu(sd_llc, target)); for_each_lower_domain(sd) { sg = sd->groups; @@ -2693,8 +2691,6 @@ next: } while (sg != sd->groups); } done: - rcu_read_unlock(); - return target; } |