Messages in this thread |  | | | From | "Stephen C. Biggs" <> | | Date | Tue, 27 Aug 2002 17:59:09 -0700 | | Subject | Re: [PATCH] sched.c |
| |
[Empty post?]
On 28 Aug 2002 at 2:47, Daniel Phillips wrote:
> On Wednesday 28 August 2002 02:13, Daniel Phillips wrote: > > On Tuesday 27 August 2002 20:35, Andrew Morton wrote: > > > Lahti Oy wrote: > > > > - for (i = 0; i < NR_CPUS; i++) > > > > + for (i = NR_CPUS; i; i--) > > > > sum += cpu_rq(i)->nr_running; > > > > > > Off-by-one there. You'd want > > > > > > for (i = NR_CPUS; --i >= 0; ) > > > > > > or something similarly foul ;) > > > > int i = NR_CPUS; > > > > while (--i)
for (i = NR_CPUS - 1; i >= 0; i--)
unless i is unsigned... - 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/
|  |