[OE-core] [PATCH 1/2] armv8: update to use armv8-a tune

[akuster808]

On 07/31/2017 12:04 PM, Khem Raj wrote:
>
> On 7/31/17 10:51 AM, Mark Hatle wrote:
>> On 7/31/17 12:40 PM, akuster808 wrote:
>>>
>>> On 07/31/2017 10:31 AM, Mark Hatle wrote:
>>>> On 7/31/17 12:16 PM, Armin Kuster wrote:
>>>>> Signed-off-by: Armin Kuster <akuster808 at gmail.com>
>>>>> ---
>>>>>   meta/conf/machine/include/arm/arch-armv8.inc | 25 +++++++++++++++++++++++++
>>>>>   1 file changed, 25 insertions(+)
>>>>>
>>>>> diff --git a/meta/conf/machine/include/arm/arch-armv8.inc b/meta/conf/machine/include/arm/arch-armv8.inc
>>>>> index 5e832fa..dc1ba5e 100644
>>>>> --- a/meta/conf/machine/include/arm/arch-armv8.inc
>>>>> +++ b/meta/conf/machine/include/arm/arch-armv8.inc
>>>>> @@ -1 +1,26 @@
>>>>> +DEFAULTTUNE ?= "armv8-a"
>>>> do we want the '-a'?  The other arm (7) are of the format armv7a (no '-').
>>> works for me either way.
>>>
>>> While we are at it. How would we want ‘armv8.1-a’, ‘armv8.2-a’, ‘armv8.3-a'
>>> formated as?
>> My preference is to drop the '-'.  As for the '.', I'm not sure.. not something
>> we've run across before.
>>
>> We could just drop it (the '.'), but it really depends on if armv81a would
>> confuse someone (familiar with arm) or not.
> I would suggest to also sync  with other distros and ensure that we dont
> do something different. Its very costly later. Since applications get
> ported to most common combination
Sync what part? naming of file? ( sorry lost regarding your comment)

- armin
>
>> --Mark
>>
>>> - armin
>>>> --Mark
>>>>
>>>>> +
>>>>> +TUNEVALID[armv8-a] = "Enable instructions for ARMv8-a"
>>>>> +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'armv8-a', ' -march=armv8-a', '', d)}"
>>>>> +
>>>>> +TUNEVALID[crypto] = "Enable crypto instructions for ARMv8-a"
>>>>> +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'crypto', '+crypto', '', d)}"
>>>>> +
>>>>> +MACHINEOVERRIDES =. "${@bb.utils.contains('TUNE_FEATURES', 'armv8-a', 'armv8a:', '' ,d)}"
>>>>> +
>>>>>   require conf/machine/include/arm/arch-arm64.inc
>>>>> +
>>>>> +AVAILTUNES += "armv8-a armv8-a-crypto"
>>>>> +ARMPKGARCH_tune-armv8-a ?= "armv8-a"
>>>>> +
>>>>> +# Enable Crypto extension. This also enables Advanced SIMD and floating-point instructions
>>>>> +ARMPKGARCH_tune-armv8-a-crypto ?= "armv8-a"
>>>>> +
>>>>> +TUNE_FEATURES_tune-armv8-a = "aarch64 armv8-a"
>>>>> +TUNE_FEATURES_tune-armv8-a-crypto = "${TUNE_FEATURES_tune-armv8-a} crypto"
>>>>> +
>>>>> +PACKAGE_EXTRA_ARCHS_tune-armv8-a = "aarch64 armv8-a"
>>>>> +PACKAGE_EXTRA_ARCHS_tune-armv8-a-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8-a} armv8-a-crypto"
>>>>> +
>>>>> +BASE_LIB_tune-armv8-a = "${BASE_LIB_tune-aarch64}"
>>>>> +BASE_LIB_tune-armv8-a-crypto = "${BASE_LIB_tune-aarch64}"
>>>>>