All of lore.kernel.org
 help / color / mirror / Atom feed
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: ANNIE LI <annie.li@oracle.com>
Cc: xen-devel@lists.xensource.com, "Luck, Tony" <tony.luck@intel.com>,
	Ian Campbell <ian.campbell@citrix.com>
Subject: Re: [PATCH] ia64: fix build breakage because of conflicting u64 guest handles
Date: Mon, 5 Dec 2011 22:27:24 -0500	[thread overview]
Message-ID: <20111206032724.GC6647@phenom.dumpdata.com> (raw)
In-Reply-To: <4EDC8D57.6050900@oracle.com>

On Mon, Dec 05, 2011 at 05:22:31PM +0800, ANNIE LI wrote:
> 
> 
> On 2011-12-1 2:39, Konrad Rzeszutek Wilk wrote:
> >On Wed, Nov 30, 2011 at 10:22:37AM -0800, Luck, Tony wrote:
> >>include/xen/interface/xen.h:526: error: conflicting types for ‘__guest_handle_u64’
> >>arch/ia64/include/asm/xen/interface.h:74: error: previous declaration of ‘__guest_handle_u64’ was here
> >>
> >>Problem introduced by "xen/granttable: Introducing grant table V2 stucture"
> >>
> >>which added a new definition to include/xen/interface/xen.h for "u64".
> >>
> >>Fix: delete the ia64 arch specific definition.
> >>
> >>Signed-off-by: Tony Luck<tony.luck@intel.com>
> >>---
> >>
> >>Can someone either fold this into the above patch, or add it to the
> >>same tree that is feeding into linux-next - I saw the breakage in
> >>today's "next-20111130" tag.  Thanks.
> >Ah, I can fold it in. Thanks!
> A definition for uint64_t already existing in
> arch/x86/include/asm/xen/interface.h, 58 line:
> DEFINE_GUEST_HANDLE(uint64_t);
> Maybe it is better to remove the definition in
> include/xen/interface/xen.h of grant table v2 patch, and not change
> code of arch/ia64/include/asm/xen/interface.h.
> 
> Konrad, did you fold it already? or I revert the definition in my
> following sub-page and transitive patches?

I took Tony's patch.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

      reply	other threads:[~2011-12-06  3:27 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-30 18:22 [PATCH] ia64: fix build breakage because of conflicting u64 guest handles Luck, Tony
2011-11-30 18:39 ` Konrad Rzeszutek Wilk
2011-12-05  9:22   ` ANNIE LI
2011-12-06  3:27     ` Konrad Rzeszutek Wilk [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=20111206032724.GC6647@phenom.dumpdata.com \
    --to=konrad.wilk@oracle.com \
    --cc=annie.li@oracle.com \
    --cc=ian.campbell@citrix.com \
    --cc=tony.luck@intel.com \
    --cc=xen-devel@lists.xensource.com \
    /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.