* [edk2-BuildSpecification PATCH] Clarify alignment requirements for VPD VOID* PCDs
@ 2017-04-18 4:41 Michael Kinney
2017-04-18 4:41 ` Michael Kinney
0 siblings, 1 reply; 3+ messages in thread
From: Michael Kinney @ 2017-04-18 4:41 UTC (permalink / raw)
To: edk2-devel; +Cc: Liming Gao, Yonghong Zhu, Kevin W Shaw
https://bugzilla.tianocore.org/show_bug.cgi?id=490
GitHub branch for review:
https://github.com/mdkinney/edk2-BuildSpecification/tree/Bugzilla_490_ClarifyVpdPcdAlignment
GitHub branch compare against latest DRAFT for review:
https://github.com/tianocore-docs/edk2-BuildSpecification/compare/master...mdkinney:Bugzilla_490_ClarifyVpdPcdAlignment?w=1
Clarify when error or warning messages are generated
for VOID* PCDs when VPD offset is not aligned.
Unicode string VPD PCDs must be 2-byte aligned.
Byte array {} VPD PCDs should be 8-byte aligned,
but is only a warning message if they are not 8-byte
aligned. All other data types only require
byte-alignment.
Cc: Liming Gao <liming.gao@intel.com>
Cc: Yonghong Zhu <yonghong.zhu@intel.com>
Cc: Kevin W Shaw <kevin.w.shaw@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Michael Kinney <michael.d.kinney@intel.com>
Michael Kinney (1):
Clarify alignment requirements for VPD VOID* PCDs
7_build_environment/73_guided_tools.md | 16 ++++++++++++++--
README.md | 1 +
2 files changed, 15 insertions(+), 2 deletions(-)
--
2.6.3.windows.1
^ permalink raw reply [flat|nested] 3+ messages in thread* [edk2-BuildSpecification PATCH] Clarify alignment requirements for VPD VOID* PCDs 2017-04-18 4:41 [edk2-BuildSpecification PATCH] Clarify alignment requirements for VPD VOID* PCDs Michael Kinney @ 2017-04-18 4:41 ` Michael Kinney 2017-04-18 5:24 ` Zhu, Yonghong 0 siblings, 1 reply; 3+ messages in thread From: Michael Kinney @ 2017-04-18 4:41 UTC (permalink / raw) To: edk2-devel; +Cc: Liming Gao, Yonghong Zhu, Kevin W Shaw https://bugzilla.tianocore.org/show_bug.cgi?id=490 Clarify when error or warning messages are generated for VOID* PCDs when VPD offset is not aligned. Unicode string VPD PCDs must be 2-byte aligned. Byte array {} VPD PCDs should be 8-byte aligned, but is only a warning message if they are not 8-byte aligned. All other data types only require byte-alignment. Cc: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Cc: Kevin W Shaw <kevin.w.shaw@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Michael Kinney <michael.d.kinney@intel.com> --- 7_build_environment/73_guided_tools.md | 16 ++++++++++++++-- README.md | 1 + 2 files changed, 15 insertions(+), 2 deletions(-) diff --git a/7_build_environment/73_guided_tools.md b/7_build_environment/73_guided_tools.md index 878ae33..a8881d3 100644 --- a/7_build_environment/73_guided_tools.md +++ b/7_build_environment/73_guided_tools.md @@ -123,8 +123,20 @@ file required by the build system is provided in the appendix, VPD Tool. * ASCII strings, "string", will be byte aligned. * Unicode strings, L"string" will be two-byte aligned. - * Byte arrays, {0x00, 0x01} will be 8-byte aligned. If the developer assigns - offset values in the DSC file, the developer must follow the same rules. + * Byte arrays, {0x00, 0x01} will be 8-byte aligned. + + If the developer manually assigns offset values in the DSC file, the developer + must follow the same rules. + + ********** + **Note:** If a developer manually sets the offset of a `VOID*` PCD with + Unicode string, L"string", style to a value that is not 2-byte aligned, then + an error is generated and the build halts. + ********** + **Note:** If a developer manually sets the offset of a `VOID*` PCD with byte + array {} style to a value that is not 8-byte aligned, then a warning is + generated, but the build will continue. + ********** 2. Modify the FDF file: diff --git a/README.md b/README.md index 71c4511..8542434 100644 --- a/README.md +++ b/README.md @@ -203,3 +203,4 @@ Copyright (c) 2008-2017, Intel Corporation. All rights reserved. | 1.27 | Convert to Gitbook | April 2017 | | | [#471](https://bugzilla.tianocore.org/show_bug.cgi?id=471) Build spec: only copy the "TianoCore" Userextension section into "As Built" INF | | | | [#472](https://bugzilla.tianocore.org/show_bug.cgi?id=472) [Build Spec] Extend macro usage in the !include statements for DSC/FDF files | | +| | [#490](https://bugzilla.tianocore.org/show_bug.cgi?id=490) Build Spec: specify the alignment requirements for VOID* PCDs stored in a PCD section | | -- 2.6.3.windows.1 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [edk2-BuildSpecification PATCH] Clarify alignment requirements for VPD VOID* PCDs 2017-04-18 4:41 ` Michael Kinney @ 2017-04-18 5:24 ` Zhu, Yonghong 0 siblings, 0 replies; 3+ messages in thread From: Zhu, Yonghong @ 2017-04-18 5:24 UTC (permalink / raw) To: Kinney, Michael D, edk2-devel@lists.01.org Cc: Gao, Liming, Shaw, Kevin W, Zhu, Yonghong Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> Best Regards, Zhu Yonghong -----Original Message----- From: Kinney, Michael D Sent: Tuesday, April 18, 2017 12:41 PM To: edk2-devel@lists.01.org Cc: Gao, Liming <liming.gao@intel.com>; Zhu, Yonghong <yonghong.zhu@intel.com>; Shaw, Kevin W <kevin.w.shaw@intel.com> Subject: [edk2-BuildSpecification PATCH] Clarify alignment requirements for VPD VOID* PCDs https://bugzilla.tianocore.org/show_bug.cgi?id=490 Clarify when error or warning messages are generated for VOID* PCDs when VPD offset is not aligned. Unicode string VPD PCDs must be 2-byte aligned. Byte array {} VPD PCDs should be 8-byte aligned, but is only a warning message if they are not 8-byte aligned. All other data types only require byte-alignment. Cc: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Cc: Kevin W Shaw <kevin.w.shaw@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Michael Kinney <michael.d.kinney@intel.com> --- 7_build_environment/73_guided_tools.md | 16 ++++++++++++++-- README.md | 1 + 2 files changed, 15 insertions(+), 2 deletions(-) diff --git a/7_build_environment/73_guided_tools.md b/7_build_environment/73_guided_tools.md index 878ae33..a8881d3 100644 --- a/7_build_environment/73_guided_tools.md +++ b/7_build_environment/73_guided_tools.md @@ -123,8 +123,20 @@ file required by the build system is provided in the appendix, VPD Tool. * ASCII strings, "string", will be byte aligned. * Unicode strings, L"string" will be two-byte aligned. - * Byte arrays, {0x00, 0x01} will be 8-byte aligned. If the developer assigns - offset values in the DSC file, the developer must follow the same rules. + * Byte arrays, {0x00, 0x01} will be 8-byte aligned. + + If the developer manually assigns offset values in the DSC file, the + developer must follow the same rules. + + ********** + **Note:** If a developer manually sets the offset of a `VOID*` PCD + with Unicode string, L"string", style to a value that is not 2-byte + aligned, then an error is generated and the build halts. + ********** + **Note:** If a developer manually sets the offset of a `VOID*` PCD + with byte array {} style to a value that is not 8-byte aligned, then + a warning is generated, but the build will continue. + ********** 2. Modify the FDF file: diff --git a/README.md b/README.md index 71c4511..8542434 100644 --- a/README.md +++ b/README.md @@ -203,3 +203,4 @@ Copyright (c) 2008-2017, Intel Corporation. All rights reserved. | 1.27 | Convert to Gitbook | April 2017 | | | [#471](https://bugzilla.tianocore.org/show_bug.cgi?id=471) Build spec: only copy the "TianoCore" Userextension section into "As Built" INF | | | | [#472](https://bugzilla.tianocore.org/show_bug.cgi?id=472) [Build Spec] Extend macro usage in the !include statements for DSC/FDF files | | +| | [#490](https://bugzilla.tianocore.org/show_bug.cgi?id=490) Build Spec: specify the alignment requirements for VOID* PCDs stored in a PCD section | | -- 2.6.3.windows.1 ^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-04-18 5:24 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-04-18 4:41 [edk2-BuildSpecification PATCH] Clarify alignment requirements for VPD VOID* PCDs Michael Kinney 2017-04-18 4:41 ` Michael Kinney 2017-04-18 5:24 ` Zhu, Yonghong
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox