diff options
author | Michael D Kinney <michael.d.kinney@intel.com> | 2019-05-08 15:19:35 -0700 |
---|---|---|
committer | Michael D Kinney <michael.d.kinney@intel.com> | 2019-05-09 19:58:05 -0700 |
commit | e5a0cf9125313966fdd42014a0a92e8c5d350db0 (patch) | |
tree | 10302a73ba4a49c2266a578351e750f794903d0b | |
parent | 24990e84d32385ef72fcbd9018ed61ec3cdd84cd (diff) | |
download | edk2-e5a0cf9125313966fdd42014a0a92e8c5d350db0.zip edk2-e5a0cf9125313966fdd42014a0a92e8c5d350db0.tar.gz edk2-e5a0cf9125313966fdd42014a0a92e8c5d350db0.tar.bz2 |
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.sun@intel.com>
Cc: Yi Qian <yi.qian@intel.com>
Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
Reviewed-by: Zailiang Sun <zailiang.sun@intel.com>
Reviewed-by: Yi Qian <yi.qian@intel.com>
-rw-r--r-- | Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.c b/Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.c index 28de845..3583e32 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);
//
|