[Skiboot] [PATCH] build: Fix linker script for builtin Kernel

Klaus Heinrich Kiwi klaus at linux.vnet.ibm.com
Fri Dec 11 22:18:30 AEDT 2020



On 12/10/2020 9:47 PM, Klaus Heinrich Kiwi wrote:
> 
> 
> On 12/10/2020 6:00 PM, Klaus Heinrich Kiwi wrote:
> 
>>       /* Optional kernel image */
>>          . = ALIGN(PAGE_SIZE);
>>          .builtin_kernel : {
>> @@ -243,6 +241,8 @@ SECTIONS
>>           __builtin_kernel_end = .;
>>       }
>>
>> +    DEBUG_SECTIONS
>> +
> 
> FYI, an IBM colleague from the compiler team (Segher) mentioned we could add (NOLOAD)
> to the sections undere DEBUG_SECTIONS macro instead, and that could save us some
> memory as well..
> 
> I'll test this tomorrow and send a V2 if successful.

I did a few tests with the (NOLOAD) for the debug sections, but that alone wasn't sufficient
to fix original issue.. additionally, I couldn't note any memory / size differences in the final
binary.

So unless Segher has other suggestions, I think we will need the patch above after all.

Thanks,


>   -Klaus

-- 
Klaus Heinrich Kiwi <klaus at linux.vnet.ibm.com>


More information about the Skiboot mailing list