Re: [OE-core] [RFC PATCH 2/3] tcmode-default: use go-binary-native by default

["Chen Qi" <Qi.Chen@windriver.com>]

[-- Attachment #1: Type: text/plain, Size: 3080 bytes --]

On 08/13/2020 01:13 AM, Bruce Ashfield wrote:
>
>
> On Wed, Aug 12, 2020 at 5:25 AM Chen Qi <Qi.Chen@windriver.com 
> <mailto:Qi.Chen@windriver.com>> wrote:
>
>     Hi Ross,
>
>     I've just tracked down to this patch about k8s build failure for arm.
>     I'm now setting the preferred provider to 'go-native' to avoid k8s
>     build regression.
>
>     go-native contains patches like
>     0001-allow-CC-and-CXX-to-have-multiple-words.patch.
>     These patches seem to make sense.
>
>     Do you have any idea how to deal with such situation?
>
>
> We've had lots of problems with 32bit arm and the new go compiler. I 
> just confirmed that k8s still builds 64 bit.
>
> And if this is the gnu-stubs.h issue, it's the same thing.
>
> It might be time to just exclude 32bit arm from some of the bigger 
> stack in meta-virt, since getting them to respect the cross go 
> compiler is getting harder and harder.
>
> That being said, I hunted down a few more incorrect 'go' calls in the 
> k8s build scripts, and can now build 32 bit again. But as for runtime, 
> I can't say if it works or not.
>

That's good news :)
But for excluding 32bit arm support, I have some doubt. Because as far 
as I know, there are some 32bit arm BSPs that are in use, e.g. 
xilinx-zynq and ti-am335x.

Best Regards,
Chen Qi

> Once I complete builds on other archs, I'll push the change to meta-virt.
>
> Bruce
>
>
>
>     Best Regards,
>     Chen Qi
>
>     On 06/17/2020 01:05 AM, Ross Burton wrote:
>>     From: Ross Burton<ross.burton@arm.com> <mailto:ross.burton@arm.com>
>>
>>     go-binary-native is faster and more portable than go-native, so use it
>>     by default.
>>
>>     Signed-off-by: Ross Burton<ross.burton@arm.com> <mailto:ross.burton@arm.com>
>>     ---
>>       meta/conf/distro/include/tcmode-default.inc | 4 ++++
>>       1 file changed, 4 insertions(+)
>>
>>     diff --git a/meta/conf/distro/include/tcmode-default.inc b/meta/conf/distro/include/tcmode-default.inc
>>     index ab4333144b..7103e10d3b 100644
>>     --- a/meta/conf/distro/include/tcmode-default.inc
>>     +++ b/meta/conf/distro/include/tcmode-default.inc
>>     @@ -66,6 +66,10 @@ PREFERRED_VERSION_qemu ?= "${QEMUVERSION}"
>>       PREFERRED_VERSION_qemu-native ?= "${QEMUVERSION}"
>>       PREFERRED_VERSION_nativesdk-qemu ?= "${QEMUVERSION}"
>>       
>>     +# Bootstrap Go using a binary release fromgolang.org <http://golang.org>.  If you want to bootstrap
>>     +# from source using the C-implemented Go 1.4 (only supports x86-64 hosts) then use
>>     +# go-native.
>>     +PREFERRED_PROVIDER_go-native ?= "go-binary-native"
>>       PREFERRED_VERSION_virtual/${TARGET_PREFIX}go ?= "${GOVERSION}"
>>       PREFERRED_VERSION_go-cross-${TUNE_PKGARCH} ?= "${GOVERSION}"
>>       PREFERRED_VERSION_go-crosssdk-${SDK_ARCH} ?= "${GOVERSION}"
>>
>>
>
>     
>
>
>
> -- 
> - Thou shalt not follow the NULL pointer, for chaos and madness await 
> thee at its end
> - "Use the force Harry" - Gandalf, Star Trek II
>


[-- Attachment #2: Type: text/html, Size: 6425 bytes --]