On Wed, 27 Feb 2008, Thomas Gleixner wrote:> +static inline int phys_addr_valid(unsigned long addr)> +{> + return addr < (1 << boot_cpu_data.x86_phys_bits);That needs to be:+ return addr < (1UL << boot_cpu_data.x86_phys_bits);Where is my brown paperbag ? I know that GCC _is_ stupid, but I neverremember. tglx