From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] Analysis of build failures, december 26th
Date: Sun, 29 Dec 2013 18:06:27 +0100 [thread overview]
Message-ID: <20131229180627.2633245f@skate> (raw)
In-Reply-To: <E6BF041E97966E4DB955F0DF883AD2D685F39E91@de02wembxa.internal.synopsys.com>
Dear Mischa Jonker,
On Sun, 29 Dec 2013 12:58:53 +0000, Mischa Jonker wrote:
> > Seems to be a uClibc issue on ARC:
> >
> > e4defrag.c:203:12: error: conflicting types for 'posix_fadvise'
> >
> > Mischa, can you have a look?
> >
> OK, I will have a look.
Thanks!
> > The package uses pthread_setaffinity_np(), which is only available in NPTL.
> > Options:
> >
> > */ change iozone to not use pthread_setaffinity_np()
> >
> > */ have an additional kconfig knob that tells whether the thread
> > implementation is NPTL or not and disallow iozone when !NPTL
> >
>
> I believe it has been suggested before to introduce HAS_PTHREADS_NATIVE and let it depend on this.
> http://patchwork.ozlabs.org/patch/258714/
> http://patchwork.ozlabs.org/patch/290224/
>
> Back then you were reworking internal toolchain build infrastructure; do you want me to make a new patch that introduces this (and also adds an option for external toolchain where applicable?)
After thinking about it, I believe it's a little bit too heavy to add
yet another Kconfig knob to know whether the toolchain has NPTL support
or not. Since for the moment, only iozone is affected by this problem,
I believe an iozone-specific solution is better.
So I've just sent a patch against iozone that adds a dummy empty
pthread_setaffinity_np() function when the uClibc doesn't use NPTL.
Since most likely the absence of NPTL is going to be on non-SMP
architectures, it's not a big problem to not have
pthread_setaffinity_np().
Let me know what you think about it.
> 1st issue is about length of some instructions being counted incorrectly, which makes compiler select short branch instructions where they don't fit,
> 2nd issue is about if-conversion trying to predicate instructions that cannot be predicated.
>
> I have been filing internal bug reports for these and I believe both issues are fixed. We just need to update git commit ID for gcc, binutils, etc.
> Let me discuss which exact release should be used and get back to you (i.e. I'll submit a patch with version bump of the tools).
> Everyone is out for the holiday season now, so it may take another week or so.
Sure, no problem.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
next prev parent reply other threads:[~2013-12-29 17:06 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-27 7:30 [Buildroot] [autobuild.buildroot.net] Build results for 2013-12-26 Thomas Petazzoni
2013-12-27 9:35 ` [Buildroot] Analysis of build failures, december 26th Thomas Petazzoni
2013-12-27 10:01 ` Baruch Siach
2013-12-27 13:31 ` Fatih Aşıcı
2013-12-27 13:33 ` Thomas Petazzoni
2013-12-27 13:57 ` Fatih Aşıcı
2013-12-27 13:59 ` Thomas Petazzoni
2013-12-27 14:22 ` Fatih Aşıcı
2013-12-27 14:24 ` Thomas Petazzoni
2013-12-29 12:26 ` Baruch Siach
2013-12-29 12:58 ` Mischa Jonker
2013-12-29 17:06 ` Thomas Petazzoni [this message]
2013-12-30 8:12 ` Fatih Aşıcı
2013-12-30 9:40 ` Fatih Aşıcı
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=20131229180627.2633245f@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox