From nobody Thu Jul 3 14:44:33 2025 Delivered-To: importer@patchew.org Received-SPF: none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) client-ip=198.145.21.10; envelope-from=edk2-devel-bounces@lists.01.org; helo=ml01.01.org; Authentication-Results: mx.zohomail.com; spf=none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) smtp.mailfrom=edk2-devel-bounces@lists.01.org Return-Path: Received: from ml01.01.org (ml01.01.org [198.145.21.10]) by mx.zohomail.com with SMTPS id 1503543829093167.63408386118067; Wed, 23 Aug 2017 20:03:49 -0700 (PDT) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id EFAA321D2E658; Wed, 23 Aug 2017 20:01:05 -0700 (PDT) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 863DF21D2E648 for ; Wed, 23 Aug 2017 20:01:02 -0700 (PDT) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 23 Aug 2017 20:03:36 -0700 Received: from ydong10-win10.ccr.corp.intel.com ([10.239.158.51]) by orsmga002.jf.intel.com with ESMTP; 23 Aug 2017 20:03:35 -0700 X-Original-To: edk2-devel@lists.01.org X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.41,419,1498546800"; d="scan'208";a="127637287" From: Eric Dong To: edk2-devel@lists.01.org Date: Thu, 24 Aug 2017 11:03:27 +0800 Message-Id: <1503543807-6304-5-git-send-email-eric.dong@intel.com> X-Mailer: git-send-email 2.7.0.windows.1 In-Reply-To: <1503543807-6304-1-git-send-email-eric.dong@intel.com> References: <1503543807-6304-1-git-send-email-eric.dong@intel.com> Subject: [edk2] [Patch 4/4] UefiCpuPkg: Update default for PcdCpuProcTraceMemSize/PcdCpuProcTraceOutputScheme. X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Michael Kinney , Ruiyu Ni MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" X-ZohoMail: RSF_4 Z_629925259 SPT_0 Content-Type: text/plain; charset="utf-8" These two definitions have redundant definition which can be handle by code. This patch update them to follow new code definitions. V2: Add more comments for the PCDs and keep consistent in .dec and .uni fil= es. Cc: Michael Kinney Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Eric Dong --- UefiCpuPkg/UefiCpuPkg.dec | 22 ++++++++++++---------- UefiCpuPkg/UefiCpuPkg.uni | 16 +++++++++------- 2 files changed, 21 insertions(+), 17 deletions(-) diff --git a/UefiCpuPkg/UefiCpuPkg.dec b/UefiCpuPkg/UefiCpuPkg.dec index b4e099d..3bd8740 100644 --- a/UefiCpuPkg/UefiCpuPkg.dec +++ b/UefiCpuPkg/UefiCpuPkg.dec @@ -286,7 +286,9 @@ gUefiCpuPkgTokenSpaceGuid.PcdCpuFeaturesSetting|{0x00, 0x00, 0x00, 0x00,= 0x00, 0x00, 0x00, 0x00}|VOID*|0x00000019 =20 ## Contains the size of memory required when CPU processor trace is enab= led.

- # Default value is 0x10 which disables the processor trace.
+ # Processor trace is enabled through set BIT44(CPU_FEATURE_PROC_TRACE) = in PcdCpuFeaturesSetting.

+ # This PCD is ignored if CPU processor trace is disabled.

+ # Default value is 0x00 which means 4KB of memory is allocated if CPU p= rocessor trace is enabled.
# 0x0 - 4K.
# 0x1 - 8K.
# 0x2 - 16K.
@@ -303,19 +305,19 @@ # 0xD - 32M.
# 0xE - 64M.
# 0xF - 128M.
- # 0x10 - ProcTraceMemDisable.
- # @Prompt The memory size used for processor trace. - # @ValidRange 0x80000001 | 0 - 0x10 - gUefiCpuPkgTokenSpaceGuid.PcdCpuProcTraceMemSize|0x10|UINT32|0x60000012 + # @Prompt The memory size used for processor trace if processor trace is= enabled. + # @ValidRange 0x80000001 | 0 - 0xF + gUefiCpuPkgTokenSpaceGuid.PcdCpuProcTraceMemSize|0x0|UINT32|0x60000012 =20 ## Contains the processor trace output scheme when CPU processor trace i= s enabled.

- # Default value is 2 which disables the processor trace.
+ # Processor trace is enabled through set BIT44(CPU_FEATURE_PROC_TRACE) = in PcdCpuFeaturesSetting.

+ # This PCD is ignored if CPU processor trace is disabled.

+ # Default value is 0 which means single range output scheme will be use= d if CPU processor trace is enabled.
# 0 - Single Range output scheme.
# 1 - ToPA(Table of physical address) scheme.
- # 2 - Invalid scheme.
- # @Prompt The processor trace output scheme. - # @ValidRange 0x80000001 | 0 - 2 - gUefiCpuPkgTokenSpaceGuid.PcdCpuProcTraceOutputScheme|0x2|UINT8|0x600000= 15 + # @Prompt The processor trace output scheme used when processor trace is= enabled. + # @ValidRange 0x80000001 | 0 - 1 + gUefiCpuPkgTokenSpaceGuid.PcdCpuProcTraceOutputScheme|0x0|UINT8|0x600000= 15 =20 [UserExtensions.TianoCore."ExtraFiles"] UefiCpuPkgExtra.uni diff --git a/UefiCpuPkg/UefiCpuPkg.uni b/UefiCpuPkg/UefiCpuPkg.uni index 858e4a7..9472b18 100644 --- a/UefiCpuPkg/UefiCpuPkg.uni +++ b/UefiCpuPkg/UefiCpuPkg.uni @@ -197,8 +197,10 @@ =20 #string STR_gUefiCpuPkgTokenSpaceGuid_PcdCpuProcTraceMemSize_PROMPT #lang= uage en-US "Memory size used by Processor Trace feature." =20 -#string STR_gUefiCpuPkgTokenSpaceGuid_PcdCpuProcTraceMemSize_HELP #langua= ge en-US "User input the memory size can be used by processor trace feature= .

\n" - = "Default value is 0x10 which disables the processor memory trace.<= BR>\n" +#string STR_gUefiCpuPkgTokenSpaceGuid_PcdCpuProcTraceMemSize_HELP #langua= ge en-US "User input the size of memory required when CPU processor trace i= s enabled.

\n" + = "Processor trace is enabled through set BIT44(CPU_FEATURE_PROC_TRA= CE) in PcdCpuFeaturesSetting.

\n" + = "This PCD is ignored if CPU processor trace is disabled.

\n" + = "Default value is 0x00 which means 4KB of memory is allocated if C= PU processor trace is enabled.
\n" = "0x0 - 4K.
\n" = "0x1 - 8K.
\n" = "0x2 - 16K.
\n" @@ -215,12 +217,12 @@ = "0xD - 32M.
\n" = "0xE - 64M.
\n" = "0xF - 128M.
\n" - = "0x10 - ProcTraceMemDisable.
\n" =20 #string STR_gUefiCpuPkgTokenSpaceGuid_PcdCpuProcTraceOutputScheme_PROMPT = #language en-US "Processor Trace output scheme type." =20 -#string STR_gUefiCpuPkgTokenSpaceGuid_PcdCpuProcTraceOutputScheme_HELP #l= anguage en-US "User input the processor trace output scheme type.

\n" - = "Default value is 2 which disables the processor memory trace= .
\n" +#string STR_gUefiCpuPkgTokenSpaceGuid_PcdCpuProcTraceOutputScheme_HELP #l= anguage en-US "User input the processor trace output scheme when CPU proces= sor trace is enabled.

\n" + = "Processor trace is enabled through set BIT44(CPU_FEATURE_PRO= C_TRACE) in PcdCpuFeaturesSetting.

\n" + = "This PCD is ignored if CPU processor trace is disabled.
<= BR>\n" + = "Default value is 0 which means single range output scheme wi= ll be used if CPU processor trace is enabled.
\n" = "0 - Single Range output scheme.
\n" - = "1 - ToPA(Table of physical address) scheme.
\n" - = "2 - Invalid scheme.
\n" \ No newline at end of file + = "1 - ToPA(Table of physical address) scheme.
\n" \ No newline at end of file --=20 2.7.0.windows.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel