[SeaBIOS] [PATCH 2/2] fw/pci: map memory and IO regions for multiple pci root buses

Kevin O'Connor kevin at koconnor.net
Mon Nov 24 19:01:00 CET 2014


On Mon, Nov 24, 2014 at 12:21:31PM -0500, Kevin O'Connor wrote:
> On Mon, Nov 24, 2014 at 03:28:52PM +0100, Gerd Hoffmann wrote:
> > I think I would try to reuse the existing code which does the same for
> > bridges.  Reuse "struct pci_bus" to add one more level (and maybe rename
> > the struct), then have the resource propagation code in
> > pci_bios_check_devices() do the job.
> > 
> > That will need some reorganization, because the simple "struct pci_bus
> > *busses" array indexed by bus number will not work any more.
> 
> Why not just pretend that the extra root PCI buses are children of bus
> zero (for resource sizing and mapping purposes).

Thinking about this further, an even easier route may be to just place
all devices on the extra root PCI buses in busses[0].  If I understand
it correctly, when a system has more than one PCI root bus, all the
root buses snoop all mem and io accesses, so there isn't a requirement
for the io/mem regions to be continuous for that bus.  So, if all the
devices are thrown into busses[0], I think it may just work.

Specifically, I'm thinking about something like:

--- a/src/fw/pciinit.c
+++ b/src/fw/pciinit.c
@@ -668,6 +668,9 @@ static int pci_bios_check_devices(struct pci_bus *busses)
             busses[pci->secondary_bus].bus_dev = pci;
 
         struct pci_bus *bus = &busses[pci_bdf_to_bus(pci->bdf)];
+        if (!bus->bus_dev)
+            // Treat devices on root buses as if they are on the 0 root bus
+            bus = &busses[0];
         int i;
         for (i = 0; i < PCI_NUM_REGIONS; i++) {
             if ((pci->class == PCI_CLASS_BRIDGE_PCI) &&

-Kevin



More information about the SeaBIOS mailing list