From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) (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 7DCC821A0483B for ; Wed, 19 Apr 2017 20:07:00 -0700 (PDT) Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga104.jf.intel.com with ESMTP; 19 Apr 2017 20:06:59 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.37,223,1488873600"; d="scan'208";a="79043404" Received: from fmsmsx105.amr.corp.intel.com ([10.18.124.203]) by orsmga004.jf.intel.com with ESMTP; 19 Apr 2017 20:06:59 -0700 Received: from FMSMSX109.amr.corp.intel.com (10.18.116.9) by FMSMSX105.amr.corp.intel.com (10.18.124.203) with Microsoft SMTP Server (TLS) id 14.3.319.2; Wed, 19 Apr 2017 20:06:58 -0700 Received: from shsmsx151.ccr.corp.intel.com (10.239.6.50) by fmsmsx109.amr.corp.intel.com (10.18.116.9) with Microsoft SMTP Server (TLS) id 14.3.319.2; Wed, 19 Apr 2017 20:06:58 -0700 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.117]) by SHSMSX151.ccr.corp.intel.com ([169.254.3.224]) with mapi id 14.03.0319.002; Thu, 20 Apr 2017 11:06:54 +0800 From: "Zhu, Yonghong" To: "Kinney, Michael D" , "edk2-devel@lists.01.org" CC: "Gao, Liming" , "Shaw, Kevin W" , "Zhu, Yonghong" Thread-Topic: [edk2-BuildSpecification PATCH 3/3] Add report section for mixed PCD access Thread-Index: AQHSuUzO/BfkVgSQxECTtkEXd1Cpt6HNkPPg Date: Thu, 20 Apr 2017 03:06:54 +0000 Message-ID: References: <1492634248-15292-1-git-send-email-michael.d.kinney@intel.com> <1492634248-15292-4-git-send-email-michael.d.kinney@intel.com> In-Reply-To: <1492634248-15292-4-git-send-email-michael.d.kinney@intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiZmZiN2MwNGUtNTU4MS00MDg3LWJmMDgtZTBhNWIzOTFiMDk4IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE1LjkuNi42IiwiVHJ1c3RlZExhYmVsSGFzaCI6IlpJQWF4Ujk2K25kXC9BRjFjem1zeEh4RWs1Y2gwSWg4YlN2RXdTVlYrWDBnPSJ9 x-ctpclassification: CTP_IC dlp-product: dlpe-windows dlp-version: 10.0.102.7 dlp-reaction: no-action x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [edk2-BuildSpecification PATCH 3/3] Add report section for mixed PCD access 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: , X-List-Received-Date: Thu, 20 Apr 2017 03:07:00 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hi Mike, Is this section 13.5 but not 13.6 ? others are good to me. Reviewed-by: Yo= nghong Zhu =20 ## 13.6 Mixed PCD Section Best Regards, Zhu Yonghong -----Original Message----- From: Kinney, Michael D=20 Sent: Thursday, April 20, 2017 4:37 AM To: edk2-devel@lists.01.org Cc: Gao, Liming ; Zhu, Yonghong ; Shaw, Kevin W Subject: [edk2-BuildSpecification PATCH 3/3] Add report section for mixed P= CD access https://bugzilla.tianocore.org/show_bug.cgi?id=3D485 The build tools currently require a PCD to use the same access method from = all modules. This update still requires modules built from sources to use = the same access method, but binary modules are allowed to use alternate acc= ess methods. Cc: Liming Gao Cc: Yonghong Zhu Cc: Kevin W Shaw Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Michael Kinney --- 13_build_reports/133_output.md | 1 + 13_build_reports/135_mixed_pcd_section.md | 54 ++++++++++++++++++= ++++ 13_build_reports/136_global_pcd_section.md | 14 +++--- 13_build_reports/137_fd_section.md | 10 ++-- 13_build_reports/138_module_section.md | 16 +++---- .../139_execution_order_prediction_section.md | 4 +- README.md | 1 + SUMMARY.md | 9 ++-- 8 files changed, 83 insertions(+), 26 deletions(-) create mode 100644 13_= build_reports/135_mixed_pcd_section.md diff --git a/13_build_reports/133_output.md b/13_build_reports/133_output.m= d index 5363b7f..ce64efd 100644 --- a/13_build_reports/133_output.md +++ b/13_build_reports/133_output.md @@ -51,6 +51,7 @@ The layout of the text report file: =20 ``` |---- Platform summary + |----- Mixed PCD section |----- Global PCD section |----- FD section* |---- FD Region sub-section* diff --git a/13_build_reports/135_mixed_pcd_section.md b/13_build_reports/1= 35_mixed_pcd_section.md new file mode 100644 index 0000000..98d6662 --- /dev/null +++ b/13_build_reports/135_mixed_pcd_section.md @@ -0,0 +1,54 @@ + + +## 13.6 Mixed PCD Section + +There is an optional sub-section that, when present, lists the PCDs in=20 +the platform that use multiple access methods. This sub-section is only=20 +present if there are Binary modules included in the platform build and=20 +the binary module uses a different PCD access method than other modules=20 +in the same platform build. + +The sub-section header is: + +``` +>=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D +>=3D=3D=3D=3D=3D=3D=3D=3D=3D< +The following PCDs use different access methods: +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D +.. (List of PCDs) +<=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D +=3D=3D=3D=3D=3D=3D=3D=3D=3D> +``` + +Format for the entries in this section: + +``` +. +``` diff --git a/13_build_reports/136_global_pcd_section.md b/13_build_reports/= 136_global_pcd_section.md index 664949e..f24f97a 100644 --- a/13_build_reports/136_global_pcd_section.md +++ b/13_build_reports/136_global_pcd_section.md @@ -1,5 +1,5 @@ =20 -## 13.5 Global PCD Section +## 13.6 Global PCD Section =20 This section contains the information for all PCDs whose values are the sa= me for all modules in a platform. The content of global PCD sub-section is= grouped @@ -47,7 +47,7 @@ gEfiMdeModulePkgTokenSpaceGuid =20 Each global PCD item contains one or more lines: =20 -### 13.5.1 Required line +### 13.6.1 Required line =20 The first line is required: =20 @@ -67,9 +67,9 @@ The first line is required: DEC DEFAULT =3D 0x0 ``` =20 -### 13.5.2 Optional lines +### 13.6.2 Optional lines =20 -#### 13.5.2.1 Dynamic/DynamicEx +#### 13.6.2.1 Dynamic/DynamicEx =20 * if `` is DYN-HII =20 @@ -93,7 +93,7 @@ The first line is required: 0x0001FFF ``` =20 -#### 13.5.2.2 Default (optional) line +#### 13.6.2.2 Default (optional) line =20 The second optional line is present if the value from the DEC was overridd= en. It is formatted as follows: @@ -107,7 +107,7 @@ It is formatted as follows: DEC DEFAULT =3D 0x0 ``` =20 -#### 13.5.2.3 Additional optional lines +#### 13.6.2.3 Additional optional lines =20 Additional lines are optional and show if the PCD's value was obtained fro= m the INF file. This will be listed if the module's final PCD value is not= the same diff --git a/13_build_reports/137_fd_section.md b/13_build_report= s/137_fd_section.md index 81b8831..49c2695 100644 --- a/13_build_reports/137_fd_section.md +++ b/13_build_reports/137_fd_section.md @@ -1,5 +1,5 @@ =20 -## 13.6 FD Section +## 13.7 FD Section =20 This section contains platform flash device information and its layout. =20 -### 13.6.1 FD Section Header +### 13.7.1 FD Section Header =20 Given that a platform may have multi-Firmware device, this section may app= ear more than once in the output report. The section header lists the name= of FD @@ -59,7 +59,7 @@ Size: 0x2a0000(2688KB) <=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= > ``` =20 -### 13.6.2 FD Region Sub-section +### 13.7.2 FD Region Sub-section =20 This sub-section contains FD region information of platform flash device. = If the region is a firmware volume, it lists the set of modules and its sp= ace @@ -122,7 +122,7 @@ Size: 0x2000 (8K) ...(More list of FD regions) ``` =20 -### 13.6.3 VPD PCD Sub-section +### 13.7.3 VPD PCD Sub-section =20 This section lists, in Offset order, every VPD PCD specified in the DSC fi= le. The line format for this section is PcdName SkuId Offset PcdSize PcdValue. diff --git a/13_build_reports/138_module_section.md b/13_build_reports/138_= module_section.md index 96200a2..23e039b 100644 --- a/13_build_reports/138_module_section.md +++ b/13_build_reports/138_module_section.md @@ -1,5 +1,5 @@ =20 -## 13.7 Module Section +## 13.8 Module Section =20 Module section lists all modules involved in the platform build. If the **EXECUTION_ORDER** option is specified in **-Y** option, the module secti= ons are sorted according to their PEI or DXE dispatch order; otherwise the= module sections are listed according to their DSC position. =20 -### 13.7.1 Module Section Summary +### 13.8.1 Module Section Summary =20 This sub-section lists the module basic information: Module name: INF file= , file GUID, module size, module build time stamp and driver type. @@ -99,7 +99,7 @@ Driver Type: 0x7 (DRIVER) <=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= > ``` =20 -### 13.7.2 Library Sub-section +### 13.8.2 Library Sub-section =20 This sub-section, which follows each Module Summary section, holds the in= formation for all libraries used in this module. If it is an EDKII style @@= -208,7 +208,7 @@ s:\edk2\EdkCompatibilityPkg\Foundation\Framework\Guid\Edk= FrameworkGuidLib.inf option. ********** =20 -### 13.7.3 PCD Sub-section +### 13.8.3 PCD Sub-section =20 This sub-section (following the Module Summary information) holds the inf= ormation for all PCDs used in this module. The content of module PCD @@ -28= 1,7 +281,7 @@ Each PCD may contain up to four lines: option. ********** =20 -### 13.7.4 DEPEX Sub-section +### 13.8.4 DEPEX Sub-section =20 This sub-section (following the Module Summary information) holds module = dependency expression (DEPEX) information. The sub-section header holds the= @@ -353,7 +353,7 @@ EFI_HII_DATABASE_PROTOCOL_GUID option. ********** =20 -### 13.7.5 Build Flags Sub-section +### 13.8.5 Build Flags Sub-section =20 This sub-section (following the Module Summary information) holds module b= uild flags information. The sub-section header holds the module tool chain= tag and @@ -387,7 +387,7 @@ EHs-c- /GR- /GF /Gy /Zi /Gm `-Y` option. ********** =20 -### 13.7.6 Fixed Address Prediction Sub-section +### 13.8.6 Fixed Address Prediction Sub-section =20 This sub-section (following the Module Summary information) contains modul= e notification function information. All the notification functions are li= sted diff --git a/13_build_reports/139_execution_order_prediction_section.m= d b/13_build_reports/139_execution_order_prediction_section.md index c4e4a2b..7a2f277 100644 --- a/13_build_reports/139_execution_order_prediction_section.md +++ b/13_build_reports/139_execution_order_prediction_section.md @@ -1,5 +1,5 @@ =20 -## 13.8 Execution Order Prediction Section +## 13.9 Execution Order Prediction Section =20 This section contains platform level prediction for the execution flow. Ea= ch phase list the following triple in their predicted order: diff --git a/README.md b/README.md index 21bb782..c511971 100644 --- a/README.md +++ b/README.md @@ -206,3 +206,4 @@ Copyright (c) 2008-2017, Intel Corporation. All rights = reserved. | | [#483](https://bugzilla.tianocore.org/show_bug.cgi?id=3D483= ) Build Spec: add description for merge Pre-build and Post-build into build= process = = |= | | | [#490](https://bugzilla.tianocore.org/show_bug.cgi?id=3D490= ) Build Spec: specify the alignment requirements for VOID* PCDs stored in a= PCD section = = |= | | | [#479](https://bugzilla.tianocore.org/show_bug.cgi?id=3D479= ) Build spec: add description for nmake long command line handling = = = |= | +| | [#485](https://bugzilla.tianocore.org/show_bug.cgi?id=3D485= ) Build spec: add support for mixed Pcd access method = = = |= | diff --git a/SUMMARY.md b/SUMMARY.md index e374319..386fb4f 100644 --- a/SUMMARY.md +++ b/SUMMARY.md @@ -107,10 +107,11 @@ * [13.2 Sample Launch Steps: NT32 platform](13_build_reports/132_sample_= launch_steps_nt32_platform.md#132-sample-launch-steps-nt32-platform) * [13.3 Output](13_build_reports/133_output.md#133-output) * [13.4 Platform Summary](13_build_reports/134_platform_summary.md#134-p= latform-summary) - * [13.5 Global PCD Section](13_build_reports/135_global_pcd_section.md#1= 35-global-pcd-section) - * [13.6 FD Section](13_build_reports/136_fd_section.md#136-fd-section) - * [13.7 Module Section](13_build_reports/137_module_section.md#137-modul= e-section) - * [13.8 Execution Order Prediction Section](13_build_reports/138_executi= on_order_prediction_section.md#138-execution-order-prediction-section) + * [13.5 Mixed PCD=20 + Section](13_build_reports/135_mixed_pcd_section.md#135-mixed-pcd-secti + on) + * [13.6 Global PCD=20 + Section](13_build_reports/136_global_pcd_section.md#136-global-pcd-sec + tion) + * [13.7 FD=20 + Section](13_build_reports/137_fd_section.md#137-fd-section) + * [13.8 Module=20 + Section](13_build_reports/138_module_section.md#138-module-section) + * [13.9 Execution Order Prediction=20 + Section](13_build_reports/139_execution_order_prediction_section.md#13 + 9-execution-order-prediction-section) * [Appendix A Variables](appendix_a_variables.md#appendix-a-variables) * [Appendix B tools_def.txt](appendix_b_toolsdef_txt.md#appendix-b-tools_d= eftxt) * [Appendix C target.txt](appendix_c_targettxt.md#appendix-c-targettxt) -- 2.6.3.windows.1