Strange comment in sched.c

Michael Hayes (mike@aiinc.ca)
Thu, 20 Feb 2003 11:20:18 -0800 (PST)


This comment at sched.c:843 is a bit confusing:

* We fend off statistical fluctuations in runqueue lengths by
* saving the runqueue length during the previous load-balancing
* operation and using the smaller one the current and saved lengths.

First, it's ungrammatical ("...the smaller one the..."). Second,
the code immediately following it seems to be choosing the _larger_
of the two values, not the smaller:

if (idle || (this_rq->nr_running > this_rq->prev_nr_running[this_cpu]))
nr_running = this_rq->nr_running;
else
nr_running = this_rq->prev_nr_running[this_cpu];

-
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/