]> www.infradead.org Git - users/dwmw2/qemu.git/commit
hw/pci/pci-hmp-cmds: Avoid displaying bogus size in 'info pci'
authorPhilippe Mathieu-Daudé <philmd@linaro.org>
Thu, 1 Aug 2024 13:14:49 +0000 (15:14 +0200)
committerMichael S. Tsirkin <mst@redhat.com>
Wed, 11 Sep 2024 13:46:04 +0000 (09:46 -0400)
commitaa37616fb4eed38f3fc26507faf2433c998e0073
treeb523cdd78f81bc612a2b6d139ec204d4a958ace9
parente228d62b4af29bca698ec57efdceb46f392f5444
hw/pci/pci-hmp-cmds: Avoid displaying bogus size in 'info pci'

When BAR aren't mapped, we get:

  (qemu) info pci
    Bus  0, device   0, function 0:
      Host bridge: PCI device dead:beef
        ...
        BAR4: 32 bit memory at 0xffffffffffffffff [0x00000ffe].
        BAR5: I/O at 0xffffffffffffffff [0x0ffe].

Check the BAR is mapped comparing its address to PCI_BAR_UNMAPPED
which is what the PCI layer uses for unmapped BARs.
See pci_bar_address and pci_update_mappings implementations and
in "hw/pci/pci.h":

  typedef struct PCIIORegion {
      pcibus_t addr; /* current PCI mapping address. -1 means not mapped */
  #define PCI_BAR_UNMAPPED (~(pcibus_t)0)
      ...

This improves the logging, not displaying bogus sizes:

  (qemu) info pci
    Bus  0, device   0, function 0:
      Host bridge: PCI device dead:beef
        ...
      BAR4: 32 bit memory (not mapped)
      BAR5: I/O (not mapped)

Remove trailing dot which is not used in other commands format.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20240801131449.51328-1-philmd@linaro.org>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
hw/pci/pci-hmp-cmds.c