| Message ID | 20200812192758.25587-1-sean.j.christopherson@intel.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | KVM: x86/mmu: Zap orphaned kids for nested TDP MMU | expand |
On Wed, Aug 12, 2020 at 12:28 PM Sean Christopherson <sean.j.christopherson@intel.com> wrote: > > As promised, albeit a few days late. > > Ben, I kept your performance numbers even though it this version has > non-trivial differences relative to what you tested. I assume we'll need > a v3 anyways if this doesn't provide the advertised performance benefits. > > Ben Gardon (1): > KVM: x86/MMU: Recursively zap nested TDP SPs when zapping last/only > parent > > Sean Christopherson (1): > KVM: x86/mmu: Move flush logic from mmu_page_zap_pte() to > FNAME(invlpg) > > arch/x86/kvm/mmu/mmu.c | 38 ++++++++++++++++++++++------------ > arch/x86/kvm/mmu/paging_tmpl.h | 7 +++++-- > 2 files changed, 30 insertions(+), 15 deletions(-) > Thanks for sending this revised series Sean. This all looks good to me. I think the main performance difference between this series and the original patch I sent is only zapping nested TDP shadow pages, but I expect it to behave more or less the same since the number of direct TDP pages is pretty bounded. > > -- > 2.28.0 >
As promised, albeit a few days late. Ben, I kept your performance numbers even though it this version has non-trivial differences relative to what you tested. I assume we'll need a v3 anyways if this doesn't provide the advertised performance benefits. Ben Gardon (1): KVM: x86/MMU: Recursively zap nested TDP SPs when zapping last/only parent Sean Christopherson (1): KVM: x86/mmu: Move flush logic from mmu_page_zap_pte() to FNAME(invlpg) arch/x86/kvm/mmu/mmu.c | 38 ++++++++++++++++++++++------------ arch/x86/kvm/mmu/paging_tmpl.h | 7 +++++-- 2 files changed, 30 insertions(+), 15 deletions(-)