[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <4BA40864.3080204@googlemail.com>
Date: Sat, 20 Mar 2010 00:27:32 +0100
From: René Bolldorf <xsecute@...glemail.com>
To: "H. Peter Anvin" <hpa@...or.com>
CC: linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/5] arch/x86/kernel/head_32.S Warning: shift count out
of range (32 is not between 0 and 31)
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@...glemail.com>
>>
>> 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@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists