diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2009-07-02 15:05:10 -0400 | 
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-07-02 15:05:10 -0400 | 
| commit | 43644679a1e80f53e6e0155ab75b1093ba3c0365 (patch) | |
| tree | 8c06f272fe6290c9031adc7f176711af12435902 /fs/buffer.c | |
| parent | 42ca4fb69126dd9d0e25112edca1cacf846aa5c3 (diff) | |
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>
Diffstat (limited to 'fs/buffer.c')
0 files changed, 0 insertions, 0 deletions
