Skip to content

Commit

Permalink
kvm: x86/mmu: Don't clear write flooding for direct SP
Browse files Browse the repository at this point in the history
Although there is no harm, but there is no point to clear write
flooding for direct SP.

Signed-off-by: Lai Jiangshan <jiangshan.ljs@antgroup.com>
Link: https://lore.kernel.org/r/20230105100310.6700-1-jiangshanlai@gmail.com
Signed-off-by: Sean Christopherson <seanjc@google.com>
  • Loading branch information
Lai Jiangshan authored and sean-jc committed Jan 19, 2023
1 parent 6458637 commit 5ee0c37
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions arch/x86/kvm/mmu/paging_tmpl.h
Original file line number Diff line number Diff line change
Expand Up @@ -642,12 +642,12 @@ static int FNAME(fetch)(struct kvm_vcpu *vcpu, struct kvm_page_fault *fault,
if (WARN_ON(!VALID_PAGE(vcpu->arch.mmu->root.hpa)))
goto out_gpte_changed;

for (shadow_walk_init(&it, vcpu, fault->addr);
shadow_walk_okay(&it) && it.level > gw->level;
shadow_walk_next(&it)) {
for_each_shadow_entry(vcpu, fault->addr, it) {
gfn_t table_gfn;

clear_sp_write_flooding_count(it.sptep);
if (it.level == gw->level)
break;

table_gfn = gw->table_gfn[it.level - 2];
access = gw->pt_access[it.level - 2];
Expand Down Expand Up @@ -692,8 +692,6 @@ static int FNAME(fetch)(struct kvm_vcpu *vcpu, struct kvm_page_fault *fault,
trace_kvm_mmu_spte_requested(fault);

for (; shadow_walk_okay(&it); shadow_walk_next(&it)) {
clear_sp_write_flooding_count(it.sptep);

/*
* We cannot overwrite existing page tables with an NX
* large page, as the leaf could be executable.
Expand Down

0 comments on commit 5ee0c37

Please sign in to comment.