On 08/04/2011 08:05 AM, Avi Kivity wrote:
This is a hack, for devices that have a back-channel to read this
address back outside the normal configuration mechanisms, such
as VMware svga.

Reviewed-by: Richard Henderson<r...@twiddle.net>
Signed-off-by: Avi Kivity<a...@redhat.com>

Can we add a comment to the header file to this effect?

Michael, if you ever have some free time and are feeling generous, adding docs to the pci functions would be quite helpful.

Regards,

Anthony Liguori

---
  hw/pci.c |    5 +++++
  hw/pci.h |    1 +
  2 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/hw/pci.c b/hw/pci.c
index 36db58b..912f849 100644
--- a/hw/pci.c
+++ b/hw/pci.c
@@ -952,6 +952,11 @@ void pci_register_bar_region(PCIDevice *pci_dev, int 
region_num,
      pci_dev->io_regions[region_num].memory = memory;
  }

+pcibus_t pci_get_bar_addr(PCIDevice *pci_dev, int region_num)
+{
+    return pci_dev->io_regions[region_num].addr;
+}
+
  static void pci_bridge_filter(PCIDevice *d, pcibus_t *addr, pcibus_t *size,
                                uint8_t type)
  {
diff --git a/hw/pci.h b/hw/pci.h
index c51156d..64282ad 100644
--- a/hw/pci.h
+++ b/hw/pci.h
@@ -207,6 +207,7 @@ void pci_register_bar_simple(PCIDevice *pci_dev, int 
region_num,
                               pcibus_t size, uint8_t attr, ram_addr_t 
ram_addr);
  void pci_register_bar_region(PCIDevice *pci_dev, int region_num,
                               uint8_t attr, MemoryRegion *memory);
+pcibus_t pci_get_bar_addr(PCIDevice *pci_dev, int region_num);

  int pci_add_capability(PCIDevice *pdev, uint8_t cap_id,
                         uint8_t offset, uint8_t size);

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to