|
 |
76daa3 |
From 38ee1508c565a3bc78e6505f4839b964cc39d060 Mon Sep 17 00:00:00 2001
|
|
|
76daa3 |
From: Peter Xu <peterx@redhat.com>
|
|
|
76daa3 |
Date: Tue, 6 Jun 2017 06:57:30 +0200
|
|
|
76daa3 |
Subject: [PATCH 07/17] msix: trace control bit write op
|
|
|
76daa3 |
MIME-Version: 1.0
|
|
|
76daa3 |
Content-Type: text/plain; charset=UTF-8
|
|
|
76daa3 |
Content-Transfer-Encoding: 8bit
|
|
|
76daa3 |
|
|
|
76daa3 |
RH-Author: Peter Xu <peterx@redhat.com>
|
|
|
76daa3 |
Message-id: <1496732251-27935-3-git-send-email-peterx@redhat.com>
|
|
|
76daa3 |
Patchwork-id: 75492
|
|
|
76daa3 |
O-Subject: [RHEL7.4 qemu-kvm-rhev PATCH 2/3] msix: trace control bit write op
|
|
|
76daa3 |
Bugzilla: 1448813
|
|
|
76daa3 |
RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Xiao Wang <jasowang@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Michael S. Tsirkin <mst@redhat.com>
|
|
|
76daa3 |
|
|
|
76daa3 |
Meanwhile, abstract a function to detect msix masked bit.
|
|
|
76daa3 |
|
|
|
76daa3 |
Signed-off-by: Peter Xu <peterx@redhat.com>
|
|
|
76daa3 |
Message-Id: <1494309644-18743-3-git-send-email-peterx@redhat.com>
|
|
|
76daa3 |
Acked-by: Michael S. Tsirkin <mst@redhat.com>
|
|
|
76daa3 |
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
|
|
|
76daa3 |
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
|
|
|
76daa3 |
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
76daa3 |
(cherry picked from commit a226e71aadec51d639528fdefd020428eeab2cc2)
|
|
|
76daa3 |
Signed-off-by: Peter Xu <peterx@redhat.com>
|
|
|
76daa3 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
76daa3 |
---
|
|
|
76daa3 |
hw/pci/msix.c | 11 +++++++++--
|
|
|
76daa3 |
hw/pci/trace-events | 3 +++
|
|
|
76daa3 |
2 files changed, 12 insertions(+), 2 deletions(-)
|
|
|
76daa3 |
|
|
|
76daa3 |
diff --git a/hw/pci/msix.c b/hw/pci/msix.c
|
|
|
76daa3 |
index bb54e8b..fc5fe51 100644
|
|
|
76daa3 |
--- a/hw/pci/msix.c
|
|
|
76daa3 |
+++ b/hw/pci/msix.c
|
|
|
76daa3 |
@@ -22,6 +22,7 @@
|
|
|
76daa3 |
#include "hw/xen/xen.h"
|
|
|
76daa3 |
#include "qemu/range.h"
|
|
|
76daa3 |
#include "qapi/error.h"
|
|
|
76daa3 |
+#include "trace.h"
|
|
|
76daa3 |
|
|
|
76daa3 |
#define MSIX_CAP_LENGTH 12
|
|
|
76daa3 |
|
|
|
76daa3 |
@@ -130,10 +131,14 @@ static void msix_handle_mask_update(PCIDevice *dev, int vector, bool was_masked)
|
|
|
76daa3 |
}
|
|
|
76daa3 |
}
|
|
|
76daa3 |
|
|
|
76daa3 |
+static bool msix_masked(PCIDevice *dev)
|
|
|
76daa3 |
+{
|
|
|
76daa3 |
+ return dev->config[dev->msix_cap + MSIX_CONTROL_OFFSET] & MSIX_MASKALL_MASK;
|
|
|
76daa3 |
+}
|
|
|
76daa3 |
+
|
|
|
76daa3 |
static void msix_update_function_masked(PCIDevice *dev)
|
|
|
76daa3 |
{
|
|
|
76daa3 |
- dev->msix_function_masked = !msix_enabled(dev) ||
|
|
|
76daa3 |
- (dev->config[dev->msix_cap + MSIX_CONTROL_OFFSET] & MSIX_MASKALL_MASK);
|
|
|
76daa3 |
+ dev->msix_function_masked = !msix_enabled(dev) || msix_masked(dev);
|
|
|
76daa3 |
}
|
|
|
76daa3 |
|
|
|
76daa3 |
/* Handle MSI-X capability config write. */
|
|
|
76daa3 |
@@ -148,6 +153,8 @@ void msix_write_config(PCIDevice *dev, uint32_t addr,
|
|
|
76daa3 |
return;
|
|
|
76daa3 |
}
|
|
|
76daa3 |
|
|
|
76daa3 |
+ trace_msix_write_config(dev->name, msix_enabled(dev), msix_masked(dev));
|
|
|
76daa3 |
+
|
|
|
76daa3 |
was_masked = dev->msix_function_masked;
|
|
|
76daa3 |
msix_update_function_masked(dev);
|
|
|
76daa3 |
|
|
|
76daa3 |
diff --git a/hw/pci/trace-events b/hw/pci/trace-events
|
|
|
76daa3 |
index 2b9cf24..83c8f5a 100644
|
|
|
76daa3 |
--- a/hw/pci/trace-events
|
|
|
76daa3 |
+++ b/hw/pci/trace-events
|
|
|
76daa3 |
@@ -7,3 +7,6 @@ pci_update_mappings_add(void *d, uint32_t bus, uint32_t slot, uint32_t func, int
|
|
|
76daa3 |
# hw/pci/pci_host.c
|
|
|
76daa3 |
pci_cfg_read(const char *dev, unsigned devid, unsigned fnid, unsigned offs, unsigned val) "%s %02u:%u @0x%x -> 0x%x"
|
|
|
76daa3 |
pci_cfg_write(const char *dev, unsigned devid, unsigned fnid, unsigned offs, unsigned val) "%s %02u:%u @0x%x <- 0x%x"
|
|
|
76daa3 |
+
|
|
|
76daa3 |
+# hw/pci/msix.c
|
|
|
76daa3 |
+msix_write_config(char *name, bool enabled, bool masked) "dev %s enabled %d masked %d"
|
|
|
76daa3 |
--
|
|
|
76daa3 |
1.8.3.1
|
|
|
76daa3 |
|