devicetree-compiler.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Gibson <david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org>
To: Paolo Bonzini <pbonzini-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Cc: David Gibson <dgibson-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
	devicetree-compiler-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH] fix libfdt_env.h for RHEL7
Date: Wed, 8 Mar 2017 12:44:00 +1100	[thread overview]
Message-ID: <20170308014400.GE19967@umbus.fritz.box> (raw)
In-Reply-To: <88992ad1-f92f-7006-3b78-6b2e631f4110-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

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

On Mon, Mar 06, 2017 at 01:52:54PM +0100, Paolo Bonzini wrote:
> 
> 
> On 06/03/2017 02:25, David Gibson wrote:
> > I'm not sure which your primary concern is:
> > 
> > 1) For the RHEL release itself.  I don't think updating the qemu module
> > is terribly relevant for this, since we build against the packaged
> > libfdt rather than the qemu submodule.  We could ask Mirek to pull the
> > define fix into the downstream libfdt package.
> > 
> > 2) For building upstream qemu on RHEL, using the submodule.  I don't
> > have a good answer for this one.
> 
> Yes, it is for the submodule.  Because QEMU now requires 1.4.2, the
> submodule is needed to compile QEMU on past RHEL releases.  Maybe we can
> "#undef __bitwise" or something like that.

Hm.  I wonder why I haven't hit that with my own recent builds.  I
guess they were on Power, maybe the problematic symbol is only on the
x86 libc.

The problem should go away in not too long - Mirek is packaging dtc
1.4.3 for RHEL.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

      parent reply	other threads:[~2017-03-08  1:44 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-03 16:29 [PATCH] fix libfdt_env.h for RHEL7 Paolo Bonzini
     [not found] ` <1488558597-21364-1-git-send-email-pbonzini-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2017-03-06  1:25   ` David Gibson
     [not found]     ` <20170306122557.71669b69-K0bRW+63XPQe6aEkudXLsA@public.gmane.org>
2017-03-06 12:52       ` Paolo Bonzini
     [not found]         ` <88992ad1-f92f-7006-3b78-6b2e631f4110-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2017-03-08  1:44           ` David Gibson [this message]

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=20170308014400.GE19967@umbus.fritz.box \
    --to=david-xt8fgy+axnrb3ne2bgzf6laj5h9x9tb+@public.gmane.org \
    --cc=devicetree-compiler-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=dgibson-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=pbonzini-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    /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;
as well as URLs for NNTP newsgroup(s).