Re: [GIT PULL] scheduler fix

From: Ingo Molnar
Date: Thu Apr 08 2010 - 14:52:51 EST



* Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote:

>
>
> On Thu, 8 Apr 2010, Ingo Molnar wrote:
> >
> > So i'd suggest changing nr_cpu_ids to unsigned int [unless there's some strong
> > reason to have it signed] plus doing something like:
> >
> > if (len < (nr_cpu_ids >> BITS_PER_BYTE_BITS))
>
> No workee.
>
> It really should round up.

Indeed.

> If you worry about code generation, I'd suggest looking at whether
> nr_cpu_ids could just be made unsigned.
>
> Anyway, this all was _not_ the point of my original email. I really don't
> care about this particular instance. I care more about the whole "in general
> people should think _way_ more about validating user-supplied arguments than
> clearly happened this time".

Yeah, no argument about that, point taken and accepted.

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