On 04/30/19 21:31, Michael D Kinney wrote: > The CpuMpPei module uses a services from the CpuLib class, > but the CpuLib class is missing from the INF file. This > update is required to use the new MpInitLibUp instance that > does not use the CpuLib class. > > Cc: Eric Dong <eric.d...@intel.com> > Cc: Ray Ni <ray...@intel.com> > Cc: Laszlo Ersek <ler...@redhat.com> > Signed-off-by: Michael D Kinney <michael.d.kin...@intel.com> > --- > UefiCpuPkg/CpuMpPei/CpuMpPei.inf | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/UefiCpuPkg/CpuMpPei/CpuMpPei.inf > b/UefiCpuPkg/CpuMpPei/CpuMpPei.inf > index 14ec75f254..4fc4ea77ba 100644 > --- a/UefiCpuPkg/CpuMpPei/CpuMpPei.inf > +++ b/UefiCpuPkg/CpuMpPei/CpuMpPei.inf > @@ -1,7 +1,7 @@ > ## @file > # CPU driver installs CPU PI Multi-processor PPI. > # > -# Copyright (c) 2015 - 2018, Intel Corporation. All rights reserved.<BR> > +# Copyright (c) 2015 - 2019, Intel Corporation. All rights reserved.<BR> > # SPDX-License-Identifier: BSD-2-Clause-Patent > # > ## > @@ -43,6 +43,7 @@ [LibraryClasses] > CpuExceptionHandlerLib > MpInitLib > BaseMemoryLib > + CpuLib > > [Ppis] > gEfiPeiMpServicesPpiGuid ## PRODUCES >
Right, CpuMpPei calls CpuFlushTlb(), but its INF file doesn't require CpuLib. And this issue has been masked thus far because both DxeMpInitLib and PeiMpInitLib depend on CpuLib. Reviewed-by: Laszlo Ersek <ler...@redhat.com> Thanks, Laszlo -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#39877): https://edk2.groups.io/g/devel/message/39877 Mute This Topic: https://groups.io/mt/31424371/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-