All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] gcc10: Don't default back to -fcommon
@ 2020-08-04  8:20 Jacob Kroon
  2020-08-05  0:14 ` [OE-core] " Khem Raj
  0 siblings, 1 reply; 25+ messages in thread
From: Jacob Kroon @ 2020-08-04  8:20 UTC (permalink / raw)
  To: openembedded-core

It has been almost 3 months since the upgrade to gcc 10. Switch back to
relying on gcc default configuration (-fno-common).

This reverts OE-Core commit 951e859b1e8297970278c539e989b8a6d06a9cb3.

Signed-off-by: Jacob Kroon <jacob.kroon@gmail.com>
---
 meta/recipes-devtools/gcc/gcc-configure-common.inc | 1 -
 1 file changed, 1 deletion(-)

diff --git a/meta/recipes-devtools/gcc/gcc-configure-common.inc b/meta/recipes-devtools/gcc/gcc-configure-common.inc
index 6cb40fcb74..a64c4caf00 100644
--- a/meta/recipes-devtools/gcc/gcc-configure-common.inc
+++ b/meta/recipes-devtools/gcc/gcc-configure-common.inc
@@ -34,7 +34,6 @@ EXTRA_OECONF = "\
     --program-prefix=${TARGET_PREFIX} \
     --without-local-prefix \
     --disable-install-libiberty \
-    --with-specs=%{!fno-common:%{!fcommon:-fcommon}} \
     ${EXTRA_OECONF_BASE} \
     ${EXTRA_OECONF_GCC_FLOAT} \
     ${EXTRA_OECONF_PATHS} \

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-04  8:20 [PATCH] gcc10: Don't default back to -fcommon Jacob Kroon
@ 2020-08-05  0:14 ` Khem Raj
  2020-08-05  5:05   ` Jacob Kroon
  0 siblings, 1 reply; 25+ messages in thread
From: Khem Raj @ 2020-08-05  0:14 UTC (permalink / raw)
  To: Jacob Kroon; +Cc: Patches and discussions about the oe-core layer

On Tue, Aug 4, 2020 at 1:20 AM Jacob Kroon <jacob.kroon@gmail.com> wrote:
>
> It has been almost 3 months since the upgrade to gcc 10. Switch back to
> relying on gcc default configuration (-fno-common).
>

have you done a world build with this? In core+meta-oe I still see 70+
recipes failing
what will be good is if people start sending fixes before switching
the defaults.

> This reverts OE-Core commit 951e859b1e8297970278c539e989b8a6d06a9cb3.
>
> Signed-off-by: Jacob Kroon <jacob.kroon@gmail.com>
> ---
>  meta/recipes-devtools/gcc/gcc-configure-common.inc | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/meta/recipes-devtools/gcc/gcc-configure-common.inc b/meta/recipes-devtools/gcc/gcc-configure-common.inc
> index 6cb40fcb74..a64c4caf00 100644
> --- a/meta/recipes-devtools/gcc/gcc-configure-common.inc
> +++ b/meta/recipes-devtools/gcc/gcc-configure-common.inc
> @@ -34,7 +34,6 @@ EXTRA_OECONF = "\
>      --program-prefix=${TARGET_PREFIX} \
>      --without-local-prefix \
>      --disable-install-libiberty \
> -    --with-specs=%{!fno-common:%{!fcommon:-fcommon}} \
>      ${EXTRA_OECONF_BASE} \
>      ${EXTRA_OECONF_GCC_FLOAT} \
>      ${EXTRA_OECONF_PATHS} \
> 

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-05  0:14 ` [OE-core] " Khem Raj
@ 2020-08-05  5:05   ` Jacob Kroon
  2020-08-05  6:00     ` Khem Raj
  0 siblings, 1 reply; 25+ messages in thread
From: Jacob Kroon @ 2020-08-05  5:05 UTC (permalink / raw)
  To: Khem Raj; +Cc: Patches and discussions about the oe-core layer

On 8/5/20 2:14 AM, Khem Raj wrote:
> On Tue, Aug 4, 2020 at 1:20 AM Jacob Kroon <jacob.kroon@gmail.com> wrote:
>>
>> It has been almost 3 months since the upgrade to gcc 10. Switch back to
>> relying on gcc default configuration (-fno-common).
>>
> 
> have you done a world build with this? In core+meta-oe I still see 70+
> recipes failing
> what will be good is if people start sending fixes before switching
> the defaults.
> 

Ah, that I haven't, only the images I use. Is there a list of failing 
recipes available somewhere ?

Jacob

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-05  5:05   ` Jacob Kroon
@ 2020-08-05  6:00     ` Khem Raj
  2020-08-05  6:17       ` Khem Raj
  0 siblings, 1 reply; 25+ messages in thread
From: Khem Raj @ 2020-08-05  6:00 UTC (permalink / raw)
  To: Jacob Kroon; +Cc: Patches and discussions about the oe-core layer

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

On Tue, Aug 4, 2020 at 10:05 PM Jacob Kroon <jacob.kroon@gmail.com> wrote:

> On 8/5/20 2:14 AM, Khem Raj wrote:
> > On Tue, Aug 4, 2020 at 1:20 AM Jacob Kroon <jacob.kroon@gmail.com>
> wrote:
> >>
> >> It has been almost 3 months since the upgrade to gcc 10. Switch back to
> >> relying on gcc default configuration (-fno-common).
> >>
> >
> > have you done a world build with this? In core+meta-oe I still see 70+
> > recipes failing
> > what will be good is if people start sending fixes before switching
> > the defaults.
> >
>
> Ah, that I haven't, only the images I use. Is there a list of failing
> recipes available somewhere ?


I think I posted the list when I submitted the gcc10 upgrade but this patch
is already in master-next so Richard will soon share the Oe-core list
atleast

>
>
> Jacob
>

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

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-05  6:00     ` Khem Raj
@ 2020-08-05  6:17       ` Khem Raj
  2020-08-05 13:15         ` Jacob Kroon
  2020-08-15 22:10         ` Richard Purdie
  0 siblings, 2 replies; 25+ messages in thread
From: Khem Raj @ 2020-08-05  6:17 UTC (permalink / raw)
  To: Jacob Kroon; +Cc: Patches and discussions about the oe-core layer

here is a recent build with clang11 which also switched to using
-fno-common and errors are pretty much same you will see with gcc10
with this patch to revert the -fcommon defaults.

https://errors.yoctoproject.org/Errors/Build/107234/?limit=100

I was waiting for upstream packages to get fixed before switching the
defaults back to -fno-common.

On Tue, Aug 4, 2020 at 11:00 PM Khem Raj <raj.khem@gmail.com> wrote:
>
>
>
> On Tue, Aug 4, 2020 at 10:05 PM Jacob Kroon <jacob.kroon@gmail.com> wrote:
>>
>> On 8/5/20 2:14 AM, Khem Raj wrote:
>> > On Tue, Aug 4, 2020 at 1:20 AM Jacob Kroon <jacob.kroon@gmail.com> wrote:
>> >>
>> >> It has been almost 3 months since the upgrade to gcc 10. Switch back to
>> >> relying on gcc default configuration (-fno-common).
>> >>
>> >
>> > have you done a world build with this? In core+meta-oe I still see 70+
>> > recipes failing
>> > what will be good is if people start sending fixes before switching
>> > the defaults.
>> >
>>
>> Ah, that I haven't, only the images I use. Is there a list of failing
>> recipes available somewhere ?
>
>
> I think I posted the list when I submitted the gcc10 upgrade but this patch is already in master-next so Richard will soon share the Oe-core list atleast
>>
>>
>>
>> Jacob

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-05  6:17       ` Khem Raj
@ 2020-08-05 13:15         ` Jacob Kroon
  2020-08-05 13:26           ` Richard Purdie
  2020-08-15 22:10         ` Richard Purdie
  1 sibling, 1 reply; 25+ messages in thread
From: Jacob Kroon @ 2020-08-05 13:15 UTC (permalink / raw)
  To: Khem Raj; +Cc: Patches and discussions about the oe-core layer

On 8/5/20 8:17 AM, Khem Raj wrote:
> here is a recent build with clang11 which also switched to using
> -fno-common and errors are pretty much same you will see with gcc10
> with this patch to revert the -fcommon defaults.
> 
> https://errors.yoctoproject.org/Errors/Build/107234/?limit=100
> 
> I was waiting for upstream packages to get fixed before switching the
> defaults back to -fno-common.
> 

Yes I understand, maybe its better to hold this for now.

Having a list of recipes that needs fixing would probably speed things 
up. I can have a look at some of them.

Jacob

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-05 13:15         ` Jacob Kroon
@ 2020-08-05 13:26           ` Richard Purdie
  0 siblings, 0 replies; 25+ messages in thread
From: Richard Purdie @ 2020-08-05 13:26 UTC (permalink / raw)
  To: Jacob Kroon, Khem Raj; +Cc: Patches and discussions about the oe-core layer

On Wed, 2020-08-05 at 15:15 +0200, Jacob Kroon wrote:
> On 8/5/20 8:17 AM, Khem Raj wrote:
> > here is a recent build with clang11 which also switched to using
> > -fno-common and errors are pretty much same you will see with gcc10
> > with this patch to revert the -fcommon defaults.
> > 
> > https://errors.yoctoproject.org/Errors/Build/107234/?limit=100
> > 
> > I was waiting for upstream packages to get fixed before switching
> > the
> > defaults back to -fno-common.
> > 
> 
> Yes I understand, maybe its better to hold this for now.
> 
> Having a list of recipes that needs fixing would probably speed
> things up. I can have a look at some of them.

I ran this through with just OE-Core and the list of failures is:

https://autobuilder.yoctoproject.org/typhoon/#/builders/52/builds/2237

I'm a little frustrated that patches like this get sent and them people
like Khem and I have to spot and avoid the problems as there isn't
testing :/.

I'll drop the patch for now until more of these issues are resolved.

Cheers,

Richard


^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-05  6:17       ` Khem Raj
  2020-08-05 13:15         ` Jacob Kroon
@ 2020-08-15 22:10         ` Richard Purdie
  2020-08-16  1:56           ` Khem Raj
  1 sibling, 1 reply; 25+ messages in thread
From: Richard Purdie @ 2020-08-15 22:10 UTC (permalink / raw)
  To: Khem Raj, Jacob Kroon; +Cc: Patches and discussions about the oe-core layer

On Tue, 2020-08-04 at 23:17 -0700, Khem Raj wrote:
> here is a recent build with clang11 which also switched to using
> -fno-common and errors are pretty much same you will see with gcc10
> with this patch to revert the -fcommon defaults.
> 
> https://errors.yoctoproject.org/Errors/Build/107234/?limit=100
> 
> I was waiting for upstream packages to get fixed before switching the
> defaults back to -fno-common.

Khem requested I revisit this. I see at least:

a) perf failures pretty much everywhere
b) kexec-tools in no-x11 build
c) cpio build failures in testimage

https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/1264

the world builds haven't completed yet.

So looks much better but still a few things need fixes.

Cheers,

Richard



^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-15 22:10         ` Richard Purdie
@ 2020-08-16  1:56           ` Khem Raj
  2020-08-16  2:41             ` Khem Raj
  0 siblings, 1 reply; 25+ messages in thread
From: Khem Raj @ 2020-08-16  1:56 UTC (permalink / raw)
  To: Richard Purdie
  Cc: Jacob Kroon, Patches and discussions about the oe-core layer

On Sat, Aug 15, 2020 at 3:10 PM Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
>
> On Tue, 2020-08-04 at 23:17 -0700, Khem Raj wrote:
> > here is a recent build with clang11 which also switched to using
> > -fno-common and errors are pretty much same you will see with gcc10
> > with this patch to revert the -fcommon defaults.
> >
> > https://errors.yoctoproject.org/Errors/Build/107234/?limit=100
> >
> > I was waiting for upstream packages to get fixed before switching the
> > defaults back to -fno-common.
>
> Khem requested I revisit this. I see at least:
>
> a) perf failures pretty much everywhere

This is interesting

| x86_64-poky-linux-gcc: error: unrecognized command-line option
'-m64/include/uapi/asm-generic/errno.h'

I wonder if its some kernel update which is causing this, it seems to
be missing a space in commandline somewhere

> b) kexec-tools in no-x11 build

I think I dealt with this in meta-oe perhaps we need to use -fcommon in cflags.

> c) cpio build failures in testimage
>
cpio needs a patch to build see
https://git.openembedded.org/openembedded-core/commit/?id=ca6e5511b59b585de793ff6424c7c6ccfcd8e61e
I guess we need to either patch cpio with this patch on target before
building or pass -fcommon to the test so this patch is not needed

Thanks for the run, I will try to look into it, I think for cpio build
failure perhaps we need to pass

> https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/1264
>
> the world builds haven't completed yet.
>
> So looks much better but still a few things need fixes.
>
> Cheers,
>
> Richard
>
>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-16  1:56           ` Khem Raj
@ 2020-08-16  2:41             ` Khem Raj
  2020-08-16 13:47               ` Richard Purdie
  0 siblings, 1 reply; 25+ messages in thread
From: Khem Raj @ 2020-08-16  2:41 UTC (permalink / raw)
  To: Richard Purdie, Bruce Ashfield
  Cc: Jacob Kroon, Patches and discussions about the oe-core layer

Bruce,

We need https://github.com/torvalds/linux/commit/168200b6d6ea0cb5765943ec5da5b8149701f36a
backported to 5.4 kernels for perf to build with -fno-common can you
queue this in your kernel updates as soon as you can ?

Thanks
-Khem

On Sat, Aug 15, 2020 at 6:56 PM Khem Raj <raj.khem@gmail.com> wrote:
>
> On Sat, Aug 15, 2020 at 3:10 PM Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
> >
> > On Tue, 2020-08-04 at 23:17 -0700, Khem Raj wrote:
> > > here is a recent build with clang11 which also switched to using
> > > -fno-common and errors are pretty much same you will see with gcc10
> > > with this patch to revert the -fcommon defaults.
> > >
> > > https://errors.yoctoproject.org/Errors/Build/107234/?limit=100
> > >
> > > I was waiting for upstream packages to get fixed before switching the
> > > defaults back to -fno-common.
> >
> > Khem requested I revisit this. I see at least:
> >
> > a) perf failures pretty much everywhere
>
> This is interesting
>
> | x86_64-poky-linux-gcc: error: unrecognized command-line option
> '-m64/include/uapi/asm-generic/errno.h'
>
> I wonder if its some kernel update which is causing this, it seems to
> be missing a space in commandline somewhere
>
> > b) kexec-tools in no-x11 build
>
> I think I dealt with this in meta-oe perhaps we need to use -fcommon in cflags.
>
> > c) cpio build failures in testimage
> >
> cpio needs a patch to build see
> https://git.openembedded.org/openembedded-core/commit/?id=ca6e5511b59b585de793ff6424c7c6ccfcd8e61e
> I guess we need to either patch cpio with this patch on target before
> building or pass -fcommon to the test so this patch is not needed
>
> Thanks for the run, I will try to look into it, I think for cpio build
> failure perhaps we need to pass
>
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/1264
> >
> > the world builds haven't completed yet.
> >
> > So looks much better but still a few things need fixes.
> >
> > Cheers,
> >
> > Richard
> >
> >

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-16  2:41             ` Khem Raj
@ 2020-08-16 13:47               ` Richard Purdie
  2020-08-16 22:55                 ` Bruce Ashfield
  2020-08-16 23:29                 ` Khem Raj
  0 siblings, 2 replies; 25+ messages in thread
From: Richard Purdie @ 2020-08-16 13:47 UTC (permalink / raw)
  To: Khem Raj, Bruce Ashfield
  Cc: Jacob Kroon, Patches and discussions about the oe-core layer

On Sat, 2020-08-15 at 19:41 -0700, Khem Raj wrote:
> Bruce,
> 
> We need 
> https://github.com/torvalds/linux/commit/168200b6d6ea0cb5765943ec5da5b8149701f36a
> backported to 5.4 kernels for perf to build with -fno-common can you
> queue this in your kernel updates as soon as you can ?

I think we have other active kernels which will also need it as even
with Khem's patch we still see failures:

https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/1267

Cheers,

Richard


^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-16 13:47               ` Richard Purdie
@ 2020-08-16 22:55                 ` Bruce Ashfield
  2020-08-16 23:14                   ` Khem Raj
  2020-08-17 12:27                   ` Richard Purdie
  2020-08-16 23:29                 ` Khem Raj
  1 sibling, 2 replies; 25+ messages in thread
From: Bruce Ashfield @ 2020-08-16 22:55 UTC (permalink / raw)
  To: Richard Purdie
  Cc: Khem Raj, Jacob Kroon, Patches and discussions about the oe-core layer

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

On Sun, Aug 16, 2020 at 9:47 AM Richard Purdie <
richard.purdie@linuxfoundation.org> wrote:

> On Sat, 2020-08-15 at 19:41 -0700, Khem Raj wrote:
> > Bruce,
> >
> > We need
> >
> https://github.com/torvalds/linux/commit/168200b6d6ea0cb5765943ec5da5b8149701f36a
> > backported to 5.4 kernels for perf to build with -fno-common can you
> > queue this in your kernel updates as soon as you can ?
>
> I think we have other active kernels which will also need it as even
> with Khem's patch we still see failures:
>
>
I've taken care of linux-yocto now, I don't think there are any active use
cases for those kernels that won't be covered.

Cheers,

Bruce



> https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/1267
>
> Cheers,
>
> Richard
>
>

-- 
- 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: 2164 bytes --]

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-16 22:55                 ` Bruce Ashfield
@ 2020-08-16 23:14                   ` Khem Raj
  2020-08-17 12:27                   ` Richard Purdie
  1 sibling, 0 replies; 25+ messages in thread
From: Khem Raj @ 2020-08-16 23:14 UTC (permalink / raw)
  To: Bruce Ashfield
  Cc: Richard Purdie, Jacob Kroon,
	Patches and discussions about the oe-core layer

Awesome, thanks!

On Sun, Aug 16, 2020 at 3:55 PM Bruce Ashfield <bruce.ashfield@gmail.com> wrote:
>
>
>
> On Sun, Aug 16, 2020 at 9:47 AM Richard Purdie <richard.purdie@linuxfoundation.org> wrote:
>>
>> On Sat, 2020-08-15 at 19:41 -0700, Khem Raj wrote:
>> > Bruce,
>> >
>> > We need
>> > https://github.com/torvalds/linux/commit/168200b6d6ea0cb5765943ec5da5b8149701f36a
>> > backported to 5.4 kernels for perf to build with -fno-common can you
>> > queue this in your kernel updates as soon as you can ?
>>
>> I think we have other active kernels which will also need it as even
>> with Khem's patch we still see failures:
>>
>
> I've taken care of linux-yocto now, I don't think there are any active use cases for those kernels that won't be covered.
>
> Cheers,
>
> Bruce
>
>
>>
>> https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/1267
>>
>> Cheers,
>>
>> Richard
>>
>
>
> --
> - Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end
> - "Use the force Harry" - Gandalf, Star Trek II
>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-16 13:47               ` Richard Purdie
  2020-08-16 22:55                 ` Bruce Ashfield
@ 2020-08-16 23:29                 ` Khem Raj
  1 sibling, 0 replies; 25+ messages in thread
From: Khem Raj @ 2020-08-16 23:29 UTC (permalink / raw)
  To: Richard Purdie
  Cc: Bruce Ashfield, Jacob Kroon,
	Patches and discussions about the oe-core layer

On Sun, Aug 16, 2020 at 6:47 AM Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
>
> On Sat, 2020-08-15 at 19:41 -0700, Khem Raj wrote:
> > Bruce,
> >
> > We need
> > https://github.com/torvalds/linux/commit/168200b6d6ea0cb5765943ec5da5b8149701f36a
> > backported to 5.4 kernels for perf to build with -fno-common can you
> > queue this in your kernel updates as soon as you can ?
>
> I think we have other active kernels which will also need it as even
> with Khem's patch we still see failures:
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/1267
>

I see

meta_ide.MetaIDE.test_meta_ide_can_build_cpio_project: FAILED (64.00s)

It seems the sdk fix for cpio is not working properly

> Cheers,
>
> Richard
>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-16 22:55                 ` Bruce Ashfield
  2020-08-16 23:14                   ` Khem Raj
