diff options
| author | rsun3 <rsun3@6f19259b-4bc3-4df7-8a09-765794883524> | 2010-11-08 07:36:12 +0000 |
|---|---|---|
| committer | rsun3 <rsun3@6f19259b-4bc3-4df7-8a09-765794883524> | 2010-11-08 07:36:12 +0000 |
| commit | 13fd0d50b5f53a6a3cc0b294593ef8500cd9b985 (patch) | |
| tree | c984ee09b01a801164801fbf565fd0a0f926c13c /MdeModulePkg/Bus | |
| parent | fefefa4cb15eed75dbe3d4867768893bf6d96a30 (diff) | |
| download | edk2-13fd0d50b5f53a6a3cc0b294593ef8500cd9b985.zip edk2-13fd0d50b5f53a6a3cc0b294593ef8500cd9b985.tar.gz edk2-13fd0d50b5f53a6a3cc0b294593ef8500cd9b985.tar.bz2 | |
Per the UEFI spec, if a PCI controller is a P2P bridge, then the I/O, Memory and Bus Master bits in the Command register of the PCI configuration header should be placed in the enabled state by the PCI Bus driver.
But current PCI bus driver has a bug that if a P2P bridge has no child devices detected during enumeration, the bits won’t be set. This may impact PCI hot plug capable bridges because the OS may re-assign resources for them causing the reserved resource by the firmware will be overwritten.
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@11013 6f19259b-4bc3-4df7-8a09-765794883524
Diffstat (limited to 'MdeModulePkg/Bus')
| -rw-r--r-- | MdeModulePkg/Bus/Pci/PciBusDxe/PciDeviceSupport.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/MdeModulePkg/Bus/Pci/PciBusDxe/PciDeviceSupport.c b/MdeModulePkg/Bus/Pci/PciBusDxe/PciDeviceSupport.c index 1ad863c..d0b1725 100644 --- a/MdeModulePkg/Bus/Pci/PciBusDxe/PciDeviceSupport.c +++ b/MdeModulePkg/Bus/Pci/PciBusDxe/PciDeviceSupport.c @@ -694,7 +694,7 @@ StartPciDevicesOnBridge ( //
// If it is a PPB
//
- if (!IsListEmpty (&PciIoDevice->ChildList)) {
+ if (IS_PCI_BRIDGE (&PciIoDevice->Pci)) {
Status = StartPciDevicesOnBridge (
Controller,
PciIoDevice,
@@ -746,7 +746,7 @@ StartPciDevicesOnBridge ( (*NumberOfChildren)++;
}
- if (!IsListEmpty (&PciIoDevice->ChildList)) {
+ if (IS_PCI_BRIDGE (&PciIoDevice->Pci)) {
Status = StartPciDevicesOnBridge (
Controller,
PciIoDevice,
|