[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openembedded-core
Subject:    Re: [OE-core] [PATCH 1/2] armv8: update to use armv8-a tune
From:       Khem Raj <raj.khem () gmail ! com>
Date:       2017-07-31 19:04:19
Message-ID: e0e65de3-e49a-b1af-9bd7-67a6d33ba859 () gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/signed)]

[Attachment #4 (multipart/mixed)]


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


["signature.asc" (application/pgp-signature)]

-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[prev in list] [next in list] [prev in thread] [next in thread]