[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [RFC v5 5/6] pci_expander_bridge: add config_write callback
From: |
Zihan Yang |
Subject: |
[Qemu-devel] [RFC v5 5/6] pci_expander_bridge: add config_write callback for pxb-pcie |
Date: |
Mon, 17 Sep 2018 22:57:36 +0800 |
This allows SeaBIOS to configure MCFG base during initialization.
The mcfg size is calculated using based on desired bus number
Signed-off-by: Zihan Yang <address@hidden>
---
hw/pci-bridge/pci_expander_bridge.c | 37 +++++++++++++++++++++++++++++
include/hw/pci-bridge/pci_expander_bridge.h | 7 ++++++
2 files changed, 44 insertions(+)
diff --git a/hw/pci-bridge/pci_expander_bridge.c
b/hw/pci-bridge/pci_expander_bridge.c
index 1e1999d..09413df 100644
--- a/hw/pci-bridge/pci_expander_bridge.c
+++ b/hw/pci-bridge/pci_expander_bridge.c
@@ -20,6 +20,7 @@
#include "hw/pci-bridge/pci_expander_bridge.h"
#include "qemu/range.h"
#include "qemu/error-report.h"
+#include "qemu/units.h"
#include "sysemu/numa.h"
#include "qapi/visitor.h"
@@ -395,6 +396,41 @@ static void pxb_dev_exitfn(PCIDevice *pci_dev)
pxb_dev_list = g_list_remove(pxb_dev_list, pxb);
}
+static void pxb_pcie_update_pciexbar(PXBDev *pxb)
+{
+ PCIDevice *pci_dev = PCI_DEVICE(pxb);
+ PCIBus *pxb_bus = pci_get_bus(pci_dev);
+ BusState *bus = BUS(QLIST_FIRST(&pxb_bus->child));
+ PCIExpressHost *pehb = PCIE_HOST_BRIDGE(bus->parent);
+
+ uint64_t addr;
+ uint32_t length;
+ int enable;
+
+ addr = pci_get_quad(pci_dev->config + PXB_PCIE_HOST_BRIDGE_MCFG_BAR);
+ enable = addr & PXB_PCIE_HOST_BRIDGE_ENABLE;
+ addr &= PXB_PCIE_HOST_BRIDGE_PCIEXBAR_ADMSK;
+ length = (pxb->max_bus - pxb->bus_nr + 1) * MiB;
+
+ pcie_host_mmcfg_update(pehb, enable, addr, length);
+}
+
+static void pxb_pcie_config_write(PCIDevice *d, uint32_t address,
+ uint32_t val, int len)
+{
+ PXBDev *pxb = PXB_PCIE_DEV(d);
+
+ pci_default_write_config(d, address, val, len);
+
+ /* Since desired bus number is provided by pxb-pcie,
+ we just need seabios to decide the mcfg_base for us.
+ mcfg_size can be calculated easily */
+ if (ranges_overlap(address, len, PXB_PCIE_HOST_BRIDGE_MCFG_BAR,
+ PXB_PCIE_HOST_BRIDGE_MCFG_BAR_SIZE)) {
+ pxb_pcie_update_pciexbar(pxb);
+ }
+}
+
static Property pxb_dev_properties[] = {
/* Note: 0 is not a legal PXB bus number. */
DEFINE_PROP_UINT8(PROP_PXB_BUS_NR, PXBDev, bus_nr, 0),
@@ -459,6 +495,7 @@ static void pxb_pcie_dev_class_init(ObjectClass *klass,
void *data)
k->vendor_id = PCI_VENDOR_ID_REDHAT;
k->device_id = PCI_DEVICE_ID_REDHAT_PXB_PCIE;
k->class_id = PCI_CLASS_BRIDGE_HOST;
+ k->config_write = pxb_pcie_config_write;
dc->desc = "PCI Express Expander Bridge";
dc->props = pxb_pcie_dev_properties;
diff --git a/include/hw/pci-bridge/pci_expander_bridge.h
b/include/hw/pci-bridge/pci_expander_bridge.h
index e017c62..5be0a8e 100644
--- a/include/hw/pci-bridge/pci_expander_bridge.h
+++ b/include/hw/pci-bridge/pci_expander_bridge.h
@@ -2,6 +2,7 @@
#define HW_PCI_EXPANDER_H
#include "hw/pci/pcie_host.h"
+#include "hw/pci-host/q35.h"
#define TYPE_PXB_PCIE_HOST "pxb-pcie-host"
#define PXB_PCIE_HOST_DEVICE(obj) \
@@ -13,4 +14,10 @@ typedef struct PXBPCIEHost {
/*< public >*/
} PXBPCIEHost;
+#define PXB_PCIE_HOST_BRIDGE_MCFG_BAR 0x50 /* 64bit register
*/
+#define PXB_PCIE_HOST_BRIDGE_MCFG_BAR_SIZE 8
+#define PXB_PCIE_HOST_BRIDGE_ENABLE MCH_HOST_BRIDGE_PCIEXBAREN
+/* The mcfg_base of pxb-pcie is not 256MB-aligned, but MB-aligned */
+#define PXB_PCIE_HOST_BRIDGE_PCIEXBAR_ADMSK Q35_MASK(64, 35, 20)
+
#endif
--
2.7.4
- [Qemu-devel] [RFC v5 0/6] pci_expander_brdige: support separate pci domain for pxb-pcie, Zihan Yang, 2018/09/17
- [Qemu-devel] [RFC v5 2/6] pci_expander_bridge: add domain_nr and max_bus property for pxb-pcie, Zihan Yang, 2018/09/17
- [Qemu-devel] [RFC v5 1/6] pci_expander_bridge: add type TYPE_PXB_PCIE_HOST, Zihan Yang, 2018/09/17
- [Qemu-devel] [RFC v5 3/6] acpi-build: allocate mcfg for pxb-pcie host bridges, Zihan Yang, 2018/09/17
- [Qemu-devel] [RFC v5 4/6] i386/acpi-build: describe new pci domain in AML, Zihan Yang, 2018/09/17
- [Qemu-devel] [RFC v5 5/6] pci_expander_bridge: add config_write callback for pxb-pcie,
Zihan Yang <=
- [Qemu-devel] [RFC v5 6/6] pci_expander_bridge: inform seabios of desired mcfg size via hidden bar, Zihan Yang, 2018/09/17
- Re: [Qemu-devel] [RFC v5 0/6] pci_expander_brdige: support separate pci domain for pxb-pcie, Michael S. Tsirkin, 2018/09/18
- Re: [Qemu-devel] [RFC v5 0/6] pci_expander_brdige: support separate pci domain for pxb-pcie, Zihan Yang, 2018/09/18
- Re: [Qemu-devel] [RFC v5 0/6] pci_expander_brdige: support separate pci domain for pxb-pcie, Gerd Hoffmann, 2018/09/19
- Re: [Qemu-devel] [RFC v5 0/6] pci_expander_brdige: support separate pci domain for pxb-pcie, Zihan Yang, 2018/09/19
- Re: [Qemu-devel] [RFC v5 0/6] pci_expander_brdige: support separate pci domain for pxb-pcie, Gerd Hoffmann, 2018/09/20
- Re: [Qemu-devel] [RFC v5 0/6] pci_expander_brdige: support separate pci domain for pxb-pcie, Marcel Apfelbaum, 2018/09/20
- Re: [Qemu-devel] [RFC v5 0/6] pci_expander_brdige: support separate pci domain for pxb-pcie, Zihan Yang, 2018/09/24
Re: [Qemu-devel] [RFC v5 0/6] pci_expander_brdige: support separate pci domain for pxb-pcie, Marcel Apfelbaum, 2018/09/20