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.

Reply via email to