[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 4/8] intel_iommu: add iotlb/context cache statistics
From: |
Peter Xu |
Subject: |
[Qemu-devel] [PATCH 4/8] intel_iommu: add iotlb/context cache statistics |
Date: |
Tue, 27 Jun 2017 17:03:35 +0800 |
Add statistics for the VT-d IOMMU DMA remapping.
Now "info iommu" shows us this for extra:
Statistics: iotlb=26.35% (6689/25388), context=99.99% (18697/18699)
Signed-off-by: Peter Xu <address@hidden>
---
hw/i386/intel_iommu.c | 21 +++++++++++++++++++++
include/hw/i386/intel_iommu.h | 10 ++++++++++
2 files changed, 31 insertions(+)
diff --git a/hw/i386/intel_iommu.c b/hw/i386/intel_iommu.c
index 39f772a..45d0919 100644
--- a/hw/i386/intel_iommu.c
+++ b/hw/i386/intel_iommu.c
@@ -37,6 +37,11 @@
#include "kvm_i386.h"
#include "trace.h"
+static void vtd_reset_stats(IntelIOMMUState *s)
+{
+ memset(&s->cache_stat, 0, sizeof(s->cache_stat));
+}
+
static void vtd_define_quad(IntelIOMMUState *s, hwaddr addr, uint64_t val,
uint64_t wmask, uint64_t w1cmask)
{
@@ -1095,9 +1100,12 @@ static bool vtd_do_iommu_translate(VTDAddressSpace
*vtd_as, PCIBus *bus,
*/
assert(!vtd_is_interrupt_addr(addr));
+ s->cache_stat.iotlb_total++;
+
/* Try to fetch slpte form IOTLB */
iotlb_entry = vtd_lookup_iotlb(s, source_id, addr);
if (iotlb_entry) {
+ s->cache_stat.iotlb_hit++;
trace_vtd_iotlb_page_hit(source_id, addr, iotlb_entry->slpte,
iotlb_entry->domain_id);
slpte = iotlb_entry->slpte;
@@ -1107,8 +1115,11 @@ static bool vtd_do_iommu_translate(VTDAddressSpace
*vtd_as, PCIBus *bus,
goto out;
}
+ s->cache_stat.context_total++;
+
/* Try to fetch context-entry from cache first */
if (cc_entry->context_cache_gen == s->context_cache_gen) {
+ s->cache_stat.context_hit++;
trace_vtd_iotlb_cc_hit(bus_num, devfn, cc_entry->context_entry.hi,
cc_entry->context_entry.lo,
cc_entry->context_cache_gen);
@@ -2875,6 +2886,7 @@ static void vtd_init(IntelIOMMUState *s)
vtd_reset_context_cache(s);
vtd_reset_iotlb(s);
+ vtd_reset_stats(s);
/* Define registers with default values and bit semantics */
vtd_define_long(s, DMAR_VER_REG, 0x10UL, 0, 0);
@@ -3022,6 +3034,15 @@ static void vtd_info_dump(X86IOMMUState *x86_iommu,
Monitor *mon,
}
DUMP("\n");
+ DUMP("Statistics: iotlb=%.2lf%% (%"PRIu64"/%"PRIu64"), "
+ "context=%.2lf%% (%"PRIu64"/%"PRIu64")\n",
+ (double)s->cache_stat.iotlb_hit /
+ s->cache_stat.iotlb_total * 100,
+ s->cache_stat.iotlb_hit, s->cache_stat.iotlb_total,
+ (double)s->cache_stat.context_hit /
+ s->cache_stat.context_total * 100,
+ s->cache_stat.context_hit, s->cache_stat.context_total);
+
DUMP("Caching-mode: %s\n", s->caching_mode ? "enabled" : "disabled");
DUMP("Misc: next_frr=%d, context_gen=%d, buggy_eim=%d\n",
s->next_frcd_reg, s->context_cache_gen, s->buggy_eim);
diff --git a/include/hw/i386/intel_iommu.h b/include/hw/i386/intel_iommu.h
index 3e51876..fc69ff3 100644
--- a/include/hw/i386/intel_iommu.h
+++ b/include/hw/i386/intel_iommu.h
@@ -255,6 +255,13 @@ struct IntelIOMMUNotifierNode {
QLIST_ENTRY(IntelIOMMUNotifierNode) next;
};
+typedef struct IOMMUCacheStats {
+ uint64_t iotlb_hit;
+ uint64_t iotlb_total;
+ uint64_t context_hit;
+ uint64_t context_total;
+} IOMMUCacheStats;
+
/* The iommu (DMAR) device state struct */
struct IntelIOMMUState {
X86IOMMUState x86_iommu;
@@ -302,6 +309,9 @@ struct IntelIOMMUState {
bool intr_eime; /* Extended interrupt mode enabled */
OnOffAuto intr_eim; /* Toggle for EIM cabability */
bool buggy_eim; /* Force buggy EIM unless eim=off */
+
+ /* For statistics */
+ IOMMUCacheStats cache_stat;
};
/* Find the VTD Address space associated with the given bus pointer,
--
2.7.4
- [Qemu-devel] [PATCH 0/8] VT-d: some enhancements on iotlb and tools, Peter Xu, 2017/06/27
- [Qemu-devel] [PATCH 1/8] intel_iommu: fix VTD_PAGE_MASK, Peter Xu, 2017/06/27
- [Qemu-devel] [PATCH 2/8] hmp: add info iommu, Peter Xu, 2017/06/27
- [Qemu-devel] [PATCH 3/8] intel_iommu: support "info iommu", Peter Xu, 2017/06/27
- [Qemu-devel] [PATCH 5/8] intel_iommu: hmp: allow "-c" for "info iommu", Peter Xu, 2017/06/27
- [Qemu-devel] [PATCH 4/8] intel_iommu: add iotlb/context cache statistics,
Peter Xu <=
- [Qemu-devel] [PATCH 7/8] intel_iommu: use access_flags for iotlb, Peter Xu, 2017/06/27
- [Qemu-devel] [PATCH 6/8] intel_iommu: let iotlb size tunable, Peter Xu, 2017/06/27
- [Qemu-devel] [PATCH 8/8] intel_iommu: implement mru list for iotlb, Peter Xu, 2017/06/27
- Re: [Qemu-devel] [PATCH 0/8] VT-d: some enhancements on iotlb and tools, Peter Xu, 2017/06/27
- Re: [Qemu-devel] [PATCH 0/8] VT-d: some enhancements on iotlb and tools, Michael S. Tsirkin, 2017/06/27
- Re: [Qemu-devel] [PATCH 0/8] VT-d: some enhancements on iotlb and tools, no-reply, 2017/06/27