RE: [PATCH] KVM: x86: always set accessed bit in shadow PTEs

From: Zhang, Yang Z
Date: Mon Nov 16 2015 - 19:59:38 EST


Paolo Bonzini wrote on 2015-11-13:
> Commit 7a1638ce4220 ("nEPT: Redefine EPT-specific link_shadow_page()",
> 2013-08-05) says:
>
> Since nEPT doesn't support A/D bit, we should not set those bit
> when building the shadow page table.
> but this is not necessary. Even though nEPT doesn't support A/D
> bits, and hence the vmcs12 EPT pointer will never enable them,
> we always use them for shadow page tables if available (see
> construct_eptp in vmx.c). So we can set the A/D bits freely
> in the shadow page table.
>
> This patch hence basically reverts commit 7a1638ce4220.
>
> Cc: Yang Zhang <yang.z.zhang@xxxxxxxxx>
> Cc: Takuya Yoshikawa <yoshikawa_takuya_b1@xxxxxxxxxxxxx>
> Signed-off-by: Paolo Bonzini <pbonzini@xxxxxxxxxx>

Looks good to me.

Best regards,
Yang


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/