From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=fail (domain: intel.com, ip: , mailfrom: eric.dong@intel.com) Received: from mga17.intel.com (mga17.intel.com []) by groups.io with SMTP; Sun, 04 Aug 2019 23:44:03 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by fmsmga107.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 04 Aug 2019 23:44:02 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,348,1559545200"; d="scan'208";a="175501299" Received: from ydong10-win10.ccr.corp.intel.com ([10.239.158.133]) by fmsmga007.fm.intel.com with ESMTP; 04 Aug 2019 23:44:02 -0700 From: "Dong, Eric" To: devel@edk2.groups.io Cc: Ray Ni , Laszlo Ersek Subject: [Patch v2 2/2] UefiCpuPkg/PiSmmCpuDxeSmm: Remove debug message. Date: Mon, 5 Aug 2019 14:43:58 +0800 Message-Id: <20190805064359.29332-3-eric.dong@intel.com> X-Mailer: git-send-email 2.21.0.windows.1 In-Reply-To: <20190805064359.29332-1-eric.dong@intel.com> References: <20190805064359.29332-1-eric.dong@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit This debug message may be called by BSP and APs. It may caused ASSERT when APs call this debug code. In order to avoid system boot assert, Remove this debug message. Signed-off-by: Eric Dong Cc: Ray Ni Cc: Laszlo Ersek --- UefiCpuPkg/PiSmmCpuDxeSmm/CpuS3.c | 22 +--------------------- 1 file changed, 1 insertion(+), 21 deletions(-) diff --git a/UefiCpuPkg/PiSmmCpuDxeSmm/CpuS3.c b/UefiCpuPkg/PiSmmCpuDxeSmm/CpuS3.c index 2cfc61b2c6..d20bc4aae6 100644 --- a/UefiCpuPkg/PiSmmCpuDxeSmm/CpuS3.c +++ b/UefiCpuPkg/PiSmmCpuDxeSmm/CpuS3.c @@ -1,7 +1,7 @@ /** @file Code for Processor S3 restoration -Copyright (c) 2006 - 2018, Intel Corporation. All rights reserved.
+Copyright (c) 2006 - 2019, Intel Corporation. All rights reserved.
SPDX-License-Identifier: BSD-2-Clause-Patent **/ @@ -90,8 +90,6 @@ UINT8 mApHltLoopCodeTemplate[] = { 0xEB, 0xFC // jmp $-2 }; -CHAR16 *mRegisterTypeStr[] = {L"MSR", L"CR", L"MMIO", L"CACHE", L"SEMAP", L"INVALID" }; - /** Sync up the MTRR values for all processors. @@ -189,7 +187,6 @@ ProgramProcessorRegister ( UINT32 PackageThreadsCount; UINT32 CurrentThread; UINTN ProcessorIndex; - UINTN ThreadIndex; UINTN ValidThreadCount; UINT32 *ValidCoreCountPerPackage; @@ -202,23 +199,6 @@ ProgramProcessorRegister ( RegisterTableEntry = &RegisterTableEntryHead[Index]; - DEBUG_CODE_BEGIN (); - if (ApLocation != NULL) { - AcquireSpinLock (&CpuFlags->ConsoleLogLock); - ThreadIndex = ApLocation->Package * CpuStatus->MaxCoreCount * CpuStatus->MaxThreadCount + - ApLocation->Core * CpuStatus->MaxThreadCount + - ApLocation->Thread; - DEBUG (( - DEBUG_INFO, - "Processor = %lu, Entry Index %lu, Type = %s!\n", - (UINT64)ThreadIndex, - (UINT64)Index, - mRegisterTypeStr[MIN ((REGISTER_TYPE)RegisterTableEntry->RegisterType, InvalidReg)] - )); - ReleaseSpinLock (&CpuFlags->ConsoleLogLock); - } - DEBUG_CODE_END (); - // // Check the type of specified register // -- 2.21.0.windows.1