Re: [PATCH 2/5] arch/x86/kernel/head_32.S Warning: shift count outof range (32 is not between 0 and 31)

From: Renà Bolldorf
Date: Fri Mar 19 2010 - 19:27:31 EST


On 03/19/10 22:45, H. Peter Anvin wrote:
On 03/18/2010 03:08 PM, Renï Bolldorf wrote:
Signed-off-by: Rene Bolldorf<xsecute@xxxxxxxxxxxxxx>

diff --git a/arch/x86/kernel/head_32.S b/arch/x86/kernel/head_32.S
index 37c3d4b..8bfba5e 100644
--- a/arch/x86/kernel/head_32.S
+++ b/arch/x86/kernel/head_32.S
@@ -62,7 +62,7 @@

/* Enough space to fit pagetables for the low memory linear map */
MAPPING_BEYOND_END = \
- PAGE_TABLE_SIZE(((1<<32) - __PAGE_OFFSET)>> PAGE_SHIFT)<< PAGE_SHIFT
+ PAGE_TABLE_SIZE(((1<<31) - __PAGE_OFFSET)>> PAGE_SHIFT)<< PAGE_SHIFT

/*
* Worst-case size of the kernel mapping we need to make:

NAK... this is in fact doubly wrong... the assembler handles this case
just fine -- it's a .S file after all -- and the result is wrong.

If you want to change it change it to 0x100000000.

-hpa

The funny thing is even if i change to 0x100000000 or 0 etc., the compiler warns me. Is this normal or a compiler issue?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/