[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 02/21] hw/pci/pci.c: Turn DPRINTF into trace events
From: |
Michael Tokarev |
Subject: |
[PULL 02/21] hw/pci/pci.c: Turn DPRINTF into trace events |
Date: |
Wed, 7 May 2025 21:03:53 +0300 |
From: Gustavo Romero <gustavo.romero@linaro.org>
Remove PCI_DPRINTF() macro and use trace events instead.
Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
Reviewed-by: Michael Tokarev <mjt@tls.msk.ru>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
---
hw/pci/pci.c | 19 ++++++-------------
hw/pci/trace-events | 4 ++++
2 files changed, 10 insertions(+), 13 deletions(-)
diff --git a/hw/pci/pci.c b/hw/pci/pci.c
index fe38c4c028..352b3d12c8 100644
--- a/hw/pci/pci.c
+++ b/hw/pci/pci.c
@@ -54,13 +54,6 @@
#include "hw/xen/xen.h"
#include "hw/i386/kvm/xen_evtchn.h"
-//#define DEBUG_PCI
-#ifdef DEBUG_PCI
-# define PCI_DPRINTF(format, ...) printf(format, ## __VA_ARGS__)
-#else
-# define PCI_DPRINTF(format, ...) do { } while (0)
-#endif
-
bool pci_available = true;
static char *pcibus_get_dev_path(DeviceState *dev);
@@ -2439,12 +2432,12 @@ static void pci_patch_ids(PCIDevice *pdev, uint8_t
*ptr, uint32_t size)
/* Only a valid rom will be patched. */
rom_magic = pci_get_word(ptr);
if (rom_magic != 0xaa55) {
- PCI_DPRINTF("Bad ROM magic %04x\n", rom_magic);
+ trace_pci_bad_rom_magic(rom_magic, 0xaa55);
return;
}
pcir_offset = pci_get_word(ptr + 0x18);
if (pcir_offset + 8 >= size || memcmp(ptr + pcir_offset, "PCIR", 4)) {
- PCI_DPRINTF("Bad PCIR offset 0x%x or signature\n", pcir_offset);
+ trace_pci_bad_pcir_offset(pcir_offset);
return;
}
@@ -2453,8 +2446,8 @@ static void pci_patch_ids(PCIDevice *pdev, uint8_t *ptr,
uint32_t size)
rom_vendor_id = pci_get_word(ptr + pcir_offset + 4);
rom_device_id = pci_get_word(ptr + pcir_offset + 6);
- PCI_DPRINTF("%s: ROM id %04x%04x / PCI id %04x%04x\n", pdev->romfile,
- vendor_id, device_id, rom_vendor_id, rom_device_id);
+ trace_pci_rom_and_pci_ids(pdev->romfile, vendor_id, device_id,
+ rom_vendor_id, rom_device_id);
checksum = ptr[6];
@@ -2462,7 +2455,7 @@ static void pci_patch_ids(PCIDevice *pdev, uint8_t *ptr,
uint32_t size)
/* Patch vendor id and checksum (at offset 6 for etherboot roms). */
checksum += (uint8_t)rom_vendor_id + (uint8_t)(rom_vendor_id >> 8);
checksum -= (uint8_t)vendor_id + (uint8_t)(vendor_id >> 8);
- PCI_DPRINTF("ROM checksum %02x / %02x\n", ptr[6], checksum);
+ trace_pci_rom_checksum_change(ptr[6], checksum);
ptr[6] = checksum;
pci_set_word(ptr + pcir_offset + 4, vendor_id);
}
@@ -2471,7 +2464,7 @@ static void pci_patch_ids(PCIDevice *pdev, uint8_t *ptr,
uint32_t size)
/* Patch device id and checksum (at offset 6 for etherboot roms). */
checksum += (uint8_t)rom_device_id + (uint8_t)(rom_device_id >> 8);
checksum -= (uint8_t)device_id + (uint8_t)(device_id >> 8);
- PCI_DPRINTF("ROM checksum %02x / %02x\n", ptr[6], checksum);
+ trace_pci_rom_checksum_change(ptr[6], checksum);
ptr[6] = checksum;
pci_set_word(ptr + pcir_offset + 6, device_id);
}
diff --git a/hw/pci/trace-events b/hw/pci/trace-events
index 6a9968962f..02c80d3ec8 100644
--- a/hw/pci/trace-events
+++ b/hw/pci/trace-events
@@ -6,6 +6,10 @@ pci_pm_transition(const char *dev, uint32_t bus, uint32_t
slot, uint32_t func, u
pci_update_mappings_del(const char *dev, uint32_t bus, uint32_t slot, uint32_t
func, int bar, uint64_t addr, uint64_t size) "%s %02x:%02x.%x
%d,0x%"PRIx64"+0x%"PRIx64
pci_update_mappings_add(const char *dev, uint32_t bus, uint32_t slot, uint32_t
func, int bar, uint64_t addr, uint64_t size) "%s %02x:%02x.%x
%d,0x%"PRIx64"+0x%"PRIx64
pci_route_irq(int dev_irq, const char *dev_path, int parent_irq, const char
*parent_path) "IRQ %d @%s -> IRQ %d @%s"
+pci_bad_rom_magic(uint16_t bad_rom_magic, uint16_t good_rom_magic) "Bad ROM
magic number: %04"PRIX16". Should be: %04"PRIX16
+pci_bad_pcir_offset(uint16_t pcir_offset) "Bad PCIR offset 0x%"PRIx16" or
signature"
+pci_rom_and_pci_ids(char *romfile, uint16_t vendor_id, uint16_t device_id,
uint16_t rom_vendor_id, uint16_t rom_device_id) "%s: ROM ID
%04"PRIx16":%04"PRIx16" | PCI ID %04"PRIx16":%04"PRIx16
+pci_rom_checksum_change(uint8_t old_checksum, uint8_t new_checksum) "ROM
checksum changed from %02"PRIx8" to %02"PRIx8
# pci_host.c
pci_cfg_read(const char *dev, uint32_t bus, uint32_t slot, uint32_t func,
unsigned offs, unsigned val) "%s %02x:%02x.%x @0x%x -> 0x%x"
--
2.39.5
- [PULL 00/21] Trivial patches for 2025-05-07, Michael Tokarev, 2025/05/07
- [PULL 01/21] linux-user/syscall.c: add translation logic for epoll_pwait2 syscall, Michael Tokarev, 2025/05/07
- [PULL 03/21] hw/isa/ich9: Remove stray empty comment, Michael Tokarev, 2025/05/07
- [PULL 02/21] hw/pci/pci.c: Turn DPRINTF into trace events,
Michael Tokarev <=
- [PULL 05/21] hw/i386/acpi-build: Update document reference, Michael Tokarev, 2025/05/07
- [PULL 04/21] hw/i386/acpi-build: Fix typo and grammar in comment, Michael Tokarev, 2025/05/07
- [PULL 06/21] hw/acpi/ich9: Remove ICH9_DEBUG macro, Michael Tokarev, 2025/05/07
- [PULL 07/21] hw/i386/acpi-build: Fix typo in function name, Michael Tokarev, 2025/05/07
- [PULL 08/21] hw/core/machine: Fix indentation, Michael Tokarev, 2025/05/07
- [PULL 09/21] qom/object: Fix typo in comment, Michael Tokarev, 2025/05/07
- [PULL 10/21] hw/net/e1000: Remove stray empty comment in header, Michael Tokarev, 2025/05/07
- [PULL 12/21] hw/acpi/pcihp: Fix typo in function name, Michael Tokarev, 2025/05/07
- [PULL 14/21] block.c: fix "the the" typo in comment, Michael Tokarev, 2025/05/07
- [PULL 11/21] hw/pci-host/gpex-acpi: Fix typo in comment, Michael Tokarev, 2025/05/07