Re: [Xen-devel] [PATCHv1 1/3] x86/ept: remove unnecessary sync after resolving misconfigured entries

[Andrew Cooper <andrew.cooper3@xxxxxxxxxx>]

On 06/11/15 17:37, David Vrabel wrote:
> When using EPT, type changes are done with the following steps:
>
> 1. Set entry as invalid (misconfigured) by settings a reserved memory
> type.
>
> 2. Flush all EPT and combined translations (ept_sync_domain()).
>
> 3. Fixup misconfigured entries as required (on EPT_MISCONFIG vmexits or
> when explicitly setting an entry.
>
> Since resolve_misconfig() only updates entries that were misconfigured,
> there is no need to invalidate any translations since the hardware
> does not cache misconfigured translations (vol 3, section 28.3.2).
>
> Remove the unnecessary (and very expensive) ept_sync_domain() calls).
>
> Signed-off-by: David Vrabel <david.vrabel@xxxxxxxxxx>

Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel