On Fri, Nov 14, 2008 at 9:56 AM, ron minnich <[EMAIL PROTECTED]> wrote: > On Fri, Nov 14, 2008 at 9:53 AM, Stefan Reinauer <[EMAIL PROTECTED]> wrote: >> Carl-Daniel Hailfinger wrote: >>> On 14.11.2008 18:14, [EMAIL PROTECTED] 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 -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
