|
|
|
|
|
|
|
|
|
|
xen-ia64-devel
[Xen-devel] Re: [PATCH] xen: compilation fix for ia64 support.
To: |
Jeremy Fitzhardinge <jeremy@xxxxxxxx>, "Luck, Tony" <tony.luck@xxxxxxxxx> |
Subject: |
[Xen-devel] Re: [PATCH] xen: compilation fix for ia64 support. |
From: |
Isaku Yamahata <yamahata@xxxxxxxxxxxxx> |
Date: |
Fri, 4 Jul 2008 11:41:09 +0900 |
Cc: |
xen-ia64-devel@xxxxxxxxxxxxxxxxxxx, xen-devel@xxxxxxxxxxxxxxxxxxx, linux-ia64@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx |
Delivery-date: |
Thu, 03 Jul 2008 19:41:30 -0700 |
Envelope-to: |
www-data@xxxxxxxxxxxxxxxxxxx |
In-reply-to: |
<486D49C7.5070702@xxxxxxxx> |
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> |
References: |
<20080703070952.GB15683%yamahata@xxxxxxxxxxxxx> <486D49C7.5070702@xxxxxxxx> |
Sender: |
xen-devel-bounces@xxxxxxxxxxxxxxxxxxx |
User-agent: |
Mutt/1.5.6i |
On Thu, Jul 03, 2008 at 02:51:03PM -0700, Jeremy Fitzhardinge wrote:
> Isaku Yamahata wrote:
> >don't include asm/pvclock-abi.h directly, but asm/xen/pvclock-abi.h
> >At this moment linux ia64 won't accept asm/pvclock-abi.h because
> >there's no user of it.
>
> Huh? Is this a ia64 maintainer policy? Aren't you about to start using it?
Ah sorry, that was only what I expected. I haven't asked the maintainer yet.
I'm asking now.
> > Instead make it include asm/xen/pvclock-abi.h
> >and create asm-x86/xen/pvclock-abi.h which includes asm-x86/pvclock-abi.h.
> >
>
> I'm not very keen on crufting up asm-x86/xen/ for the ia64 code.
> Couldn't you have an asm-ia64/xen/pvclock-abi.h which includes
> <asm-x86/pvclock-abi.h>?
I'm file with that. In fact I don't have strong opinion here
as long as the linux ia64 maintainer, Tony, is fine.
Tony, what do you think about the following patch?
- path
asm-ia64/pvclock-abi.h versus asm-ia64/xen/pvclock-abi.h
- including asm-x86/pvclock-abi.h
>From fa958c98713f193eeab2cbe9bab1676059b850a3 Mon Sep 17 00:00:00 2001
From: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>
Date: Fri, 4 Jul 2008 11:29:33 +0900
Subject: [PATCH] ia64/xen: add a neccessary header file to copmle
include/xen/interface/xen.h
Create include/asm-ia64/pvclock-abi.h to compile which includes
include/asm-x86/pvclock-abi.h because ia64/xen uses same struture.
Hopefully include/asm-x86/pvclock-abi.h would be moved to somewhere
more generic.
Signed-off-by: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>
---
include/asm-ia64/pvclock-abi.h | 5 +++++
1 files changed, 5 insertions(+), 0 deletions(-)
create mode 100644 include/asm-ia64/pvclock-abi.h
diff --git a/include/asm-ia64/pvclock-abi.h b/include/asm-ia64/pvclock-abi.h
new file mode 100644
index 0000000..1c34d11
--- /dev/null
+++ b/include/asm-ia64/pvclock-abi.h
@@ -0,0 +1,5 @@
+/*
+ * use same streucute to x86's
+ * Hopefully asm-x86/pvclock-abi.h would be moved to somewhere more generic.
+ */
+#include <asm-x86/pvclock-abi.h>
--
1.5.3
--
yamahata
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
|
|
|
|