回复: [edk2-devel] 回复: [PATCH v2 1/1] MdePkg/Include: Use DEBUG_FILE_PATH to specify debug file path.
Guomin: gEfiCallerBaseName is the module base name, not source file name. It can verify the build reproduce, but it can't provide the same functionality to _FILE_. Thanks Liming > -邮件原件- > 发件人: Jiang, Guomin > 发送时间: 2022年4月20日 16:58 > 收件人: Gao, Liming ; devel@edk2.groups.io > 抄送: Kinney, Michael D ; Liu, Zhiguang > > 主题: RE: [edk2-devel] 回复: [PATCH v2 1/1] MdePkg/Include: Use > DEBUG_FILE_PATH to specify debug file path. > > Hi Liming, > > I just add ```-D DEBUG_FILE_PATH=gEfiCallerBaseName``` to BuildOptions > > Thanks > Guomin > > > -Original Message- > > From: gaoliming > > Sent: Sunday, April 17, 2022 11:21 AM > > To: Jiang, Guomin ; devel@edk2.groups.io > > Cc: Kinney, Michael D ; Liu, Zhiguang > > > > Subject: 回复: [edk2-devel] 回复: [PATCH v2 1/1] MdePkg/Include: Use > > DEBUG_FILE_PATH to specify debug file path. > > > > Guomin: > > I understand the purpose. But, I don't see the complete solution to > > configure DEBUG_FILE_PATH for every source file in order to meet with the > > debug image reproducible builds. Have you verified this solution with the > real > > DEBUG_FILE_PATH for every source file? Or, you just set DEBUG_FILE_PATH > > to the same value for the different source file? > > > > Thanks > > Liming > > > -邮件原件- > > > 发件人: Jiang, Guomin > > > 发送时间: 2022年4月13日 13:43 > > > 收件人: devel@edk2.groups.io; Gao, Liming > > > > 抄送: Kinney, Michael D ; Liu, Zhiguang > > > > > > 主题: RE: [edk2-devel] 回复: [PATCH v2 1/1] MdePkg/Include: Use > > > DEBUG_FILE_PATH to specify debug file path. > > > > > > Hi Liming, > > > > > > Below is the detail why need this change. If you only care usage. > > > Please jump to "How to use it" directly. > > > > > > Why need: > > > 1. Replication build is not new, you can refer > > > https://reproducible-builds.org for detail 2. The benefit is that 1) > > > make sure the same commit generate same binary so we can sure that it > > > is not modified by anyone 2) store the binary in the repo without > > > concern of different binary with same code. > > > > > > What's barrier in EDK2 implement? > > > 1. Very depend on Toolchain(for example VS, GCC, CLANG). Different > > > toolchain have different feature set and different toolchain version > > > have different feature set. > > > 2. Deploy new toolchain need big effort, include but not limit deploy > > > it in CI CD system, update toolchain, size concern, feature change, > > > etc > > > > > > What's the change: > > > 1. The change want to address one issue that we encounter in > > > replication > > > build: the ASSERT in EDK2 will be different in different environment. > > > For > > > example: file path, back slash or forward slash, etc > > > > > > How to use it: > > > 1. If you want to keep current ASSERT string format. No action is > > > required and the change keep back compatible 2. If you want to > > > customize the ASSERT string format. You can use additional tool to > > > generate PATH and define macro in BuildOptions or tools_def. > > > > > > Note: > > > 1. Replication build need many effort: 1) address the pdb path, 2) > > > address the timestamp, etc 2. This change is not target for resolving > > > all issue in replication build 3. I think it is a small step toward > > > space even though it haven't fix every thing. > > > > > > Thank > > > Guomin > > > > > > > -Original Message- > > > > From: devel@edk2.groups.io On Behalf Of > > > > gaoliming > > > > Sent: Wednesday, April 13, 2022 9:03 AM > > > > To: Jiang, Guomin ; devel@edk2.groups.io > > > > Cc: Kinney, Michael D ; Liu, Zhiguang > > > > > > > > Subject: [edk2-devel] 回复: [PATCH v2 1/1] MdePkg/Include: Use > > > > DEBUG_FILE_PATH to specify debug file path. > > > > > > > > Guomin: > > > > Can you introduce DEBUG_FILE_PATH usage? If the developer wants > to > > > > enable this feature, how configure DEBUG_FILE_PATH? > > > > > > > > Thanks > > > > Liming > > > > > -邮件原件- > > > > > 发件人: Guomin Jiang > > > > > 发送时间: 2022年4月12日 18:25 > > > > > 收件人: devel@edk2.groups.io > > > > > 抄送: Michael D Kinney ; Liming Gao > >
回复: [edk2-devel] 回复: [PATCH v2 1/1] MdePkg/Include: Use DEBUG_FILE_PATH to specify debug file path.
Guomin: I understand the purpose. But, I don't see the complete solution to configure DEBUG_FILE_PATH for every source file in order to meet with the debug image reproducible builds. Have you verified this solution with the real DEBUG_FILE_PATH for every source file? Or, you just set DEBUG_FILE_PATH to the same value for the different source file? Thanks Liming > -邮件原件- > 发件人: Jiang, Guomin > 发送时间: 2022年4月13日 13:43 > 收件人: devel@edk2.groups.io; Gao, Liming > 抄送: Kinney, Michael D ; Liu, Zhiguang > > 主题: RE: [edk2-devel] 回复: [PATCH v2 1/1] MdePkg/Include: Use > DEBUG_FILE_PATH to specify debug file path. > > Hi Liming, > > Below is the detail why need this change. If you only care usage. Please jump > to "How to use it" directly. > > Why need: > 1. Replication build is not new, you can refer https://reproducible-builds.org > for detail > 2. The benefit is that 1) make sure the same commit generate same binary so > we can sure that it is not modified by anyone 2) store the binary in the repo > without concern of different binary with same code. > > What's barrier in EDK2 implement? > 1. Very depend on Toolchain(for example VS, GCC, CLANG). Different toolchain > have different feature set and different toolchain version have different > feature set. > 2. Deploy new toolchain need big effort, include but not limit deploy it in CI > CD system, update toolchain, size concern, feature change, etc > > What's the change: > 1. The change want to address one issue that we encounter in replication > build: the ASSERT in EDK2 will be different in different environment. For > example: file path, back slash or forward slash, etc > > How to use it: > 1. If you want to keep current ASSERT string format. No action is required and > the change keep back compatible > 2. If you want to customize the ASSERT string format. You can use additional > tool to generate PATH and define macro in BuildOptions or tools_def. > > Note: > 1. Replication build need many effort: 1) address the pdb path, 2) address the > timestamp, etc > 2. This change is not target for resolving all issue in replication build > 3. I think it is a small step toward space even though it haven't fix every > thing. > > Thank > Guomin > > > -Original Message- > > From: devel@edk2.groups.io On Behalf Of > > gaoliming > > Sent: Wednesday, April 13, 2022 9:03 AM > > To: Jiang, Guomin ; devel@edk2.groups.io > > Cc: Kinney, Michael D ; Liu, Zhiguang > > > > Subject: [edk2-devel] 回复: [PATCH v2 1/1] MdePkg/Include: Use > > DEBUG_FILE_PATH to specify debug file path. > > > > Guomin: > > Can you introduce DEBUG_FILE_PATH usage? If the developer wants to > > enable this feature, how configure DEBUG_FILE_PATH? > > > > Thanks > > Liming > > > -邮件原件- > > > 发件人: Guomin Jiang > > > 发送时间: 2022年4月12日 18:25 > > > 收件人: devel@edk2.groups.io > > > 抄送: Michael D Kinney ; Liming Gao > > > ; Zhiguang Liu > > > 主题: [PATCH v2 1/1] MdePkg/Include: Use DEBUG_FILE_PATH to specify > > > debug file path. > > > > > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3840 > > > > > > Use DEBUG_FILE_PATH to control ASSERT path > > > > > > Motivation and Goal: > > > 1. Make replication build more easy and less toolchain dependency 2. > > > Consume the ASSERT string easy for downstream 3. Make code more > clear > > > > > > Cc: Michael D Kinney > > > Cc: Liming Gao > > > Cc: Zhiguang Liu > > > Signed-off-by: Guomin Jiang > > > --- > > > MdePkg/Include/Library/DebugLib.h | 34 > > > +-- > > > 1 file changed, 23 insertions(+), 11 deletions(-) > > > > > > diff --git a/MdePkg/Include/Library/DebugLib.h > > > b/MdePkg/Include/Library/DebugLib.h > > > index 8d3d08638d73..a76a268a00b6 100644 > > > --- a/MdePkg/Include/Library/DebugLib.h > > > +++ b/MdePkg/Include/Library/DebugLib.h > > > @@ -8,7 +8,7 @@ > > >of size reduction when compiler optimization is disabled. If > > > MDEPKG_NDEBUG is > > >defined, then debug and assert related macros wrapped by it are the > > > NULL implementations. > > > > > > -Copyright (c) 2006 - 2020, Intel Corporation. All rights > > > reserved. > > > +Copyright (c) 2006 - 2022, Intel Corporation. All rights > > > +reserved. > > > SPDX-License-Identifier: BSD-2-Clause-Patent > > > > > > **/ > > > @@ -85,6 +85,26 @@ SPDX-License