Re: [Xen-devel] [PATCHv8] x86/ept: defer the invalidation until the p2m lock is released

[George Dunlap <george.dunlap@xxxxxxxxxx>]

On Tue, Apr 12, 2016 at 5:19 PM, David Vrabel <david.vrabel@xxxxxxxxxx> wrote:
> Holding the p2m lock while calling ept_sync_domain() is very expensive
> since it does an on_selected_cpus() call.  IPIs on many socket
> machines can be very slow and on_selected_cpus() is serialized.
>
> It is safe to defer the invalidate until the p2m lock is released
> except for two cases:
>
> 1. When freeing a page table page (since partial translations may be
>    cached).
> 2. When reclaiming a zero page as part of PoD.
>
> For these cases, add p2m_tlb_flush_sync() calls which will immediately
> perform the invalidate before the page is freed or reclaimed.
>
> Signed-off-by: David Vrabel <david.vrabel@xxxxxxxxxx>

Looks good, thanks:

Reviewed-by: George Dunlap <geroge.dunlap@xxxxxxxxxx>

 -George

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