Hello community, here is the log from the commit of package ovmf for openSUSE:Factory checked in at 2017-10-23 16:39:00 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/ovmf (Old) and /work/SRC/openSUSE:Factory/.ovmf.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "ovmf" Mon Oct 23 16:39:00 2017 rev:18 rq:534326 version:2017+git1505340320.5afa5b8159 Changes: -------- --- /work/SRC/openSUSE:Factory/ovmf/ovmf.changes 2017-09-15 21:05:47.918092492 +0200 +++ /work/SRC/openSUSE:Factory/.ovmf.new/ovmf.changes 2017-10-23 16:39:01.292755474 +0200 @@ -1,0 +2,7 @@ +Mon Oct 16 09:02:28 UTC 2017 - [email protected] + +- Update ovmf-gdb-symbols.patch to avoid some symbols from being + removed (bsc#1063463) +- Add needssslcertforbuild back. It's useful for the devel projects. + +------------------------------------------------------------------- ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ ovmf.spec ++++++ --- /var/tmp/diff_new_pack.M2Nrbc/_old 2017-10-23 16:39:02.532697446 +0200 +++ /var/tmp/diff_new_pack.M2Nrbc/_new 2017-10-23 16:39:02.536697259 +0200 @@ -14,6 +14,7 @@ # Please submit bugfixes or comments via http://bugs.opensuse.org/ # +# needssslcertforbuild %undefine _build_create_debug ++++++ ovmf-gdb-symbols.patch ++++++ --- /var/tmp/diff_new_pack.M2Nrbc/_old 2017-10-23 16:39:02.684690334 +0200 +++ /var/tmp/diff_new_pack.M2Nrbc/_new 2017-10-23 16:39:02.688690146 +0200 @@ -1,7 +1,7 @@ -From 983c7fbc2444cbbbf3b82ae07a56232075a412bb Mon Sep 17 00:00:00 2001 +From 6f04d3c7e2fd042e1d367d9c4c0a9d19080f0e19 Mon Sep 17 00:00:00 2001 From: Gary Ching-Pang Lin <[email protected]> Date: Tue, 24 Jun 2014 11:57:32 +0800 -Subject: [PATCH 1/2] Add DebugPkg +Subject: [PATCH 1/3] Add DebugPkg --- DebugPkg/DebugPkg.dec | 34 +++++ @@ -14,10 +14,11 @@ create mode 100644 DebugPkg/GdbSyms/GdbSyms.inf create mode 100644 DebugPkg/Scripts/gdb_uefi.py -Index: ovmf-2017+git1472049752.ea2f21e/DebugPkg/DebugPkg.dec -=================================================================== +diff --git a/DebugPkg/DebugPkg.dec b/DebugPkg/DebugPkg.dec +new file mode 100644 +index 0000000000..e12401de3e --- /dev/null -+++ ovmf-2017+git1472049752.ea2f21e/DebugPkg/DebugPkg.dec ++++ b/DebugPkg/DebugPkg.dec @@ -0,0 +1,34 @@ +## @file +# Debug package - various useful stuff for debugging. @@ -53,10 +54,11 @@ + +[LibraryClasses] + -Index: ovmf-2017+git1472049752.ea2f21e/DebugPkg/GdbSyms/GdbSyms.c -=================================================================== +diff --git a/DebugPkg/GdbSyms/GdbSyms.c b/DebugPkg/GdbSyms/GdbSyms.c +new file mode 100644 +index 0000000000..2551dfab90 --- /dev/null -+++ ovmf-2017+git1472049752.ea2f21e/DebugPkg/GdbSyms/GdbSyms.c ++++ b/DebugPkg/GdbSyms/GdbSyms.c @@ -0,0 +1,70 @@ +/** @file + @@ -128,10 +130,11 @@ +} + + -Index: ovmf-2017+git1472049752.ea2f21e/DebugPkg/GdbSyms/GdbSyms.inf -=================================================================== +diff --git a/DebugPkg/GdbSyms/GdbSyms.inf b/DebugPkg/GdbSyms/GdbSyms.inf +new file mode 100644 +index 0000000000..afb78871b8 --- /dev/null -+++ ovmf-2017+git1472049752.ea2f21e/DebugPkg/GdbSyms/GdbSyms.inf ++++ b/DebugPkg/GdbSyms/GdbSyms.inf @@ -0,0 +1,57 @@ +## @file +# @@ -190,11 +193,12 @@ +[Depex] + TRUE + -Index: ovmf-2017+git1472049752.ea2f21e/DebugPkg/Scripts/gdb_uefi.py -=================================================================== +diff --git a/DebugPkg/Scripts/gdb_uefi.py b/DebugPkg/Scripts/gdb_uefi.py +new file mode 100644 +index 0000000000..dac215c538 --- /dev/null -+++ ovmf-2017+git1472049752.ea2f21e/DebugPkg/Scripts/gdb_uefi.py -@@ -0,0 +1,350 @@ ++++ b/DebugPkg/Scripts/gdb_uefi.py +@@ -0,0 +1,348 @@ +""" +Allows loading TianoCore symbols into a GDB session attached to EFI +Firmware. @@ -215,7 +219,6 @@ +import array +import getopt +import binascii -+import re + +__license__ = "BSD" +__version = "1.0.0" @@ -441,7 +444,6 @@ + base = base + opt['SizeOfHeaders'] + if sym_name != self.EINVAL: + sym_name = sym_name.cast (self.ptype('CHAR8')).string () -+ sym_name = re.sub(r"\.dll$", ".debug", sym_name) + syms.append ("add-symbol-file %s 0x%x" % \ + (sym_name, + long (base))) @@ -545,12 +547,85 @@ +ReloadUefi () + + -Index: ovmf-2017+git1472049752.ea2f21e/OvmfPkg/OvmfPkgX64.dsc -=================================================================== ---- ovmf-2017+git1472049752.ea2f21e.orig/OvmfPkg/OvmfPkgX64.dsc -+++ ovmf-2017+git1472049752.ea2f21e/OvmfPkg/OvmfPkgX64.dsc -@@ -821,3 +821,4 @@ +-- +2.14.2 + + +From 670d526765a9b0f3f6b3f64f4f51a3aa03028e88 Mon Sep 17 00:00:00 2001 +From: Gary Ching-Pang Lin <[email protected]> +Date: Tue, 24 Jun 2014 11:59:02 +0800 +Subject: [PATCH 2/3] Compile DebugPkg and load .debug files + +--- + DebugPkg/Scripts/gdb_uefi.py | 2 ++ + OvmfPkg/OvmfPkgX64.dsc | 1 + + 2 files changed, 3 insertions(+) + +diff --git a/DebugPkg/Scripts/gdb_uefi.py b/DebugPkg/Scripts/gdb_uefi.py +index dac215c538..3db87a4de4 100644 +--- a/DebugPkg/Scripts/gdb_uefi.py ++++ b/DebugPkg/Scripts/gdb_uefi.py +@@ -18,6 +18,7 @@ images were converted from MACH-O or ELF binaries. + import array + import getopt + import binascii ++import re + + __license__ = "BSD" + __version = "1.0.0" +@@ -243,6 +244,7 @@ class ReloadUefi (gdb.Command): + base = base + opt['SizeOfHeaders'] + if sym_name != self.EINVAL: + sym_name = sym_name.cast (self.ptype('CHAR8')).string () ++ sym_name = re.sub(r"\.dll$", ".debug", sym_name) + syms.append ("add-symbol-file %s 0x%x" % \ + (sym_name, + long (base))) +diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc +index 36c60fc19c..d59a48e695 100644 +--- a/OvmfPkg/OvmfPkgX64.dsc ++++ b/OvmfPkg/OvmfPkgX64.dsc +@@ -885,3 +885,4 @@ [Components] NULL|MdeModulePkg/Library/VarCheckUefiLib/VarCheckUefiLib.inf } !endif + DebugPkg/GdbSyms/GdbSyms.inf +-- +2.14.2 + + +From bab77f28b89595536135c3437d8fe680e0ea2320 Mon Sep 17 00:00:00 2001 +From: Gary Lin <[email protected]> +Date: Mon, 16 Oct 2017 16:32:27 +0800 +Subject: [PATCH 3/3] DebugPkg: Print the local variables + +With the "--whole-archive" linker option, several symbols in GdbSyms.dll +such as EFI_SYSTEM_TABLE_POINTER would be dropped, and the gdb_uefi +script could not find those symbols anymore. This commit add a few +useless DEBUG statements to keep those symbols. +--- + DebugPkg/GdbSyms/GdbSyms.c | 8 ++++++++ + 1 file changed, 8 insertions(+) + +diff --git a/DebugPkg/GdbSyms/GdbSyms.c b/DebugPkg/GdbSyms/GdbSyms.c +index 2551dfab90..0d591e5a8f 100644 +--- a/DebugPkg/GdbSyms/GdbSyms.c ++++ b/DebugPkg/GdbSyms/GdbSyms.c +@@ -64,6 +64,14 @@ Initialize ( + (UINTN) &EIDCME | + 1 + ; ++ DEBUG ((DEBUG_VERBOSE, "%a: %llx\n", __FUNCTION__, &ESTP)); ++ DEBUG ((DEBUG_VERBOSE, "%a: %llx\n", __FUNCTION__, &EDIITH)); ++ DEBUG ((DEBUG_VERBOSE, "%a: %llx\n", __FUNCTION__, &EIDH)); ++ DEBUG ((DEBUG_VERBOSE, "%a: %llx\n", __FUNCTION__, &EIOHU)); ++ DEBUG ((DEBUG_VERBOSE, "%a: %llx\n", __FUNCTION__, &EIDDE)); ++ DEBUG ((DEBUG_VERBOSE, "%a: %llx\n", __FUNCTION__, &EIDCNE)); ++ DEBUG ((DEBUG_VERBOSE, "%a: %llx\n", __FUNCTION__, &EIDCRE)); ++ DEBUG ((DEBUG_VERBOSE, "%a: %llx\n", __FUNCTION__, &EIDCME)); + return !!Dummy & EFI_SUCCESS; + } + +-- +2.14.2 +
