Re: [PATCH] add requeue task

[Nick Piggin]

Nikita Danilov wrote:
> Nick Piggin writes:
>  > Con Kolivas wrote:
>  > > add requeue task

>  > >  /*
>  > > + * Put task to the end of the run list without the overhead of dequeue
>  > > + * followed by enqueue.
>  > > + */
>  > > +static void requeue_task(struct task_struct *p, prio_array_t *array)
>  > > +{
>  > > +	list_del(&p->run_list);
>  > > +	list_add_tail(&p->run_list, array->queue + p->prio);
>  > > +}
>
> Shouldn't this be
>
> list_move_tail(&p->run_list, array->queue + p->prio);
>
> ?

I think it should indeed.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/