From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] Help to fix Microblaze issues in Buildroot
Date: Wed, 20 Nov 2013 11:10:27 +0100 [thread overview]
Message-ID: <20131120111027.472fe84b@skate> (raw)
In-Reply-To: <CAM+bi4t7Z4RWQ=6r=qOUOXKpqV76hRSogU0iBp12FKvN+fGviA@mail.gmail.com>
Dear Alvaro Gamez,
On Wed, 20 Nov 2013 11:03:59 +0100, Alvaro Gamez wrote:
> Sorry for the big delay on this issue.
>
> Now that I've spent a few hours looking at the code, I can firmly say this
> is a compiler bug.
>
> I've studied beecrypt's sha384.c code (openssl and others implement exactly
> the same code from a common source) and I've managed to reduce the code to
> what I think is a minimal example of bug triggering code.
>
> After requesting gcc to preprocess the source, the same error still
> happens. Almost any minimal change I've tried to the preprocessed source
> makes the error go away (indexes or values seem to be irrelevant, I've just
> mantained the numbers the code came from). An optimization flag is required
> to trigger the bug (-Os, -O1,2,3). I haven't been successful trying to find
> which specific optimization flag is the one that causes the error.
>
> I've manually merged both (original and preprocessed files) to ease the
> analysis of the original source code and the resulting code. I'm sorry I
> can't provide any more information, I really don't have any grasp of gcc's
> internals.
Thanks! In fact, Spenser submitted patches to add support for
Microblaze in the internal toolchain backend. This is useful because
the pre-built external toolchains that we have been using until now are
quite old, and not maintained by Xilinx.
Therefore, the plan is, I believe:
(1) Merge Spenser's patches to add Microblaze internal toolchain
support.
(2) Remove the broken Microblaze external toolchains
(3) Hopefully, convince Xilinx to also provide pre-built external
toolchains that are up-to-date. This will have to be done by people
having contacts within Xilinx.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
next prev parent reply other threads:[~2013-11-20 10:10 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-06 16:54 [Buildroot] Help to fix Microblaze issues in Buildroot Thomas Petazzoni
[not found] ` <CAM+bi4vGmKHRA3j0ua-dsOBzTuj-BPrM2+z3papeyfDjjazhgw@mail.gmail.com>
2013-11-20 10:03 ` Alvaro Gamez
2013-11-20 10:10 ` Thomas Petazzoni [this message]
2013-11-20 10:28 ` Thomas De Schampheleire
2013-11-20 10:32 ` Alvaro Gamez
2013-11-20 10:36 ` Alvaro Gamez
2013-11-20 10:41 ` Thomas Petazzoni
2013-11-20 12:40 ` Spenser Gilliland
2013-11-20 13:56 ` Alvaro Gamez
2013-11-20 14:00 ` Spenser Gilliland
2013-11-20 14:09 ` Thomas Petazzoni
2013-11-20 14:15 ` Alvaro Gamez
2013-11-20 14:18 ` Thomas Petazzoni
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=20131120111027.472fe84b@skate \
--to=thomas.petazzoni@free-electrons.com \
--cc=buildroot@busybox.net \
/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.