Reviewed-by: Zailiang Sun <zailiang....@intel.com> > -----Original Message----- > From: Kinney, Michael D > Sent: Thursday, May 9, 2019 12:31 PM > To: devel@edk2.groups.io > Cc: Sun, Zailiang <zailiang....@intel.com>; Qian, Yi <yi.q...@intel.com> > Subject: [Patch 04/16] Vlv2TbltDevicePkg/PpmPolicy: Fix call to MP Services > Protocol > > The NumberOfEnabledProcessors parameter to the MP Services Protocol > service GetNumberOfProcessors() is not optional and is not allowed to be > NULL. Add the CpuEnabledCount local variable and pass it into > GetNumberOfProcessors(). > > Cc: Zailiang Sun <zailiang....@intel.com> > Cc: Yi Qian <yi.q...@intel.com> > Signed-off-by: Michael D Kinney <michael.d.kin...@intel.com> > --- > Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.c > b/Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.c > index 28de8457ce..3583e324e6 100644 > --- a/Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.c > +++ b/Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.c > @@ -1,5 +1,5 @@ > /** > - Copyright (c) 2009 - 2013, Intel Corporation. All rights reserved.<BR> > + Copyright (c) 2009 - 2019, Intel Corporation. All rights > + reserved.<BR> > SPDX-License-Identifier: BSD-2-Clause-Patent > > > @@ -40,6 +40,7 @@ PpmPolicyEntry( > EFI_HANDLE Handle; > EFI_STATUS Status; > UINTN CpuCount; > + UINTN CpuEnabledCount; > UINT8 CPUMobileFeature; > > PCH_STEPPING Stepping; > @@ -63,7 +64,7 @@ PpmPolicyEntry( > // > // Get processor count from MP service. > // > - Status = MpService->GetNumberOfProcessors (MpService, &CpuCount, > NULL); > + Status = MpService->GetNumberOfProcessors (MpService, &CpuCount, > + &CpuEnabledCount); > ASSERT_EFI_ERROR (Status); > > // > -- > 2.21.0.windows.1
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#40293): https://edk2.groups.io/g/devel/message/40293 Mute This Topic: https://groups.io/mt/31551773/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
