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

Bin Meng bmeng.cn at gmail.com
Wed Nov 12 06:32:59 CET 2014


Hi Simon,

On Wed, Nov 12, 2014 at 1:28 PM, Simon Glass <sjg at chromium.org> wrote:
> 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?

Just in case some weird toolchain omit the "." afer "rel". Which
toolchain are you using now that exposes such behavior?

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

Regards,
Bin


More information about the U-Boot mailing list