Re: [PATCH 1/4] KVM: MMU: fix permission_fault()

From: Xiao Guangrong
Date: Wed Apr 06 2016 - 11:11:55 EST




On 04/06/2016 04:56 PM, Paolo Bonzini wrote:


On 25/03/2016 14:19, Xiao Guangrong wrote:
@@ -193,11 +193,11 @@ static inline u8 permission_fault(struct kvm_vcpu *vcpu, struct kvm_mmu *mmu,
((pte_access & PT_USER_MASK) << (PFERR_RSVD_BIT - PT_USER_SHIFT));

One more tweak is needed in the line above; pfec - 1 must become pfec &
~1, because you've removed the

pfec |= PFERR_PRESENT_MASK;

line. Applied with this change.

Yes, indeed. Thanks for your fix.