sched: remove set_leftmost()
authorIngo Molnar <mingo@elte.hu>
Mon, 15 Oct 2007 15:00:11 +0000 (17:00 +0200)
committerIngo Molnar <mingo@elte.hu>
Mon, 15 Oct 2007 15:00:11 +0000 (17:00 +0200)
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>
kernel/sched_fair.c

index 91664d665c0ff938adb8306f2aca5af62d216fd5..48c69211888f3eca93a14881d55320020b74a916 100644 (file)
@@ -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);
 }