* [PATCH v2 0/1] Add a tune file for cortexa53, valid for 32 bits and 64 bits
@ 2016-11-19 12:50 Herve Jourdain
2016-11-19 12:50 ` [PATCH v2 1/1] " Herve Jourdain
2016-11-20 8:07 ` [PATCH v2 0/1] " Herve Jourdain
0 siblings, 2 replies; 5+ messages in thread
From: Herve Jourdain @ 2016-11-19 12:50 UTC (permalink / raw)
To: openembedded-core
Tune file for ARM Cortex-A53, supporting 32 bits and 64 bits configuration
Herve Jourdain (1):
Add a tune file for cortexa53, valid for 32 bits and 64 bits
meta/conf/machine/include/tune-cortexa53.inc | 22 ++++++++++++++++++++++
1 file changed, 22 insertions(+)
create mode 100644 meta/conf/machine/include/tune-cortexa53.inc
--
2.7.4
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v2 1/1] Add a tune file for cortexa53, valid for 32 bits and 64 bits
2016-11-19 12:50 [PATCH v2 0/1] Add a tune file for cortexa53, valid for 32 bits and 64 bits Herve Jourdain
@ 2016-11-19 12:50 ` Herve Jourdain
2016-11-19 17:50 ` Khem Raj
2016-11-20 8:07 ` [PATCH v2 0/1] " Herve Jourdain
1 sibling, 1 reply; 5+ messages in thread
From: Herve Jourdain @ 2016-11-19 12:50 UTC (permalink / raw)
To: openembedded-core
Signed-off-by: Herve Jourdain <herve.jourdain@neuf.fr>
---
meta/conf/machine/include/tune-cortexa53.inc | 22 ++++++++++++++++++++++
1 file changed, 22 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 0000000..ff5edb6
--- /dev/null
+++ b/meta/conf/machine/include/tune-cortexa53.inc
@@ -0,0 +1,22 @@
+DEFAULTTUNE ?= "cortexa53-32"
+
+require conf/machine/include/arm/arch-armv8.inc
+
+TUNEVALID[cortexa53] = "Enable Cortex-A53 specific processor optimizations"
+TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'cortexa53', ' -mcpu=cortex-a53 ', '',d)}"
+
+# Extra tune selections
+AVAILTUNES += "cortexa53-32"
+ARMPKGARCH_tune-cortexa53-32 ?= "cortexa53"
+TUNE_FEATURES_tune-cortexa53-32 = "${TUNE_FEATURES_tune-armv7ve-neon-vfpv4} cortexa53"
+BASE_LIB_tune-cortexa53-32 = "lib"
+TUNE_PKGARCH_tune-cortexa53-32 = "cortexa53-32"
+PACKAGE_EXTRA_ARCHS_tune-cortexa53-32 = "${PACKAGE_EXTRA_ARCHS_tune-armv7ve-neon-vfpv4} cortexa53-32 cortexa53-neon-vfpv4"
+
+AVAILTUNES += "cortexa53-64"
+ARMPKGARCH_tune-cortexa53-64 ?= "cortexa53"
+TUNE_FEATURES_tune-cortexa53-64 = "${TUNE_FEATURES_tune-aarch64} cortexa53"
+BASE_LIB_tune-cortexa53-64 = "lib64"
+TUNE_PKGARCH_tune-cortexa53-64 = "cortexa53-64"
+PACKAGE_EXTRA_ARCHS_tune-cortexa53-64 = "${PACKAGE_EXTRA_ARCHS_tune-aarch64} cortexa53-64"
+
--
2.7.4
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH v2 1/1] Add a tune file for cortexa53, valid for 32 bits and 64 bits
2016-11-19 12:50 ` [PATCH v2 1/1] " Herve Jourdain
@ 2016-11-19 17:50 ` Khem Raj
2016-11-20 8:05 ` Herve Jourdain
0 siblings, 1 reply; 5+ messages in thread
From: Khem Raj @ 2016-11-19 17:50 UTC (permalink / raw)
To: Herve Jourdain; +Cc: Patches and discussions about the oe-core layer
This seems fine to me. There is a tune checker that Martin has, it would be good
to run that and see if this new tune is tested.
http://git.openembedded.org/openembedded-core-contrib/log/?h=jansa/tune2-test
On Sat, Nov 19, 2016 at 4:50 AM, Herve Jourdain <herve.jourdain@neuf.fr> wrote:
> Signed-off-by: Herve Jourdain <herve.jourdain@neuf.fr>
> ---
> meta/conf/machine/include/tune-cortexa53.inc | 22 ++++++++++++++++++++++
> 1 file changed, 22 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 0000000..ff5edb6
> --- /dev/null
> +++ b/meta/conf/machine/include/tune-cortexa53.inc
> @@ -0,0 +1,22 @@
> +DEFAULTTUNE ?= "cortexa53-32"
> +
> +require conf/machine/include/arm/arch-armv8.inc
> +
> +TUNEVALID[cortexa53] = "Enable Cortex-A53 specific processor optimizations"
> +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'cortexa53', ' -mcpu=cortex-a53 ', '',d)}"
> +
> +# Extra tune selections
> +AVAILTUNES += "cortexa53-32"
> +ARMPKGARCH_tune-cortexa53-32 ?= "cortexa53"
> +TUNE_FEATURES_tune-cortexa53-32 = "${TUNE_FEATURES_tune-armv7ve-neon-vfpv4} cortexa53"
> +BASE_LIB_tune-cortexa53-32 = "lib"
> +TUNE_PKGARCH_tune-cortexa53-32 = "cortexa53-32"
> +PACKAGE_EXTRA_ARCHS_tune-cortexa53-32 = "${PACKAGE_EXTRA_ARCHS_tune-armv7ve-neon-vfpv4} cortexa53-32 cortexa53-neon-vfpv4"
> +
> +AVAILTUNES += "cortexa53-64"
> +ARMPKGARCH_tune-cortexa53-64 ?= "cortexa53"
> +TUNE_FEATURES_tune-cortexa53-64 = "${TUNE_FEATURES_tune-aarch64} cortexa53"
> +BASE_LIB_tune-cortexa53-64 = "lib64"
> +TUNE_PKGARCH_tune-cortexa53-64 = "cortexa53-64"
> +PACKAGE_EXTRA_ARCHS_tune-cortexa53-64 = "${PACKAGE_EXTRA_ARCHS_tune-aarch64} cortexa53-64"
> +
> --
> 2.7.4
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v2 1/1] Add a tune file for cortexa53, valid for 32 bits and 64 bits
2016-11-19 17:50 ` Khem Raj
@ 2016-11-20 8:05 ` Herve Jourdain
0 siblings, 0 replies; 5+ messages in thread
From: Herve Jourdain @ 2016-11-20 8:05 UTC (permalink / raw)
To: 'Khem Raj'
Cc: 'Patches and discussions about the oe-core layer'
Hi Khem,
Unfortunately, it has a few issues in 32bits mode...
Basically, cortex-a53 is an armv8+crc...
I'm sending an updated version - v3 - soon.
But I'll run the tune checker first. At least, the new version I'm coming up with and plan to send soon seems to work fine now with 32bits - and I shouldn't have broken the 64bits support in the process, but I will also compile it.
Herve
-----Original Message-----
From: Khem Raj [mailto:raj.khem@gmail.com]
Sent: samedi 19 novembre 2016 18:51
To: Herve Jourdain <herve.jourdain@neuf.fr>
Cc: Patches and discussions about the oe-core layer <openembedded-core@lists.openembedded.org>
Subject: Re: [OE-core] [oe-core][PATCH v2 1/1] Add a tune file for cortexa53, valid for 32 bits and 64 bits
This seems fine to me. There is a tune checker that Martin has, it would be good to run that and see if this new tune is tested.
http://git.openembedded.org/openembedded-core-contrib/log/?h=jansa/tune2-test
On Sat, Nov 19, 2016 at 4:50 AM, Herve Jourdain <herve.jourdain@neuf.fr> wrote:
> Signed-off-by: Herve Jourdain <herve.jourdain@neuf.fr>
> ---
> meta/conf/machine/include/tune-cortexa53.inc | 22
> ++++++++++++++++++++++
> 1 file changed, 22 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 0000000..ff5edb6
> --- /dev/null
> +++ b/meta/conf/machine/include/tune-cortexa53.inc
> @@ -0,0 +1,22 @@
> +DEFAULTTUNE ?= "cortexa53-32"
> +
> +require conf/machine/include/arm/arch-armv8.inc
> +
> +TUNEVALID[cortexa53] = "Enable Cortex-A53 specific processor optimizations"
> +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'cortexa53', ' -mcpu=cortex-a53 ', '',d)}"
> +
> +# Extra tune selections
> +AVAILTUNES += "cortexa53-32"
> +ARMPKGARCH_tune-cortexa53-32 ?= "cortexa53"
> +TUNE_FEATURES_tune-cortexa53-32 = "${TUNE_FEATURES_tune-armv7ve-neon-vfpv4} cortexa53"
> +BASE_LIB_tune-cortexa53-32 = "lib"
> +TUNE_PKGARCH_tune-cortexa53-32 = "cortexa53-32"
> +PACKAGE_EXTRA_ARCHS_tune-cortexa53-32 = "${PACKAGE_EXTRA_ARCHS_tune-armv7ve-neon-vfpv4} cortexa53-32 cortexa53-neon-vfpv4"
> +
> +AVAILTUNES += "cortexa53-64"
> +ARMPKGARCH_tune-cortexa53-64 ?= "cortexa53"
> +TUNE_FEATURES_tune-cortexa53-64 = "${TUNE_FEATURES_tune-aarch64} cortexa53"
> +BASE_LIB_tune-cortexa53-64 = "lib64"
> +TUNE_PKGARCH_tune-cortexa53-64 = "cortexa53-64"
> +PACKAGE_EXTRA_ARCHS_tune-cortexa53-64 = "${PACKAGE_EXTRA_ARCHS_tune-aarch64} cortexa53-64"
> +
> --
> 2.7.4
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 0/1] Add a tune file for cortexa53, valid for 32 bits and 64 bits
2016-11-19 12:50 [PATCH v2 0/1] Add a tune file for cortexa53, valid for 32 bits and 64 bits Herve Jourdain
2016-11-19 12:50 ` [PATCH v2 1/1] " Herve Jourdain
@ 2016-11-20 8:07 ` Herve Jourdain
1 sibling, 0 replies; 5+ messages in thread
From: Herve Jourdain @ 2016-11-20 8:07 UTC (permalink / raw)
To: openembedded-core
Please disregard this patch, it's not working fine for 32bits (cortex-a53
must have armv8+crc as arch), v3 coming soon.
Sorry for that.
Herve
-----Original Message-----
From: Herve Jourdain [mailto:herve.jourdain@neuf.fr]
Sent: samedi 19 novembre 2016 13:51
To: openembedded-core@lists.openembedded.org
Cc: Herve Jourdain <herve.jourdain@neuf.fr>
Subject: [oe-core][PATCH v2 0/1] Add a tune file for cortexa53, valid for 32
bits and 64 bits
Tune file for ARM Cortex-A53, supporting 32 bits and 64 bits configuration
Herve Jourdain (1):
Add a tune file for cortexa53, valid for 32 bits and 64 bits
meta/conf/machine/include/tune-cortexa53.inc | 22 ++++++++++++++++++++++
1 file changed, 22 insertions(+)
create mode 100644 meta/conf/machine/include/tune-cortexa53.inc
--
2.7.4
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-11-20 8:08 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-19 12:50 [PATCH v2 0/1] Add a tune file for cortexa53, valid for 32 bits and 64 bits Herve Jourdain
2016-11-19 12:50 ` [PATCH v2 1/1] " Herve Jourdain
2016-11-19 17:50 ` Khem Raj
2016-11-20 8:05 ` Herve Jourdain
2016-11-20 8:07 ` [PATCH v2 0/1] " Herve Jourdain
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox