From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Wolfgang Denk <wd@denx.de>
Cc: Adrian Alonso <aalonso@secretlab.ca>,
"poky@yoctoproject.org" <poky@yoctoproject.org>
Subject: Re: [PATCH 05/25] u-boot: disable -Os option not supported by gcc
Date: Sun, 23 Jan 2011 00:11:53 +0000 [thread overview]
Message-ID: <1295741513.14388.37111.camel@rex> (raw)
In-Reply-To: <20110122144831.609C3B335@gemini.denx.de>
On Sat, 2011-01-22 at 15:48 +0100, Wolfgang Denk wrote:
> In message <1295565854.14388.27977.camel@rex> you wrote:
> > > What exactly is the reason that -Os is not supported?
> >
> > Its not so much not supported, its that gcc 4.5 seems to have totally
> > broken it and for better or worse, we're on that gcc version. Until we
> > have a toolchain where the option works what do we do?
>
> Usually I try to approach problems at the root cause - the best
> solution was if we had patches for GCC to make it work. It appears
> that "-Os" is only one way to trigger issues in recent GCC version;
> plain "-O" or "-O1" trigger other problems, see for example
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45052
Right, I'm aware that recent versions of gcc have issues sadly :(. I'm
unaware of any resource available to Yocto with the right combination
of time/experience to fix that problem at source at this time though :(.
> > Well, its a case of "totally broken" or a risk of size issues on some
> > systems so I'm in favour of the latter. I agree the toolchain issues
> > need to be resolved and I'm sure they will sooner or later. I have
> > noticed upstream gcc don't seem to like the -Os option though and it
> > often seems to be the source of problems...
>
> Probably this is (at least partially) due to the fact that only such
> lunatic fringe groups as the embedded folks tend to use exotic
> features like -Os, and our word is of little interest to the GCC
> PTBs.
I don't see embedded folks as a lunatic fringe group and I would like to
see gcc care a little more about some of these use cases and options,
you don't need to convince me about this.
> Did anybody from the Yocto project / Poky / the LF / Intel or
> any other bigger player try to put his weight in?
I don't think the world works this way.
Lets take a step back and remind ourselves why Yocto exists in the first
place. Yocto exists to try and take Embedded Linux to a new level and
the fact there are people working on it shows a commitment from the LF,
Intel and other players to go and work on some hard problems. Great.
We can't fix every problem overnight, we don't have infinite resources.
Sad, but true.
For gcc specifically, consider the steps you need to take to approach
upstream about a problem. Top of the list is using recent versions. If
we try and report a bug upstream in an old version we'll just get told
to try with the latest. We have moved all our architectures to a modern
version of gcc although I think we're still a little behind. We're now
trying to figure out exactly how the land lies and make sure we have
something usable in the meantime (as you've noticed there are problems).
As and when we track down issues, they will get reported upstream and if
Yocto becomes a strong player representing embedded, I'd hope those
reports will get acted on. It will take time for Yocto to become a
significant player and it will take help from the wider community to
support that. We're not there yet.
FWIW, off the top of my head I think we have already fed a couple of
issues into the gcc community in one way or another.
We are open to suggestions on the issues where effort is needed. As I've
said above we have finite resources and some requirements from the
people providing those resources. We're going to do the best we can with
what we have.
This kind of issue is one reason I do want to keep up following upstream
versions so we can interact with the upstream more quickly and
effectively. I appreciate there are downsides to it too but I think in
the long run if we can keep on top of things and stay up to date it will
assist us in the long run immensely as we'll be able to feed back
information to the upstream much more quickly and efficiently if any
problems do appear.
So please try and help Yocto achieve these things but keep in mind its
not as simple as you make out above. There is a lot to do and help is
appreciated, gcc is just one facet of what we need to work on.
Cheers,
Richard
next prev parent reply other threads:[~2011-01-23 0:12 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-17 20:29 [PATCH 01/25] xilinx-kernel: bbclass to handle linux kernel hw/sw integration Adrian Alonso
2011-01-17 20:29 ` [PATCH 02/25] xilinx-boot: bbclass to handle u-boot " Adrian Alonso
2011-01-17 20:29 ` [PATCH 03/25] gcc: crosssdk unset TARGET_FPU Adrian Alonso
2011-01-17 23:55 ` Ilya Yanok
2011-01-18 0:59 ` Adrian Alonso
2011-01-18 1:45 ` Ilya Yanok
2011-01-20 2:11 ` Ilya Yanok
2011-01-20 17:35 ` Saul Wold
2011-01-20 17:53 ` Adrian Alonso
2011-01-21 0:43 ` Ilya Yanok
2011-01-21 0:47 ` [PATCH] native,nativesdk,crosssdk: reset TARGET_FPU Ilya Yanok
2011-01-17 20:29 ` [PATCH 04/25] u-boot: update version use xilinx-boot bbclass Adrian Alonso
2011-01-17 20:29 ` [PATCH 05/25] u-boot: disable -Os option not supported by gcc Adrian Alonso
2011-01-20 18:53 ` Darren Hart
2011-01-20 18:55 ` Darren Hart
2011-01-20 21:41 ` Adrian Alonso
2011-01-20 23:05 ` Wolfgang Denk
2011-01-20 23:24 ` Richard Purdie
2011-01-22 14:48 ` Wolfgang Denk
2011-01-23 0:11 ` Richard Purdie [this message]
2011-01-25 9:09 ` Wolfgang Denk
2011-01-25 22:49 ` Richard Purdie
2011-01-17 20:29 ` [PATCH 06/25] virtex4: fix machine name in u-boot preferred version Adrian Alonso
2011-01-17 20:29 ` [PATCH 07/25] virtex5: " Adrian Alonso
2011-01-17 20:29 ` [PATCH 08/25] xilinx-boot: add ml405 in default target configuration Adrian Alonso
2011-01-17 20:29 ` [PATCH 09/25] u-boot-xilinx: ml405 uartlite support Adrian Alonso
2011-01-17 20:29 ` [PATCH 10/25] xilinx-boot: split configure function Adrian Alonso
2011-01-17 20:29 ` [PATCH 11/25] xilinx-boot: add do_mk_xparam function Adrian Alonso
2011-01-17 20:29 ` [PATCH 12/25] u-boot-xilinx: ml507 uartlite support Adrian Alonso
2011-01-17 20:29 ` [PATCH 13/25] u-boot-xilinx: remove hardcoded include path Adrian Alonso
2011-01-17 20:29 ` [PATCH 14/25] linux-xilinx: update kernel version 2.6.37 rc4 Adrian Alonso
2011-01-17 20:29 ` [PATCH 15/25] linux-xilinx: remove hardcoded include path Adrian Alonso
2011-01-17 20:29 ` [PATCH 16/25] linux-xilinx: add license files checksum Adrian Alonso
2011-01-17 20:29 ` [PATCH 17/25] device-table: add tty uartlite static dev nodes Adrian Alonso
2011-01-17 20:29 ` [PATCH 18/25] u-boot-xilinx: add license files checksum Adrian Alonso
2011-01-17 20:29 ` [PATCH 19/25] tune-ppc405: append ppc440 in PACKAGE_EXTRA_ARCHS Adrian Alonso
2011-01-17 20:29 ` [PATCH 20/25] tune-ppc440: append ppc405 " Adrian Alonso
2011-01-17 20:29 ` [PATCH 21/25] virtex4: update default serial console Adrian Alonso
2011-01-17 20:29 ` [PATCH 22/25] virtex5: " Adrian Alonso
2011-01-17 20:29 ` [PATCH 23/25] defconfig: virtex4 config for linux 2.6.37.rc4 Adrian Alonso
2011-01-17 20:29 ` [PATCH 24/25] defconfig: virtex5 " Adrian Alonso
2011-01-17 20:29 ` [PATCH 25/25] openssl: powerpc gnueabi toolchain support Adrian Alonso
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1295741513.14388.37111.camel@rex \
--to=richard.purdie@linuxfoundation.org \
--cc=aalonso@secretlab.ca \
--cc=poky@yoctoproject.org \
--cc=wd@denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.