Re: [GIT PULL] scheduler fix

From: Linus Torvalds
Date: Thu Apr 08 2010 - 14:40:56 EST




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.

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".

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