@ 2020-08-17 12:27                   ` Richard Purdie
  2020-08-17 14:15                     ` Bruce Ashfield
  2020-08-17 19:47                     ` Khem Raj
  1 sibling, 2 replies; 25+ messages in thread
From: Richard Purdie @ 2020-08-17 12:27 UTC (permalink / raw)
  To: Bruce Ashfield
  Cc: Khem Raj, Jacob Kroon, Patches and discussions about the oe-core layer

On Sun, 2020-08-16 at 18:55 -0400, Bruce Ashfield wrote:
> 
> 
> On Sun, Aug 16, 2020 at 9:47 AM Richard Purdie <
> richard.purdie@linuxfoundation.org> wrote:
> > On Sat, 2020-08-15 at 19:41 -0700, Khem Raj wrote:
> > > Bruce,
> > > 
> > > We need 
> > > 
> > https://github.com/torvalds/linux/commit/168200b6d6ea0cb5765943ec5da5b8149701f36a
> > > backported to 5.4 kernels for perf to build with -fno-common can
> > you
> > > queue this in your kernel updates as soon as you can ?
> > 
> > I think we have other active kernels which will also need it as
> > even
> > with Khem's patch we still see failures:
> > 
> 
> I've taken care of linux-yocto now, I don't think there are any
> active use cases for those kernels that won't be covered.

I bumped the meta-yocto BSPs with a quickly hacked up patch but we
still see:

https://autobuilder.yoctoproject.org/typhoon/#/builders/65/builds/2339

so I think there may be another patch we need somewhere...

(which matches what we saw with Khem's _append which also failed).

There is still some piece of this we're missing, maybe on non-IA.

Cheers,

Richard


^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-17 12:27                   ` Richard Purdie
@ 2020-08-17 14:15                     ` Bruce Ashfield
  2020-08-17 14:45                       ` Richard Purdie
  2020-08-17 19:47                     ` Khem Raj
  1 sibling, 1 reply; 25+ messages in thread
From: Bruce Ashfield @ 2020-08-17 14:15 UTC (permalink / raw)
  To: Richard Purdie
  Cc: Khem Raj, Jacob Kroon, Patches and discussions about the oe-core layer

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

On Mon, Aug 17, 2020 at 8:27 AM Richard Purdie <
richard.purdie@linuxfoundation.org> wrote:

> On Sun, 2020-08-16 at 18:55 -0400, Bruce Ashfield wrote:
> >
> >
> > On Sun, Aug 16, 2020 at 9:47 AM Richard Purdie <
> > richard.purdie@linuxfoundation.org> wrote:
> > > On Sat, 2020-08-15 at 19:41 -0700, Khem Raj wrote:
> > > > Bruce,
> > > >
> > > > We need
> > > >
> > >
> https://github.com/torvalds/linux/commit/168200b6d6ea0cb5765943ec5da5b8149701f36a
> > > > backported to 5.4 kernels for perf to build with -fno-common can
> > > you
> > > > queue this in your kernel updates as soon as you can ?
> > >
> > > I think we have other active kernels which will also need it as
> > > even
> > > with Khem's patch we still see failures:
> > >
> >
> > I've taken care of linux-yocto now, I don't think there are any
> > active use cases for those kernels that won't be covered.
>
> I bumped the meta-yocto BSPs with a quickly hacked up patch but we
> still see:
>
>
I have local bumps for the yocto-bsps as well, let me try a perf build
against beaglebone to see if I can see anything.

If I build master-next, will I get all the required gcc patches to trigger
the issue ?

Bruce



> https://autobuilder.yoctoproject.org/typhoon/#/builders/65/builds/2339
>
> so I think there may be another patch we need somewhere...
>
> (which matches what we saw with Khem's _append which also failed).
>
> There is still some piece of this we're missing, maybe on non-IA.
>
> Cheers,
>
> Richard
>
>

-- 
- 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: 3157 bytes --]

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-17 14:15                     ` Bruce Ashfield
@ 2020-08-17 14:45                       ` Richard Purdie
  0 siblings, 0 replies; 25+ messages in thread
From: Richard Purdie @ 2020-08-17 14:45 UTC (permalink / raw)
  To: Bruce Ashfield
  Cc: Khem Raj, Jacob Kroon, Patches and discussions about the oe-core layer

On Mon, 2020-08-17 at 10:15 -0400, Bruce Ashfield wrote:
> 
> 
> On Mon, Aug 17, 2020 at 8:27 AM Richard Purdie <
> richard.purdie@linuxfoundation.org> wrote:
> > On Sun, 2020-08-16 at 18:55 -0400, Bruce Ashfield wrote:
> > > 
> > > 
> > > On Sun, Aug 16, 2020 at 9:47 AM Richard Purdie <
> > > richard.purdie@linuxfoundation.org> wrote:
> > > > On Sat, 2020-08-15 at 19:41 -0700, Khem Raj wrote:
> > > > > Bruce,
> > > > > 
> > > > > We need 
> > > > > 
> > > > 
> > https://github.com/torvalds/linux/commit/168200b6d6ea0cb5765943ec5da5b8149701f36a
> > > > > backported to 5.4 kernels for perf to build with -fno-common
> > can
> > > > you
> > > > > queue this in your kernel updates as soon as you can ?
> > > > 
> > > > I think we have other active kernels which will also need it as
> > > > even
> > > > with Khem's patch we still see failures:
> > > > 
> > > 
> > > I've taken care of linux-yocto now, I don't think there are any
> > > active use cases for those kernels that won't be covered.
> > 
> > I bumped the meta-yocto BSPs with a quickly hacked up patch but we
> > still see:
> > 
> 
> I have local bumps for the yocto-bsps as well, let me try a perf
> build against beaglebone to see if I can see anything.
> 
> If I build master-next, will I get all the required gcc patches to
> trigger the issue ?

You need:

http://git.yoctoproject.org/cgit.cgi/poky/commit/?h=master-next&id=c79101ffff8a9cc70daefe737e16b773190bb6ca

without its revert (which is also in -next atm).

Cheers,

Richard


^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-17 12:27                   ` Richard Purdie
  2020-08-17 14:15                     ` Bruce Ashfield
@ 2020-08-17 19:47                     ` Khem Raj
  2020-08-17 19:55                       ` Khem Raj
  2020-08-18 11:57                       ` Martin Jansa
  1 sibling, 2 replies; 25+ messages in thread
From: Khem Raj @ 2020-08-17 19:47 UTC (permalink / raw)
  To: Richard Purdie
  Cc: Bruce Ashfield, Jacob Kroon,
	Patches and discussions about the oe-core layer

