Ray, Sorry I missed this patch. It looks good to me. I have tested it and it resolves some slow boot issues I was seeing.
Please add more details to the commit message, since this patch fixes a bug in a prior patch: This issue is only observed when DEBUG_CODE() macros are enabled and was introduced by the following patch: [edk2] [Patch] MdeModulePkg: Fix a PciBusDxe hot plug bug SVN revision 18658 Reviewed-by: Michael Kinney <[email protected]> Mike >-----Original Message----- >From: edk2-devel [mailto:[email protected]] On Behalf Of >Ruiyu Ni >Sent: Tuesday, October 27, 2015 10:21 PM >To: [email protected] >Cc: Ni, Ruiyu; Tian, Feng >Subject: [edk2] [Patch] MdeModulePkg: Do not dump NULL padding resource >descriptor > >Contributed-under: TianoCore Contribution Agreement 1.0 >Signed-off-by: Ruiyu Ni <[email protected]> >Cc: Feng Tian <[email protected]> >--- > MdeModulePkg/Bus/Pci/PciBusDxe/PciEnumeratorSupport.c | 4 ++++ > 1 file changed, 4 insertions(+) > >diff --git a/MdeModulePkg/Bus/Pci/PciBusDxe/PciEnumeratorSupport.c >b/MdeModulePkg/Bus/Pci/PciBusDxe/PciEnumeratorSupport.c >index 030ef42..a6ade26 100644 >--- a/MdeModulePkg/Bus/Pci/PciBusDxe/PciEnumeratorSupport.c >+++ b/MdeModulePkg/Bus/Pci/PciBusDxe/PciEnumeratorSupport.c >@@ -340,6 +340,10 @@ DumpPpbPaddingResource ( > EFI_ACPI_ADDRESS_SPACE_DESCRIPTOR *Descriptor; > PCI_BAR_TYPE Type; > >+ if (PciIoDevice->ResourcePaddingDescriptors == NULL) { >+ return; >+ } >+ > if (ResourceType == PciBarTypeIo16 || ResourceType == PciBarTypeIo32) { > ResourceType = PciBarTypeIo; > } >-- >1.9.5.msysgit.1 > >_______________________________________________ >edk2-devel mailing list >[email protected] >https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
