[gdb/f13/master] * Tue Aug 3 2010 Jan Kratochvil <jan.kratochvil at redhat.com> - 7.1-31.fc13 - Fix long delay printing

Jan Kratochvil jkratoch at fedoraproject.org
Tue Aug 3 19:43:48 UTC 2010


commit 90924695e78774ca97f1832a50904914ea86bebf
Author: Jan Kratochvil <jan.kratochvil at redhat.com>
Date:   Tue Aug 3 21:43:28 2010 +0200

    * Tue Aug  3 2010 Jan Kratochvil <jan.kratochvil at redhat.com> - 7.1-31.fc13
    - Fix long delay printing an overflown stack backtrace (BZ 620012, Tom Tromey).

 gdb-bz620012-slow-long-bt.patch |   45 +++++++++++++++++++++++++++++++++++++++
 gdb.spec                        |    9 +++++++-
 2 files changed, 53 insertions(+), 1 deletions(-)
---
diff --git a/gdb-bz620012-slow-long-bt.patch b/gdb-bz620012-slow-long-bt.patch
new file mode 100644
index 0000000..9a1bf35
--- /dev/null
+++ b/gdb-bz620012-slow-long-bt.patch
@@ -0,0 +1,45 @@
+--- a/gdb/stack.c
++++ b/gdb/stack.c
+@@ -1309,24 +1309,24 @@ backtrace_command_1 (char *count_exp, int show_locals, int from_tty)
+   else
+     count = -1;
+ 
+-  {
+-    struct partial_symtab *ps;
+-
+-    /* Read in symbols for all of the frames.  Need to do this
+-       unconditionally to ensure that psymbols are read.  Also need to
+-       do this in a separate pass so that "Reading in symbols for xxx"
+-       messages don't screw up the appearance of the backtrace.  Also
+-       if people have strong opinions against reading symbols for
+-       backtrace this may have to be an option.  */
+-    i = count;
+-    for (fi = trailing; fi != NULL && i--; fi = get_prev_frame (fi))
+-      {
+-	QUIT;
+-	ps = find_pc_psymtab (get_frame_address_in_block (fi));
+-	if (info_verbose && ps)
+-	  PSYMTAB_TO_SYMTAB (ps); /* Force syms to come in.  */
+-      }
+-  }
++  if (info_verbose)
++    {
++      struct partial_symtab *ps;
++
++      /* Read in symbols for all of the frames.  Need to do this in a
++         separate pass so that "Reading in symbols for xxx" messages
++         don't screw up the appearance of the backtrace.  Also if
++         people have strong opinions against reading symbols for
++         backtrace this may have to be an option.  */
++      i = count;
++      for (fi = trailing; fi != NULL && i--; fi = get_prev_frame (fi))
++	{
++	  QUIT;
++	  ps = find_pc_psymtab (get_frame_address_in_block (fi));
++	  if (ps)
++	    PSYMTAB_TO_SYMTAB (ps); /* Force syms to come in.  */
++	}
++    }
+ 
+   for (i = 0, fi = trailing; fi && count--; i++, fi = get_prev_frame (fi))
+     {
diff --git a/gdb.spec b/gdb.spec
index a62d508..5bfe7d0 100644
--- a/gdb.spec
+++ b/gdb.spec
@@ -36,7 +36,7 @@ Version: 7.1
 
 # The release always contains a leading reserved number, start it at 1.
 # `upstream' is not a part of `name' to stay fully rpm dependencies compatible for the testing.
-Release: 30%{?_with_upstream:.upstream}%{dist}
+Release: 31%{?_with_upstream:.upstream}%{dist}
 
 License: GPLv3+ and GPLv3+ with exceptions and GPLv2+ and GPLv2+ with exceptions and GPL+ and LGPLv2+ and GFDL and BSD and Public Domain
 Group: Development/Debuggers
@@ -536,6 +536,9 @@ Patch490: gdb-bz614604-bt-cfi-without-die.patch
 # Fix crash in gdbpy_is_string (BZ 611569, Tom Tromey).
 Patch492: gdb-bz611569-gdbpy_is_string.patch
 
+# Fix long delay printing an overflown stack backtrace (BZ 620012, Tom Tromey).
+Patch494: gdb-bz620012-slow-long-bt.patch
+
 BuildRequires: ncurses-devel%{?_isa} texinfo gettext flex bison expat-devel%{?_isa}
 Requires: readline%{?_isa}
 BuildRequires: readline-devel%{?_isa}
@@ -841,6 +844,7 @@ rm -f gdb/jv-exp.c gdb/m2-exp.c gdb/objc-exp.c gdb/p-exp.c
 %patch489 -p1
 %patch490 -p1
 %patch492 -p1
+%patch494 -p1
 
 %patch415 -p1
 %patch393 -p1
@@ -1173,6 +1177,9 @@ fi
 %endif
 
 %changelog
+* Tue Aug  3 2010 Jan Kratochvil <jan.kratochvil at redhat.com> - 7.1-31.fc13
+- Fix long delay printing an overflown stack backtrace (BZ 620012, Tom Tromey).
+
 * Thu Jul 22 2010 Jan Kratochvil <jan.kratochvil at redhat.com> - 7.1-30.fc13
 - Fix crash in gdbpy_is_string (BZ 611569, Tom Tromey).
 


More information about the scm-commits mailing list