Commit 0615dec5 authored by Josh Aas's avatar Josh Aas Committed by Linus Torvalds

[PATCH] sched: remove outdated/misleading comments

This patch removes two outdated/misleading comments from the CPU scheduler.

1) The first comment removed is simply incorrect.  The function it
   comments on is not used for what the comments says it is anymore.

2) The second comment is a leftover from when the "if" block it comments
   on contained a goto.  It does not any more, and the comment doesn't make
   sense.

There isn't really a reason to add different comments, though someone might
feel differently in the case of the second one.  I'll leave adding a
comment to anybody who wants to - more important to just get rid of them
now.
Signed-off-by: default avatarJosh Aas <josha@sgi.com>
Acked-by: default avatarIngo Molnar <mingo@elte.hu>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 0cc0f9fc
...@@ -601,11 +601,6 @@ static void requeue_task(struct task_struct *p, prio_array_t *array) ...@@ -601,11 +601,6 @@ static void requeue_task(struct task_struct *p, prio_array_t *array)
list_move_tail(&p->run_list, array->queue + p->prio); list_move_tail(&p->run_list, array->queue + p->prio);
} }
/*
* Used by the migration code - we pull tasks from the head of the
* remote queue so we want these tasks to show up at the head of the
* local queue:
*/
static inline void enqueue_task_head(struct task_struct *p, prio_array_t *array) static inline void enqueue_task_head(struct task_struct *p, prio_array_t *array)
{ {
list_add(&p->run_list, array->queue + p->prio); list_add(&p->run_list, array->queue + p->prio);
...@@ -2573,10 +2568,7 @@ asmlinkage void __sched schedule(void) ...@@ -2573,10 +2568,7 @@ asmlinkage void __sched schedule(void)
if (unlikely(prev->flags & PF_DEAD)) if (unlikely(prev->flags & PF_DEAD))
prev->state = EXIT_DEAD; prev->state = EXIT_DEAD;
/*
* if entering off of a kernel preemption go straight
* to picking the next task.
*/
switch_count = &prev->nivcsw; switch_count = &prev->nivcsw;
if (prev->state && !(preempt_count() & PREEMPT_ACTIVE)) { if (prev->state && !(preempt_count() & PREEMPT_ACTIVE)) {
switch_count = &prev->nvcsw; switch_count = &prev->nvcsw;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment