FW: 1 GB lowmem

[Rune Torgersen]

Kumar Gala wrote:
> On May 21, 2008, at 4:24 PM, Rune Torgersen wrote:
> 
>> Kumar Gala wrote:
>>> On May 21, 2008, at 3:55 PM, Rune Torgersen wrote:
>>>> Argh... Found it. Had to set CONFIG_TASK_SIZE to 0x80000000. Now it
>>>> works in both vaniulla an d RT kernel.
>>> 
>>> We should really add some sanity check on CONFIG_TASK_SIZE vs
>>> KERNEL_START.
>> 
>> Something like this?
>> Wording sould probablyy be a bit different/cleare.
>> 
>> diff --git a/include/asm-powerpc/processor.h
>> b/include/asm-powerpc/processor.h
>> index fd98ca9..72e0e3f 100644
>> --- a/include/asm-powerpc/processor.h
>> +++ b/include/asm-powerpc/processor.h
>> @@ -81,6 +81,10 @@ extern struct task_struct *last_task_used_altivec;
>> extern struct task_struct *last_task_used_spe;
>> 
>> #ifdef CONFIG_PPC32
>> +
>> +#if CONFIG_TASK_SIZE > CONFIG_KERNEL_START
>> +#error User TASK_SIZE overlaps with KERNEL_START address
>> +#endif
>> #define TASK_SIZE      (CONFIG_TASK_SIZE)
>> 
>> /* This decides where the kernel will search for a free chunk of vm
>> 
> 
> something like that would be good.  is there really anything PPC32
> specific about it? 
> 
Probably not. Could this be done in the Kconfig file instead?. Don't
know if Kconfig supports doing a comparison.