From c856dbfb24dcca457dfe8daadfe5a657f60e79f7 Mon Sep 17 00:00:00 2001

From: Peter Xu <peterx@redhat.com>

Date: Fri, 12 Oct 2018 07:58:44 +0100

Subject: [PATCH 14/17] intel-iommu: trace domain id during page walk

RH-Author: Peter Xu <peterx@redhat.com>

Message-id: <20181012075846.25449-8-peterx@redhat.com>

Patchwork-id: 82680

O-Subject: [RHEL-8 qemu-kvm PATCH 7/9] intel-iommu: trace domain id during page walk

Bugzilla: 1450712

RH-Acked-by: Auger Eric <eric.auger@redhat.com>

RH-Acked-by: Xiao Wang <jasowang@redhat.com>

RH-Acked-by: Michael S. Tsirkin <mst@redhat.com>

This patch only modifies the trace points.

Previously we were tracing page walk levels.  They are redundant since

we have page mask (size) already.  Now we trace something much more

useful which is the domain ID of the page walking.  That can be very

useful when we trace more than one devices on the same system, so that

we can know which map is for which domain.

CC: QEMU Stable <qemu-stable@nongnu.org>

Signed-off-by: Peter Xu <peterx@redhat.com>

Reviewed-by: Michael S. Tsirkin <mst@redhat.com>

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>

(cherry picked from commit d118c06ebbee2d23ddf873cae4a809311aa61310)

Signed-off-by: Peter Xu <peterx@redhat.com>

Signed-off-by: Danilo C. L. de Paula <ddepaula@redhat.com>

---

 hw/i386/intel_iommu.c | 16 ++++++++++------

 hw/i386/trace-events  |  2 +-

 2 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/hw/i386/intel_iommu.c b/hw/i386/intel_iommu.c

index a882894..61bb3d3 100644

--- a/hw/i386/intel_iommu.c

+++ b/hw/i386/intel_iommu.c

@@ -756,6 +756,7 @@ typedef int (*vtd_page_walk_hook)(IOMMUTLBEntry *entry, void *private);

  * @notify_unmap: whether we should notify invalid entries

  * @as: VT-d address space of the device

  * @aw: maximum address width

+ * @domain: domain ID of the page walk

  */

 typedef struct {

     VTDAddressSpace *as;

@@ -763,17 +764,18 @@ typedef struct {

     void *private;

     bool notify_unmap;

     uint8_t aw;

+    uint16_t domain_id;

 } vtd_page_walk_info;

-static int vtd_page_walk_one(IOMMUTLBEntry *entry, int level,

-                             vtd_page_walk_info *info)

+static int vtd_page_walk_one(IOMMUTLBEntry *entry, vtd_page_walk_info *info)

 {

     vtd_page_walk_hook hook_fn = info->hook_fn;

     void *private = info->private;

     assert(hook_fn);

-    trace_vtd_page_walk_one(level, entry->iova, entry->translated_addr,

-                            entry->addr_mask, entry->perm);

+    trace_vtd_page_walk_one(info->domain_id, entry->iova,

+                            entry->translated_addr, entry->addr_mask,

+                            entry->perm);

     return hook_fn(entry, private);

 }

@@ -844,7 +846,7 @@ static int vtd_page_walk_level(dma_addr_t addr, uint64_t start,

                 trace_vtd_page_walk_skip_perm(iova, iova_next);

                 goto next;

             }

-            ret = vtd_page_walk_one(&entry, level, info);

+            ret = vtd_page_walk_one(&entry, info);

             if (ret < 0) {

                 return ret;

             }

@@ -856,7 +858,7 @@ static int vtd_page_walk_level(dma_addr_t addr, uint64_t start,

                      * Translated address is meaningless, zero it.

                      */

                     entry.translated_addr = 0x0;

-                    ret = vtd_page_walk_one(&entry, level, info);

+                    ret = vtd_page_walk_one(&entry, info);

                     if (ret < 0) {

                         return ret;

                     }

@@ -1466,6 +1468,7 @@ static void vtd_iotlb_page_invalidate_notify(IntelIOMMUState *s,

                     .notify_unmap = true,

                     .aw = s->aw_bits,

                     .as = vtd_as,

+                    .domain_id = domain_id,

                 };

                 /*

@@ -2947,6 +2950,7 @@ static void vtd_iommu_replay(IOMMUMemoryRegion *iommu_mr, IOMMUNotifier *n)

                 .notify_unmap = false,

                 .aw = s->aw_bits,

                 .as = vtd_as,

+                .domain_id = VTD_CONTEXT_ENTRY_DID(ce.hi),

             };

             vtd_page_walk(&ce, 0, ~0ULL, &info;;

diff --git a/hw/i386/trace-events b/hw/i386/trace-events

index 22d4464..ca23ba9 100644

--- a/hw/i386/trace-events

+++ b/hw/i386/trace-events

@@ -39,7 +39,7 @@ vtd_fault_disabled(void) "Fault processing disabled for context entry"

 vtd_replay_ce_valid(uint8_t bus, uint8_t dev, uint8_t fn, uint16_t domain, uint64_t hi, uint64_t lo) "replay valid context device %02"PRIx8":%02"PRIx8".%02"PRIx8" domain 0x%"PRIx16" hi 0x%"PRIx64" lo 0x%"PRIx64

 vtd_replay_ce_invalid(uint8_t bus, uint8_t dev, uint8_t fn) "replay invalid context device %02"PRIx8":%02"PRIx8".%02"PRIx8

 vtd_page_walk_level(uint64_t addr, uint32_t level, uint64_t start, uint64_t end) "walk (base=0x%"PRIx64", level=%"PRIu32") iova range 0x%"PRIx64" - 0x%"PRIx64

-vtd_page_walk_one(uint32_t level, uint64_t iova, uint64_t gpa, uint64_t mask, int perm) "detected page level 0x%"PRIx32" iova 0x%"PRIx64" -> gpa 0x%"PRIx64" mask 0x%"PRIx64" perm %d"

+vtd_page_walk_one(uint16_t domain, uint64_t iova, uint64_t gpa, uint64_t mask, int perm) "domain 0x%"PRIu16" iova 0x%"PRIx64" -> gpa 0x%"PRIx64" mask 0x%"PRIx64" perm %d"

 vtd_page_walk_skip_read(uint64_t iova, uint64_t next) "Page walk skip iova 0x%"PRIx64" - 0x%"PRIx64" due to unable to read"

 vtd_page_walk_skip_perm(uint64_t iova, uint64_t next) "Page walk skip iova 0x%"PRIx64" - 0x%"PRIx64" due to perm empty"

 vtd_page_walk_skip_reserve(uint64_t iova, uint64_t next) "Page walk skip iova 0x%"PRIx64" - 0x%"PRIx64" due to rsrv set"

-- 

1.8.3.1