Hi Eric, On 11/06/17 13:26, Eric Dong wrote: > The Protocol/Ppi used in the drivers but the it not add them > in the dependence section, it may cause driver assert. > This patch series add the missed Protocol/Ppi. > > Eric Dong (2): > UefiCpuPkg/CpuFeaturesDxe.inf: Add missed Depex protocol. > UefiCpuPkg/CpuFeaturesPei.inf: Add missed Depex Ppi. > > UefiCpuPkg/CpuFeatures/CpuFeaturesDxe.inf | 2 +- > UefiCpuPkg/CpuFeatures/CpuFeaturesPei.inf | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) >
(hopefully I'm commenting soon enough,) where exactly are the MP protocol and PPI used in UefiCpuPkg/CpuFeatures/? I grepped the directory for both protocol and PPI, and I found no matches. If the dependencies are incurred via library instances, then those library instances should spell out the depex. The only library class used by CpuFeaturesDxe.inf and CpuFeaturesPei.inf, for which multi-processing looks remotely relevant, is: RegisterCpuFeaturesLib. For this class, two instances appear to exist: UefiCpuPkg/Library/RegisterCpuFeaturesLib/DxeRegisterCpuFeaturesLib.inf UefiCpuPkg/Library/RegisterCpuFeaturesLib/PeiRegisterCpuFeaturesLib.inf The DXE instance consumes gEfiMpServiceProtocolGuid alright, but it also spells out the protocol in the depex: [Depex] gEfiMpServiceProtocolGuid AND gEdkiiCpuFeaturesSetDoneGuid The PEI instance is similar, wrt. the PPI: [Depex] gEfiPeiMpServicesPpiGuid AND gEdkiiCpuFeaturesSetDoneGuid So, this series appears unnecessary. What am I missing? Thanks Laszlo _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel