[SeaBIOS] [PATCH V2 2/2] fw/pci: map memory and IO regions for multiple pci root buses
Kevin O'Connor
kevin at koconnor.net
Mon Feb 16 17:07:36 CET 2015
On Mon, Feb 16, 2015 at 05:43:31PM +0200, Marcel Apfelbaum wrote:
> On 02/16/2015 04:55 PM, Kevin O'Connor wrote:
> >On Mon, Feb 16, 2015 at 11:31:03AM +0200, Marcel Apfelbaum wrote:
> >>For resource sizing and mapping purposes treat devices on extra root
> >>buses as if they are on the default root bus (bus 0).
> >
> Hi Kevin,
> Thank you for the review.
>
> >Remind me - did you need the resources for each extra root bus to be
> >continuous (ie, not interspersed with resources allocated to devices
> >on the root bus)? If I read your patch correctly, it wont segregate
> >resources on the extra buses.
>
> I did need them continuous for prev implementation.
> I modified the QEMU code to be generic allowing mem/IO
> regions not to be continuous (per PCI root bus).
> The ACPI and the Operating systems do not require this,
> so the correct way is to achieve better resources utilization.
Okay, thanks.
BTW, do you need the res_on_default_bus flag? Would the below work
(totally untested)?
-Kevin
--- 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)
+ // Resources for all root busses go in busses[0]
+ bus = &busses[0];
int i;
for (i = 0; i < PCI_NUM_REGIONS; i++) {
if ((pci->class == PCI_CLASS_BRIDGE_PCI) &&
@@ -698,6 +701,9 @@ static int pci_bios_check_devices(struct pci_bus *busses)
if (!s->bus_dev)
continue;
struct pci_bus *parent = &busses[pci_bdf_to_bus(s->bus_dev->bdf)];
+ if (!parent->bus_dev)
+ // Resources for all root busses go in busses[0]
+ parent = &busses[0];
int type;
int hotplug_support = pci_bus_hotplug_support(s);
for (type = 0; type < PCI_REGION_TYPE_COUNT; type++) {
More information about the SeaBIOS
mailing list