From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) (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 477E521A04830 for ; Wed, 19 Apr 2017 13:37:34 -0700 (PDT) Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by orsmga105.jf.intel.com with ESMTP; 19 Apr 2017 13:37:34 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.37,222,1488873600"; d="scan'208";a="91513147" Received: from mdkinney-mobl.amr.corp.intel.com ([10.241.98.51]) by fmsmga005.fm.intel.com with ESMTP; 19 Apr 2017 13:37:33 -0700 From: Michael Kinney To: edk2-devel@lists.01.org Cc: Liming Gao , Yonghong Zhu , Kevin W Shaw Date: Wed, 19 Apr 2017 13:37:28 -0700 Message-Id: <1492634248-15292-4-git-send-email-michael.d.kinney@intel.com> X-Mailer: git-send-email 2.6.3.windows.1 In-Reply-To: <1492634248-15292-1-git-send-email-michael.d.kinney@intel.com> References: <1492634248-15292-1-git-send-email-michael.d.kinney@intel.com> Subject: [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: Wed, 19 Apr 2017 20:37:34 -0000 https://bugzilla.tianocore.org/show_bug.cgi?id=485 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 access 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.md 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: ``` |---- 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/135_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 the +platform that use multiple access methods. This sub-section is only present if +there are Binary modules included in the platform build and the binary module +uses a different PCD access method than other modules in the same platform +build. + +The sub-section header is: + +``` +>===============================================================================< +The following PCDs use different access methods: +================================================================================= +.. (List of PCDs) +<===============================================================================> +``` + +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 @@ -## 13.5 Global PCD Section +## 13.6 Global PCD Section This section contains the information for all PCDs whose values are the same for all modules in a platform. The content of global PCD sub-section is grouped @@ -47,7 +47,7 @@ gEfiMdeModulePkgTokenSpaceGuid Each global PCD item contains one or more lines: -### 13.5.1 Required line +### 13.6.1 Required line The first line is required: @@ -67,9 +67,9 @@ The first line is required: DEC DEFAULT = 0x0 ``` -### 13.5.2 Optional lines +### 13.6.2 Optional lines -#### 13.5.2.1 Dynamic/DynamicEx +#### 13.6.2.1 Dynamic/DynamicEx * if `` is DYN-HII @@ -93,7 +93,7 @@ The first line is required: 0x0001FFF ``` -#### 13.5.2.2 Default (optional) line +#### 13.6.2.2 Default (optional) line The second optional line is present if the value from the DEC was overridden. It is formatted as follows: @@ -107,7 +107,7 @@ It is formatted as follows: DEC DEFAULT = 0x0 ``` -#### 13.5.2.3 Additional optional lines +#### 13.6.2.3 Additional optional lines Additional lines are optional and show if the PCD's value was obtained from 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_reports/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 @@ -## 13.6 FD Section +## 13.7 FD Section This section contains platform flash device information and its layout. -### 13.6.1 FD Section Header +### 13.7.1 FD Section Header Given that a platform may have multi-Firmware device, this section may appear more than once in the output report. The section header lists the name of FD @@ -59,7 +59,7 @@ Size: 0x2a0000(2688KB) <==========================================================================> ``` -### 13.6.2 FD Region Sub-section +### 13.7.2 FD Region Sub-section 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 space @@ -122,7 +122,7 @@ Size: 0x2000 (8K) ...(More list of FD regions) ``` -### 13.6.3 VPD PCD Sub-section +### 13.7.3 VPD PCD Sub-section This section lists, in Offset order, every VPD PCD specified in the DSC file. 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 @@ -## 13.7 Module Section +## 13.8 Module Section Module section lists all modules involved in the platform build. If the **EXECUTION_ORDER** option is specified in **-Y** option, the module sections are sorted according to their PEI or DXE dispatch order; otherwise the module sections are listed according to their DSC position. -### 13.7.1 Module Section Summary +### 13.8.1 Module Section Summary 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) <==========================================================================> ``` -### 13.7.2 Library Sub-section +### 13.8.2 Library Sub-section This sub-section, which follows each Module Summary section, holds the information for all libraries used in this module. If it is an EDKII style @@ -208,7 +208,7 @@ s:\edk2\EdkCompatibilityPkg\Foundation\Framework\Guid\EdkFrameworkGuidLib.inf option. ********** -### 13.7.3 PCD Sub-section +### 13.8.3 PCD Sub-section This sub-section (following the Module Summary information) holds the information for all PCDs used in this module. The content of module PCD @@ -281,7 +281,7 @@ Each PCD may contain up to four lines: option. ********** -### 13.7.4 DEPEX Sub-section +### 13.8.4 DEPEX Sub-section 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. ********** -### 13.7.5 Build Flags Sub-section +### 13.8.5 Build Flags Sub-section This sub-section (following the Module Summary information) holds module build 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. ********** -### 13.7.6 Fixed Address Prediction Sub-section +### 13.8.6 Fixed Address Prediction Sub-section This sub-section (following the Module Summary information) contains module notification function information. All the notification functions are listed diff --git a/13_build_reports/139_execution_order_prediction_section.md 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 @@ -## 13.8 Execution Order Prediction Section +## 13.9 Execution Order Prediction Section This section contains platform level prediction for the execution flow. Each 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=483) Build Spec: add description for merge Pre-build and Post-build into build process | | | | [#490](https://bugzilla.tianocore.org/show_bug.cgi?id=490) Build Spec: specify the alignment requirements for VOID* PCDs stored in a PCD section | | | | [#479](https://bugzilla.tianocore.org/show_bug.cgi?id=479) Build spec: add description for nmake long command line handling | | +| | [#485](https://bugzilla.tianocore.org/show_bug.cgi?id=485) 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-platform-summary) - * [13.5 Global PCD Section](13_build_reports/135_global_pcd_section.md#135-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-module-section) - * [13.8 Execution Order Prediction Section](13_build_reports/138_execution_order_prediction_section.md#138-execution-order-prediction-section) + * [13.5 Mixed PCD Section](13_build_reports/135_mixed_pcd_section.md#135-mixed-pcd-section) + * [13.6 Global PCD Section](13_build_reports/136_global_pcd_section.md#136-global-pcd-section) + * [13.7 FD Section](13_build_reports/137_fd_section.md#137-fd-section) + * [13.8 Module Section](13_build_reports/138_module_section.md#138-module-section) + * [13.9 Execution Order Prediction Section](13_build_reports/139_execution_order_prediction_section.md#139-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_deftxt) * [Appendix C target.txt](appendix_c_targettxt.md#appendix-c-targettxt) -- 2.6.3.windows.1