On 14.11.2008 19:04, ron minnich wrote: > 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. >
Compilation may work, but RAMinit will fail during runtime if you do that. Regards, Carl-Daniel -- http://www.hailfinger.org/ -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
