Re: [Xen-devel][PATCH][RESEND] PV drivers for HVMguests

["Kasai Takanori" <kasai.takanori@xxxxxxxxxxxxxx>]

Hi Ian,

I confirmed this patch.
However, this patch has not been applied yet.

Do you think that this patch has another problem?
If this patch has no problem, could you apply this patch?

Thanks

--
Takanori Kasai


----- Original Message ----- 
From: "Kasai Takanori" <kasai.takanori@xxxxxxxxxxxxxx>
To: "Ian Campbell" <Ian.Campbell@xxxxxxxxxxxxx>
Cc: "xen-ia64-devel" <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
Sent: Saturday, October 28, 2006 3:30 PM
Subject: [Xen-ia64-devel] Re: [Xen-devel][PATCH][RESEND] PV drivers for 
HVMguests

> Hi Ian,
>
> > I've attached my current patch including the ptrace fix from above and
> > the comments from below. Can you give it a go?
> I confirmed this patch. (cs:12004)
> I tested compiling it and simple test on RHEL4U2 and RHEL4U4.
>
> Thanks
>
> --
> Takanori Kasai
>
>
> As follows:
> ------------
> diff -r cf05aabe6e65 linux-2.6-xen-sparse/include/asm-ia64/hypercall.h
> --- a/linux-2.6-xen-sparse/include/asm-ia64/hypercall.h Wed Oct 25 15:29:00 
> 2006 -0600
> +++ b/linux-2.6-xen-sparse/include/asm-ia64/hypercall.h Sat Oct 28 13:30:39 
> 2006 +0900
> @@ -39,6 +39,10 @@
>
> #include <asm/xen/xcom_hcall.h>
> struct xencomm_handle;
> +
> +#ifdef HAVE_XEN_PLATFORM_COMPAT_H
> +#include <xen/platform-compat.h>
> +#endif
>
> /*
>  * Assembler stubs for hyper-calls.
> diff -r cf05aabe6e65 linux-2.6-xen-sparse/include/asm-ia64/xen/xencomm.h
> --- a/linux-2.6-xen-sparse/include/asm-ia64/xen/xencomm.h Wed Oct 25 15:29:00 
> 2006 -0600
> +++ b/linux-2.6-xen-sparse/include/asm-ia64/xen/xencomm.h Sat Oct 28 13:42:40 
> 2006 +0900
> @@ -20,6 +20,10 @@
> #define _LINUX_XENCOMM_H_
>
> #include <xen/interface/xencomm.h>
> +
> +#ifdef HAVE_XEN_PLATFORM_COMPAT_H
> +#include <xen/platform-compat.h>
> +#endif
>
> #define XENCOMM_MINI_ADDRS 3
> struct xencomm_mini {
> diff -r cf05aabe6e65 
> unmodified_drivers/linux-2.6/compat-include/asm-generic/pgtable-nopud.h
> --- a/unmodified_drivers/linux-2.6/compat-include/asm-generic/pgtable-nopud.h 
> Wed Oct 25 15:29:00 2006 -0600
> +++ b/unmodified_drivers/linux-2.6/compat-include/asm-generic/pgtable-nopud.h 
> Sat Oct 28 12:50:07 2006 +0900
> @@ -9,6 +9,7 @@
> #define pud_offset(d, va)     d
> #define pud_none(pud)         0
> #define pud_present(pud)      1
> +#define pud_bad(pud)          0
> #define PTRS_PER_PUD          1
>
> #endif /* _PGTABLE_NOPUD_H */
> diff -r cf05aabe6e65 
> unmodified_drivers/linux-2.6/compat-include/xen/platform-compat.h
> --- a/unmodified_drivers/linux-2.6/compat-include/xen/platform-compat.h Wed 
> Oct 25 15:29:00 2006 -0600
> +++ b/unmodified_drivers/linux-2.6/compat-include/xen/platform-compat.h Sat 
> Oct 28 13:16:35 2006 +0900
> @@ -25,6 +25,21 @@
> #define NET_IP_ALIGN 0
> #endif
>
> +#if defined(_LINUX_ERR_H) && !defined(IS_ERR_VALUE)
> +#define IS_ERR_VALUE(x) unlikely((x) > (unsigned long)-1000L)
> +#endif
> +
> +#if defined(_ASM_IA64_PGTABLE_H) && !defined(_PGTABLE_NOPUD_H)
> +#include <asm-generic/pgtable-nopud.h>
> +#endif
> +
> +/* Some kernels have this typedef backported so we cannot reliably
> + * detect based on version number, hence we forcibly #define it.
> + */
> +#if defined(__LINUX_TYPES_H) || defined(__LINUX_GFP_H)
> +#define gfp_t unsigned
> +#endif
> +
> #if defined(_LINUX_FS_H) && LINUX_VERSION_CODE < KERNEL_VERSION(2,6,9)
> #define nonseekable_open(inode, filp) /* Nothing to do */
> #endif
>
>
>
> _______________________________________________
> Xen-ia64-devel mailing list
> Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-ia64-devel


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