On Sat, 19 Jan 2002, Erich Focht wrote:
> In the load_balance() function the initial value for max_load should
> better be set to 1 instead of 0 in order to avoid finding 'busiest'
> runqueues with only one task. This avoids taking the spin-locks
> unnecessarily for the case idle=1.
agreed.
> Another issue: I don't understand how prev_max_load works, I think
> that the comments in load_balance are not true any more and the
> comparison to prev_max_load can be dropped. [...]
you are right, and this changed recently. Since we do not search for
multiple queues anymore when balancing, this variable can be dropped.
I've added both of your suggestions to my tree.
Ingo
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Wed Jan 23 2002 - 21:00:43 EST