diff options
author | Ingo Molnar <mingo@elte.hu> | 2007-10-15 17:00:11 +0200 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2007-10-15 17:00:11 +0200 |
commit | 57cb499df26d80ec11cd49e56d20835334ac4ab9 (patch) | |
tree | 1affcb6b68f5428e7c96121fec806d1074b0c1a2 | |
parent | 2ddbf952508fb9911036c484a87f6351106b917c (diff) | |
download | op-kernel-dev-57cb499df26d80ec11cd49e56d20835334ac4ab9.zip op-kernel-dev-57cb499df26d80ec11cd49e56d20835334ac4ab9.tar.gz |
sched: remove set_leftmost()
Lee Schermerhorn noticed that set_leftmost() contains dead code,
remove this.
Reported-by: Lee Schermerhorn <Lee.Schermerhorn@hp.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Reviewed-by: Thomas Gleixner <tglx@linutronix.de>
-rw-r--r-- | kernel/sched_fair.c | 14 |
1 files changed, 2 insertions, 12 deletions
diff --git a/kernel/sched_fair.c b/kernel/sched_fair.c index 91664d6..48c6921 100644 --- a/kernel/sched_fair.c +++ b/kernel/sched_fair.c @@ -124,16 +124,6 @@ max_vruntime(u64 min_vruntime, u64 vruntime) return min_vruntime; } -static inline void -set_leftmost(struct cfs_rq *cfs_rq, struct rb_node *leftmost) -{ - struct sched_entity *se; - - cfs_rq->rb_leftmost = leftmost; - if (leftmost) - se = rb_entry(leftmost, struct sched_entity, run_node); -} - static inline s64 entity_key(struct cfs_rq *cfs_rq, struct sched_entity *se) { @@ -175,7 +165,7 @@ __enqueue_entity(struct cfs_rq *cfs_rq, struct sched_entity *se) * used): */ if (leftmost) - set_leftmost(cfs_rq, &se->run_node); + cfs_rq->rb_leftmost = &se->run_node; rb_link_node(&se->run_node, parent, link); rb_insert_color(&se->run_node, &cfs_rq->tasks_timeline); @@ -185,7 +175,7 @@ static void __dequeue_entity(struct cfs_rq *cfs_rq, struct sched_entity *se) { if (cfs_rq->rb_leftmost == &se->run_node) - set_leftmost(cfs_rq, rb_next(&se->run_node)); + cfs_rq->rb_leftmost = rb_next(&se->run_node); rb_erase(&se->run_node, &cfs_rq->tasks_timeline); } |