Re: [edk2] [Patch] BaseTools: Update GenFw to clear unused debug entry generated by VS tool chain

2017-07-05 Thread Laszlo Ersek
I understand it now. The article at was of great help. On 07/05/17 15:29, Laszlo Ersek wrote: > On 07/05/17 14:42, Laszlo Ersek wrote: >> On 07/03/17 07:21, Liming Gao wrote: >>> @@ -2886,11 +2891,23 @@ Returns: >>> >>>if (DebugDirectory

Re: [edk2] [Patch] BaseTools: Update GenFw to clear unused debug entry generated by VS tool chain

2017-07-05 Thread Leif Lindholm
Many thanks for tracking this down Laszlo - we'd stumbled over it ourselves this morning. Liming, Mike, Andrew: this currently makes the BaseTools unusable with gcc toolchains. Can we revert this commit until this has been resolved? Best Regards, Leif On Wed, Jul 05, 2017 at 02:42:15PM +0200, L

Re: [edk2] [Patch] BaseTools: Update GenFw to clear unused debug entry generated by VS tool chain

2017-07-05 Thread Laszlo Ersek
On 07/05/17 14:42, Laszlo Ersek wrote: > Hi Liming, > > Gerd's Jenkins CI reported a GenFw segfault, with this patch applied. > > I can reproduce the segfault locally. This is the command line: > >> GenFw \ >> -e DXE_RUNTIME_DRIVER \ >> -o >> .../Build/OvmfX64/NOOPT_GCC48/X64/MdeModulePkg/U

Re: [edk2] [Patch] BaseTools: Update GenFw to clear unused debug entry generated by VS tool chain

2017-07-05 Thread Laszlo Ersek
Hi Liming, Gerd's Jenkins CI reported a GenFw segfault, with this patch applied. I can reproduce the segfault locally. This is the command line: > GenFw \ > -e DXE_RUNTIME_DRIVER \ > -o > .../Build/OvmfX64/NOOPT_GCC48/X64/MdeModulePkg/Universal/ReportStatusCodeRouter/RuntimeDxe/ReportStatus

Re: [edk2] [Patch] BaseTools: Update GenFw to clear unused debug entry generated by VS tool chain

2017-07-02 Thread Zhu, Yonghong
Reviewed-by: Yonghong Zhu Best Regards, Zhu Yonghong -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Liming Gao Sent: Monday, July 03, 2017 1:21 PM To: edk2-devel@lists.01.org Subject: [edk2] [Patch] BaseTools: Update GenFw to clear unused

[edk2] [Patch] BaseTools: Update GenFw to clear unused debug entry generated by VS tool chain

2017-07-02 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=600 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao --- BaseTools/Source/C/GenFw/GenFw.c | 27 ++- 1 file changed, 22 insertions(+), 5 deletions(-) diff --git a/BaseTools/Source/C/GenFw/GenF