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
_______________________________________________
SeaBIOS mailing list
SeaBIOS@seabios.org
http://www.seabios.org/mailman/listinfo/seabios
