WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] Fix ia64 tools build

To: Keir Fraser <keir@xxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Fix ia64 tools build
From: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>
Date: Fri, 29 Oct 2010 14:12:21 +0100
Cc: KUWAMURA Shin'ya <kuwa@xxxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 29 Oct 2010 06:13:27 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C8F03C83.89B8%keir@xxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Organization: Citrix Systems, Inc.
References: <C8F03C83.89B8%keir@xxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Fri, 2010-10-29 at 08:55 +0100, Keir Fraser wrote:
> On 29/10/2010 08:39, "KUWAMURA Shin'ya" <kuwa@xxxxxxxxxxxxxx> wrote:
> 
> > Ping?
> > 
> > This patch is still required for ia64 tools.
> > 
> > Best regards,
> 
> I think it would ne neater to have your own
> tools/libxc/ia64/ac_ia64_tools.h, and then conditionally include that from
> Xen's acenv.h (e.g., dependent on __XEN_TOOLS__). Or revert the bits of c/s
> 22066 that broke the ia64 build in the first place -- possibly the changes
> were based on a false assumption?

Yeah, I thought it was unused. I may have confused it with
tools/libxc/ia64/acpi/platform/aclinux.h which is in fact just a symlink
to xen/include/acpi/platform/aclinux.h.

Note that tools/libxc is now licensed as LGPL so before simply
reinstating the old file it probably needs to be confirmed that it is ok
to license that way. (that said the contents of the file don't seem
particularly exciting, a cleanroom rewrite based on compiling and
observing the errors would be pretty trivial)

>  Cc'ing the author of that patch, Ian
> Campbell. In any case, ifdef'ing the bulk of aclinux.h is just gross, I
> don't like that fix.

Why do we even need what appears to be a simplistic OS abstraction layer
in the tools first place?

Ian.


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

<Prev in Thread] Current Thread [Next in Thread>