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

[Sheng Yang <sheng@linux.intel.com>]

On Tuesday 04 August 2009 00:47:16 Marcelo Tosatti wrote:
> On Mon, Aug 03, 2009 at 02:17:27PM +0800, Sheng Yang wrote:
> > Copy u64 from guest result in chaos.
>
> Why?

My fault in userspace...

-- 
regards
Yang, Sheng

>
> > 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