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

[Khem Raj]

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

> 
> --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}"
>>>>
>>
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 163 bytes
Desc: OpenPGP digital signature
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20170731/1800fd19/attachment-0002.sig>