Re: [PATCH] KVM: x86: fixup set_efer()

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

On Wednesday 08 September 2010 18:04:18 Avi Kivity wrote:
>   On 09/04/2010 04:29 PM, Hillf Danton wrote:
> > The second call to kvm_mmu_reset_context() seems unnecessary and is
> > removed.
> > 
> > 
> > @@ -783,10 +783,6 @@ static int set_efer(struct kvm_vcpu *vcp
> > vcpu->arch.mmu.base_role.nxe = (efer & EFER_NX) && !tdp_enabled;
> > kvm_mmu_reset_context(vcpu);
> > - /* Update reserved bits */
> > - if ((efer ^ old_efer) & EFER_NX)
> > - kvm_mmu_reset_context(vcpu);
> > -
> > return 0;
> > 
> >  }
> 
> Hm.  As far as I can tell, it's the first call that is unnecessary.
> I'll look at the history and try to understand why it was introduced.

Oops, seems my patch failed to remove the old line...

--
regards
Yang, Sheng