x86: fix power-of-2 round_up/round_down macros
authorLinus Torvalds <torvalds@linux-foundation.org>
Thu, 2 Jul 2009 19:05:10 +0000 (12:05 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 2 Jul 2009 19:05:10 +0000 (12:05 -0700)
commit43644679a1e80f53e6e0155ab75b1093ba3c0365
tree8c06f272fe6290c9031adc7f176711af12435902
parent42ca4fb69126dd9d0e25112edca1cacf846aa5c3
x86: fix power-of-2 round_up/round_down macros

These macros had two bugs:
 - the type of the mask was not correctly expanded to the full size of
   the argument being expanded, resulting in possible loss of high bits
   when mixing types.
 - the alignment argument was evaluated twice, despite the macro looking
   like a fancy function (but it really does need to be a macro, since
   it works on arbitrary integer types)

Noticed by Peter Anvin, and with a fix that is a modification of his
suggestion (bug noticed by Yinghai Lu).

Cc: Peter Anvin <hpa@zytor.com>
Cc: Yinghai Lu <yinghai@kernel.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/x86/include/asm/proto.h