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

ron minnich rminnich at gmail.com
Fri Nov 14 19:04:06 CET 2008


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.

ron




More information about the coreboot mailing list