| Message ID | 20170420182028.31964-2-raj.khem@gmail.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [1/5] boost: Upgrade to 1.64.0 | expand |
On Thu, Apr 20, 2017 at 11:20 AM, Khem Raj <raj.khem@gmail.com> wrote: > Signed-off-by: Khem Raj <raj.khem@gmail.com> > --- > ...rop-setting-mcpu-to-cortex-a8-on-arm-arch.patch | 108 +++++++++++++++++++++ > meta/recipes-devtools/valgrind/valgrind_3.12.0.bb | 3 +- > 2 files changed, 110 insertions(+), 1 deletion(-) > create mode 100644 meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch > > diff --git a/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch b/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch > new file mode 100644 > index 00000000000..9f1da7bac68 > --- /dev/null > +++ b/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch > @@ -0,0 +1,108 @@ > +From 715cf122388f3527afa5649cebf9f1522c240693 Mon Sep 17 00:00:00 2001 > +From: Khem Raj <raj.khem@gmail.com> > +Date: Thu, 20 Apr 2017 10:11:16 -0700 > +Subject: [PATCH] makefiles: Drop setting -mcpu to cortex-a8 on arm > + architecture > + > +We can not assume that all arches armv7+ are cortex-a8 only > +it fails to build for rpi which is armv7ve based (cortex-a8) cpu > +implementation. > +Fixes > +| cc1: warning: switch -mcpu=cortex-a8 conflicts with -march=armv7ve switch The valgrind tests are built for fixed CPU targets which are known to support the instructions being tested (since those instructions may not be supported by the target CPU which valgrind itself is being built for). It may be safer to add -march=armv7a to the -mcpu=cortex-a8 tests (and ensure that -march=armv7ve doesn't leak through) rather than remove -mcpu=cortex-a8. See the comments in: http://git.openembedded.org/openembedded-core/commit/?id=3e94af90d90f6db6bab66ae87a47b31f3a0474f6 The change to the toplevel Makefile.all.am looks OK. > +Upstream-Status: Pending > + > +Signed-off-by: Khem Raj <raj.khem@gmail.com> > +--- > + Makefile.all.am | 6 +++--- > + helgrind/tests/Makefile.am | 6 +++--- > + none/tests/arm/Makefile.am | 18 +++++++++--------- > + 3 files changed, 15 insertions(+), 15 deletions(-) > + > +diff --git a/Makefile.all.am b/Makefile.all.am > +index 02059a3..c7c4700 100644 > +--- a/Makefile.all.am > ++++ b/Makefile.all.am > +@@ -197,11 +197,11 @@ AM_CCASFLAGS_PPC64LE_LINUX = @FLAG_M64@ -g > + > + AM_FLAG_M3264_ARM_LINUX = @FLAG_M32@ > + AM_CFLAGS_ARM_LINUX = @FLAG_M32@ \ > +- $(AM_CFLAGS_BASE) -marm -mcpu=cortex-a8 > ++ $(AM_CFLAGS_BASE) -marm > + AM_CFLAGS_PSO_ARM_LINUX = @FLAG_M32@ $(AM_CFLAGS_BASE) \ > +- -marm -mcpu=cortex-a8 $(AM_CFLAGS_PSO_BASE) > ++ -marm $(AM_CFLAGS_PSO_BASE) > + AM_CCASFLAGS_ARM_LINUX = @FLAG_M32@ \ > +- -marm -mcpu=cortex-a8 -g > ++ -marm -g > + > + AM_FLAG_M3264_ARM64_LINUX = @FLAG_M64@ > + AM_CFLAGS_ARM64_LINUX = @FLAG_M64@ $(AM_CFLAGS_BASE) > +diff --git a/helgrind/tests/Makefile.am b/helgrind/tests/Makefile.am > +index df82169..07eb66a 100644 > +--- a/helgrind/tests/Makefile.am > ++++ b/helgrind/tests/Makefile.am > +@@ -189,9 +189,9 @@ if ! VGCONF_PLATFORMS_INCLUDE_X86_DARWIN > + endif > + > + if VGCONF_PLATFORMS_INCLUDE_ARM_LINUX > +-annotate_hbefore_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 > +-tc07_hbl1_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 > +-tc08_hbl2_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 > ++annotate_hbefore_CFLAGS = $(AM_CFLAGS) > ++tc07_hbl1_CFLAGS = $(AM_CFLAGS) > ++tc08_hbl2_CFLAGS = $(AM_CFLAGS) > + else > + annotate_hbefore_CFLAGS = $(AM_CFLAGS) > + tc07_hbl1_CFLAGS = $(AM_CFLAGS) > +diff --git a/none/tests/arm/Makefile.am b/none/tests/arm/Makefile.am > +index 024eb6d..ccecb90 100644 > +--- a/none/tests/arm/Makefile.am > ++++ b/none/tests/arm/Makefile.am > +@@ -52,10 +52,10 @@ allexec_CFLAGS = $(AM_CFLAGS) @FLAG_W_NO_NONNULL@ > + # need special helping w.r.t -mfpu and -mfloat-abi, though. > + # Also force -O0 since -O takes hundreds of MB of memory > + # for v6intThumb.c. > +-v6intARM_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -marm > +-v6intThumb_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -mthumb > ++v6intARM_CFLAGS = $(AM_CFLAGS) -g -O0 -marm > ++v6intThumb_CFLAGS = $(AM_CFLAGS) -g -O0 -mthumb > + > +-v6media_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -mthumb > ++v6media_CFLAGS = $(AM_CFLAGS) -g -O0 -mthumb > + > + v8crypto_a_CFLAGS = $(AM_CFLAGS) -g -O0 -mfpu=crypto-neon-fp-armv8 -marm > + v8crypto_t_CFLAGS = $(AM_CFLAGS) -g -O0 -mfpu=crypto-neon-fp-armv8 -mthumb > +@@ -65,23 +65,23 @@ v8memory_a_CFLAGS = $(AM_CFLAGS) -g -O0 \ > + v8memory_t_CFLAGS = $(AM_CFLAGS) -g -O0 \ > + -march=armv8-a -mfpu=crypto-neon-fp-armv8 -mthumb > + > +-vfp_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ > ++vfp_CFLAGS = $(AM_CFLAGS) -g -O0 \ > + -mfpu=neon \ > + -mthumb > + > + > +-neon128_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ > ++neon128_CFLAGS = $(AM_CFLAGS) -g -O0 \ > + -mfpu=neon \ > + -mthumb > + > +-neon64_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ > ++neon64_CFLAGS = $(AM_CFLAGS) -g -O0 \ > + -mfpu=neon \ > + -mthumb > + > + intdiv_CFLAGS = $(AM_CFLAGS) -g -march=armv7ve -mcpu=cortex-a15 -mthumb > +-ldrt_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -mthumb > +-ldrt_arm_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -marm > ++ldrt_CFLAGS = $(AM_CFLAGS) -g -mthumb > ++ldrt_arm_CFLAGS = $(AM_CFLAGS) -g -marm > + > +-vcvt_fixed_float_VFP_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -mfpu=vfpv3 > ++vcvt_fixed_float_VFP_CFLAGS = $(AM_CFLAGS) -g -mfpu=vfpv3 > + > + vfpv4_fma_CFLAGS = $(AM_CFLAGS) -g -O0 -march=armv7ve -mcpu=cortex-a15 -mfpu=vfpv4 -marm > +-- > +2.12.2 > + > diff --git a/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb b/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb > index 62a96354cd0..a07c8767ce3 100644 > --- a/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb > +++ b/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb > @@ -23,7 +23,8 @@ SRC_URI = "http://www.valgrind.org/downloads/valgrind-${PV}.tar.bz2 \ > file://use-appropriate-march-mcpu-mfpu-for-ARM-test-apps.patch \ > file://avoid-neon-for-targets-which-don-t-support-it.patch \ > file://valgrind-make-ld-XXX.so-strlen-intercept-optional.patch \ > -" > + file://0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch \ > + " > SRC_URI_append_libc-musl = "\ > file://0001-fix-build-for-musl-targets.patch \ > " > -- > 2.12.2 > > -- > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-core -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
On Thu, Apr 20, 2017 at 5:56 PM, Andre McCurdy <armccurdy@gmail.com> wrote: > On Thu, Apr 20, 2017 at 11:20 AM, Khem Raj <raj.khem@gmail.com> wrote: >> Signed-off-by: Khem Raj <raj.khem@gmail.com> >> --- >> ...rop-setting-mcpu-to-cortex-a8-on-arm-arch.patch | 108 +++++++++++++++++++++ >> meta/recipes-devtools/valgrind/valgrind_3.12.0.bb | 3 +- >> 2 files changed, 110 insertions(+), 1 deletion(-) >> create mode 100644 meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch >> >> diff --git a/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch b/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch >> new file mode 100644 >> index 00000000000..9f1da7bac68 >> --- /dev/null >> +++ b/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch >> @@ -0,0 +1,108 @@ >> +From 715cf122388f3527afa5649cebf9f1522c240693 Mon Sep 17 00:00:00 2001 >> +From: Khem Raj <raj.khem@gmail.com> >> +Date: Thu, 20 Apr 2017 10:11:16 -0700 >> +Subject: [PATCH] makefiles: Drop setting -mcpu to cortex-a8 on arm >> + architecture >> + >> +We can not assume that all arches armv7+ are cortex-a8 only >> +it fails to build for rpi which is armv7ve based (cortex-a8) cpu >> +implementation. >> +Fixes >> +| cc1: warning: switch -mcpu=cortex-a8 conflicts with -march=armv7ve switch > > The valgrind tests are built for fixed CPU targets which are known to > support the instructions being tested (since those instructions may > not be supported by the target CPU which valgrind itself is being > built for). > > It may be safer to add -march=armv7a to the -mcpu=cortex-a8 tests (and > ensure that -march=armv7ve doesn't leak through) rather than remove > -mcpu=cortex-a8. OE always passes mcpu flags, therefore it wont be an issue unless you are saying thats not the case. For upstreaming it might be OK, but I think this will need a different for upstreaming since it should configure mcpu/march based on some configure input or some such instead of hardcoding it. > > See the comments in: > > http://git.openembedded.org/openembedded-core/commit/?id=3e94af90d90f6db6bab66ae87a47b31f3a0474f6 > > The change to the toplevel Makefile.all.am looks OK. > >> +Upstream-Status: Pending >> + >> +Signed-off-by: Khem Raj <raj.khem@gmail.com> >> +--- >> + Makefile.all.am | 6 +++--- >> + helgrind/tests/Makefile.am | 6 +++--- >> + none/tests/arm/Makefile.am | 18 +++++++++--------- >> + 3 files changed, 15 insertions(+), 15 deletions(-) >> + >> +diff --git a/Makefile.all.am b/Makefile.all.am >> +index 02059a3..c7c4700 100644 >> +--- a/Makefile.all.am >> ++++ b/Makefile.all.am >> +@@ -197,11 +197,11 @@ AM_CCASFLAGS_PPC64LE_LINUX = @FLAG_M64@ -g >> + >> + AM_FLAG_M3264_ARM_LINUX = @FLAG_M32@ >> + AM_CFLAGS_ARM_LINUX = @FLAG_M32@ \ >> +- $(AM_CFLAGS_BASE) -marm -mcpu=cortex-a8 >> ++ $(AM_CFLAGS_BASE) -marm >> + AM_CFLAGS_PSO_ARM_LINUX = @FLAG_M32@ $(AM_CFLAGS_BASE) \ >> +- -marm -mcpu=cortex-a8 $(AM_CFLAGS_PSO_BASE) >> ++ -marm $(AM_CFLAGS_PSO_BASE) >> + AM_CCASFLAGS_ARM_LINUX = @FLAG_M32@ \ >> +- -marm -mcpu=cortex-a8 -g >> ++ -marm -g >> + >> + AM_FLAG_M3264_ARM64_LINUX = @FLAG_M64@ >> + AM_CFLAGS_ARM64_LINUX = @FLAG_M64@ $(AM_CFLAGS_BASE) >> +diff --git a/helgrind/tests/Makefile.am b/helgrind/tests/Makefile.am >> +index df82169..07eb66a 100644 >> +--- a/helgrind/tests/Makefile.am >> ++++ b/helgrind/tests/Makefile.am >> +@@ -189,9 +189,9 @@ if ! VGCONF_PLATFORMS_INCLUDE_X86_DARWIN >> + endif >> + >> + if VGCONF_PLATFORMS_INCLUDE_ARM_LINUX >> +-annotate_hbefore_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 >> +-tc07_hbl1_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 >> +-tc08_hbl2_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 >> ++annotate_hbefore_CFLAGS = $(AM_CFLAGS) >> ++tc07_hbl1_CFLAGS = $(AM_CFLAGS) >> ++tc08_hbl2_CFLAGS = $(AM_CFLAGS) >> + else >> + annotate_hbefore_CFLAGS = $(AM_CFLAGS) >> + tc07_hbl1_CFLAGS = $(AM_CFLAGS) >> +diff --git a/none/tests/arm/Makefile.am b/none/tests/arm/Makefile.am >> +index 024eb6d..ccecb90 100644 >> +--- a/none/tests/arm/Makefile.am >> ++++ b/none/tests/arm/Makefile.am >> +@@ -52,10 +52,10 @@ allexec_CFLAGS = $(AM_CFLAGS) @FLAG_W_NO_NONNULL@ >> + # need special helping w.r.t -mfpu and -mfloat-abi, though. >> + # Also force -O0 since -O takes hundreds of MB of memory >> + # for v6intThumb.c. >> +-v6intARM_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -marm >> +-v6intThumb_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -mthumb >> ++v6intARM_CFLAGS = $(AM_CFLAGS) -g -O0 -marm >> ++v6intThumb_CFLAGS = $(AM_CFLAGS) -g -O0 -mthumb >> + >> +-v6media_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -mthumb >> ++v6media_CFLAGS = $(AM_CFLAGS) -g -O0 -mthumb >> + >> + v8crypto_a_CFLAGS = $(AM_CFLAGS) -g -O0 -mfpu=crypto-neon-fp-armv8 -marm >> + v8crypto_t_CFLAGS = $(AM_CFLAGS) -g -O0 -mfpu=crypto-neon-fp-armv8 -mthumb >> +@@ -65,23 +65,23 @@ v8memory_a_CFLAGS = $(AM_CFLAGS) -g -O0 \ >> + v8memory_t_CFLAGS = $(AM_CFLAGS) -g -O0 \ >> + -march=armv8-a -mfpu=crypto-neon-fp-armv8 -mthumb >> + >> +-vfp_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ >> ++vfp_CFLAGS = $(AM_CFLAGS) -g -O0 \ >> + -mfpu=neon \ >> + -mthumb >> + >> + >> +-neon128_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ >> ++neon128_CFLAGS = $(AM_CFLAGS) -g -O0 \ >> + -mfpu=neon \ >> + -mthumb >> + >> +-neon64_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ >> ++neon64_CFLAGS = $(AM_CFLAGS) -g -O0 \ >> + -mfpu=neon \ >> + -mthumb >> + >> + intdiv_CFLAGS = $(AM_CFLAGS) -g -march=armv7ve -mcpu=cortex-a15 -mthumb >> +-ldrt_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -mthumb >> +-ldrt_arm_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -marm >> ++ldrt_CFLAGS = $(AM_CFLAGS) -g -mthumb >> ++ldrt_arm_CFLAGS = $(AM_CFLAGS) -g -marm >> + >> +-vcvt_fixed_float_VFP_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -mfpu=vfpv3 >> ++vcvt_fixed_float_VFP_CFLAGS = $(AM_CFLAGS) -g -mfpu=vfpv3 >> + >> + vfpv4_fma_CFLAGS = $(AM_CFLAGS) -g -O0 -march=armv7ve -mcpu=cortex-a15 -mfpu=vfpv4 -marm >> +-- >> +2.12.2 >> + >> diff --git a/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb b/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb >> index 62a96354cd0..a07c8767ce3 100644 >> --- a/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb >> +++ b/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb >> @@ -23,7 +23,8 @@ SRC_URI = "http://www.valgrind.org/downloads/valgrind-${PV}.tar.bz2 \ >> file://use-appropriate-march-mcpu-mfpu-for-ARM-test-apps.patch \ >> file://avoid-neon-for-targets-which-don-t-support-it.patch \ >> file://valgrind-make-ld-XXX.so-strlen-intercept-optional.patch \ >> -" >> + file://0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch \ >> + " >> SRC_URI_append_libc-musl = "\ >> file://0001-fix-build-for-musl-targets.patch \ >> " >> -- >> 2.12.2 >> >> -- >> _______________________________________________ >> Openembedded-core mailing list >> Openembedded-core@lists.openembedded.org >> http://lists.openembedded.org/mailman/listinfo/openembedded-core -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
On Thu, Apr 20, 2017 at 6:06 PM, Khem Raj <raj.khem@gmail.com> wrote: > On Thu, Apr 20, 2017 at 5:56 PM, Andre McCurdy <armccurdy@gmail.com> wrote: >> On Thu, Apr 20, 2017 at 11:20 AM, Khem Raj <raj.khem@gmail.com> wrote: >>> Signed-off-by: Khem Raj <raj.khem@gmail.com> >>> --- >>> ...rop-setting-mcpu-to-cortex-a8-on-arm-arch.patch | 108 +++++++++++++++++++++ >>> meta/recipes-devtools/valgrind/valgrind_3.12.0.bb | 3 +- >>> 2 files changed, 110 insertions(+), 1 deletion(-) >>> create mode 100644 meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch >>> >>> diff --git a/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch b/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch >>> new file mode 100644 >>> index 00000000000..9f1da7bac68 >>> --- /dev/null >>> +++ b/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch >>> @@ -0,0 +1,108 @@ >>> +From 715cf122388f3527afa5649cebf9f1522c240693 Mon Sep 17 00:00:00 2001 >>> +From: Khem Raj <raj.khem@gmail.com> >>> +Date: Thu, 20 Apr 2017 10:11:16 -0700 >>> +Subject: [PATCH] makefiles: Drop setting -mcpu to cortex-a8 on arm >>> + architecture >>> + >>> +We can not assume that all arches armv7+ are cortex-a8 only >>> +it fails to build for rpi which is armv7ve based (cortex-a8) cpu >>> +implementation. >>> +Fixes >>> +| cc1: warning: switch -mcpu=cortex-a8 conflicts with -march=armv7ve switch >> >> The valgrind tests are built for fixed CPU targets which are known to >> support the instructions being tested (since those instructions may >> not be supported by the target CPU which valgrind itself is being >> built for). >> >> It may be safer to add -march=armv7a to the -mcpu=cortex-a8 tests (and >> ensure that -march=armv7ve doesn't leak through) rather than remove >> -mcpu=cortex-a8. > > OE always passes mcpu flags, therefore it wont be an issue unless you > are saying thats not the case. OE won't pass an -mcpu flag for generic DEFAULTTUNE values such as "armv7athf". > For upstreaming it might be OK, but I > think this will need a different for upstreaming since it should > configure mcpu/march based on some configure input or some such > instead of hardcoding it. For the ../tests/... Makefiles, the mcpu/march flags should absolutely be hardcoded. e.g. the test for "idiv" should always be built in exactly the same way, regardless of whether or not the target arch you configure and build valgrind itself for is armv7ve. >> >> See the comments in: >> >> http://git.openembedded.org/openembedded-core/commit/?id=3e94af90d90f6db6bab66ae87a47b31f3a0474f6 >> >> The change to the toplevel Makefile.all.am looks OK. >> >>> +Upstream-Status: Pending >>> + >>> +Signed-off-by: Khem Raj <raj.khem@gmail.com> >>> +--- >>> + Makefile.all.am | 6 +++--- >>> + helgrind/tests/Makefile.am | 6 +++--- >>> + none/tests/arm/Makefile.am | 18 +++++++++--------- >>> + 3 files changed, 15 insertions(+), 15 deletions(-) >>> + >>> +diff --git a/Makefile.all.am b/Makefile.all.am >>> +index 02059a3..c7c4700 100644 >>> +--- a/Makefile.all.am >>> ++++ b/Makefile.all.am >>> +@@ -197,11 +197,11 @@ AM_CCASFLAGS_PPC64LE_LINUX = @FLAG_M64@ -g >>> + >>> + AM_FLAG_M3264_ARM_LINUX = @FLAG_M32@ >>> + AM_CFLAGS_ARM_LINUX = @FLAG_M32@ \ >>> +- $(AM_CFLAGS_BASE) -marm -mcpu=cortex-a8 >>> ++ $(AM_CFLAGS_BASE) -marm >>> + AM_CFLAGS_PSO_ARM_LINUX = @FLAG_M32@ $(AM_CFLAGS_BASE) \ >>> +- -marm -mcpu=cortex-a8 $(AM_CFLAGS_PSO_BASE) >>> ++ -marm $(AM_CFLAGS_PSO_BASE) >>> + AM_CCASFLAGS_ARM_LINUX = @FLAG_M32@ \ >>> +- -marm -mcpu=cortex-a8 -g >>> ++ -marm -g >>> + >>> + AM_FLAG_M3264_ARM64_LINUX = @FLAG_M64@ >>> + AM_CFLAGS_ARM64_LINUX = @FLAG_M64@ $(AM_CFLAGS_BASE) >>> +diff --git a/helgrind/tests/Makefile.am b/helgrind/tests/Makefile.am >>> +index df82169..07eb66a 100644 >>> +--- a/helgrind/tests/Makefile.am >>> ++++ b/helgrind/tests/Makefile.am >>> +@@ -189,9 +189,9 @@ if ! VGCONF_PLATFORMS_INCLUDE_X86_DARWIN >>> + endif >>> + >>> + if VGCONF_PLATFORMS_INCLUDE_ARM_LINUX >>> +-annotate_hbefore_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 >>> +-tc07_hbl1_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 >>> +-tc08_hbl2_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 >>> ++annotate_hbefore_CFLAGS = $(AM_CFLAGS) >>> ++tc07_hbl1_CFLAGS = $(AM_CFLAGS) >>> ++tc08_hbl2_CFLAGS = $(AM_CFLAGS) >>> + else >>> + annotate_hbefore_CFLAGS = $(AM_CFLAGS) >>> + tc07_hbl1_CFLAGS = $(AM_CFLAGS) >>> +diff --git a/none/tests/arm/Makefile.am b/none/tests/arm/Makefile.am >>> +index 024eb6d..ccecb90 100644 >>> +--- a/none/tests/arm/Makefile.am >>> ++++ b/none/tests/arm/Makefile.am >>> +@@ -52,10 +52,10 @@ allexec_CFLAGS = $(AM_CFLAGS) @FLAG_W_NO_NONNULL@ >>> + # need special helping w.r.t -mfpu and -mfloat-abi, though. >>> + # Also force -O0 since -O takes hundreds of MB of memory >>> + # for v6intThumb.c. >>> +-v6intARM_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -marm >>> +-v6intThumb_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -mthumb >>> ++v6intARM_CFLAGS = $(AM_CFLAGS) -g -O0 -marm >>> ++v6intThumb_CFLAGS = $(AM_CFLAGS) -g -O0 -mthumb >>> + >>> +-v6media_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -mthumb >>> ++v6media_CFLAGS = $(AM_CFLAGS) -g -O0 -mthumb >>> + >>> + v8crypto_a_CFLAGS = $(AM_CFLAGS) -g -O0 -mfpu=crypto-neon-fp-armv8 -marm >>> + v8crypto_t_CFLAGS = $(AM_CFLAGS) -g -O0 -mfpu=crypto-neon-fp-armv8 -mthumb >>> +@@ -65,23 +65,23 @@ v8memory_a_CFLAGS = $(AM_CFLAGS) -g -O0 \ >>> + v8memory_t_CFLAGS = $(AM_CFLAGS) -g -O0 \ >>> + -march=armv8-a -mfpu=crypto-neon-fp-armv8 -mthumb >>> + >>> +-vfp_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ >>> ++vfp_CFLAGS = $(AM_CFLAGS) -g -O0 \ >>> + -mfpu=neon \ >>> + -mthumb >>> + >>> + >>> +-neon128_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ >>> ++neon128_CFLAGS = $(AM_CFLAGS) -g -O0 \ >>> + -mfpu=neon \ >>> + -mthumb >>> + >>> +-neon64_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ >>> ++neon64_CFLAGS = $(AM_CFLAGS) -g -O0 \ >>> + -mfpu=neon \ >>> + -mthumb >>> + >>> + intdiv_CFLAGS = $(AM_CFLAGS) -g -march=armv7ve -mcpu=cortex-a15 -mthumb >>> +-ldrt_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -mthumb >>> +-ldrt_arm_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -marm >>> ++ldrt_CFLAGS = $(AM_CFLAGS) -g -mthumb >>> ++ldrt_arm_CFLAGS = $(AM_CFLAGS) -g -marm >>> + >>> +-vcvt_fixed_float_VFP_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -mfpu=vfpv3 >>> ++vcvt_fixed_float_VFP_CFLAGS = $(AM_CFLAGS) -g -mfpu=vfpv3 >>> + >>> + vfpv4_fma_CFLAGS = $(AM_CFLAGS) -g -O0 -march=armv7ve -mcpu=cortex-a15 -mfpu=vfpv4 -marm >>> +-- >>> +2.12.2 >>> + >>> diff --git a/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb b/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb >>> index 62a96354cd0..a07c8767ce3 100644 >>> --- a/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb >>> +++ b/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb >>> @@ -23,7 +23,8 @@ SRC_URI = "http://www.valgrind.org/downloads/valgrind-${PV}.tar.bz2 \ >>> file://use-appropriate-march-mcpu-mfpu-for-ARM-test-apps.patch \ >>> file://avoid-neon-for-targets-which-don-t-support-it.patch \ >>> file://valgrind-make-ld-XXX.so-strlen-intercept-optional.patch \ >>> -" >>> + file://0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch \ >>> + " >>> SRC_URI_append_libc-musl = "\ >>> file://0001-fix-build-for-musl-targets.patch \ >>> " >>> -- >>> 2.12.2 >>> >>> -- >>> _______________________________________________ >>> Openembedded-core mailing list >>> Openembedded-core@lists.openembedded.org >>> http://lists.openembedded.org/mailman/listinfo/openembedded-core -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
On Thu, Apr 20, 2017 at 6:29 PM, Andre McCurdy <armccurdy@gmail.com> wrote: > On Thu, Apr 20, 2017 at 6:06 PM, Khem Raj <raj.khem@gmail.com> wrote: >> On Thu, Apr 20, 2017 at 5:56 PM, Andre McCurdy <armccurdy@gmail.com> wrote: >>> On Thu, Apr 20, 2017 at 11:20 AM, Khem Raj <raj.khem@gmail.com> wrote: >>>> Signed-off-by: Khem Raj <raj.khem@gmail.com> >>>> --- >>>> ...rop-setting-mcpu-to-cortex-a8-on-arm-arch.patch | 108 +++++++++++++++++++++ >>>> meta/recipes-devtools/valgrind/valgrind_3.12.0.bb | 3 +- >>>> 2 files changed, 110 insertions(+), 1 deletion(-) >>>> create mode 100644 meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch >>>> >>>> diff --git a/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch b/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch >>>> new file mode 100644 >>>> index 00000000000..9f1da7bac68 >>>> --- /dev/null >>>> +++ b/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch >>>> @@ -0,0 +1,108 @@ >>>> +From 715cf122388f3527afa5649cebf9f1522c240693 Mon Sep 17 00:00:00 2001 >>>> +From: Khem Raj <raj.khem@gmail.com> >>>> +Date: Thu, 20 Apr 2017 10:11:16 -0700 >>>> +Subject: [PATCH] makefiles: Drop setting -mcpu to cortex-a8 on arm >>>> + architecture >>>> + >>>> +We can not assume that all arches armv7+ are cortex-a8 only >>>> +it fails to build for rpi which is armv7ve based (cortex-a8) cpu >>>> +implementation. >>>> +Fixes >>>> +| cc1: warning: switch -mcpu=cortex-a8 conflicts with -march=armv7ve switch >>> >>> The valgrind tests are built for fixed CPU targets which are known to >>> support the instructions being tested (since those instructions may >>> not be supported by the target CPU which valgrind itself is being >>> built for). >>> >>> It may be safer to add -march=armv7a to the -mcpu=cortex-a8 tests (and >>> ensure that -march=armv7ve doesn't leak through) rather than remove >>> -mcpu=cortex-a8. >> >> OE always passes mcpu flags, therefore it wont be an issue unless you >> are saying thats not the case. > > OE won't pass an -mcpu flag for generic DEFAULTTUNE values such as "armv7athf". > >> For upstreaming it might be OK, but I >> think this will need a different for upstreaming since it should >> configure mcpu/march based on some configure input or some such >> instead of hardcoding it. > > For the ../tests/... Makefiles, the mcpu/march flags should absolutely > be hardcoded. e.g. the test for "idiv" should always be built in > exactly the same way, regardless of whether or not the target arch you > configure and build valgrind itself for is armv7ve. > what is the purpose of compiling such a test case for a machine where it will cause SIGILL ? >>> >>> See the comments in: >>> >>> http://git.openembedded.org/openembedded-core/commit/?id=3e94af90d90f6db6bab66ae87a47b31f3a0474f6 >>> >>> The change to the toplevel Makefile.all.am looks OK. >>> >>>> +Upstream-Status: Pending >>>> + >>>> +Signed-off-by: Khem Raj <raj.khem@gmail.com> >>>> +--- >>>> + Makefile.all.am | 6 +++--- >>>> + helgrind/tests/Makefile.am | 6 +++--- >>>> + none/tests/arm/Makefile.am | 18 +++++++++--------- >>>> + 3 files changed, 15 insertions(+), 15 deletions(-) >>>> + >>>> +diff --git a/Makefile.all.am b/Makefile.all.am >>>> +index 02059a3..c7c4700 100644 >>>> +--- a/Makefile.all.am >>>> ++++ b/Makefile.all.am >>>> +@@ -197,11 +197,11 @@ AM_CCASFLAGS_PPC64LE_LINUX = @FLAG_M64@ -g >>>> + >>>> + AM_FLAG_M3264_ARM_LINUX = @FLAG_M32@ >>>> + AM_CFLAGS_ARM_LINUX = @FLAG_M32@ \ >>>> +- $(AM_CFLAGS_BASE) -marm -mcpu=cortex-a8 >>>> ++ $(AM_CFLAGS_BASE) -marm >>>> + AM_CFLAGS_PSO_ARM_LINUX = @FLAG_M32@ $(AM_CFLAGS_BASE) \ >>>> +- -marm -mcpu=cortex-a8 $(AM_CFLAGS_PSO_BASE) >>>> ++ -marm $(AM_CFLAGS_PSO_BASE) >>>> + AM_CCASFLAGS_ARM_LINUX = @FLAG_M32@ \ >>>> +- -marm -mcpu=cortex-a8 -g >>>> ++ -marm -g >>>> + >>>> + AM_FLAG_M3264_ARM64_LINUX = @FLAG_M64@ >>>> + AM_CFLAGS_ARM64_LINUX = @FLAG_M64@ $(AM_CFLAGS_BASE) >>>> +diff --git a/helgrind/tests/Makefile.am b/helgrind/tests/Makefile.am >>>> +index df82169..07eb66a 100644 >>>> +--- a/helgrind/tests/Makefile.am >>>> ++++ b/helgrind/tests/Makefile.am >>>> +@@ -189,9 +189,9 @@ if ! VGCONF_PLATFORMS_INCLUDE_X86_DARWIN >>>> + endif >>>> + >>>> + if VGCONF_PLATFORMS_INCLUDE_ARM_LINUX >>>> +-annotate_hbefore_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 >>>> +-tc07_hbl1_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 >>>> +-tc08_hbl2_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 >>>> ++annotate_hbefore_CFLAGS = $(AM_CFLAGS) >>>> ++tc07_hbl1_CFLAGS = $(AM_CFLAGS) >>>> ++tc08_hbl2_CFLAGS = $(AM_CFLAGS) >>>> + else >>>> + annotate_hbefore_CFLAGS = $(AM_CFLAGS) >>>> + tc07_hbl1_CFLAGS = $(AM_CFLAGS) >>>> +diff --git a/none/tests/arm/Makefile.am b/none/tests/arm/Makefile.am >>>> +index 024eb6d..ccecb90 100644 >>>> +--- a/none/tests/arm/Makefile.am >>>> ++++ b/none/tests/arm/Makefile.am >>>> +@@ -52,10 +52,10 @@ allexec_CFLAGS = $(AM_CFLAGS) @FLAG_W_NO_NONNULL@ >>>> + # need special helping w.r.t -mfpu and -mfloat-abi, though. >>>> + # Also force -O0 since -O takes hundreds of MB of memory >>>> + # for v6intThumb.c. >>>> +-v6intARM_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -marm >>>> +-v6intThumb_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -mthumb >>>> ++v6intARM_CFLAGS = $(AM_CFLAGS) -g -O0 -marm >>>> ++v6intThumb_CFLAGS = $(AM_CFLAGS) -g -O0 -mthumb >>>> + >>>> +-v6media_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -mthumb >>>> ++v6media_CFLAGS = $(AM_CFLAGS) -g -O0 -mthumb >>>> + >>>> + v8crypto_a_CFLAGS = $(AM_CFLAGS) -g -O0 -mfpu=crypto-neon-fp-armv8 -marm >>>> + v8crypto_t_CFLAGS = $(AM_CFLAGS) -g -O0 -mfpu=crypto-neon-fp-armv8 -mthumb >>>> +@@ -65,23 +65,23 @@ v8memory_a_CFLAGS = $(AM_CFLAGS) -g -O0 \ >>>> + v8memory_t_CFLAGS = $(AM_CFLAGS) -g -O0 \ >>>> + -march=armv8-a -mfpu=crypto-neon-fp-armv8 -mthumb >>>> + >>>> +-vfp_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ >>>> ++vfp_CFLAGS = $(AM_CFLAGS) -g -O0 \ >>>> + -mfpu=neon \ >>>> + -mthumb >>>> + >>>> + >>>> +-neon128_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ >>>> ++neon128_CFLAGS = $(AM_CFLAGS) -g -O0 \ >>>> + -mfpu=neon \ >>>> + -mthumb >>>> + >>>> +-neon64_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ >>>> ++neon64_CFLAGS = $(AM_CFLAGS) -g -O0 \ >>>> + -mfpu=neon \ >>>> + -mthumb >>>> + >>>> + intdiv_CFLAGS = $(AM_CFLAGS) -g -march=armv7ve -mcpu=cortex-a15 -mthumb >>>> +-ldrt_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -mthumb >>>> +-ldrt_arm_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -marm >>>> ++ldrt_CFLAGS = $(AM_CFLAGS) -g -mthumb >>>> ++ldrt_arm_CFLAGS = $(AM_CFLAGS) -g -marm >>>> + >>>> +-vcvt_fixed_float_VFP_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -mfpu=vfpv3 >>>> ++vcvt_fixed_float_VFP_CFLAGS = $(AM_CFLAGS) -g -mfpu=vfpv3 >>>> + >>>> + vfpv4_fma_CFLAGS = $(AM_CFLAGS) -g -O0 -march=armv7ve -mcpu=cortex-a15 -mfpu=vfpv4 -marm >>>> +-- >>>> +2.12.2 >>>> + >>>> diff --git a/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb b/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb >>>> index 62a96354cd0..a07c8767ce3 100644 >>>> --- a/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb >>>> +++ b/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb >>>> @@ -23,7 +23,8 @@ SRC_URI = "http://www.valgrind.org/downloads/valgrind-${PV}.tar.bz2 \ >>>> file://use-appropriate-march-mcpu-mfpu-for-ARM-test-apps.patch \ >>>> file://avoid-neon-for-targets-which-don-t-support-it.patch \ >>>> file://valgrind-make-ld-XXX.so-strlen-intercept-optional.patch \ >>>> -" >>>> + file://0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch \ >>>> + " >>>> SRC_URI_append_libc-musl = "\ >>>> file://0001-fix-build-for-musl-targets.patch \ >>>> " >>>> -- >>>> 2.12.2 >>>> >>>> -- >>>> _______________________________________________ >>>> Openembedded-core mailing list >>>> Openembedded-core@lists.openembedded.org >>>> http://lists.openembedded.org/mailman/listinfo/openembedded-core -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
On Fri, Apr 21, 2017 at 9:32 AM, Khem Raj <raj.khem@gmail.com> wrote: > On Thu, Apr 20, 2017 at 6:29 PM, Andre McCurdy <armccurdy@gmail.com> wrote: >> On Thu, Apr 20, 2017 at 6:06 PM, Khem Raj <raj.khem@gmail.com> wrote: >>> On Thu, Apr 20, 2017 at 5:56 PM, Andre McCurdy <armccurdy@gmail.com> wrote: >>>> On Thu, Apr 20, 2017 at 11:20 AM, Khem Raj <raj.khem@gmail.com> wrote: >>>>> Signed-off-by: Khem Raj <raj.khem@gmail.com> >>>>> --- >>>>> ...rop-setting-mcpu-to-cortex-a8-on-arm-arch.patch | 108 +++++++++++++++++++++ >>>>> meta/recipes-devtools/valgrind/valgrind_3.12.0.bb | 3 +- >>>>> 2 files changed, 110 insertions(+), 1 deletion(-) >>>>> create mode 100644 meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch >>>>> >>>>> diff --git a/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch b/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch >>>>> new file mode 100644 >>>>> index 00000000000..9f1da7bac68 >>>>> --- /dev/null >>>>> +++ b/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch >>>>> @@ -0,0 +1,108 @@ >>>>> +From 715cf122388f3527afa5649cebf9f1522c240693 Mon Sep 17 00:00:00 2001 >>>>> +From: Khem Raj <raj.khem@gmail.com> >>>>> +Date: Thu, 20 Apr 2017 10:11:16 -0700 >>>>> +Subject: [PATCH] makefiles: Drop setting -mcpu to cortex-a8 on arm >>>>> + architecture >>>>> + >>>>> +We can not assume that all arches armv7+ are cortex-a8 only >>>>> +it fails to build for rpi which is armv7ve based (cortex-a8) cpu >>>>> +implementation. >>>>> +Fixes >>>>> +| cc1: warning: switch -mcpu=cortex-a8 conflicts with -march=armv7ve switch >>>> >>>> The valgrind tests are built for fixed CPU targets which are known to >>>> support the instructions being tested (since those instructions may >>>> not be supported by the target CPU which valgrind itself is being >>>> built for). >>>> >>>> It may be safer to add -march=armv7a to the -mcpu=cortex-a8 tests (and >>>> ensure that -march=armv7ve doesn't leak through) rather than remove >>>> -mcpu=cortex-a8. >>> >>> OE always passes mcpu flags, therefore it wont be an issue unless you >>> are saying thats not the case. >> >> OE won't pass an -mcpu flag for generic DEFAULTTUNE values such as "armv7athf". >> >>> For upstreaming it might be OK, but I >>> think this will need a different for upstreaming since it should >>> configure mcpu/march based on some configure input or some such >>> instead of hardcoding it. >> >> For the ../tests/... Makefiles, the mcpu/march flags should absolutely >> be hardcoded. e.g. the test for "idiv" should always be built in >> exactly the same way, regardless of whether or not the target arch you >> configure and build valgrind itself for is armv7ve. >> > > what is the purpose of compiling such a test case for a machine > where it will cause SIGILL ? I don't think it will when executed via valgrind, since valgrind acts as an interpreter for the binaries it runs. In the past I've used valgrind to run an x86 binary which used cmov on a VIA C3 CPU which didn't support that instruction. I guess a similar approach would work for ARM - ie you could use valgrind to run an armv7ve binary on an armv7a CPU. >>>> >>>> See the comments in: >>>> >>>> http://git.openembedded.org/openembedded-core/commit/?id=3e94af90d90f6db6bab66ae87a47b31f3a0474f6 >>>> >>>> The change to the toplevel Makefile.all.am looks OK. >>>> >>>>> +Upstream-Status: Pending >>>>> + >>>>> +Signed-off-by: Khem Raj <raj.khem@gmail.com> >>>>> +--- >>>>> + Makefile.all.am | 6 +++--- >>>>> + helgrind/tests/Makefile.am | 6 +++--- >>>>> + none/tests/arm/Makefile.am | 18 +++++++++--------- >>>>> + 3 files changed, 15 insertions(+), 15 deletions(-) >>>>> + >>>>> +diff --git a/Makefile.all.am b/Makefile.all.am >>>>> +index 02059a3..c7c4700 100644 >>>>> +--- a/Makefile.all.am >>>>> ++++ b/Makefile.all.am >>>>> +@@ -197,11 +197,11 @@ AM_CCASFLAGS_PPC64LE_LINUX = @FLAG_M64@ -g >>>>> + >>>>> + AM_FLAG_M3264_ARM_LINUX = @FLAG_M32@ >>>>> + AM_CFLAGS_ARM_LINUX = @FLAG_M32@ \ >>>>> +- $(AM_CFLAGS_BASE) -marm -mcpu=cortex-a8 >>>>> ++ $(AM_CFLAGS_BASE) -marm >>>>> + AM_CFLAGS_PSO_ARM_LINUX = @FLAG_M32@ $(AM_CFLAGS_BASE) \ >>>>> +- -marm -mcpu=cortex-a8 $(AM_CFLAGS_PSO_BASE) >>>>> ++ -marm $(AM_CFLAGS_PSO_BASE) >>>>> + AM_CCASFLAGS_ARM_LINUX = @FLAG_M32@ \ >>>>> +- -marm -mcpu=cortex-a8 -g >>>>> ++ -marm -g >>>>> + >>>>> + AM_FLAG_M3264_ARM64_LINUX = @FLAG_M64@ >>>>> + AM_CFLAGS_ARM64_LINUX = @FLAG_M64@ $(AM_CFLAGS_BASE) >>>>> +diff --git a/helgrind/tests/Makefile.am b/helgrind/tests/Makefile.am >>>>> +index df82169..07eb66a 100644 >>>>> +--- a/helgrind/tests/Makefile.am >>>>> ++++ b/helgrind/tests/Makefile.am >>>>> +@@ -189,9 +189,9 @@ if ! VGCONF_PLATFORMS_INCLUDE_X86_DARWIN >>>>> + endif >>>>> + >>>>> + if VGCONF_PLATFORMS_INCLUDE_ARM_LINUX >>>>> +-annotate_hbefore_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 >>>>> +-tc07_hbl1_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 >>>>> +-tc08_hbl2_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 >>>>> ++annotate_hbefore_CFLAGS = $(AM_CFLAGS) >>>>> ++tc07_hbl1_CFLAGS = $(AM_CFLAGS) >>>>> ++tc08_hbl2_CFLAGS = $(AM_CFLAGS) >>>>> + else >>>>> + annotate_hbefore_CFLAGS = $(AM_CFLAGS) >>>>> + tc07_hbl1_CFLAGS = $(AM_CFLAGS) >>>>> +diff --git a/none/tests/arm/Makefile.am b/none/tests/arm/Makefile.am >>>>> +index 024eb6d..ccecb90 100644 >>>>> +--- a/none/tests/arm/Makefile.am >>>>> ++++ b/none/tests/arm/Makefile.am >>>>> +@@ -52,10 +52,10 @@ allexec_CFLAGS = $(AM_CFLAGS) @FLAG_W_NO_NONNULL@ >>>>> + # need special helping w.r.t -mfpu and -mfloat-abi, though. >>>>> + # Also force -O0 since -O takes hundreds of MB of memory >>>>> + # for v6intThumb.c. >>>>> +-v6intARM_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -marm >>>>> +-v6intThumb_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -mthumb >>>>> ++v6intARM_CFLAGS = $(AM_CFLAGS) -g -O0 -marm >>>>> ++v6intThumb_CFLAGS = $(AM_CFLAGS) -g -O0 -mthumb >>>>> + >>>>> +-v6media_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -mthumb >>>>> ++v6media_CFLAGS = $(AM_CFLAGS) -g -O0 -mthumb >>>>> + >>>>> + v8crypto_a_CFLAGS = $(AM_CFLAGS) -g -O0 -mfpu=crypto-neon-fp-armv8 -marm >>>>> + v8crypto_t_CFLAGS = $(AM_CFLAGS) -g -O0 -mfpu=crypto-neon-fp-armv8 -mthumb >>>>> +@@ -65,23 +65,23 @@ v8memory_a_CFLAGS = $(AM_CFLAGS) -g -O0 \ >>>>> + v8memory_t_CFLAGS = $(AM_CFLAGS) -g -O0 \ >>>>> + -march=armv8-a -mfpu=crypto-neon-fp-armv8 -mthumb >>>>> + >>>>> +-vfp_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ >>>>> ++vfp_CFLAGS = $(AM_CFLAGS) -g -O0 \ >>>>> + -mfpu=neon \ >>>>> + -mthumb >>>>> + >>>>> + >>>>> +-neon128_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ >>>>> ++neon128_CFLAGS = $(AM_CFLAGS) -g -O0 \ >>>>> + -mfpu=neon \ >>>>> + -mthumb >>>>> + >>>>> +-neon64_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ >>>>> ++neon64_CFLAGS = $(AM_CFLAGS) -g -O0 \ >>>>> + -mfpu=neon \ >>>>> + -mthumb >>>>> + >>>>> + intdiv_CFLAGS = $(AM_CFLAGS) -g -march=armv7ve -mcpu=cortex-a15 -mthumb >>>>> +-ldrt_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -mthumb >>>>> +-ldrt_arm_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -marm >>>>> ++ldrt_CFLAGS = $(AM_CFLAGS) -g -mthumb >>>>> ++ldrt_arm_CFLAGS = $(AM_CFLAGS) -g -marm >>>>> + >>>>> +-vcvt_fixed_float_VFP_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -mfpu=vfpv3 >>>>> ++vcvt_fixed_float_VFP_CFLAGS = $(AM_CFLAGS) -g -mfpu=vfpv3 >>>>> + >>>>> + vfpv4_fma_CFLAGS = $(AM_CFLAGS) -g -O0 -march=armv7ve -mcpu=cortex-a15 -mfpu=vfpv4 -marm >>>>> +-- >>>>> +2.12.2 >>>>> + >>>>> diff --git a/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb b/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb >>>>> index 62a96354cd0..a07c8767ce3 100644 >>>>> --- a/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb >>>>> +++ b/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb >>>>> @@ -23,7 +23,8 @@ SRC_URI = "http://www.valgrind.org/downloads/valgrind-${PV}.tar.bz2 \ >>>>> file://use-appropriate-march-mcpu-mfpu-for-ARM-test-apps.patch \ >>>>> file://avoid-neon-for-targets-which-don-t-support-it.patch \ >>>>> file://valgrind-make-ld-XXX.so-strlen-intercept-optional.patch \ >>>>> -" >>>>> + file://0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch \ >>>>> + " >>>>> SRC_URI_append_libc-musl = "\ >>>>> file://0001-fix-build-for-musl-targets.patch \ >>>>> " >>>>> -- >>>>> 2.12.2 >>>>> >>>>> -- >>>>> _______________________________________________ >>>>> Openembedded-core mailing list >>>>> Openembedded-core@lists.openembedded.org >>>>> http://lists.openembedded.org/mailman/listinfo/openembedded-core -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
diff --git a/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch b/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch new file mode 100644 index 00000000000..9f1da7bac68 --- /dev/null +++ b/meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch @@ -0,0 +1,108 @@ +From 715cf122388f3527afa5649cebf9f1522c240693 Mon Sep 17 00:00:00 2001 +From: Khem Raj <raj.khem@gmail.com> +Date: Thu, 20 Apr 2017 10:11:16 -0700 +Subject: [PATCH] makefiles: Drop setting -mcpu to cortex-a8 on arm + architecture + +We can not assume that all arches armv7+ are cortex-a8 only +it fails to build for rpi which is armv7ve based (cortex-a8) cpu +implementation. +Fixes +| cc1: warning: switch -mcpu=cortex-a8 conflicts with -march=armv7ve switch + +Upstream-Status: Pending + +Signed-off-by: Khem Raj <raj.khem@gmail.com> +--- + Makefile.all.am | 6 +++--- + helgrind/tests/Makefile.am | 6 +++--- + none/tests/arm/Makefile.am | 18 +++++++++--------- + 3 files changed, 15 insertions(+), 15 deletions(-) + +diff --git a/Makefile.all.am b/Makefile.all.am +index 02059a3..c7c4700 100644 +--- a/Makefile.all.am ++++ b/Makefile.all.am +@@ -197,11 +197,11 @@ AM_CCASFLAGS_PPC64LE_LINUX = @FLAG_M64@ -g + + AM_FLAG_M3264_ARM_LINUX = @FLAG_M32@ + AM_CFLAGS_ARM_LINUX = @FLAG_M32@ \ +- $(AM_CFLAGS_BASE) -marm -mcpu=cortex-a8 ++ $(AM_CFLAGS_BASE) -marm + AM_CFLAGS_PSO_ARM_LINUX = @FLAG_M32@ $(AM_CFLAGS_BASE) \ +- -marm -mcpu=cortex-a8 $(AM_CFLAGS_PSO_BASE) ++ -marm $(AM_CFLAGS_PSO_BASE) + AM_CCASFLAGS_ARM_LINUX = @FLAG_M32@ \ +- -marm -mcpu=cortex-a8 -g ++ -marm -g + + AM_FLAG_M3264_ARM64_LINUX = @FLAG_M64@ + AM_CFLAGS_ARM64_LINUX = @FLAG_M64@ $(AM_CFLAGS_BASE) +diff --git a/helgrind/tests/Makefile.am b/helgrind/tests/Makefile.am +index df82169..07eb66a 100644 +--- a/helgrind/tests/Makefile.am ++++ b/helgrind/tests/Makefile.am +@@ -189,9 +189,9 @@ if ! VGCONF_PLATFORMS_INCLUDE_X86_DARWIN + endif + + if VGCONF_PLATFORMS_INCLUDE_ARM_LINUX +-annotate_hbefore_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 +-tc07_hbl1_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 +-tc08_hbl2_CFLAGS = $(AM_CFLAGS) -mcpu=cortex-a8 ++annotate_hbefore_CFLAGS = $(AM_CFLAGS) ++tc07_hbl1_CFLAGS = $(AM_CFLAGS) ++tc08_hbl2_CFLAGS = $(AM_CFLAGS) + else + annotate_hbefore_CFLAGS = $(AM_CFLAGS) + tc07_hbl1_CFLAGS = $(AM_CFLAGS) +diff --git a/none/tests/arm/Makefile.am b/none/tests/arm/Makefile.am +index 024eb6d..ccecb90 100644 +--- a/none/tests/arm/Makefile.am ++++ b/none/tests/arm/Makefile.am +@@ -52,10 +52,10 @@ allexec_CFLAGS = $(AM_CFLAGS) @FLAG_W_NO_NONNULL@ + # need special helping w.r.t -mfpu and -mfloat-abi, though. + # Also force -O0 since -O takes hundreds of MB of memory + # for v6intThumb.c. +-v6intARM_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -marm +-v6intThumb_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -mthumb ++v6intARM_CFLAGS = $(AM_CFLAGS) -g -O0 -marm ++v6intThumb_CFLAGS = $(AM_CFLAGS) -g -O0 -mthumb + +-v6media_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -mthumb ++v6media_CFLAGS = $(AM_CFLAGS) -g -O0 -mthumb + + v8crypto_a_CFLAGS = $(AM_CFLAGS) -g -O0 -mfpu=crypto-neon-fp-armv8 -marm + v8crypto_t_CFLAGS = $(AM_CFLAGS) -g -O0 -mfpu=crypto-neon-fp-armv8 -mthumb +@@ -65,23 +65,23 @@ v8memory_a_CFLAGS = $(AM_CFLAGS) -g -O0 \ + v8memory_t_CFLAGS = $(AM_CFLAGS) -g -O0 \ + -march=armv8-a -mfpu=crypto-neon-fp-armv8 -mthumb + +-vfp_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ ++vfp_CFLAGS = $(AM_CFLAGS) -g -O0 \ + -mfpu=neon \ + -mthumb + + +-neon128_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ ++neon128_CFLAGS = $(AM_CFLAGS) -g -O0 \ + -mfpu=neon \ + -mthumb + +-neon64_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \ ++neon64_CFLAGS = $(AM_CFLAGS) -g -O0 \ + -mfpu=neon \ + -mthumb + + intdiv_CFLAGS = $(AM_CFLAGS) -g -march=armv7ve -mcpu=cortex-a15 -mthumb +-ldrt_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -mthumb +-ldrt_arm_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -marm ++ldrt_CFLAGS = $(AM_CFLAGS) -g -mthumb ++ldrt_arm_CFLAGS = $(AM_CFLAGS) -g -marm + +-vcvt_fixed_float_VFP_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -mfpu=vfpv3 ++vcvt_fixed_float_VFP_CFLAGS = $(AM_CFLAGS) -g -mfpu=vfpv3 + + vfpv4_fma_CFLAGS = $(AM_CFLAGS) -g -O0 -march=armv7ve -mcpu=cortex-a15 -mfpu=vfpv4 -marm +-- +2.12.2 + diff --git a/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb b/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb index 62a96354cd0..a07c8767ce3 100644 --- a/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb +++ b/meta/recipes-devtools/valgrind/valgrind_3.12.0.bb @@ -23,7 +23,8 @@ SRC_URI = "http://www.valgrind.org/downloads/valgrind-${PV}.tar.bz2 \ file://use-appropriate-march-mcpu-mfpu-for-ARM-test-apps.patch \ file://avoid-neon-for-targets-which-don-t-support-it.patch \ file://valgrind-make-ld-XXX.so-strlen-intercept-optional.patch \ -" + file://0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch \ + " SRC_URI_append_libc-musl = "\ file://0001-fix-build-for-musl-targets.patch \ "
Signed-off-by: Khem Raj <raj.khem@gmail.com> --- ...rop-setting-mcpu-to-cortex-a8-on-arm-arch.patch | 108 +++++++++++++++++++++ meta/recipes-devtools/valgrind/valgrind_3.12.0.bb | 3 +- 2 files changed, 110 insertions(+), 1 deletion(-) create mode 100644 meta/recipes-devtools/valgrind/valgrind/0001-makefiles-Drop-setting-mcpu-to-cortex-a8-on-arm-arch.patch -- 2.12.2 -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core