* [PATCH V2 1/3] binutils: Let crosssdk gold linker generate 4096 btyes long .interp section
@ 2020-08-20 8:01 Khem Raj
2020-08-20 8:01 ` [PATCH V2 2/3] gcc-cross-canadian: Install gcc/g++ wrappers for musl Khem Raj
2020-08-20 8:01 ` [PATCH V2 3/3] gcc-cross-canadian: Correct the regexp to delete versioned gcc binary Khem Raj
0 siblings, 2 replies; 7+ messages in thread
From: Khem Raj @ 2020-08-20 8:01 UTC (permalink / raw)
To: openembedded-core; +Cc: Khem Raj
When gold is used as default linker in crosssdk e.g. when building SDK
binaries with LTO, the binaries do not have large enough .interp
section size and SDK relocation fails for those nativesdk binaries and libraries
which used gold for linking. This patch extends the .interp relaxation
fix to gold
Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
...s-crosssdk-Generate-relocatable-SDKs.patch | 26 +++++++++++++++----
1 file changed, 21 insertions(+), 5 deletions(-)
diff --git a/meta/recipes-devtools/binutils/binutils/0001-binutils-crosssdk-Generate-relocatable-SDKs.patch b/meta/recipes-devtools/binutils/binutils/0001-binutils-crosssdk-Generate-relocatable-SDKs.patch
index 0e62d47616..4c1d11291b 100644
--- a/meta/recipes-devtools/binutils/binutils/0001-binutils-crosssdk-Generate-relocatable-SDKs.patch
+++ b/meta/recipes-devtools/binutils/binutils/0001-binutils-crosssdk-Generate-relocatable-SDKs.patch
@@ -1,22 +1,38 @@
-From b8d182865081d17549fb9a4b9bc3062b526caf65 Mon Sep 17 00:00:00 2001
+From a0b23b160d6cfa7be4437c6e623633d76395f2ad Mon Sep 17 00:00:00 2001
From: Khem Raj <raj.khem@gmail.com>
Date: Mon, 2 Mar 2015 01:58:54 +0000
-Subject: [PATCH 01/17] binutils-crosssdk: Generate relocatable SDKs
+Subject: [PATCH 01/16] binutils-crosssdk: Generate relocatable SDKs
This patch will modify the ELF linker scripts so that the crosssdk
linker will generate binaries with a 4096 bytes PT_INTERP section. When the binaries
will be relocated, at SDK install time, the interpreter path can be easily
changed by the relocating script.
+generate larger .interp section for gold linker as well
+
Upstream-Status: Inappropriate [SDK specific]
Signed-off-by: Laurentiu Palcu <laurentiu.palcu@intel.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
+ gold/layout.cc | 2 +-
ld/genscripts.sh | 3 +++
ld/scripttempl/elf.sc | 4 ++--
- 2 files changed, 5 insertions(+), 2 deletions(-)
+ 3 files changed, 6 insertions(+), 3 deletions(-)
+diff --git a/gold/layout.cc b/gold/layout.cc
+index 13e533aaf21..b0afff16e2e 100644
+--- a/gold/layout.cc
++++ b/gold/layout.cc
+@@ -5019,7 +5019,7 @@ Layout::create_interp(const Target* target)
+ gold_assert(interp != NULL);
+ }
+
+- size_t len = strlen(interp) + 1;
++ size_t len = 4096;
+
+ Output_section_data* odata = new Output_data_const(interp, len, 1);
+
diff --git a/ld/genscripts.sh b/ld/genscripts.sh
index 03392d265c7..435689ea144 100755
--- a/ld/genscripts.sh
@@ -43,10 +59,10 @@ index 03392d265c7..435689ea144 100755
DATA_ALIGNMENT=${DATA_ALIGNMENT_}
RELOCATING=" "
diff --git a/ld/scripttempl/elf.sc b/ld/scripttempl/elf.sc
-index 0b8b32a4407..ee6b71075d7 100644
+index eb74743e5c3..c9a8a47615f 100644
--- a/ld/scripttempl/elf.sc
+++ b/ld/scripttempl/elf.sc
-@@ -140,8 +140,8 @@ if test -z "$DATA_SEGMENT_ALIGN"; then
+@@ -143,8 +143,8 @@ if test -z "$DATA_SEGMENT_ALIGN"; then
DATA_SEGMENT_RELRO_END=". = DATA_SEGMENT_RELRO_END (${SEPARATE_GOTPLT-0}, .);"
fi
fi
--
2.28.0
^ permalink raw reply related [flat|nested] 7+ messages in thread* [PATCH V2 2/3] gcc-cross-canadian: Install gcc/g++ wrappers for musl
2020-08-20 8:01 [PATCH V2 1/3] binutils: Let crosssdk gold linker generate 4096 btyes long .interp section Khem Raj
@ 2020-08-20 8:01 ` Khem Raj
2020-08-20 21:07 ` Leon Woestenberg
2020-08-21 10:51 ` [OE-core] " Richard Purdie
2020-08-20 8:01 ` [PATCH V2 3/3] gcc-cross-canadian: Correct the regexp to delete versioned gcc binary Khem Raj
1 sibling, 2 replies; 7+ messages in thread
From: Khem Raj @ 2020-08-20 8:01 UTC (permalink / raw)
To: openembedded-core; +Cc: Khem Raj, Leon Woestenberg
gcc needs -mmusl option to be passed in SDK since we ship crossdk compiler
configured for glibc by default, this helps in creating correct
compiler defaults for musl based SDK compilers
[YOCTO #13459]
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Cc: Leon Woestenberg <leon@sidebranch.com>
---
v2: Delete file before creating wrapper
.../gcc/gcc-cross-canadian.inc | 23 +++++++++++++++++++
1 file changed, 23 insertions(+)
diff --git a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
index 553ef7fe62..c5935b978d 100644
--- a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
+++ b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
@@ -139,6 +139,29 @@ do_install () {
chown -R root:root ${D}
cross_canadian_bindirlinks
+
+ for i in linux ${CANADIANEXTRAOS}
+ do
+ for v in ${CANADIANEXTRAVENDOR}
+ do
+ d=${D}${bindir}/../${TARGET_ARCH}$v-$i
+ install -d $d
+ for j in ${TARGET_PREFIX}gcc${EXEEXT} ${TARGET_PREFIX}g++${EXEEXT}
+ do
+ p=${TARGET_ARCH}$v-$i-`echo $j | sed -e s,${TARGET_PREFIX},,`
+ case $i in
+ *musl*)
+ rm -rf $d/$p
+ echo "#!/usr/bin/env sh" > $d/$p
+ echo "exec \`dirname \$0\`/../${TARGET_SYS}/$j -mmusl \$@" >> $d/$p
+ chmod 0755 $d/$p
+ ;;
+ *)
+ ;;
+ esac
+ done
+ done
+ done
}
ELFUTILS = "nativesdk-elfutils"
--
2.28.0
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH V2 2/3] gcc-cross-canadian: Install gcc/g++ wrappers for musl
2020-08-20 8:01 ` [PATCH V2 2/3] gcc-cross-canadian: Install gcc/g++ wrappers for musl Khem Raj
@ 2020-08-20 21:07 ` Leon Woestenberg
2020-08-20 22:51 ` Khem Raj
2020-08-21 10:51 ` [OE-core] " Richard Purdie
1 sibling, 1 reply; 7+ messages in thread
From: Leon Woestenberg @ 2020-08-20 21:07 UTC (permalink / raw)
To: Khem Raj; +Cc: OE Core mailing list
On Thu, Aug 20, 2020 at 10:01 AM Khem Raj <raj.khem@gmail.com> wrote:
>
> gcc needs -mmusl option to be passed in SDK since we ship crossdk compiler
> configured for glibc by default, this helps in creating correct
> compiler defaults for musl based SDK compilers
>
> [YOCTO #13459]
>
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> Cc: Leon Woestenberg <leon@sidebranch.com>
> ---
> v2: Delete file before creating wrapper
>
Thanks, v2 works and solves [YOCTO #13459]. We already independently
had the same v1->v2 fix locally, while we tested your v1 approach
(and ignored the mailing list...)
Tested-by: Leon Woestenberg <leon@sidebranch.com>
Acked-by: Leon Woestenberg <leon@sidebranch.com>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH V2 2/3] gcc-cross-canadian: Install gcc/g++ wrappers for musl
2020-08-20 21:07 ` Leon Woestenberg
@ 2020-08-20 22:51 ` Khem Raj
0 siblings, 0 replies; 7+ messages in thread
From: Khem Raj @ 2020-08-20 22:51 UTC (permalink / raw)
To: Leon Woestenberg; +Cc: OE Core mailing list
On Thu, Aug 20, 2020 at 2:07 PM Leon Woestenberg <leon@sidebranch.com> wrote:
>
> On Thu, Aug 20, 2020 at 10:01 AM Khem Raj <raj.khem@gmail.com> wrote:
> >
> > gcc needs -mmusl option to be passed in SDK since we ship crossdk compiler
> > configured for glibc by default, this helps in creating correct
> > compiler defaults for musl based SDK compilers
> >
> > [YOCTO #13459]
> >
> > Signed-off-by: Khem Raj <raj.khem@gmail.com>
> > Cc: Leon Woestenberg <leon@sidebranch.com>
> > ---
> > v2: Delete file before creating wrapper
> >
> Thanks, v2 works and solves [YOCTO #13459]. We already independently
> had the same v1->v2 fix locally, while we tested your v1 approach
> (and ignored the mailing list...)
>
> Tested-by: Leon Woestenberg <leon@sidebranch.com>
> Acked-by: Leon Woestenberg <leon@sidebranch.com>
Great thanks for confirmation, happy musl'ing :)
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [OE-core] [PATCH V2 2/3] gcc-cross-canadian: Install gcc/g++ wrappers for musl
2020-08-20 8:01 ` [PATCH V2 2/3] gcc-cross-canadian: Install gcc/g++ wrappers for musl Khem Raj
2020-08-20 21:07 ` Leon Woestenberg
@ 2020-08-21 10:51 ` Richard Purdie
2020-08-21 19:51 ` Khem Raj
1 sibling, 1 reply; 7+ messages in thread
From: Richard Purdie @ 2020-08-21 10:51 UTC (permalink / raw)
To: Khem Raj, openembedded-core; +Cc: Leon Woestenberg
On Thu, 2020-08-20 at 01:01 -0700, Khem Raj wrote:
> gcc needs -mmusl option to be passed in SDK since we ship crossdk compiler
> configured for glibc by default, this helps in creating correct
> compiler defaults for musl based SDK compilers
>
> [YOCTO #13459]
>
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> Cc: Leon Woestenberg <leon@sidebranch.com>
> ---
> v2: Delete file before creating wrapper
>
> .../gcc/gcc-cross-canadian.inc | 23 +++++++++++++++++++
> 1 file changed, 23 insertions(+)
>
> diff --git a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
> index 553ef7fe62..c5935b978d 100644
> --- a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
> +++ b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
> @@ -139,6 +139,29 @@ do_install () {
> chown -R root:root ${D}
>
> cross_canadian_bindirlinks
> +
> + for i in linux ${CANADIANEXTRAOS}
> + do
> + for v in ${CANADIANEXTRAVENDOR}
> + do
> + d=${D}${bindir}/../${TARGET_ARCH}$v-$i
> + install -d $d
> + for j in ${TARGET_PREFIX}gcc${EXEEXT} ${TARGET_PREFIX}g++${EXEEXT}
> + do
> + p=${TARGET_ARCH}$v-$i-`echo $j | sed -e s,${TARGET_PREFIX},,`
> + case $i in
> + *musl*)
> + rm -rf $d/$p
> + echo "#!/usr/bin/env sh" > $d/$p
> + echo "exec \`dirname \$0\`/../${TARGET_SYS}/$j -mmusl \$@" >> $d/$p
> + chmod 0755 $d/$p
> + ;;
> + *)
> + ;;
> + esac
> + done
> + done
> + done
> }
>
> ELFUTILS = "nativesdk-elfutils"
I've bisected this error in mingw:
https://autobuilder.yoctoproject.org/typhoon/#/builders/89/builds/2341
to this change.
Error:
Problem: package gcc-cross-canadian-i686-10.2.0-r0.i686_nativesdk_mingw32 requires /usr/bin/env, but none of the providers can be installed
- package packagegroup-cross-canadian-qemux86-1.0-r0.i686_nativesdk_mingw32 requires gcc-cross-canadian-i686, but none of the providers can be installed
- package target-sdk-provides-dummy-1.0-r0.sdk_provides_dummy_target conflicts with /bin/sh provided by nativesdk-curl-7.71.1-r0.i686_nativesdk_mingw32
- conflicting requests
(try to add '--allowerasing' to command line to replace conflicting packages or '--skip-broken' to skip uninstallable packages)
This is due to the addition of the /usr/bin/env dependency to the
generated files.
I'm trying to unravel the problems but it may mean extra fixes to both
meta-mingw and master handling of /bin/sh.
Cheers,
Richard
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [OE-core] [PATCH V2 2/3] gcc-cross-canadian: Install gcc/g++ wrappers for musl
2020-08-21 10:51 ` [OE-core] " Richard Purdie
@ 2020-08-21 19:51 ` Khem Raj
0 siblings, 0 replies; 7+ messages in thread
From: Khem Raj @ 2020-08-21 19:51 UTC (permalink / raw)
To: Richard Purdie; +Cc: Leon Woestenberg, openembedded-core
[-- Attachment #1: Type: text/plain, Size: 3555 bytes --]
On Fri, Aug 21, 2020 at 3:51 AM Richard Purdie <
richard.purdie@linuxfoundation.org> wrote:
> On Thu, 2020-08-20 at 01:01 -0700, Khem Raj wrote:
>
> > gcc needs -mmusl option to be passed in SDK since we ship crossdk
> compiler
>
> > configured for glibc by default, this helps in creating correct
>
> > compiler defaults for musl based SDK compilers
>
> >
>
> > [YOCTO #13459]
>
> >
>
> > Signed-off-by: Khem Raj <raj.khem@gmail.com>
>
> > Cc: Leon Woestenberg <leon@sidebranch.com>
>
> > ---
>
> > v2: Delete file before creating wrapper
>
> >
>
> > .../gcc/gcc-cross-canadian.inc | 23 +++++++++++++++++++
>
> > 1 file changed, 23 insertions(+)
>
> >
>
> > diff --git a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
> b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
>
> > index 553ef7fe62..c5935b978d 100644
>
> > --- a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
>
> > +++ b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
>
> > @@ -139,6 +139,29 @@ do_install () {
>
> > chown -R root:root ${D}
>
> >
>
> > cross_canadian_bindirlinks
>
> > +
>
> > + for i in linux ${CANADIANEXTRAOS}
>
> > + do
>
> > + for v in ${CANADIANEXTRAVENDOR}
>
> > + do
>
> > + d=${D}${bindir}/../${TARGET_ARCH}$v-$i
>
> > + install -d $d
>
> > + for j in ${TARGET_PREFIX}gcc${EXEEXT}
> ${TARGET_PREFIX}g++${EXEEXT}
>
> > + do
>
> > + p=${TARGET_ARCH}$v-$i-`echo $j | sed -e
> s,${TARGET_PREFIX},,`
>
> > + case $i in
>
> > + *musl*)
>
> > + rm -rf $d/$p
>
> > + echo "#!/usr/bin/env sh" > $d/$p
>
> > + echo "exec \`dirname
> \$0\`/../${TARGET_SYS}/$j -mmusl \$@" >> $d/$p
>
> > + chmod 0755 $d/$p
>
> > + ;;
>
> > + *)
>
> > + ;;
>
> > + esac
>
> > + done
>
> > + done
>
> > + done
>
> > }
>
> >
>
> > ELFUTILS = "nativesdk-elfutils"
>
>
>
> I've bisected this error in mingw:
>
>
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/89/builds/2341
>
>
>
> to this change.
>
>
>
> Error:
>
> Problem: package gcc-cross-canadian-i686-10.2.0-r0.i686_nativesdk_mingw32
> requires /usr/bin/env, but none of the providers can be installed
>
> - package
> packagegroup-cross-canadian-qemux86-1.0-r0.i686_nativesdk_mingw32 requires
> gcc-cross-canadian-i686, but none of the providers can be installed
>
> - package target-sdk-provides-dummy-1.0-r0.sdk_provides_dummy_target
> conflicts with /bin/sh provided by
> nativesdk-curl-7.71.1-r0.i686_nativesdk_mingw32
>
> - conflicting requests
>
> (try to add '--allowerasing' to command line to replace conflicting
> packages or '--skip-broken' to skip uninstallable packages)
>
>
>
> This is due to the addition of the /usr/bin/env dependency to the
>
> generated files.
>
>
>
> I'm trying to unravel the problems but it may mean extra fixes to both
>
> meta-mingw and master handling of /bin/sh.
Maybe we can call out /bin/sh directly will that work on mingw
>
>
>
>
> Cheers,
>
>
>
> Richard
>
>
>
>
[-- Attachment #2: Type: text/html, Size: 4800 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH V2 3/3] gcc-cross-canadian: Correct the regexp to delete versioned gcc binary
2020-08-20 8:01 [PATCH V2 1/3] binutils: Let crosssdk gold linker generate 4096 btyes long .interp section Khem Raj
2020-08-20 8:01 ` [PATCH V2 2/3] gcc-cross-canadian: Install gcc/g++ wrappers for musl Khem Raj
@ 2020-08-20 8:01 ` Khem Raj
1 sibling, 0 replies; 7+ messages in thread
From: Khem Raj @ 2020-08-20 8:01 UTC (permalink / raw)
To: openembedded-core; +Cc: Khem Raj
After gcc 10 this expression needed to be adjusted, its better to use a
bitbake variable which we do have readily available
Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
meta/recipes-devtools/gcc/gcc-cross-canadian.inc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
index c5935b978d..d3875c94d3 100644
--- a/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
+++ b/meta/recipes-devtools/gcc/gcc-cross-canadian.inc
@@ -106,7 +106,7 @@ do_install () {
rm -f ${D}${bindir}/*c++
# We don't care about the gcc-<version> copies
- rm -f ${D}${bindir}/*gcc-?.?*
+ rm -f ${D}${bindir}/*gcc-${BINV}*
# Cleanup empty directories which are not shipped
# we use rmdir instead of 'rm -f' to ensure the non empty directories are not deleted
--
2.28.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2020-08-21 19:51 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-20 8:01 [PATCH V2 1/3] binutils: Let crosssdk gold linker generate 4096 btyes long .interp section Khem Raj
2020-08-20 8:01 ` [PATCH V2 2/3] gcc-cross-canadian: Install gcc/g++ wrappers for musl Khem Raj
2020-08-20 21:07 ` Leon Woestenberg
2020-08-20 22:51 ` Khem Raj
2020-08-21 10:51 ` [OE-core] " Richard Purdie
2020-08-21 19:51 ` Khem Raj
2020-08-20 8:01 ` [PATCH V2 3/3] gcc-cross-canadian: Correct the regexp to delete versioned gcc binary Khem Raj
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.