diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2009-07-02 12:05:10 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-07-02 12:05:10 -0700 |
commit | 43644679a1e80f53e6e0155ab75b1093ba3c0365 (patch) | |
tree | 8c06f272fe6290c9031adc7f176711af12435902 /drivers/atm/iphase.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 'drivers/atm/iphase.c')
0 files changed, 0 insertions, 0 deletions