Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1514123

Re: [PATCH] arm64: errata: Check for --fix-cortex-a53-843419 and --fix-cortex-a53

Show all headers | View raw

On 2 November 2016 at 14:03, Will Deacon <will.deacon@arm.com> wrote:
> On Mon, Oct 31, 2016 at 12:44:14PM -0700, Markus Mayer wrote:
>> From: Markus Mayer <mmayer@broadcom.com>
>>
>> The new errata check leads to a warning with some older versions of the
>> linker that do know how to work around the errata, but still use the
>> original name of the command line option: --fix-cortex-a53. The commit
>> in question that changed the name of the option can be found at [1].
>> It looks like only "gold" is affected by this rename. Traditional "ld"
>> isn't. (There, the argument was always called --fix-cortex-a53-843419.)
>>
>> To allow older versions of gold to properly handle the erratum if they
>> can, check whether ld supports the old name of this option in addition
>> to checking the new one.
>>
>> [1] https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commitdiff;h=7a2a1c793578a8468604e661dda025ecb8d0bd20;hp=cfbf0e3c5b637d66b2b1aeadecae9c187b825b2f
>>
>> Signed-off-by: Markus Mayer <mmayer@broadcom.com>
>
> If newer versions of gold accept the correct option name, why do we care?

Because Documentation/Changes states that the minimum requirement for
binutils is 2.12. Right now, that is not really true. And not
everybody can always use the newest toolchain, for various reasons.

The question I am asking is: What do we have to lose by supporting both options?

Thanks,
-Markus

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] arm64: errata: Check for --fix-cortex-a53-843419 and --fix-cortex-a53 Markus Mayer <code@mmayer.net> - 2016-10-31 20:50 +0100
  Re: [PATCH] arm64: errata: Check for --fix-cortex-a53-843419 and --fix-cortex-a53 Markus Mayer <markus.mayer@broadcom.com> - 2016-11-02 22:10 +0100
    Re: [PATCH] arm64: errata: Check for --fix-cortex-a53-843419 and  --fix-cortex-a53 Will Deacon <will.deacon@arm.com> - 2016-11-02 22:30 +0100
      Re: [PATCH] arm64: errata: Check for --fix-cortex-a53-843419 and --fix-cortex-a53 Markus Mayer <markus.mayer@broadcom.com> - 2016-11-02 22:50 +0100
        Re: [PATCH] arm64: errata: Check for --fix-cortex-a53-843419 and  --fix-cortex-a53 Florian Fainelli <f.fainelli@gmail.com> - 2016-11-02 23:00 +0100
          Re: [PATCH] arm64: errata: Check for --fix-cortex-a53-843419 and  --fix-cortex-a53 Will Deacon <will.deacon@arm.com> - 2016-11-03 15:20 +0100
            Re: [PATCH] arm64: errata: Check for --fix-cortex-a53-843419 and  --fix-cortex-a53 Florian Fainelli <f.fainelli@gmail.com> - 2016-11-03 18:30 +0100
  Re: [PATCH] arm64: errata: Check for --fix-cortex-a53-843419 and  --fix-cortex-a53 Will Deacon <will.deacon@arm.com> - 2016-11-02 22:10 +0100

csiph-web