|   | 
      | 
  
  
      | 
      | 
  
 
     | 
    | 
  
  
     | 
    | 
  
  
    |   | 
      | 
  
  
    | 
         
xen-devel
Re: [Xen-devel] [PATCH] [HVM] Fix 64 bit PV-on-HVM driver builds
 
After further investigation, I see that the PV-on-HVM driver build was
broken only for x86_64 linux kernels PRIOR TO 2.6.11 (i.e., without a
pud_t typedef).  And my original patch breaks the build with kernels
 
= 2.6.11.  I had mistakenly thought it was a matter of CONFIG_XEN or
 
 
not.  The attached new version of this patch should work with all
(2.6) kernels.  I now test LINUX_VERSION_CODE instead of CONFIG_XEN.
I've also separated this x86_64 case from the i386 case.  We end up
including the same file
(unmodified_drivers/linux-2.6/compat-include/asm-generic/pgtable-nopud.h)
in either case, but it was pretty confusing the way I had it mixed in
with the i386 case.
Tested building against 2.6.5, 2.6.9, and 2.6.18 guest kernels.
Dave
On 6/6/07, Keir Fraser <keir@xxxxxxxxxxxxx> wrote:
 
This makes no sense. x86/64 drivers build fine for me as is.
 -- Keir
On 6/6/07 17:08, "Ben Guthro" <bguthro@xxxxxxxxxxxxxxx> wrote:
> xenbus-drivers-64bit-build-fix.patch:
> allow unmodified drivers to build under 64 bits
>
> Signed-off-by: David Lively <dlively@xxxxxxxxxxxxxxx>
> diff -r d4a0706d6747
> linux-2.6-xen-sparse/include/asm-i386/mach-xen/asm/hypervisor.h
> --- a/linux-2.6-xen-sparse/include/asm-i386/mach-xen/asm/hypervisor.h Tue May
> 15 15:14:02 2007 +0100
> +++ b/linux-2.6-xen-sparse/include/asm-i386/mach-xen/asm/hypervisor.h Thu May
> 17 15:01:35 2007 -0400
> @@ -45,13 +45,15 @@
>  #include <xen/interface/nmi.h>
>  #include <asm/ptrace.h>
>  #include <asm/page.h>
> -#if defined(__i386__)
> +
> +#if !defined(CONFIG_XEN) || defined(__i386__)
>  #  ifdef CONFIG_X86_PAE
>  #   include <asm-generic/pgtable-nopud.h>
>  #  else
>  #   include <asm-generic/pgtable-nopmd.h>
>  #  endif
>  #endif
> +
>
>  extern shared_info_t *HYPERVISOR_shared_info;
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
  
 
xenbus-drivers-64bit-build-fix.patch 
Description: Text Data 
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
 
 |   
 
 | 
    | 
  
  
    |   | 
    |