KVM: x86/mmu: Drop trace_kvm_age_page() tracepoint
Remove x86's trace_kvm_age_page() tracepoint. It's mostly redundant with the common trace_kvm_age_hva() tracepoint, and if there is a need for the extra details, e.g. gfn, referenced, etc... those details should be added to the common tracepoint so that all architectures and MMUs benefit from the info. Signed-off-by: Sean Christopherson <seanjc@google.com> Message-Id: <20210326021957.1424875-19-seanjc@google.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
parent
501b918525
commit
6dfbd6b5d5
|
@ -1502,7 +1502,6 @@ static int kvm_age_rmapp(struct kvm *kvm, struct kvm_rmap_head *rmap_head,
|
||||||
for_each_rmap_spte(rmap_head, &iter, sptep)
|
for_each_rmap_spte(rmap_head, &iter, sptep)
|
||||||
young |= mmu_spte_age(sptep);
|
young |= mmu_spte_age(sptep);
|
||||||
|
|
||||||
trace_kvm_age_page(gfn, level, slot, young);
|
|
||||||
return young;
|
return young;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -981,8 +981,6 @@ static int age_gfn_range(struct kvm *kvm, struct kvm_memory_slot *slot,
|
||||||
|
|
||||||
tdp_mmu_set_spte_no_acc_track(kvm, &iter, new_spte);
|
tdp_mmu_set_spte_no_acc_track(kvm, &iter, new_spte);
|
||||||
young = 1;
|
young = 1;
|
||||||
|
|
||||||
trace_kvm_age_page(iter.gfn, iter.level, slot, young);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
rcu_read_unlock();
|
rcu_read_unlock();
|
||||||
|
|
|
@ -255,30 +255,6 @@ TRACE_EVENT(kvm_fpu,
|
||||||
TP_printk("%s", __print_symbolic(__entry->load, kvm_fpu_load_symbol))
|
TP_printk("%s", __print_symbolic(__entry->load, kvm_fpu_load_symbol))
|
||||||
);
|
);
|
||||||
|
|
||||||
TRACE_EVENT(kvm_age_page,
|
|
||||||
TP_PROTO(ulong gfn, int level, struct kvm_memory_slot *slot, int ref),
|
|
||||||
TP_ARGS(gfn, level, slot, ref),
|
|
||||||
|
|
||||||
TP_STRUCT__entry(
|
|
||||||
__field( u64, hva )
|
|
||||||
__field( u64, gfn )
|
|
||||||
__field( u8, level )
|
|
||||||
__field( u8, referenced )
|
|
||||||
),
|
|
||||||
|
|
||||||
TP_fast_assign(
|
|
||||||
__entry->gfn = gfn;
|
|
||||||
__entry->level = level;
|
|
||||||
__entry->hva = ((gfn - slot->base_gfn) <<
|
|
||||||
PAGE_SHIFT) + slot->userspace_addr;
|
|
||||||
__entry->referenced = ref;
|
|
||||||
),
|
|
||||||
|
|
||||||
TP_printk("hva %llx gfn %llx level %u %s",
|
|
||||||
__entry->hva, __entry->gfn, __entry->level,
|
|
||||||
__entry->referenced ? "YOUNG" : "OLD")
|
|
||||||
);
|
|
||||||
|
|
||||||
#ifdef CONFIG_KVM_ASYNC_PF
|
#ifdef CONFIG_KVM_ASYNC_PF
|
||||||
DECLARE_EVENT_CLASS(kvm_async_get_page_class,
|
DECLARE_EVENT_CLASS(kvm_async_get_page_class,
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue