Porting PV-on-HVM for ia64 platform (Take 3)

Porting PV-on-HVM for ia64 platform (Take 3)

[DOI Tsunehisa]

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

Hi,

  I will post patches for PV-on-HVM on ia64 platform.

  These patch include:

   * unmodified-driver.patch
     + cut off unused codes for IPF
       - get_hypercall_stabs() is x86 specific function. We don't
         use it, and it occurrs compile error on ia64 platform.
         Thus it is cut off.
   * unmodified-build.patch
     + modify build rule for IPF
       - This patch appends a build rule for ia64 platform to
         mkbuildtree command.

Thanks,
- Tsunehisa Doi

[-- Attachment #2: unmodified-driver.patch --]
[-- Type: text/plain, Size: 1257 bytes --]

# HG changeset patch
# User Doi.Tsunehisa@jp.fujitsu.com
# Node ID 566e35701c7dd9f2aec0af470e39054810867beb
# Parent  4a17792e0a5a05caad68244abfe09948a6f2d90d
Modify unmodified_drivers code for IPF

Signed-off-by: Tsunehisa Doi <Doi.Tsunehisa@jp.fujitsu.com>
Signed-off-by: Tomonari Horikoshi <t.horikoshi@jp.fujitsu.com>

diff -r 4a17792e0a5a -r 566e35701c7d unmodified_drivers/linux-2.6/platform-pci/platform-pci.c
--- a/unmodified_drivers/linux-2.6/platform-pci/platform-pci.c	Mon Sep 04 16:04:27 2006 +0900
+++ b/unmodified_drivers/linux-2.6/platform-pci/platform-pci.c	Mon Sep 04 16:06:31 2006 +0900
@@ -115,6 +115,7 @@ unsigned long alloc_xen_mmio(unsigned lo
 	return addr;
 }
 
+#ifndef __ia64__
 /* Lifted from hvmloader.c */
 static int get_hypercall_stubs(void)
 {
@@ -160,6 +161,7 @@ static int get_hypercall_stubs(void)
 
 	return 0;
 }
+#endif /* !__ia64__ */
 
 static int __devinit platform_pci_init(struct pci_dev *pdev,
 				       const struct pci_device_id *ent)
@@ -201,10 +203,12 @@ static int __devinit platform_pci_init(s
 	platform_mmio = mmio_addr;
 	platform_mmiolen = mmio_len;
 
+#ifndef __ia64__
 	ret = get_hypercall_stubs();
 	if (ret < 0)
 		goto out;
 
+#endif /* __ia64__ */
 	
 	if ((ret = init_xen_info()))
 		goto out;

[-- Attachment #3: unmodified-build.patch --]
[-- Type: text/plain, Size: 1022 bytes --]

# HG changeset patch
# User Doi.Tsunehisa@jp.fujitsu.com
# Node ID f1682f5c0e7d8e7778da907222a3d72bc759871b
# Parent  566e35701c7dd9f2aec0af470e39054810867beb
Modify unmodified_drivers build rule for IPF

Signed-off-by: Tsunehisa Doi <Doi.Tsunehisa@jp.fujitsu.com>
Signed-off-by: Tomonari Horikoshi <t.horikoshi@jp.fujitsu.com>

diff -r 566e35701c7d -r f1682f5c0e7d unmodified_drivers/linux-2.6/mkbuildtree
--- a/unmodified_drivers/linux-2.6/mkbuildtree	Mon Sep 04 16:06:31 2006 +0900
+++ b/unmodified_drivers/linux-2.6/mkbuildtree	Mon Sep 04 16:07:19 2006 +0900
@@ -42,6 +42,12 @@ i[34567]86)
 	ln -sf ${XL}/include/asm-i386/mach-xen/asm/synch_bitops.h include/asm
 	ln -sf ${XL}/include/asm-i386/mach-xen/asm/maddr.h include/asm
 	;;
+"ia64")
+	ln -sf ${XL}/include/asm-ia64/hypervisor.h include/asm
+	ln -sf ${XL}/include/asm-ia64/hypercall.h include/asm
+	ln -sf ${XL}/include/asm-ia64/synch_bitops.h include/asm
+	ln -sf ${XL}/include/asm-ia64/maddr.h include/asm
+	;;
 *)
 	echo unknown architecture $uname
 	exit 1

[-- Attachment #4: Type: text/plain, Size: 138 bytes --]

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

Re: Porting PV-on-HVM for ia64 platform (Take 3)

[Steven Smith]

[-- Attachment #1.1: Type: text/plain, Size: 699 bytes --]

> Hi,
> 
>   I will post patches for PV-on-HVM on ia64 platform.
> 
>   These patch include:
> 
>    * unmodified-driver.patch
>      + cut off unused codes for IPF
>        - get_hypercall_stabs() is x86 specific function. We don't
>          use it, and it occurrs compile error on ia64 platform.
>          Thus it is cut off.
>    * unmodified-build.patch
>      + modify build rule for IPF
>        - This patch appends a build rule for ia64 platform to
>          mkbuildtree command.
> 
> Thanks,
> - Tsunehisa Doi
What's here looks pretty reasonable to me.  Is it sufficient by itself
to make the drivers work on IA64, or will there be more patches later
on?

Steven.

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

[-- Attachment #2: Type: text/plain, Size: 138 bytes --]

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

Re: Porting PV-on-HVM for ia64 platform (Take 3)

[Doi.Tsunehisa]

Hi Steven,

  Thank you for your comment.

You (sos22-xen) said:
> What's here looks pretty reasonable to me.  Is it sufficient by itself
> to make the drivers work on IA64, or will there be more patches later
> on?

  Yes, it's sufficient in common code.

  We had ported for ia64 platform to be same semantic as x86 platform.
Thus, its modification in common code was minimized, I think.

Thanks,
- Tsunehisa Doi