Good! I will update comment. -----Original Message----- From: Zeng, Star Sent: Thursday, July 16, 2015 9:42 AM To: Gao, Liming; edk2-devel@lists.sourceforge.net Subject: RE: [edk2] [Patch] MdeModulePkg: Skip install MemoryInfoHob if this hob exists
> -----Original Message----- > From: Liming Gao [mailto:liming....@intel.com] > Sent: Tuesday, July 14, 2015 6:06 PM > To: edk2-devel@lists.sourceforge.net > Subject: [edk2] [Patch] MdeModulePkg: Skip install MemoryInfoHob if > this hob exists How about update the subject to "MdeModulePkg DxeIpl: Skip install MemoryInfoHob if this hob exists" ? > > Add the check at DxeLoadCore() on > MdeModulePkg\Core\DxeIplPeim\DxeLoad.c > to skip install the "gEfiMemoryTypeInformationGuid" hob if it is > already installed. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Liming Gao <liming....@intel.com> > --- > MdeModulePkg/Core/DxeIplPeim/DxeLoad.c | 55 > ++++++++++++++++++---------------- > 1 file changed, 30 insertions(+), 25 deletions(-) > > diff --git a/MdeModulePkg/Core/DxeIplPeim/DxeLoad.c > b/MdeModulePkg/Core/DxeIplPeim/DxeLoad.c > index aa05076..d7d693f 100644 > --- a/MdeModulePkg/Core/DxeIplPeim/DxeLoad.c > +++ b/MdeModulePkg/Core/DxeIplPeim/DxeLoad.c > @@ -271,35 +271,40 @@ DxeLoadCore ( > // > // Now should have a HOB with the DXE core > // > } > > - Status = PeiServicesLocatePpi ( > - &gEfiPeiReadOnlyVariable2PpiGuid, > - 0, > - NULL, > - (VOID **)&Variable > - ); > - if (!EFI_ERROR (Status)) { > - DataSize = sizeof (MemoryData); > - Status = Variable->GetVariable ( > - Variable, > - > EFI_MEMORY_TYPE_INFORMATION_VARIABLE_NAME, > - &gEfiMemoryTypeInformationGuid, > - NULL, > - &DataSize, > - &MemoryData > - ); > - if (!EFI_ERROR (Status) && > ValidateMemoryTypeInfoVariable(MemoryData, DataSize)) { > - // > - // Build the GUID'd HOB for DXE > - // > - BuildGuidDataHob ( > - &gEfiMemoryTypeInformationGuid, > - MemoryData, > - DataSize > - ); > + if (GetFirstGuidHob ((CONST EFI_GUID > *)&gEfiMemoryTypeInformationGuid) == NULL) { > + // > + // Don't build GuidHob if GuidHob has been installed. Since the code below is to install GuidHob, how about reverse the sentence to "Build GuidHob if it has not been installed." ? > + // > + Status = PeiServicesLocatePpi ( > + &gEfiPeiReadOnlyVariable2PpiGuid, > + 0, > + NULL, > + (VOID **)&Variable > + ); > + if (!EFI_ERROR (Status)) { > + DataSize = sizeof (MemoryData); > + Status = Variable->GetVariable ( > + Variable, > + > EFI_MEMORY_TYPE_INFORMATION_VARIABLE_NAME, > + &gEfiMemoryTypeInformationGuid, > + NULL, > + &DataSize, > + &MemoryData > + ); > + if (!EFI_ERROR (Status) && > ValidateMemoryTypeInfoVariable(MemoryData, DataSize)) { > + // > + // Build the GUID'd HOB for DXE > + // > + BuildGuidDataHob ( > + &gEfiMemoryTypeInformationGuid, > + MemoryData, > + DataSize > + ); > + } > } > } > > // > // Look in all the FVs present in PEI and find the DXE Core > FileHandle > -- > 1.9.5.msysgit.0 > > > ---------------------------------------------------------------------- > -------- Don't Limit Your Business. Reach for the Cloud. > GigeNET's Cloud Solutions provide you with the tools and support that > you need to offload your IT needs and focus on growing your business. > Configured For All Businesses. Start Your Cloud Today. > https://www.gigenetcloud.com/ > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/edk2-devel ------------------------------------------------------------------------------ Don't Limit Your Business. Reach for the Cloud. GigeNET's Cloud Solutions provide you with the tools and support that you need to offload your IT needs and focus on growing your business. Configured For All Businesses. Start Your Cloud Today. https://www.gigenetcloud.com/ _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel
