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

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Fri Nov 14 19:37:34 CET 2008


On 14.11.2008 19:04, ron minnich wrote:
> On Fri, Nov 14, 2008 at 9:56 AM, ron minnich <rminnich at gmail.com> 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.
>>
>>     
>
> for now I'm fixing it with a section attribute.
>   

Compilation may work, but RAMinit will fail during runtime if you do that.

Regards,
Carl-Daniel

-- 
http://www.hailfinger.org/





More information about the coreboot mailing list