[PATCH] pseries: phyp dump: Variable size reserve space.

Segher Boessenkool segher at kernel.crashing.org
Thu Apr 10 05:39:59 EST 2008


>>> +	tmp = tmp & ~0x000000001FFFFFFF;

Note that this only works as you expect because the constant is
signed here -- the extra zeroes do not magically make it a 64-bit
number.  So it goes 32-bit 0x1fffffff -> 32-bit -0x20000000 ->
64-bit -0x20000000.

Please consider writing it with an "L" suffix, or "UL" even, to
reduce trickiness and surprises if ever that number is changed.


Segher




More information about the Linuxppc-dev mailing list