[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 23/23] pci/monitor: print out bridge's filtering
From: |
Michael S. Tsirkin |
Subject: |
[Qemu-devel] Re: [PATCH 23/23] pci/monitor: print out bridge's filtering values and so on. |
Date: |
Mon, 5 Oct 2009 16:24:02 +0200 |
User-agent: |
Mutt/1.5.19 (2009-01-05) |
On Mon, Oct 05, 2009 at 07:07:03PM +0900, Isaku Yamahata wrote:
> make pci_info_device() print out bridge's filtering value like
> io base/limit, subbus and subordinate bus.
>
> Signed-off-by: Isaku Yamahata <address@hidden>
> ---
> hw/pci.c | 39 +++++++++++++++++++++++++++++++++++++++
> 1 files changed, 39 insertions(+), 0 deletions(-)
>
> diff --git a/hw/pci.c b/hw/pci.c
> index 2694e83..570531c 100644
> --- a/hw/pci.c
> +++ b/hw/pci.c
> @@ -1032,7 +1032,46 @@ static void pci_info_device(PCIBus *bus, PCIDevice *d)
> d->config[PCI_INTERRUPT_LINE]);
> }
> if (class == 0x0604) {
> + uint64_t base;
> + uint64_t limit;
> +
> monitor_printf(mon, " BUS %d.\n", d->config[0x19]);
> + monitor_printf(mon, " secondary bus %d.\n",
> + d->config[PCI_SECONDARY_BUS]);
> + monitor_printf(mon, " subordinate bus %d.\n",
> + d->config[PCI_SUBORDINATE_BUS]);
> +
> + base =
> + ((uint32_t)d->config[PCI_IO_BASE] & ~PCI_IO_RANGE_TYPE_MASK) <<
> 8;
> + limit =
> + ((uint32_t)d->config[PCI_IO_LIMIT] & ~PCI_IO_RANGE_TYPE_MASK) <<
> 8;
> + if (d->config[PCI_IO_BASE] & PCI_IO_RANGE_TYPE_32) {
> + base |= pci_get_word(d->config + PCI_IO_BASE_UPPER16) << 16;
pci_get_word returns a 16 bit integer. Isn't shifting it
by 16 bit wrong?
> + limit |= pci_get_word(d->config + PCI_IO_LIMIT_UPPER16) << 16;
> + }
> + limit |= 0xfff;
> + monitor_printf(mon, " IO range [0x%04"PRIx64",
> 0x%04"PRIx64"]\n",
> + base, limit);
> +
> + base = (pci_get_word(d->config + PCI_MEMORY_BASE) &
> + PCI_MEMORY_RANGE_MASK) << 16;
> + limit = (pci_get_word(d->config + PCI_MEMORY_LIMIT) &
> + PCI_MEMORY_RANGE_MASK) << 16;
> + limit |= 0xfffff;
> + monitor_printf(mon,
> + " memory range [0x%08"PRIx64", 0x%08"PRIx64"]\n",
> + base, limit);
> + base = ((uint64_t)pci_get_word(d->config + PCI_PREF_MEMORY_BASE) &
> + PCI_PREF_RANGE_MASK) << 16;
> + limit = ((uint64_t)pci_get_word(d->config + PCI_PREF_MEMORY_LIMIT) &
> + PCI_PREF_RANGE_MASK) << 16;
> + base |=
> + (uint64_t)pci_get_long(d->config + PCI_PREF_BASE_UPPER32) << 32;
> + limit |=
> + (uint64_t)pci_get_long(d->config + PCI_PREF_LIMIT_UPPER32) << 32;
> + limit |= 0xfffff;
> + monitor_printf(mon, " prefetchable memory range "
> + "[0x%08"PRIx64", 0x%08"PRIx64"]\n", base, limit);
Here, for once, is a chance to add a helper routine to good effect :)
> }
> for(i = 0;i < PCI_NUM_REGIONS; i++) {
> r = &d->io_regions[i];
> --
> 1.6.0.2
- [Qemu-devel] Re: [PATCH 13/23] pci: make pci configuration transaction more accurate., (continued)
- [Qemu-devel] [PATCH 22/23] pci: initialize wmask according to pci header type., Isaku Yamahata, 2009/10/05
- [Qemu-devel] [PATCH 06/23] pci: define a constant to represent a unmapped bar and use it., Isaku Yamahata, 2009/10/05
- [Qemu-devel] [PATCH 08/23] pci: use helper functions to access pci config space., Isaku Yamahata, 2009/10/05
- [Qemu-devel] [PATCH 07/23] pci: helper functions to access PCIDevice::config, Isaku Yamahata, 2009/10/05
- [Qemu-devel] [PATCH 21/23] pci/brdige: qdevfy and initialize secondary bus and subordinate bus., Isaku Yamahata, 2009/10/05
- [Qemu-devel] [PATCH 23/23] pci/monitor: print out bridge's filtering values and so on., Isaku Yamahata, 2009/10/05
- [Qemu-devel] Re: [PATCH 23/23] pci/monitor: print out bridge's filtering values and so on.,
Michael S. Tsirkin <=
- [Qemu-devel] [PATCH 04/23] pci: use the symbolic constant, PCI_ROM_ADDRESS_ENABLE instead of 1., Isaku Yamahata, 2009/10/05
- [Qemu-devel] [PATCH 18/23] pci: add helper functions for pci config write function., Isaku Yamahata, 2009/10/05
- [Qemu-devel] [PATCH 09/23] pci: introduce pcibus_t to represent pci bus address/size instead of uint32_t, Isaku Yamahata, 2009/10/05
- [Qemu-devel] [PATCH 19/23] pci: use helper function in pci_default_write_config(), Isaku Yamahata, 2009/10/05
- [Qemu-devel] Re: [PATCH v4 00/23] pci: various pci clean up and pci express support., Michael S. Tsirkin, 2009/10/05
- [Qemu-devel] [PATCH 16/23] pci: pcie host and mmcfg support., Isaku Yamahata, 2009/10/05
- [Qemu-devel] Re: [PATCH 16/23] pci: pcie host and mmcfg support., Michael S. Tsirkin, 2009/10/05