* [PATCH v3 0/5] Add tune for ARMv8 and some cortex processors
@ 2018-06-15 3:22 Randy Li
2018-06-15 3:22 ` [PATCH v3 1/5] arch-armv8a.inc: add tune include for armv8 Randy Li
` (4 more replies)
0 siblings, 5 replies; 10+ messages in thread
From: Randy Li @ 2018-06-15 3:22 UTC (permalink / raw)
To: openembedded-core
I mistake some thing wrong in the previous version, I have verified
this version and it works.
I have not found a way to support big little.
Randy Li (5):
arch-armv8a.inc: add tune include for armv8
tune-cortexa32: add tunes for ARM Cortex-A32
tune-cortexa35: add tunes for ARM Cortex-A35
tune-cortexa53: add tunes for ARM Cortex-A53
tune-cortexa72: add tunes for ARM Cortex-A72
meta/conf/machine/include/arm/arch-armv8.inc | 1 -
meta/conf/machine/include/arm/arch-armv8a.inc | 28 +++++++++++++++++++++++++++
meta/conf/machine/include/tune-cortexa32.inc | 15 ++++++++++++++
meta/conf/machine/include/tune-cortexa35.inc | 15 ++++++++++++++
meta/conf/machine/include/tune-cortexa53.inc | 15 ++++++++++++++
meta/conf/machine/include/tune-cortexa72.inc | 12 ++++++++++++
6 files changed, 85 insertions(+), 1 deletion(-)
delete mode 100644 meta/conf/machine/include/arm/arch-armv8.inc
create mode 100644 meta/conf/machine/include/arm/arch-armv8a.inc
create mode 100644 meta/conf/machine/include/tune-cortexa32.inc
create mode 100644 meta/conf/machine/include/tune-cortexa35.inc
create mode 100644 meta/conf/machine/include/tune-cortexa53.inc
create mode 100644 meta/conf/machine/include/tune-cortexa72.inc
--
2.14.4
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v3 1/5] arch-armv8a.inc: add tune include for armv8
2018-06-15 3:22 [PATCH v3 0/5] Add tune for ARMv8 and some cortex processors Randy Li
@ 2018-06-15 3:22 ` Randy Li
2018-06-15 9:57 ` Nicolas Dechesne
2018-06-27 5:13 ` Khem Raj
2018-06-15 3:22 ` [PATCH v3 2/5] tune-cortexa32: add tunes for ARM Cortex-A32 Randy Li
` (3 subsequent siblings)
4 siblings, 2 replies; 10+ messages in thread
From: Randy Li @ 2018-06-15 3:22 UTC (permalink / raw)
To: openembedded-core
There are some addtional instructions apart from bare armv8,
also there is armv8.1, armv8.2.
Most the processor would support crc, except X-gene 1.
Signed-off-by: Randy Li <ayaka@soulik.info>
---
meta/conf/machine/include/arm/arch-armv8.inc | 1 -
meta/conf/machine/include/arm/arch-armv8a.inc | 28 +++++++++++++++++++++++++++
2 files changed, 28 insertions(+), 1 deletion(-)
delete mode 100644 meta/conf/machine/include/arm/arch-armv8.inc
create mode 100644 meta/conf/machine/include/arm/arch-armv8a.inc
diff --git a/meta/conf/machine/include/arm/arch-armv8.inc b/meta/conf/machine/include/arm/arch-armv8.inc
deleted file mode 100644
index 5e832fae6d..0000000000
--- a/meta/conf/machine/include/arm/arch-armv8.inc
+++ /dev/null
@@ -1 +0,0 @@
-require conf/machine/include/arm/arch-arm64.inc
diff --git a/meta/conf/machine/include/arm/arch-armv8a.inc b/meta/conf/machine/include/arm/arch-armv8a.inc
new file mode 100644
index 0000000000..323d0d7f0f
--- /dev/null
+++ b/meta/conf/machine/include/arm/arch-armv8a.inc
@@ -0,0 +1,28 @@
+DEFAULTTUNE ?= "armv8a-crc"
+
+TUNEVALID[armv8a] = "Enable instructions for ARMv8-a"
+TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'armv8a', ' -march=armv8-a', '', d)}"
+TUNEVALID[simd] = "Enable instructions for ARMv8-a Advanced SIMD and floating-point"
+TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'simd', '+simd', '', d)}"
+TUNEVALID[crc] = "Enable instructions for ARMv8-a Cyclic Redundancy Check (CRC)"
+TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'crc', '+crc', '', d)}"
+TUNEVALID[crypto] = "Enable instructions for ARMv8-a cryptographic"
+TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'crypto', '+crypto', '', d)}"
+MACHINEOVERRIDES =. "${@bb.utils.contains('TUNE_FEATURES', 'armv8a', 'armv8a:', '' ,d)}"
+
+require conf/machine/include/arm/arch-arm64.inc
+
+# Little Endian base configs
+AVAILTUNES += "armv8a armv8a-crc armv8a-crc-crypto armv8a-crypto"
+ARMPKGARCH_tune-armv8a ?= "armv8a"
+ARMPKGARCH_tune-armv8a-crc ?= "armv8a"
+ARMPKGARCH_tune-armv8a-crypto ?= "armv8a"
+ARMPKGARCH_tune-armv8a-crc-crypto ?= "armv8a"
+TUNE_FEATURES_tune-armv8a = "aarch64 armv8a simd"
+TUNE_FEATURES_tune-armv8a-crc = "${TUNE_FEATURES_tune-armv8a} crc"
+TUNE_FEATURES_tune-armv8a-crypto = "${TUNE_FEATURES_tune-armv8a} crypto"
+TUNE_FEATURES_tune-armv8a-crc-crypto = "${TUNE_FEATURES_tune-armv8a-crc} crypto"
+PACKAGE_EXTRA_ARCHS_tune-armv8a = "aarch64 armv8a simd"
+PACKAGE_EXTRA_ARCHS_tune-armv8a-crc = "${PACKAGE_EXTRA_ARCHS_tune-armv8a} crc"
+PACKAGE_EXTRA_ARCHS_tune-armv8a-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a} crypto"
+PACKAGE_EXTRA_ARCHS_tune-armv8a-crc-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc} crypto"
--
2.14.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v3 2/5] tune-cortexa32: add tunes for ARM Cortex-A32
2018-06-15 3:22 [PATCH v3 0/5] Add tune for ARMv8 and some cortex processors Randy Li
2018-06-15 3:22 ` [PATCH v3 1/5] arch-armv8a.inc: add tune include for armv8 Randy Li
@ 2018-06-15 3:22 ` Randy Li
2018-06-15 3:22 ` [PATCH v3 3/5] tune-cortexa35: add tunes for ARM Cortex-A35 Randy Li
` (2 subsequent siblings)
4 siblings, 0 replies; 10+ messages in thread
From: Randy Li @ 2018-06-15 3:22 UTC (permalink / raw)
To: openembedded-core
https://developer.arm.com/products/processors/cortex-a/cortex-a32
Signed-off-by: Randy Li <ayaka@soulik.info>
---
meta/conf/machine/include/tune-cortexa32.inc | 15 +++++++++++++++
1 file changed, 15 insertions(+)
create mode 100644 meta/conf/machine/include/tune-cortexa32.inc
diff --git a/meta/conf/machine/include/tune-cortexa32.inc b/meta/conf/machine/include/tune-cortexa32.inc
new file mode 100644
index 0000000000..22f303fae9
--- /dev/null
+++ b/meta/conf/machine/include/tune-cortexa32.inc
@@ -0,0 +1,15 @@
+DEFAULTTUNE ?= "cortexa32"
+
+require conf/machine/include/arm/arch-armv8a.inc
+
+TUNEVALID[cortexa32] = "Enable Cortex-A32 specific processor optimizations"
+TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'cortexa32', ' -mcpu=cortex-a32', '', d)}"
+
+# Little Endian base configs
+AVAILTUNES += "cortexa32 cortexa32-crypto"
+ARMPKGARCH_tune-cortexa32 = "cortexa32"
+ARMPKGARCH_tune-cortexa32-crypto = "cortexa32"
+TUNE_FEATURES_tune-cortexa32 = "${TUNE_FEATURES_tune-armv8a-crc} cortexa32"
+TUNE_FEATURES_tune-cortexa32-crypto = "${TUNE_FEATURES_tune-armv8a-crc-crypto} cortexa32"
+PACKAGE_EXTRA_ARCHS_tune-cortexa32 = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc} cortexa32"
+PACKAGE_EXTRA_ARCHS_tune-cortexa32-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc-crypto} cortexa32 cortexa32-crypto"
--
2.14.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v3 3/5] tune-cortexa35: add tunes for ARM Cortex-A35
2018-06-15 3:22 [PATCH v3 0/5] Add tune for ARMv8 and some cortex processors Randy Li
2018-06-15 3:22 ` [PATCH v3 1/5] arch-armv8a.inc: add tune include for armv8 Randy Li
2018-06-15 3:22 ` [PATCH v3 2/5] tune-cortexa32: add tunes for ARM Cortex-A32 Randy Li
@ 2018-06-15 3:22 ` Randy Li
2018-06-15 3:22 ` [PATCH v3 4/5] tune-cortexa53: add tunes for ARM Cortex-A53 Randy Li
2018-06-15 3:22 ` [PATCH v3 5/5] tune-cortexa72: add tunes for ARM Cortex-A72 Randy Li
4 siblings, 0 replies; 10+ messages in thread
From: Randy Li @ 2018-06-15 3:22 UTC (permalink / raw)
To: openembedded-core
https://developer.arm.com/products/processors/cortex-a/cortex-a35
Signed-off-by: Randy Li <ayaka@soulik.info>
---
meta/conf/machine/include/tune-cortexa35.inc | 15 +++++++++++++++
1 file changed, 15 insertions(+)
create mode 100644 meta/conf/machine/include/tune-cortexa35.inc
diff --git a/meta/conf/machine/include/tune-cortexa35.inc b/meta/conf/machine/include/tune-cortexa35.inc
new file mode 100644
index 0000000000..535a8aef15
--- /dev/null
+++ b/meta/conf/machine/include/tune-cortexa35.inc
@@ -0,0 +1,15 @@
+DEFAULTTUNE ?= "cortexa35"
+
+require conf/machine/include/arm/arch-armv8a.inc
+
+TUNEVALID[cortexa35] = "Enable Cortex-A35 specific processor optimizations"
+TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'cortexa35', ' -mcpu=cortex-a35', '', d)}"
+
+# Little Endian base configs
+AVAILTUNES += "cortexa35 cortexa35-crypto"
+ARMPKGARCH_tune-cortexa35 = "cortexa35"
+ARMPKGARCH_tune-cortexa35-crypto = "cortexa35"
+TUNE_FEATURES_tune-cortexa35 = "${TUNE_FEATURES_tune-armv8a-crc} cortexa35"
+TUNE_FEATURES_tune-cortexa35-crypto = "${TUNE_FEATURES_tune-armv8a-crc-crypto} cortexa35"
+PACKAGE_EXTRA_ARCHS_tune-cortexa35 = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc} cortexa35"
+PACKAGE_EXTRA_ARCHS_tune-cortexa35-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc-crypto} cortexa35 cortexa35-crypto"
--
2.14.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v3 4/5] tune-cortexa53: add tunes for ARM Cortex-A53
2018-06-15 3:22 [PATCH v3 0/5] Add tune for ARMv8 and some cortex processors Randy Li
` (2 preceding siblings ...)
2018-06-15 3:22 ` [PATCH v3 3/5] tune-cortexa35: add tunes for ARM Cortex-A35 Randy Li
@ 2018-06-15 3:22 ` Randy Li
2018-06-15 3:22 ` [PATCH v3 5/5] tune-cortexa72: add tunes for ARM Cortex-A72 Randy Li
4 siblings, 0 replies; 10+ messages in thread
From: Randy Li @ 2018-06-15 3:22 UTC (permalink / raw)
To: openembedded-core
It looks that the Cryptography engine is mandatory in this
platform.
https://developer.arm.com/products/processors/cortex-a/cortex-a53
Signed-off-by: Randy Li <ayaka@soulik.info>
---
meta/conf/machine/include/tune-cortexa53.inc | 15 +++++++++++++++
1 file changed, 15 insertions(+)
create mode 100644 meta/conf/machine/include/tune-cortexa53.inc
diff --git a/meta/conf/machine/include/tune-cortexa53.inc b/meta/conf/machine/include/tune-cortexa53.inc
new file mode 100644
index 0000000000..050a735d8e
--- /dev/null
+++ b/meta/conf/machine/include/tune-cortexa53.inc
@@ -0,0 +1,15 @@
+DEFAULTTUNE ?= "cortexa53"
+
+require conf/machine/include/arm/arch-armv8a.inc
+
+TUNEVALID[cortexa53] = "Enable Cortex-A53 specific processor optimizations"
+TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'cortexa53', ' -mcpu=cortex-a53', '', d)}"
+
+# Little Endian base configs
+AVAILTUNES += "cortexa53 cortexa53-crypto"
+ARMPKGARCH_tune-cortexa53 = "cortexa53"
+ARMPKGARCH_tune-cortexa53-crypto = "cortexa53"
+TUNE_FEATURES_tune-cortexa53 = "${TUNE_FEATURES_tune-armv8a-crc} cortexa53"
+TUNE_FEATURES_tune-cortexa53-crypto = "${TUNE_FEATURES_tune-armv8a-crc-crypto} cortexa53"
+PACKAGE_EXTRA_ARCHS_tune-cortexa53 = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc} cortexa53"
+PACKAGE_EXTRA_ARCHS_tune-cortexa53-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc-crypto} cortexa53 cortexa53-crypto"
--
2.14.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v3 5/5] tune-cortexa72: add tunes for ARM Cortex-A72
2018-06-15 3:22 [PATCH v3 0/5] Add tune for ARMv8 and some cortex processors Randy Li
` (3 preceding siblings ...)
2018-06-15 3:22 ` [PATCH v3 4/5] tune-cortexa53: add tunes for ARM Cortex-A53 Randy Li
@ 2018-06-15 3:22 ` Randy Li
4 siblings, 0 replies; 10+ messages in thread
From: Randy Li @ 2018-06-15 3:22 UTC (permalink / raw)
To: openembedded-core
It looks that the Cryptography engine is mandatory in this
platform.
https://developer.arm.com/products/processors/cortex-a/cortex-a72
Signed-off-by: Randy Li <ayaka@soulik.info>
---
meta/conf/machine/include/tune-cortexa72.inc | 12 ++++++++++++
1 file changed, 12 insertions(+)
create mode 100644 meta/conf/machine/include/tune-cortexa72.inc
diff --git a/meta/conf/machine/include/tune-cortexa72.inc b/meta/conf/machine/include/tune-cortexa72.inc
new file mode 100644
index 0000000000..8055492b8e
--- /dev/null
+++ b/meta/conf/machine/include/tune-cortexa72.inc
@@ -0,0 +1,12 @@
+DEFAULTTUNE ?= "cortexa72"
+
+require conf/machine/include/arm/arch-armv8a.inc
+
+TUNEVALID[cortexa72] = "Enable Cortex-A72 specific processor optimizations"
+TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'cortexa72', ' -mcpu=cortex-a72', '', d)}"
+
+# Little Endian base configs
+AVAILTUNES += "cortexa72"
+ARMPKGARCH_tune-cortexa72 = "cortexa72"
+TUNE_FEATURES_tune-cortexa72 = "${TUNE_FEATURES_tune-armv8a-crc-crypto} cortexa72"
+PACKAGE_EXTRA_ARCHS_tune-cortexa72 = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc-crypto} cortexa72"
--
2.14.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v3 1/5] arch-armv8a.inc: add tune include for armv8
2018-06-15 3:22 ` [PATCH v3 1/5] arch-armv8a.inc: add tune include for armv8 Randy Li
@ 2018-06-15 9:57 ` Nicolas Dechesne
2018-06-15 16:00 ` ayaka
2018-06-21 13:49 ` Otavio Salvador
2018-06-27 5:13 ` Khem Raj
1 sibling, 2 replies; 10+ messages in thread
From: Nicolas Dechesne @ 2018-06-15 9:57 UTC (permalink / raw)
To: Randy Li; +Cc: Patches and discussions about the oe-core layer
On Fri, Jun 15, 2018 at 5:22 AM, Randy Li <ayaka@soulik.info> wrote:
> There are some addtional instructions apart from bare armv8,
> also there is armv8.1, armv8.2.
>
> Most the processor would support crc, except X-gene 1.
>
> Signed-off-by: Randy Li <ayaka@soulik.info>
> ---
> meta/conf/machine/include/arm/arch-armv8.inc | 1 -
> meta/conf/machine/include/arm/arch-armv8a.inc | 28 +++++++++++++++++++++++++++
> 2 files changed, 28 insertions(+), 1 deletion(-)
> delete mode 100644 meta/conf/machine/include/arm/arch-armv8.inc
My previous comment still applies here. You can't just delete this
file. It's used by many BSP. and it's even used in oe-core itself:
layers/openembedded-core/meta/conf/machine/include/tune-thunderx.inc:require
conf/machine/include/arm/arch-armv8.inc
layers/openembedded-core/meta/conf/machine/qemuarm64.conf:require
conf/machine/include/arm/arch-armv8.inc
> create mode 100644 meta/conf/machine/include/arm/arch-armv8a.inc
>
> diff --git a/meta/conf/machine/include/arm/arch-armv8.inc b/meta/conf/machine/include/arm/arch-armv8.inc
> deleted file mode 100644
> index 5e832fae6d..0000000000
> --- a/meta/conf/machine/include/arm/arch-armv8.inc
> +++ /dev/null
> @@ -1 +0,0 @@
> -require conf/machine/include/arm/arch-arm64.inc
> diff --git a/meta/conf/machine/include/arm/arch-armv8a.inc b/meta/conf/machine/include/arm/arch-armv8a.inc
> new file mode 100644
> index 0000000000..323d0d7f0f
> --- /dev/null
> +++ b/meta/conf/machine/include/arm/arch-armv8a.inc
> @@ -0,0 +1,28 @@
> +DEFAULTTUNE ?= "armv8a-crc"
> +
> +TUNEVALID[armv8a] = "Enable instructions for ARMv8-a"
> +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'armv8a', ' -march=armv8-a', '', d)}"
> +TUNEVALID[simd] = "Enable instructions for ARMv8-a Advanced SIMD and floating-point"
> +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'simd', '+simd', '', d)}"
> +TUNEVALID[crc] = "Enable instructions for ARMv8-a Cyclic Redundancy Check (CRC)"
> +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'crc', '+crc', '', d)}"
> +TUNEVALID[crypto] = "Enable instructions for ARMv8-a cryptographic"
> +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'crypto', '+crypto', '', d)}"
> +MACHINEOVERRIDES =. "${@bb.utils.contains('TUNE_FEATURES', 'armv8a', 'armv8a:', '' ,d)}"
> +
> +require conf/machine/include/arm/arch-arm64.inc
> +
> +# Little Endian base configs
> +AVAILTUNES += "armv8a armv8a-crc armv8a-crc-crypto armv8a-crypto"
> +ARMPKGARCH_tune-armv8a ?= "armv8a"
> +ARMPKGARCH_tune-armv8a-crc ?= "armv8a"
> +ARMPKGARCH_tune-armv8a-crypto ?= "armv8a"
> +ARMPKGARCH_tune-armv8a-crc-crypto ?= "armv8a"
> +TUNE_FEATURES_tune-armv8a = "aarch64 armv8a simd"
> +TUNE_FEATURES_tune-armv8a-crc = "${TUNE_FEATURES_tune-armv8a} crc"
> +TUNE_FEATURES_tune-armv8a-crypto = "${TUNE_FEATURES_tune-armv8a} crypto"
> +TUNE_FEATURES_tune-armv8a-crc-crypto = "${TUNE_FEATURES_tune-armv8a-crc} crypto"
> +PACKAGE_EXTRA_ARCHS_tune-armv8a = "aarch64 armv8a simd"
> +PACKAGE_EXTRA_ARCHS_tune-armv8a-crc = "${PACKAGE_EXTRA_ARCHS_tune-armv8a} crc"
> +PACKAGE_EXTRA_ARCHS_tune-armv8a-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a} crypto"
> +PACKAGE_EXTRA_ARCHS_tune-armv8a-crc-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc} crypto"
> --
> 2.14.4
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3 1/5] arch-armv8a.inc: add tune include for armv8
2018-06-15 9:57 ` Nicolas Dechesne
@ 2018-06-15 16:00 ` ayaka
2018-06-21 13:49 ` Otavio Salvador
1 sibling, 0 replies; 10+ messages in thread
From: ayaka @ 2018-06-15 16:00 UTC (permalink / raw)
To: Nicolas Dechesne; +Cc: Patches and discussions about the oe-core layer
On 06/15/2018 05:57 PM, Nicolas Dechesne wrote:
> On Fri, Jun 15, 2018 at 5:22 AM, Randy Li <ayaka@soulik.info> wrote:
>> There are some addtional instructions apart from bare armv8,
>> also there is armv8.1, armv8.2.
>>
>> Most the processor would support crc, except X-gene 1.
>>
>> Signed-off-by: Randy Li <ayaka@soulik.info>
>> ---
>> meta/conf/machine/include/arm/arch-armv8.inc | 1 -
>> meta/conf/machine/include/arm/arch-armv8a.inc | 28 +++++++++++++++++++++++++++
>> 2 files changed, 28 insertions(+), 1 deletion(-)
>> delete mode 100644 meta/conf/machine/include/arm/arch-armv8.inc
> My previous comment still applies here. You can't just delete this
> file. It's used by many BSP. and it's even used in oe-core itself:
>
> layers/openembedded-core/meta/conf/machine/include/tune-thunderx.inc:require
> conf/machine/include/arm/arch-armv8.inc
> layers/openembedded-core/meta/conf/machine/qemuarm64.conf:require
> conf/machine/include/arm/arch-armv8.inc
I am very sorry about, should I modify those files the same time?
Since some cortex-m processors also support armv8 instructions, it would
be used to run Linux in the
future.
Also there would be armv81a, armv82a for the optimization of the
armv8.1, armv8.2.
That is why I want to rename it.
Apart from that ,would you pleased with the contents below?
>
>> create mode 100644 meta/conf/machine/include/arm/arch-armv8a.inc
>>
>> diff --git a/meta/conf/machine/include/arm/arch-armv8.inc b/meta/conf/machine/include/arm/arch-armv8.inc
>> deleted file mode 100644
>> index 5e832fae6d..0000000000
>> --- a/meta/conf/machine/include/arm/arch-armv8.inc
>> +++ /dev/null
>> @@ -1 +0,0 @@
>> -require conf/machine/include/arm/arch-arm64.inc
>> diff --git a/meta/conf/machine/include/arm/arch-armv8a.inc b/meta/conf/machine/include/arm/arch-armv8a.inc
>> new file mode 100644
>> index 0000000000..323d0d7f0f
>> --- /dev/null
>> +++ b/meta/conf/machine/include/arm/arch-armv8a.inc
>> @@ -0,0 +1,28 @@
>> +DEFAULTTUNE ?= "armv8a-crc"
>> +
>> +TUNEVALID[armv8a] = "Enable instructions for ARMv8-a"
>> +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'armv8a', ' -march=armv8-a', '', d)}"
>> +TUNEVALID[simd] = "Enable instructions for ARMv8-a Advanced SIMD and floating-point"
>> +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'simd', '+simd', '', d)}"
>> +TUNEVALID[crc] = "Enable instructions for ARMv8-a Cyclic Redundancy Check (CRC)"
>> +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'crc', '+crc', '', d)}"
>> +TUNEVALID[crypto] = "Enable instructions for ARMv8-a cryptographic"
>> +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'crypto', '+crypto', '', d)}"
>> +MACHINEOVERRIDES =. "${@bb.utils.contains('TUNE_FEATURES', 'armv8a', 'armv8a:', '' ,d)}"
>> +
>> +require conf/machine/include/arm/arch-arm64.inc
>> +
>> +# Little Endian base configs
>> +AVAILTUNES += "armv8a armv8a-crc armv8a-crc-crypto armv8a-crypto"
>> +ARMPKGARCH_tune-armv8a ?= "armv8a"
>> +ARMPKGARCH_tune-armv8a-crc ?= "armv8a"
>> +ARMPKGARCH_tune-armv8a-crypto ?= "armv8a"
>> +ARMPKGARCH_tune-armv8a-crc-crypto ?= "armv8a"
>> +TUNE_FEATURES_tune-armv8a = "aarch64 armv8a simd"
>> +TUNE_FEATURES_tune-armv8a-crc = "${TUNE_FEATURES_tune-armv8a} crc"
>> +TUNE_FEATURES_tune-armv8a-crypto = "${TUNE_FEATURES_tune-armv8a} crypto"
>> +TUNE_FEATURES_tune-armv8a-crc-crypto = "${TUNE_FEATURES_tune-armv8a-crc} crypto"
>> +PACKAGE_EXTRA_ARCHS_tune-armv8a = "aarch64 armv8a simd"
>> +PACKAGE_EXTRA_ARCHS_tune-armv8a-crc = "${PACKAGE_EXTRA_ARCHS_tune-armv8a} crc"
>> +PACKAGE_EXTRA_ARCHS_tune-armv8a-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a} crypto"
>> +PACKAGE_EXTRA_ARCHS_tune-armv8a-crc-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc} crypto"
>> --
>> 2.14.4
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3 1/5] arch-armv8a.inc: add tune include for armv8
2018-06-15 9:57 ` Nicolas Dechesne
2018-06-15 16:00 ` ayaka
@ 2018-06-21 13:49 ` Otavio Salvador
1 sibling, 0 replies; 10+ messages in thread
From: Otavio Salvador @ 2018-06-21 13:49 UTC (permalink / raw)
To: Nicolas Dechesne; +Cc: Patches and discussions about the oe-core layer
On Fri, Jun 15, 2018 at 6:57 AM, Nicolas Dechesne
<nicolas.dechesne@linaro.org> wrote:
> On Fri, Jun 15, 2018 at 5:22 AM, Randy Li <ayaka@soulik.info> wrote:
>> There are some addtional instructions apart from bare armv8,
>> also there is armv8.1, armv8.2.
>>
>> Most the processor would support crc, except X-gene 1.
>>
>> Signed-off-by: Randy Li <ayaka@soulik.info>
>> ---
>> meta/conf/machine/include/arm/arch-armv8.inc | 1 -
>> meta/conf/machine/include/arm/arch-armv8a.inc | 28 +++++++++++++++++++++++++++
>> 2 files changed, 28 insertions(+), 1 deletion(-)
>> delete mode 100644 meta/conf/machine/include/arm/arch-armv8.inc
>
> My previous comment still applies here. You can't just delete this
> file. It's used by many BSP. and it's even used in oe-core itself:
>
> layers/openembedded-core/meta/conf/machine/include/tune-thunderx.inc:require
> conf/machine/include/arm/arch-armv8.inc
> layers/openembedded-core/meta/conf/machine/qemuarm64.conf:require
> conf/machine/include/arm/arch-armv8.inc
I think the rename is fine (of course, considering it handles the
rename inside whole oe-core).
Breaking existing BSP in new release is fine. People needs to handle
it when upgrading the branches.
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3 1/5] arch-armv8a.inc: add tune include for armv8
2018-06-15 3:22 ` [PATCH v3 1/5] arch-armv8a.inc: add tune include for armv8 Randy Li
2018-06-15 9:57 ` Nicolas Dechesne
@ 2018-06-27 5:13 ` Khem Raj
1 sibling, 0 replies; 10+ messages in thread
From: Khem Raj @ 2018-06-27 5:13 UTC (permalink / raw)
To: Randy Li, openembedded-core
[-- Attachment #1.1: Type: text/plain, Size: 3584 bytes --]
On 6/14/18 8:22 PM, Randy Li wrote:
> There are some addtional instructions apart from bare armv8,
> also there is armv8.1, armv8.2.
>
> Most the processor would support crc, except X-gene 1.
I have a concern with this approach, since its mirroring the prior arm
art, I think for v8 it would be better to consolidate it around ISA defs
e.g. armv8.4-a, armv8.3-a, armv8.2-a and armv8.1-a and so on.
Moreover current approach is only considering a subset of aarch64
feature modifiers, if we want to go this route we should consider them
all as listed here
https://gcc.gnu.org/onlinedocs/gcc/AArch64-Options.html#aarch64-feature-modifiers
if there is one outlier then its better to address that in machine or
soc config instead.
>
> Signed-off-by: Randy Li <ayaka@soulik.info>
> ---
> meta/conf/machine/include/arm/arch-armv8.inc | 1 -
> meta/conf/machine/include/arm/arch-armv8a.inc | 28 +++++++++++++++++++++++++++
> 2 files changed, 28 insertions(+), 1 deletion(-)
> delete mode 100644 meta/conf/machine/include/arm/arch-armv8.inc
> create mode 100644 meta/conf/machine/include/arm/arch-armv8a.inc
>
> diff --git a/meta/conf/machine/include/arm/arch-armv8.inc b/meta/conf/machine/include/arm/arch-armv8.inc
> deleted file mode 100644
> index 5e832fae6d..0000000000
> --- a/meta/conf/machine/include/arm/arch-armv8.inc
> +++ /dev/null
> @@ -1 +0,0 @@
> -require conf/machine/include/arm/arch-arm64.inc
> diff --git a/meta/conf/machine/include/arm/arch-armv8a.inc b/meta/conf/machine/include/arm/arch-armv8a.inc
> new file mode 100644
> index 0000000000..323d0d7f0f
> --- /dev/null
> +++ b/meta/conf/machine/include/arm/arch-armv8a.inc
> @@ -0,0 +1,28 @@
> +DEFAULTTUNE ?= "armv8a-crc"
> +
> +TUNEVALID[armv8a] = "Enable instructions for ARMv8-a"
> +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'armv8a', ' -march=armv8-a', '', d)}"
> +TUNEVALID[simd] = "Enable instructions for ARMv8-a Advanced SIMD and floating-point"
> +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'simd', '+simd', '', d)}"
> +TUNEVALID[crc] = "Enable instructions for ARMv8-a Cyclic Redundancy Check (CRC)"
> +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'crc', '+crc', '', d)}"
> +TUNEVALID[crypto] = "Enable instructions for ARMv8-a cryptographic"
> +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'crypto', '+crypto', '', d)}"
> +MACHINEOVERRIDES =. "${@bb.utils.contains('TUNE_FEATURES', 'armv8a', 'armv8a:', '' ,d)}"
> +
> +require conf/machine/include/arm/arch-arm64.inc
> +
> +# Little Endian base configs
> +AVAILTUNES += "armv8a armv8a-crc armv8a-crc-crypto armv8a-crypto"
> +ARMPKGARCH_tune-armv8a ?= "armv8a"
> +ARMPKGARCH_tune-armv8a-crc ?= "armv8a"
> +ARMPKGARCH_tune-armv8a-crypto ?= "armv8a"
> +ARMPKGARCH_tune-armv8a-crc-crypto ?= "armv8a"
> +TUNE_FEATURES_tune-armv8a = "aarch64 armv8a simd"
> +TUNE_FEATURES_tune-armv8a-crc = "${TUNE_FEATURES_tune-armv8a} crc"
> +TUNE_FEATURES_tune-armv8a-crypto = "${TUNE_FEATURES_tune-armv8a} crypto"
> +TUNE_FEATURES_tune-armv8a-crc-crypto = "${TUNE_FEATURES_tune-armv8a-crc} crypto"
> +PACKAGE_EXTRA_ARCHS_tune-armv8a = "aarch64 armv8a simd"
> +PACKAGE_EXTRA_ARCHS_tune-armv8a-crc = "${PACKAGE_EXTRA_ARCHS_tune-armv8a} crc"
> +PACKAGE_EXTRA_ARCHS_tune-armv8a-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a} crypto"
> +PACKAGE_EXTRA_ARCHS_tune-armv8a-crc-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc} crypto"
>
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 201 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2018-06-27 5:13 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-15 3:22 [PATCH v3 0/5] Add tune for ARMv8 and some cortex processors Randy Li
2018-06-15 3:22 ` [PATCH v3 1/5] arch-armv8a.inc: add tune include for armv8 Randy Li
2018-06-15 9:57 ` Nicolas Dechesne
2018-06-15 16:00 ` ayaka
2018-06-21 13:49 ` Otavio Salvador
2018-06-27 5:13 ` Khem Raj
2018-06-15 3:22 ` [PATCH v3 2/5] tune-cortexa32: add tunes for ARM Cortex-A32 Randy Li
2018-06-15 3:22 ` [PATCH v3 3/5] tune-cortexa35: add tunes for ARM Cortex-A35 Randy Li
2018-06-15 3:22 ` [PATCH v3 4/5] tune-cortexa53: add tunes for ARM Cortex-A53 Randy Li
2018-06-15 3:22 ` [PATCH v3 5/5] tune-cortexa72: add tunes for ARM Cortex-A72 Randy Li
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.