Subject: [PATCH][2.5] make kernel scheduler use list_move_tail (1 occ)

This makes the kernel scheduler use list_move_tail instead of more code

--- linus-2.5/kernel/sched.c Sun Jun 9 04:17:27 2002
+++ thunder-2.5/kernel/sched.c Sun Jun 9 06:58:51 2002
@@ -1360,8 +1360,7 @@
* then just requeue the task to the end of the runqueue:
*/
if (likely(current->prio == MAX_PRIO-1 || rt_task(current))) {
- list_del(&current->run_list);
- list_add_tail(&current->run_list, array->queue + current->prio);
+ list_move_tail(&current->run_list, array->queue + current->prio);
} else {
list_del(&current->run_list);
if (list_empty(array->queue + current->prio))

--
Lightweight patch manager using pine. If you have any objections, tell me.


2002-06-09 17:27:02

by Robert Love

[permalink] [raw]
Subject: Re: [PATCH][2.5] make kernel scheduler use list_move_tail (1 occ)

On Sun, 2002-06-09 at 10:22, Robert Love wrote:

> I guess this would be fine if I knew what list_move_tail did... or if I
> had it in my tree:

Oh I see there was an earlier patch that added these new macros,
sorry... that should of been mentioned :)

Yes, this is fine with me.

Robert Love





2002-06-09 17:22:41

by Robert Love

[permalink] [raw]
Subject: Re: [PATCH][2.5] make kernel scheduler use list_move_tail (1 occ)

On Sun, 2002-06-09 at 07:22, Lightweight patch manager wrote:
> This makes the kernel scheduler use list_move_tail instead of more code
>
> [...]
>
> - list_del(&current->run_list);
> - list_add_tail(&current->run_list, array->queue + current->prio);
> + list_move_tail(&current->run_list, array->queue + current->prio);

I guess this would be fine if I knew what list_move_tail did... or if I
had it in my tree:

[10:11:06]rml@sinai:~/kernel-2.5/linux$ grep -r list_move_tail *
[10:12:07]rml@sinai:~/kernel-2.5/linux$

???

Robert Love

2002-06-09 17:50:05

by Thunder from the hill

[permalink] [raw]
Subject: Re: [PATCH][2.5] make kernel scheduler use list_move_tail (1 occ)

Hi,

On 9 Jun 2002, Robert Love wrote:
> I guess this would be fine if I knew what list_move_tail did... or if I
> had it in my tree:

There was a patch introducing it this morning, maybe you missed it.

http://marc.theaimsgroup.com/?l=linux-kernel&m=102362676727017&w=2

Regards,
Thunder
--
ship is leaving right on time | Thunder from the hill at ngforever
empty harbour, wave goodbye |
evacuation of the isle | free inhabitant not directly
caveman's paintings drowning | belonging anywhere