Re: [edk2-devel] [PATCH 1/1] MdeModulePkg: Enhance bus scan for all root bridge instances

["Ni, Ray" <ray.ni@intel.com>]

Reviewed-by: Ray Ni <ray.ni@intel.com>

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Foster
> Nong
> Sent: Friday, July 22, 2022 5:10 PM
> To: devel@edk2.groups.io
> Cc: Nong, Foster <foster.nong@intel.com>
> Subject: [edk2-devel] [PATCH 1/1] MdeModulePkg: Enhance bus scan for all
> root bridge instances
> 
> Ref:https://bugzilla.tianocore.org/show_bug.cgi?id=4000
> 
> Change flow to bus scan all root bridge instances even when any
> one root bridge meet bus resource OUT_OF_RESOURCE case.
> thus platform handler  of "EfiPciHostBridgeEndBusAllocation" has
> an chance to do relative pci bus rebalance to handle this case.
> 
> Signed-off-by: Foster Nong <foster.nong@intel.com>
> ---
>  MdeModulePkg/Bus/Pci/PciBusDxe/PciLib.c | 14 ++++++++++++--
>  1 file changed, 12 insertions(+), 2 deletions(-)
> 
> diff --git a/MdeModulePkg/Bus/Pci/PciBusDxe/PciLib.c
> b/MdeModulePkg/Bus/Pci/PciBusDxe/PciLib.c
> index 63d149b3b8c0..a89f912de85b 100644
> --- a/MdeModulePkg/Bus/Pci/PciBusDxe/PciLib.c
> +++ b/MdeModulePkg/Bus/Pci/PciBusDxe/PciLib.c
> @@ -1528,6 +1528,7 @@ PciHostBridgeEnumerator (
>    UINT8                              StartBusNumber;
>    LIST_ENTRY                         RootBridgeList;
>    LIST_ENTRY                         *Link;
> +  EFI_STATUS                         RootBridgeEnumerationStatus;
> 
>    if (FeaturePcdGet (PcdPciBusHotplugDeviceSupport)) {
>      InitializeHotPlugSupport ();
> @@ -1546,6 +1547,7 @@ PciHostBridgeEnumerator (
> 
>    DEBUG ((DEBUG_INFO, "PCI Bus First Scanning\n"));
>    RootBridgeHandle = NULL;
> +  RootBridgeEnumerationStatus = EFI_SUCCESS;
>    while (PciResAlloc->GetNextRootBridge (PciResAlloc, &RootBridgeHandle)
> == EFI_SUCCESS) {
>      //
>      // if a root bridge instance is found, create root bridge device for it
> @@ -1572,7 +1574,7 @@ PciHostBridgeEnumerator (
>      }
> 
>      if (EFI_ERROR (Status)) {
> -      return Status;
> +      RootBridgeEnumerationStatus = Status;
>      }
>    }
> 
> @@ -1581,6 +1583,10 @@ PciHostBridgeEnumerator (
>    //
>    NotifyPhase (PciResAlloc, EfiPciHostBridgeEndBusAllocation);
> 
> +  if (EFI_ERROR (RootBridgeEnumerationStatus)) {
> +      return RootBridgeEnumerationStatus;
> +  }
> +
>    if ((gPciHotPlugInit != NULL) && FeaturePcdGet
> (PcdPciBusHotplugDeviceSupport)) {
>      //
>      // Reset all assigned PCI bus number in all PPB
> @@ -1659,7 +1665,7 @@ PciHostBridgeEnumerator (
> 
>        DestroyRootBridge (RootBridgeDev);
>        if (EFI_ERROR (Status)) {
> -        return Status;
> +        RootBridgeEnumerationStatus = Status;
>        }
>      }
> 
> @@ -1667,6 +1673,10 @@ PciHostBridgeEnumerator (
>      // Notify the bus allocation phase is to end for the 2nd time
>      //
>      NotifyPhase (PciResAlloc, EfiPciHostBridgeEndBusAllocation);
> +
> +    if (EFI_ERROR (RootBridgeEnumerationStatus)) {
> +      return RootBridgeEnumerationStatus;
> +    }
>    }
> 
>    //
> --
> 2.26.2.windows.1
> 
> 
> 
> 
>