Re: [PATCH] KVM: Fix EPT identity IOCTL on 32pae

From: Marcelo Tosatti <mtosatti@redhat.com>
To: Sheng Yang <sheng@linux.intel.com>
Cc: Avi Kivity <avi@redhat.com>, kvm@vger.kernel.org
Subject: Re: [PATCH] KVM: Fix EPT identity IOCTL on 32pae
Date: Mon, 3 Aug 2009 13:47:16 -0300	[thread overview]
Message-ID: <20090803164716.GA3851@amt.cnet> (raw)
In-Reply-To: <1249280247-6732-1-git-send-email-sheng@linux.intel.com>

On Mon, Aug 03, 2009 at 02:17:27PM +0800, Sheng Yang wrote:
> Copy u64 from guest result in chaos.

Why?

> 
> Also fix a mistake of still using old macro rather than new variable().
> 
> Signed-off-by: Sheng Yang <sheng@linux.intel.com>
> ---
>  arch/x86/kvm/vmx.c |    2 +-
>  arch/x86/kvm/x86.c |    2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c
> index 7931c72..c5aaa1b 100644
> --- a/arch/x86/kvm/vmx.c
> +++ b/arch/x86/kvm/vmx.c
> @@ -2199,7 +2199,7 @@ static int alloc_identity_pagetable(struct kvm *kvm)
>  		goto out;
>  
>  	kvm->arch.ept_identity_pagetable = gfn_to_page(kvm,
> -			VMX_EPT_IDENTITY_PAGETABLE_ADDR >> PAGE_SHIFT);
> +			kvm->arch.ept_identity_map_addr >> PAGE_SHIFT);
>  out:
>  	up_write(&kvm->slots_lock);
>  	return r;
> diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
> index 2539e9a..977b705 100644
> --- a/arch/x86/kvm/x86.c
> +++ b/arch/x86/kvm/x86.c
> @@ -2178,7 +2178,7 @@ long kvm_arch_vm_ioctl(struct file *filp,
>  			goto out;
>  		break;
>  	case KVM_SET_IDENTITY_MAP_ADDR: {
> -		u64 ident_addr;
> +		unsigned long ident_addr;
>  
>  		r = -EFAULT;
>  		if (copy_from_user(&ident_addr, argp, sizeof ident_addr))
> -- 
> 1.5.4.5