On Mon, Aug 17, 2020 at 5:27 AM Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
>
> On Sun, 2020-08-16 at 18:55 -0400, Bruce Ashfield wrote:
> >
> >
> > On Sun, Aug 16, 2020 at 9:47 AM Richard Purdie <
> > richard.purdie@linuxfoundation.org> wrote:
> > > On Sat, 2020-08-15 at 19:41 -0700, Khem Raj wrote:
> > > > Bruce,
> > > >
> > > > We need
> > > >
> > > https://github.com/torvalds/linux/commit/168200b6d6ea0cb5765943ec5da5b8149701f36a
> > > > backported to 5.4 kernels for perf to build with -fno-common can
> > > you
> > > > queue this in your kernel updates as soon as you can ?
> > >
> > > I think we have other active kernels which will also need it as
> > > even
> > > with Khem's patch we still see failures:
> > >
> >
> > I've taken care of linux-yocto now, I don't think there are any
> > active use cases for those kernels that won't be covered.
>
> I bumped the meta-yocto BSPs with a quickly hacked up patch but we
> still see:
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/65/builds/2339
>
> so I think there may be another patch we need somewhere...
>
> (which matches what we saw with Khem's _append which also failed).
>

there seems to be concaternation of compiler options without space

| arm-poky-linux-gnueabi-gcc: error: unrecognized argument in option
'-mfpu=neon/include/uapi/asm-generic/errno.h'
| arm-poky-linux-gnueabi-gcc: note: valid arguments to '-mfpu=' are:
auto crypto-neon-fp-armv8 fp-armv8 fpv4-sp-d16 fpv5-d16 fpv5-sp-d16
neon neon-fp-armv8 neon-fp16 neon-vfpv3 neon-vfpv4 vfp vfp3 vfpv2
vfpv3 vfpv3-d16 vfpv3-d16-fp16 vfpv3-fp16 vfpv3xd vfpv3xd-fp16 vfpv4
vfpv4-d16
| arm-poky-linux-gnueabi-gcc: fatal error: no input files

but it also showed up on x86_64 in one of your prior builds, so I
wonder if its something in kernel build system or is it processing
TUNE_CCARGS in special ways.

> There is still some piece of this we're missing, maybe on non-IA.
>
> Cheers,
>
> Richard
>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-17 19:47                     ` Khem Raj
@ 2020-08-17 19:55                       ` Khem Raj
  2020-08-17 21:07                         ` Bruce Ashfield
  2020-08-18 11:57                       ` Martin Jansa
  1 sibling, 1 reply; 25+ messages in thread
From: Khem Raj @ 2020-08-17 19:55 UTC (permalink / raw)
  To: Richard Purdie
  Cc: Bruce Ashfield, Jacob Kroon,
	Patches and discussions about the oe-core layer

On Mon, Aug 17, 2020 at 12:47 PM Khem Raj <raj.khem@gmail.com> wrote:
>
> On Mon, Aug 17, 2020 at 5:27 AM Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
> >
> > On Sun, 2020-08-16 at 18:55 -0400, Bruce Ashfield wrote:
> > >
> > >
> > > On Sun, Aug 16, 2020 at 9:47 AM Richard Purdie <
> > > richard.purdie@linuxfoundation.org> wrote:
> > > > On Sat, 2020-08-15 at 19:41 -0700, Khem Raj wrote:
> > > > > Bruce,
> > > > >
> > > > > We need
> > > > >
> > > > https://github.com/torvalds/linux/commit/168200b6d6ea0cb5765943ec5da5b8149701f36a
> > > > > backported to 5.4 kernels for perf to build with -fno-common can
> > > > you
> > > > > queue this in your kernel updates as soon as you can ?
> > > >
> > > > I think we have other active kernels which will also need it as
> > > > even
> > > > with Khem's patch we still see failures:
> > > >
> > >
> > > I've taken care of linux-yocto now, I don't think there are any
> > > active use cases for those kernels that won't be covered.
> >
> > I bumped the meta-yocto BSPs with a quickly hacked up patch but we
> > still see:
> >
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/65/builds/2339
> >
> > so I think there may be another patch we need somewhere...
> >
> > (which matches what we saw with Khem's _append which also failed).
> >
>
> there seems to be concaternation of compiler options without space
>
> | arm-poky-linux-gnueabi-gcc: error: unrecognized argument in option
> '-mfpu=neon/include/uapi/asm-generic/errno.h'
> | arm-poky-linux-gnueabi-gcc: note: valid arguments to '-mfpu=' are:
> auto crypto-neon-fp-armv8 fp-armv8 fpv4-sp-d16 fpv5-d16 fpv5-sp-d16
> neon neon-fp-armv8 neon-fp16 neon-vfpv3 neon-vfpv4 vfp vfp3 vfpv2
> vfpv3 vfpv3-d16 vfpv3-d16-fp16 vfpv3-fp16 vfpv3xd vfpv3xd-fp16 vfpv4
> vfpv4-d16
> | arm-poky-linux-gnueabi-gcc: fatal error: no input files
>
> but it also showed up on x86_64 in one of your prior builds, so I
> wonder if its something in kernel build system or is it processing
> TUNE_CCARGS in special ways.
>
> > There is still some piece of this we're missing, maybe on non-IA.
> >

Perhaps these kernel backports might help

https://github.com/torvalds/linux/commit/7597ce89b3ed239f7a3408b930d2a6c7a4c938a1
https://github.com/torvalds/linux/commit/6d1f916265aa615218d341ba5d3fd39dfe931cde

> > Cheers,
> >
> > Richard
> >

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-17 19:55                       ` Khem Raj
@ 2020-08-17 21:07                         ` Bruce Ashfield
  0 siblings, 0 replies; 25+ messages in thread
From: Bruce Ashfield @ 2020-08-17 21:07 UTC (permalink / raw)
  To: Khem Raj
  Cc: Richard Purdie, Jacob Kroon,
	Patches and discussions about the oe-core layer

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

On Mon, Aug 17, 2020 at 3:55 PM Khem Raj <raj.khem@gmail.com> wrote:

> On Mon, Aug 17, 2020 at 12:47 PM Khem Raj <raj.khem@gmail.com> wrote:
> >
> > On Mon, Aug 17, 2020 at 5:27 AM Richard Purdie
> > <richard.purdie@linuxfoundation.org> wrote:
> > >
> > > On Sun, 2020-08-16 at 18:55 -0400, Bruce Ashfield wrote:
> > > >
> > > >
> > > > On Sun, Aug 16, 2020 at 9:47 AM Richard Purdie <
> > > > richard.purdie@linuxfoundation.org> wrote:
> > > > > On Sat, 2020-08-15 at 19:41 -0700, Khem Raj wrote:
> > > > > > Bruce,
> > > > > >
> > > > > > We need
> > > > > >
> > > > >
> https://github.com/torvalds/linux/commit/168200b6d6ea0cb5765943ec5da5b8149701f36a
> > > > > > backported to 5.4 kernels for perf to build with -fno-common can
> > > > > you
> > > > > > queue this in your kernel updates as soon as you can ?
> > > > >
> > > > > I think we have other active kernels which will also need it as
> > > > > even
> > > > > with Khem's patch we still see failures:
> > > > >
> > > >
> > > > I've taken care of linux-yocto now, I don't think there are any
> > > > active use cases for those kernels that won't be covered.
> > >
> > > I bumped the meta-yocto BSPs with a quickly hacked up patch but we
> > > still see:
> > >
> > > https://autobuilder.yoctoproject.org/typhoon/#/builders/65/builds/2339
> > >
> > > so I think there may be another patch we need somewhere...
> > >
> > > (which matches what we saw with Khem's _append which also failed).
> > >
> >
> > there seems to be concaternation of compiler options without space
> >
> > | arm-poky-linux-gnueabi-gcc: error: unrecognized argument in option
> > '-mfpu=neon/include/uapi/asm-generic/errno.h'
> > | arm-poky-linux-gnueabi-gcc: note: valid arguments to '-mfpu=' are:
> > auto crypto-neon-fp-armv8 fp-armv8 fpv4-sp-d16 fpv5-d16 fpv5-sp-d16
> > neon neon-fp-armv8 neon-fp16 neon-vfpv3 neon-vfpv4 vfp vfp3 vfpv2
> > vfpv3 vfpv3-d16 vfpv3-d16-fp16 vfpv3-fp16 vfpv3xd vfpv3xd-fp16 vfpv4
> > vfpv4-d16
> > | arm-poky-linux-gnueabi-gcc: fatal error: no input files
> >
> > but it also showed up on x86_64 in one of your prior builds, so I
> > wonder if its something in kernel build system or is it processing
> > TUNE_CCARGS in special ways.
> >
> > > There is still some piece of this we're missing, maybe on non-IA.
> > >
>
> Perhaps these kernel backports might help
>
>
> https://github.com/torvalds/linux/commit/7597ce89b3ed239f7a3408b930d2a6c7a4c938a1
>
> https://github.com/torvalds/linux/commit/6d1f916265aa615218d341ba5d3fd39dfe931cde


I unfortunately had to clean and rebuild my development machine, so only
now started a reference h/w build to see what's up with perf.

I'll follow up tomorrow when I have some input.

Cheers,

Bruce



>
>
> > > Cheers,
> > >
> > > Richard
> > >
>


-- 
- 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: 5300 bytes --]

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-17 19:47                     ` Khem Raj
  2020-08-17 19:55                       ` Khem Raj
@ 2020-08-18 11:57                       ` Martin Jansa
  2020-08-18 12:21                         ` Bruce Ashfield
  1 sibling, 1 reply; 25+ messages in thread
From: Martin Jansa @ 2020-08-18 11:57 UTC (permalink / raw)
  To: Khem Raj
  Cc: Richard Purdie, Bruce Ashfield, Jacob Kroon,
	Patches and discussions about the oe-core layer

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

Yes, I'm seeing this weird concatenation with raspberrypi builds as well:

find: unknown predicate `-mthumb/arch'
arm-webos-linux-gnueabi-gcc: error: unrecognized command-line option
'-mthumb/include/uapi/asm-generic/errno.h'
arm-webos-linux-gnueabi-gcc: fatal error: no input files
compilation terminated.

But in this case it isn't fatal for the perf build, I've noticed it only
because it was failing with 5.4.51 currently in meta-raspberrypi and after
upgrade to 5.4.58 perf built OK, but this error message is still there in
log.do_compile.

On Mon, Aug 17, 2020 at 9:47 PM Khem Raj <raj.khem@gmail.com> wrote:

> On Mon, Aug 17, 2020 at 5:27 AM Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
> >
> > On Sun, 2020-08-16 at 18:55 -0400, Bruce Ashfield wrote:
> > >
> > >
> > > On Sun, Aug 16, 2020 at 9:47 AM Richard Purdie <
> > > richard.purdie@linuxfoundation.org> wrote:
> > > > On Sat, 2020-08-15 at 19:41 -0700, Khem Raj wrote:
> > > > > Bruce,
> > > > >
> > > > > We need
> > > > >
> > > >
> https://github.com/torvalds/linux/commit/168200b6d6ea0cb5765943ec5da5b8149701f36a
> > > > > backported to 5.4 kernels for perf to build with -fno-common can
> > > > you
> > > > > queue this in your kernel updates as soon as you can ?
> > > >
> > > > I think we have other active kernels which will also need it as
> > > > even
> > > > with Khem's patch we still see failures:
> > > >
> > >
> > > I've taken care of linux-yocto now, I don't think there are any
> > > active use cases for those kernels that won't be covered.
> >
> > I bumped the meta-yocto BSPs with a quickly hacked up patch but we
> > still see:
> >
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/65/builds/2339
> >
> > so I think there may be another patch we need somewhere...
> >
> > (which matches what we saw with Khem's _append which also failed).
> >
>
> there seems to be concaternation of compiler options without space
>
> | arm-poky-linux-gnueabi-gcc: error: unrecognized argument in option
> '-mfpu=neon/include/uapi/asm-generic/errno.h'
> | arm-poky-linux-gnueabi-gcc: note: valid arguments to '-mfpu=' are:
> auto crypto-neon-fp-armv8 fp-armv8 fpv4-sp-d16 fpv5-d16 fpv5-sp-d16
> neon neon-fp-armv8 neon-fp16 neon-vfpv3 neon-vfpv4 vfp vfp3 vfpv2
> vfpv3 vfpv3-d16 vfpv3-d16-fp16 vfpv3-fp16 vfpv3xd vfpv3xd-fp16 vfpv4
> vfpv4-d16
> | arm-poky-linux-gnueabi-gcc: fatal error: no input files
>
> but it also showed up on x86_64 in one of your prior builds, so I
> wonder if its something in kernel build system or is it processing
> TUNE_CCARGS in special ways.
>
> > There is still some piece of this we're missing, maybe on non-IA.
> >
> > Cheers,
> >
> > Richard
> >
> 
>

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

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-18 11:57                       ` Martin Jansa
@ 2020-08-18 12:21                         ` Bruce Ashfield
  2020-08-18 16:03                           ` Martin Jansa
  0 siblings, 1 reply; 25+ messages in thread
From: Bruce Ashfield @ 2020-08-18 12:21 UTC (permalink / raw)
  To: Martin Jansa
  Cc: Khem Raj, Richard Purdie, Jacob Kroon,
	Patches and discussions about the oe-core layer

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

On Tue, Aug 18, 2020 at 7:57 AM Martin Jansa <martin.jansa@gmail.com> wrote:

> Yes, I'm seeing this weird concatenation with raspberrypi builds as well:
>
> find: unknown predicate `-mthumb/arch'
> arm-webos-linux-gnueabi-gcc: error: unrecognized command-line option
> '-mthumb/include/uapi/asm-generic/errno.h'
> arm-webos-linux-gnueabi-gcc: fatal error: no input files
> compilation terminated.
>

confirmed as the same with my beaglebone build that ran overnight.

I'll have a closer look and send a fix (hopefully) .. but if anyone else is
already doing this, please let me know and I'll work on something else!

Cheers,

Bruce



>
> But in this case it isn't fatal for the perf build, I've noticed it only
> because it was failing with 5.4.51 currently in meta-raspberrypi and after
> upgrade to 5.4.58 perf built OK, but this error message is still there in
> log.do_compile.
>
> On Mon, Aug 17, 2020 at 9:47 PM Khem Raj <raj.khem@gmail.com> wrote:
>
>> On Mon, Aug 17, 2020 at 5:27 AM Richard Purdie
>> <richard.purdie@linuxfoundation.org> wrote:
>> >
>> > On Sun, 2020-08-16 at 18:55 -0400, Bruce Ashfield wrote:
>> > >
>> > >
>> > > On Sun, Aug 16, 2020 at 9:47 AM Richard Purdie <
>> > > richard.purdie@linuxfoundation.org> wrote:
>> > > > On Sat, 2020-08-15 at 19:41 -0700, Khem Raj wrote:
>> > > > > Bruce,
>> > > > >
>> > > > > We need
>> > > > >
>> > > >
>> https://github.com/torvalds/linux/commit/168200b6d6ea0cb5765943ec5da5b8149701f36a
>> > > > > backported to 5.4 kernels for perf to build with -fno-common can
>> > > > you
>> > > > > queue this in your kernel updates as soon as you can ?
>> > > >
>> > > > I think we have other active kernels which will also need it as
>> > > > even
>> > > > with Khem's patch we still see failures:
>> > > >
>> > >
>> > > I've taken care of linux-yocto now, I don't think there are any
>> > > active use cases for those kernels that won't be covered.
>> >
>> > I bumped the meta-yocto BSPs with a quickly hacked up patch but we
>> > still see:
>> >
>> > https://autobuilder.yoctoproject.org/typhoon/#/builders/65/builds/2339
>> >
>> > so I think there may be another patch we need somewhere...
>> >
>> > (which matches what we saw with Khem's _append which also failed).
>> >
>>
>> there seems to be concaternation of compiler options without space
>>
>> | arm-poky-linux-gnueabi-gcc: error: unrecognized argument in option
>> '-mfpu=neon/include/uapi/asm-generic/errno.h'
>> | arm-poky-linux-gnueabi-gcc: note: valid arguments to '-mfpu=' are:
>> auto crypto-neon-fp-armv8 fp-armv8 fpv4-sp-d16 fpv5-d16 fpv5-sp-d16
>> neon neon-fp-armv8 neon-fp16 neon-vfpv3 neon-vfpv4 vfp vfp3 vfpv2
>> vfpv3 vfpv3-d16 vfpv3-d16-fp16 vfpv3-fp16 vfpv3xd vfpv3xd-fp16 vfpv4
>> vfpv4-d16
>> | arm-poky-linux-gnueabi-gcc: fatal error: no input files
>>
>> but it also showed up on x86_64 in one of your prior builds, so I
>> wonder if its something in kernel build system or is it processing
>> TUNE_CCARGS in special ways.
>>
>> > There is still some piece of this we're missing, maybe on non-IA.
>> >
>> > Cheers,
>> >
>> > Richard
>> >
>> 
>>
>

-- 
- 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: 5457 bytes --]

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-18 12:21                         ` Bruce Ashfield
@ 2020-08-18 16:03                           ` Martin Jansa
  2020-08-18 16:34                             ` Khem Raj
  2020-08-18 17:05                             ` Bruce Ashfield
  0 siblings, 2 replies; 25+ messages in thread
From: Martin Jansa @ 2020-08-18 16:03 UTC (permalink / raw)
  To: Bruce Ashfield
  Cc: Khem Raj, Richard Purdie, Jacob Kroon,
	Patches and discussions about the oe-core layer

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

The perf issue with find and cflags concatenation is fixed in upstream by:
https://github.com/torvalds/linux/commit/e4ffd066ff440a57097e9140fa9e16ceef905de8

Adding V=1 to perf's EXTRA_OEMAKE makes it easier to see, sending a patch
now.

On Tue, Aug 18, 2020 at 2:21 PM Bruce Ashfield <bruce.ashfield@gmail.com>
wrote:

>
>
> On Tue, Aug 18, 2020 at 7:57 AM Martin Jansa <martin.jansa@gmail.com>
> wrote:
>
>> Yes, I'm seeing this weird concatenation with raspberrypi builds as well:
>>
>> find: unknown predicate `-mthumb/arch'
>> arm-webos-linux-gnueabi-gcc: error: unrecognized command-line option
>> '-mthumb/include/uapi/asm-generic/errno.h'
>> arm-webos-linux-gnueabi-gcc: fatal error: no input files
>> compilation terminated.
>>
>
> confirmed as the same with my beaglebone build that ran overnight.
>
> I'll have a closer look and send a fix (hopefully) .. but if anyone else
> is already doing this, please let me know and I'll work on something else!
>
> Cheers,
>
> Bruce
>
>
>
>>
>> But in this case it isn't fatal for the perf build, I've noticed it only
>> because it was failing with 5.4.51 currently in meta-raspberrypi and after
>> upgrade to 5.4.58 perf built OK, but this error message is still there in
>> log.do_compile.
>>
>> On Mon, Aug 17, 2020 at 9:47 PM Khem Raj <raj.khem@gmail.com> wrote:
>>
>>> On Mon, Aug 17, 2020 at 5:27 AM Richard Purdie
>>> <richard.purdie@linuxfoundation.org> wrote:
>>> >
>>> > On Sun, 2020-08-16 at 18:55 -0400, Bruce Ashfield wrote:
>>> > >
>>> > >
>>> > > On Sun, Aug 16, 2020 at 9:47 AM Richard Purdie <
>>> > > richard.purdie@linuxfoundation.org> wrote:
>>> > > > On Sat, 2020-08-15 at 19:41 -0700, Khem Raj wrote:
>>> > > > > Bruce,
>>> > > > >
>>> > > > > We need
>>> > > > >
>>> > > >
>>> https://github.com/torvalds/linux/commit/168200b6d6ea0cb5765943ec5da5b8149701f36a
>>> > > > > backported to 5.4 kernels for perf to build with -fno-common can
>>> > > > you
>>> > > > > queue this in your kernel updates as soon as you can ?
>>> > > >
>>> > > > I think we have other active kernels which will also need it as
>>> > > > even
>>> > > > with Khem's patch we still see failures:
>>> > > >
>>> > >
>>> > > I've taken care of linux-yocto now, I don't think there are any
>>> > > active use cases for those kernels that won't be covered.
>>> >
>>> > I bumped the meta-yocto BSPs with a quickly hacked up patch but we
>>> > still see:
>>> >
>>> > https://autobuilder.yoctoproject.org/typhoon/#/builders/65/builds/2339
>>> >
>>> > so I think there may be another patch we need somewhere...
>>> >
>>> > (which matches what we saw with Khem's _append which also failed).
>>> >
>>>
>>> there seems to be concaternation of compiler options without space
>>>
>>> | arm-poky-linux-gnueabi-gcc: error: unrecognized argument in option
>>> '-mfpu=neon/include/uapi/asm-generic/errno.h'
>>> | arm-poky-linux-gnueabi-gcc: note: valid arguments to '-mfpu=' are:
>>> auto crypto-neon-fp-armv8 fp-armv8 fpv4-sp-d16 fpv5-d16 fpv5-sp-d16
>>> neon neon-fp-armv8 neon-fp16 neon-vfpv3 neon-vfpv4 vfp vfp3 vfpv2
>>> vfpv3 vfpv3-d16 vfpv3-d16-fp16 vfpv3-fp16 vfpv3xd vfpv3xd-fp16 vfpv4
>>> vfpv4-d16
>>> | arm-poky-linux-gnueabi-gcc: fatal error: no input files
>>>
>>> but it also showed up on x86_64 in one of your prior builds, so I
>>> wonder if its something in kernel build system or is it processing
>>> TUNE_CCARGS in special ways.
>>>
>>> > There is still some piece of this we're missing, maybe on non-IA.
>>> >
>>> > Cheers,
>>> >
>>> > Richard
>>> >
>>> 
>>>
>>
>
> --
> - 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: 6191 bytes --]

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-18 16:03                           ` Martin Jansa
@ 2020-08-18 16:34                             ` Khem Raj
  2020-08-18 17:05                             ` Bruce Ashfield
  1 sibling, 0 replies; 25+ messages in thread
From: Khem Raj @ 2020-08-18 16:34 UTC (permalink / raw)
  To: Martin Jansa
  Cc: Bruce Ashfield, Richard Purdie, Jacob Kroon,
	Patches and discussions about the oe-core layer

On Tue, Aug 18, 2020 at 9:04 AM Martin Jansa <martin.jansa@gmail.com> wrote:
>
> The perf issue with find and cflags concatenation is fixed in upstream by:
> https://github.com/torvalds/linux/commit/e4ffd066ff440a57097e9140fa9e16ceef905de8
>

this should fix it. thanks for nailing it,  I guess older kernels will
have tough time with gcc10

> Adding V=1 to perf's EXTRA_OEMAKE makes it easier to see, sending a patch now.
>
> On Tue, Aug 18, 2020 at 2:21 PM Bruce Ashfield <bruce.ashfield@gmail.com> wrote:
>>
>>
>>
>> On Tue, Aug 18, 2020 at 7:57 AM Martin Jansa <martin.jansa@gmail.com> wrote:
>>>
>>> Yes, I'm seeing this weird concatenation with raspberrypi builds as well:
>>>
>>> find: unknown predicate `-mthumb/arch'
>>> arm-webos-linux-gnueabi-gcc: error: unrecognized command-line option '-mthumb/include/uapi/asm-generic/errno.h'
>>> arm-webos-linux-gnueabi-gcc: fatal error: no input files
>>> compilation terminated.
>>
>>
>> confirmed as the same with my beaglebone build that ran overnight.
>>
>> I'll have a closer look and send a fix (hopefully) .. but if anyone else is already doing this, please let me know and I'll work on something else!
>>
>> Cheers,
>>
>> Bruce
>>
>>
>>>
>>>
>>> But in this case it isn't fatal for the perf build, I've noticed it only because it was failing with 5.4.51 currently in meta-raspberrypi and after upgrade to 5.4.58 perf built OK, but this error message is still there in log.do_compile.
>>>
>>> On Mon, Aug 17, 2020 at 9:47 PM Khem Raj <raj.khem@gmail.com> wrote:
>>>>
>>>> On Mon, Aug 17, 2020 at 5:27 AM Richard Purdie
>>>> <richard.purdie@linuxfoundation.org> wrote:
>>>> >
>>>> > On Sun, 2020-08-16 at 18:55 -0400, Bruce Ashfield wrote:
>>>> > >
>>>> > >
>>>> > > On Sun, Aug 16, 2020 at 9:47 AM Richard Purdie <
>>>> > > richard.purdie@linuxfoundation.org> wrote:
>>>> > > > On Sat, 2020-08-15 at 19:41 -0700, Khem Raj wrote:
>>>> > > > > Bruce,
>>>> > > > >
>>>> > > > > We need
>>>> > > > >
>>>> > > > https://github.com/torvalds/linux/commit/168200b6d6ea0cb5765943ec5da5b8149701f36a
>>>> > > > > backported to 5.4 kernels for perf to build with -fno-common can
>>>> > > > you
>>>> > > > > queue this in your kernel updates as soon as you can ?
>>>> > > >
>>>> > > > I think we have other active kernels which will also need it as
>>>> > > > even
>>>> > > > with Khem's patch we still see failures:
>>>> > > >
>>>> > >
>>>> > > I've taken care of linux-yocto now, I don't think there are any
>>>> > > active use cases for those kernels that won't be covered.
>>>> >
>>>> > I bumped the meta-yocto BSPs with a quickly hacked up patch but we
>>>> > still see:
>>>> >
>>>> > https://autobuilder.yoctoproject.org/typhoon/#/builders/65/builds/2339
>>>> >
>>>> > so I think there may be another patch we need somewhere...
>>>> >
>>>> > (which matches what we saw with Khem's _append which also failed).
>>>> >
>>>>
>>>> there seems to be concaternation of compiler options without space
>>>>
>>>> | arm-poky-linux-gnueabi-gcc: error: unrecognized argument in option
>>>> '-mfpu=neon/include/uapi/asm-generic/errno.h'
>>>> | arm-poky-linux-gnueabi-gcc: note: valid arguments to '-mfpu=' are:
>>>> auto crypto-neon-fp-armv8 fp-armv8 fpv4-sp-d16 fpv5-d16 fpv5-sp-d16
>>>> neon neon-fp-armv8 neon-fp16 neon-vfpv3 neon-vfpv4 vfp vfp3 vfpv2
>>>> vfpv3 vfpv3-d16 vfpv3-d16-fp16 vfpv3-fp16 vfpv3xd vfpv3xd-fp16 vfpv4
>>>> vfpv4-d16
>>>> | arm-poky-linux-gnueabi-gcc: fatal error: no input files
>>>>
>>>> but it also showed up on x86_64 in one of your prior builds, so I
>>>> wonder if its something in kernel build system or is it processing
>>>> TUNE_CCARGS in special ways.
>>>>
>>>> > There is still some piece of this we're missing, maybe on non-IA.
>>>> >
>>>> > Cheers,
>>>> >
>>>> > Richard
>>>> >
>>>> 
>>
>>
>>
>> --
>> - Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end
>> - "Use the force Harry" - Gandalf, Star Trek II
>>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon
  2020-08-18 16:03                           ` Martin Jansa
  2020-08-18 16:34                             ` Khem Raj
@ 2020-08-18 17:05                             ` Bruce Ashfield
  1 sibling, 0 replies; 25+ messages in thread
From: Bruce Ashfield @ 2020-08-18 17:05 UTC (permalink / raw)
  To: Martin Jansa
  Cc: Khem Raj, Richard Purdie, Jacob Kroon,
	Patches and discussions about the oe-core layer

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

On Tue, Aug 18, 2020 at 12:04 PM Martin Jansa <martin.jansa@gmail.com>
wrote:

> The perf issue with find and cflags concatenation is fixed in upstream by:
>
> https://github.com/torvalds/linux/commit/e4ffd066ff440a57097e9140fa9e16ceef905de8
>
> Adding V=1 to perf's EXTRA_OEMAKE makes it easier to see, sending a patch
> now.
>
>
Yah. I did the same thing here, was chasing the issue through the generated
files, but hadn't tracked down the root cause yet.

Looks good to me. I'll chuck away my debug.

Bruce



> On Tue, Aug 18, 2020 at 2:21 PM Bruce Ashfield <bruce.ashfield@gmail.com>
> wrote:
>
>>
>>
>> On Tue, Aug 18, 2020 at 7:57 AM Martin Jansa <martin.jansa@gmail.com>
>> wrote:
>>
>>> Yes, I'm seeing this weird concatenation with raspberrypi builds as well:
>>>
>>> find: unknown predicate `-mthumb/arch'
>>> arm-webos-linux-gnueabi-gcc: error: unrecognized command-line option
>>> '-mthumb/include/uapi/asm-generic/errno.h'
>>> arm-webos-linux-gnueabi-gcc: fatal error: no input files
>>> compilation terminated.
>>>
>>
>> confirmed as the same with my beaglebone build that ran overnight.
>>
>> I'll have a closer look and send a fix (hopefully) .. but if anyone else
>> is already doing this, please let me know and I'll work on something else!
>>
>> Cheers,
>>
>> Bruce
>>
>>
>>
>>>
>>> But in this case it isn't fatal for the perf build, I've noticed it only
>>> because it was failing with 5.4.51 currently in meta-raspberrypi and after
>>> upgrade to 5.4.58 perf built OK, but this error message is still there in
>>> log.do_compile.
>>>
>>> On Mon, Aug 17, 2020 at 9:47 PM Khem Raj <raj.khem@gmail.com> wrote:
>>>
>>>> On Mon, Aug 17, 2020 at 5:27 AM Richard Purdie
>>>> <richard.purdie@linuxfoundation.org> wrote:
>>>> >
>>>> > On Sun, 2020-08-16 at 18:55 -0400, Bruce Ashfield wrote:
>>>> > >
>>>> > >
>>>> > > On Sun, Aug 16, 2020 at 9:47 AM Richard Purdie <
>>>> > > richard.purdie@linuxfoundation.org> wrote:
>>>> > > > On Sat, 2020-08-15 at 19:41 -0700, Khem Raj wrote:
>>>> > > > > Bruce,
>>>> > > > >
>>>> > > > > We need
>>>> > > > >
>>>> > > >
>>>> https://github.com/torvalds/linux/commit/168200b6d6ea0cb5765943ec5da5b8149701f36a
>>>> > > > > backported to 5.4 kernels for perf to build with -fno-common can
>>>> > > > you
>>>> > > > > queue this in your kernel updates as soon as you can ?
>>>> > > >
>>>> > > > I think we have other active kernels which will also need it as
>>>> > > > even
>>>> > > > with Khem's patch we still see failures:
>>>> > > >
>>>> > >
>>>> > > I've taken care of linux-yocto now, I don't think there are any
>>>> > > active use cases for those kernels that won't be covered.
>>>> >
>>>> > I bumped the meta-yocto BSPs with a quickly hacked up patch but we
>>>> > still see:
>>>> >
>>>> >
>>>> https://autobuilder.yoctoproject.org/typhoon/#/builders/65/builds/2339
>>>> >
>>>> > so I think there may be another patch we need somewhere...
>>>> >
>>>> > (which matches what we saw with Khem's _append which also failed).
>>>> >
>>>>
>>>> there seems to be concaternation of compiler options without space
>>>>
>>>> | arm-poky-linux-gnueabi-gcc: error: unrecognized argument in option
>>>> '-mfpu=neon/include/uapi/asm-generic/errno.h'
>>>> | arm-poky-linux-gnueabi-gcc: note: valid arguments to '-mfpu=' are:
>>>> auto crypto-neon-fp-armv8 fp-armv8 fpv4-sp-d16 fpv5-d16 fpv5-sp-d16
>>>> neon neon-fp-armv8 neon-fp16 neon-vfpv3 neon-vfpv4 vfp vfp3 vfpv2
>>>> vfpv3 vfpv3-d16 vfpv3-d16-fp16 vfpv3-fp16 vfpv3xd vfpv3xd-fp16 vfpv4
>>>> vfpv4-d16
>>>> | arm-poky-linux-gnueabi-gcc: fatal error: no input files
>>>>
>>>> but it also showed up on x86_64 in one of your prior builds, so I
>>>> wonder if its something in kernel build system or is it processing
>>>> TUNE_CCARGS in special ways.
>>>>
>>>> > There is still some piece of this we're missing, maybe on non-IA.
>>>> >
>>>> > Cheers,
>>>> >
>>>> > Richard
>>>> >
>>>> 
>>>>
>>>
>>
>> --
>> - Thou shalt not follow the NULL pointer, for chaos and madness await
>> thee at its end
>> - "Use the force Harry" - Gandalf, Star Trek II
>>
>>

-- 
- 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: 7381 bytes --]

^ permalink raw reply	[flat|nested] 25+ messages in thread

end of thread, other threads:[~2020-08-18 17:05 UTC | newest]

Thread overview: 25+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-04  8:20 [PATCH] gcc10: Don't default back to -fcommon Jacob Kroon
2020-08-05  0:14 ` [OE-core] " Khem Raj
2020-08-05  5:05   ` Jacob Kroon
2020-08-05  6:00     ` Khem Raj
2020-08-05  6:17       ` Khem Raj
2020-08-05 13:15         ` Jacob Kroon
2020-08-05 13:26           ` Richard Purdie
2020-08-15 22:10         ` Richard Purdie
2020-08-16  1:56           ` Khem Raj
2020-08-16  2:41             ` Khem Raj
2020-08-16 13:47               ` Richard Purdie
2020-08-16 22:55                 ` Bruce Ashfield
2020-08-16 23:14                   ` Khem Raj
2020-08-17 12:27                   ` Richard Purdie
2020-08-17 14:15                     ` Bruce Ashfield
2020-08-17 14:45                       ` Richard Purdie
2020-08-17 19:47                     ` Khem Raj
2020-08-17 19:55                       ` Khem Raj
2020-08-17 21:07                         ` Bruce Ashfield
2020-08-18 11:57                       ` Martin Jansa
2020-08-18 12:21                         ` Bruce Ashfield
2020-08-18 16:03                           ` Martin Jansa
2020-08-18 16:34                             ` Khem Raj
2020-08-18 17:05                             ` Bruce Ashfield
2020-08-16 23:29                 ` 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.