On Tue, 6 Mar 2007, David Howells wrote:
> @@ -159,8 +175,8 @@ unsigned long __roundup_pow_of_two(unsigned long n)
>  #define roundup_pow_of_two(n)                        \
>  (                                            \
>       __builtin_constant_p(n) ? (             \
> -             (n == 1) ? 0 :                  \
> -             (1UL << (ilog2((n) - 1) + 1))   \
> +             (n == 1) ? 1 :                  \
> +             (1UL << ilog2_up(n))            \
>                                  ) :          \
>       __roundup_pow_of_two(n)                 \

That seems bogus. "n == 1" should give "0", no?

                Linus
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to