[U-Boot] [PATCH 07/33] x86: Ensure that all relocation data is included in the image

Simon Glass sjg at chromium.org
Wed Nov 12 06:28:42 CET 2014


Hi Bin,

On 11 November 2014 22:21, Bin Meng <bmeng.cn at gmail.com> wrote:
> On Wed, Nov 12, 2014 at 8:17 AM, Simon Glass <sjg at chromium.org> wrote:
>> Some toolchains put the relocation data into separate sections. Adjust the
>> linker script to catch this case. Without relocation data, U-Boot will not
>> boot.
>>
>> Signed-off-by: Simon Glass <sjg at chromium.org>
>> ---
>>
>>  arch/x86/cpu/u-boot.lds | 4 +++-
>>  1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/x86/cpu/u-boot.lds b/arch/x86/cpu/u-boot.lds
>> index f48bff5..95ab7dd 100644
>> --- a/arch/x86/cpu/u-boot.lds
>> +++ b/arch/x86/cpu/u-boot.lds
>> @@ -44,7 +44,9 @@ SECTIONS
>>
>>         . = ALIGN(4);
>>         __rel_dyn_start = .;
>> -       .rel.dyn : { *(.rel.dyn) }
>> +       .rel.dyn : {
>> +               *(.rel.*)
>
> Should it be more generic to use *(.rel*) here?

I suspect that would be safe, but why?

>
>> +       }
>>         __rel_dyn_end = .;
>>         . = ALIGN(4);
>>         _end = .;
>> --

Regards,
Simon


More information about the U-Boot mailing list