[coreboot] r1026 - in coreboot-v3: mainboard/kontron/986lcd-m southbridge/intel/i82801gx

[Stefan Reinauer]

ron minnich wrote:
> On Fri, Nov 14, 2008 at 9:53 AM, Stefan Reinauer <stepan at coresystems.de> wrote:
>   
>> Carl-Daniel Hailfinger wrote:
>>     
>>> On 14.11.2008 18:14, svn at coreboot.org wrote:
>>>
>>>       
>>>> Author: rminnich
>>>> New Revision: 1026
>>>>
>>>> /home/rminnich/coreboot-v3/build/coreboot.initram_partiallylinked.o: section .data.rel.ro.local: dual_channel_slew_group_lookup.3242 single_channel_slew_group_lookup.3243
>>>>
>>>>
>>>>         
>>> Basic rule: If you want to have arrays of pointers in initram, you lose.
>>> Pointers are not relocatable by definition. const is not going to help
>>> you there.
>>>
>>>       
>> Hm. This is bad. 'nother regression in v3 that wasn't in v2.
>>
>>     
>
> it's not acceptable as a rule. We have to fix it one way or another.
> Why gcc is marking it as writeable is a puzzle but points to a bug
> somewhere.
>
>   

We could try and use indices instead of pointers. That's the easiest
rewrite i can think of to get the problem done without wasting the code.

This is clearly a gcc weakness. This stuff should not happen when
compiling PIC. That's what PIC is for.

-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.de  • http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866