Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH/next 1/1] package/libffi: fix build failure with powerpc7
Date: Tue, 10 Dec 2019 23:18:15 +0100	[thread overview]
Message-ID: <20191210231815.5756578e@windsurf.home> (raw)
In-Reply-To: <20a72d62-aecb-be18-9bb7-2f69115e14ba@mind.be>

Hello Fabrice,

On Tue, 3 Dec 2019 23:53:06 +0100
Arnout Vandecappelle <arnout@mind.be> wrote:

> On 24/11/2019 10:03, Fabrice Fontaine wrote:
> > Fixes:
> >  - http://autobuild.buildroot.org/results/5c9dd8fb3b6a128882b6250f197c80232d8a3b53
> > 
> > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>  
> 
>  Upstream merged a different fix [1] which TBH doesn't look as good... Could you
> check with upstream what is the better solution?
> 
>  Regards,
>  Arnout
> 
> [1]
> https://github.com/libffi/libffi/pull/532/commits/2138adb2a5b6a0bc2fe3518e0645eacc89b6f392

Fabrice, have you had the chance to check with upstream what is the
most correct fix?

Also, your commit title says it fixes it for powerpc7, but the issue
also appears with older powerpc platforms, and indeed the upstream
commit says it fixes the issue for powerpc7 and older platforms.

Since the issue pops up quite a lot in the autobuilders, it would be
nice if we could finalize this patch and merge it.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

  reply	other threads:[~2019-12-10 22:18 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-11-24  9:03 [Buildroot] [PATCH/next 1/1] package/libffi: fix build failure with powerpc7 Fabrice Fontaine
2019-11-24 19:28 ` Yann E. MORIN
2019-11-24 19:57   ` Fabrice Fontaine
2019-12-03 22:53 ` Arnout Vandecappelle
2019-12-10 22:18   ` Thomas Petazzoni [this message]
2019-12-11  7:57     ` Fabrice Fontaine
2019-12-23 20:22 ` 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=20191210231815.5756578e@windsurf.home \
    --to=thomas.petazzoni@bootlin.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