This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[commit] Fix gdbserver qXfer:libraries-svr4 regression in special cases
- From: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- To: gdb-patches at sourceware dot org
- Cc: Paul Pluzhnikov <ppluzhnikov at google dot com>, Pedro Alves <palves at redhat dot com>
- Date: Mon, 28 May 2012 22:38:30 +0200
- Subject: [commit] Fix gdbserver qXfer:libraries-svr4 regression in special cases
- References: <20120303182730.GA14062@host2.jankratochvil.net> <4F54E15E.4020004@redhat.com> <20120410200604.GA1777@host2.jankratochvil.net>
On Tue, 10 Apr 2012 22:06:04 +0200, Jan Kratochvil wrote:
> So I plan to use the former patch of mine to just always fall-back to the
> non-accelerated linux-nat reading of memory.
Checked that one in.
Thanks,
Jan
http://sourceware.org/ml/gdb-cvs/2012-05/msg00200.html
--- src/gdb/ChangeLog 2012/05/24 22:14:35 1.14294
+++ src/gdb/ChangeLog 2012/05/28 20:37:28 1.14295
@@ -1,3 +1,8 @@
+2012-05-28 Jan Kratochvil <jan.kratochvil@redhat.com>
+
+ * solib-svr4.c (svr4_current_sos): New comment on
+ svr4_current_sos_via_xfer_libraries fall back.
+
2012-05-24 Jan Kratochvil <jan.kratochvil@redhat.com>
* psymtab.c (lookup_symbol_aux_psymtabs): New variable stab_best. Use
--- src/gdb/gdbserver/ChangeLog 2012/05/24 16:51:37 1.620
+++ src/gdb/gdbserver/ChangeLog 2012/05/28 20:37:29 1.621
@@ -1,3 +1,7 @@
+2012-05-28 Jan Kratochvil <jan.kratochvil@redhat.com>
+
+ * linux-low.c (linux_qxfer_libraries_svr4): Return -1 if R_DEBUG is -1.
+
2012-05-24 Pedro Alves <palves@redhat.com>
PR gdb/7205
--- src/gdb/testsuite/ChangeLog 2012/05/25 21:54:15 1.3208
+++ src/gdb/testsuite/ChangeLog 2012/05/28 20:37:29 1.3209
@@ -1,3 +1,9 @@
+2012-05-28 Jan Kratochvil <jan.kratochvil@redhat.com>
+
+ * gdb.server/solib-list-lib.c: New file.
+ * gdb.server/solib-list-main.c: New file.
+ * gdb.server/solib-list.exp: New file.
+
2012-05-25 Maciej W. Rozycki <macro@codesourcery.com>
* gdb.trace/unavailable.exp (gdb_collect_globals_test): Update
--- src/gdb/solib-svr4.c 2012/05/24 16:51:36 1.162
+++ src/gdb/solib-svr4.c 2012/05/28 20:37:29 1.163
@@ -1260,6 +1260,14 @@
int ignore_first;
struct svr4_library_list library_list;
+ /* Fall back to manual examination of the target if the packet is not
+ supported or gdbserver failed to find DT_DEBUG. gdb.server/solib-list.exp
+ tests a case where gdbserver cannot find the shared libraries list while
+ GDB itself is able to find it via SYMFILE_OBJFILE.
+
+ Unfortunately statically linked inferiors will also fall back through this
+ suboptimal code path. */
+
if (svr4_current_sos_via_xfer_libraries (&library_list))
{
if (library_list.main_lm)
--- src/gdb/gdbserver/linux-low.c 2012/05/24 16:51:37 1.220
+++ src/gdb/gdbserver/linux-low.c 2012/05/28 20:37:29 1.221
@@ -5651,7 +5651,13 @@
if (priv->r_debug == 0)
priv->r_debug = get_r_debug (pid, is_elf64);
- if (priv->r_debug == (CORE_ADDR) -1 || priv->r_debug == 0)
+ /* We failed to find DT_DEBUG. Such situation will not change for this
+ inferior - do not retry it. Report it to GDB as E01, see for the reasons
+ at the GDB solib-svr4.c side. */
+ if (priv->r_debug == (CORE_ADDR) -1)
+ return -1;
+
+ if (priv->r_debug == 0)
{
document = xstrdup ("<library-list-svr4 version=\"1.0\"/>\n");
}
--- src/gdb/testsuite/gdb.server/solib-list-lib.c
+++ src/gdb/testsuite/gdb.server/solib-list-lib.c 2012-05-28 20:37:52.373434000 +0000
@@ -0,0 +1,28 @@
+/* This testcase is part of GDB, the GNU debugger.
+
+ Copyright 2012 Free Software Foundation, Inc.
+
+ This program is free software; you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation; either version 3 of the License, or
+ (at your option) any later version.
+
+ This program is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ GNU General Public License for more details.
+
+ You should have received a copy of the GNU General Public License
+ along with this program. If not, see <http://www.gnu.org/licenses/>. */
+
+#include <signal.h>
+
+static int libvar = 23;
+
+int
+libfunc (void)
+{
+ raise (SIGUSR1);
+
+ return libvar;
+}
--- src/gdb/testsuite/gdb.server/solib-list-main.c
+++ src/gdb/testsuite/gdb.server/solib-list-main.c 2012-05-28 20:37:52.639133000 +0000
@@ -0,0 +1,25 @@
+/* This testcase is part of GDB, the GNU debugger.
+
+ Copyright 2012 Free Software Foundation, Inc.
+
+ This program is free software; you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation; either version 3 of the License, or
+ (at your option) any later version.
+
+ This program is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ GNU General Public License for more details.
+
+ You should have received a copy of the GNU General Public License
+ along with this program. If not, see <http://www.gnu.org/licenses/>. */
+
+extern int libfunc (void);
+
+int
+main (void)
+{
+ libfunc ();
+ return 0;
+}
--- src/gdb/testsuite/gdb.server/solib-list.exp
+++ src/gdb/testsuite/gdb.server/solib-list.exp 2012-05-28 20:37:52.903888000 +0000
@@ -0,0 +1,104 @@
+# Copyright 2010-2012 Free Software Foundation, Inc.
+
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program. If not, see <http://www.gnu.org/licenses/>. */
+
+# In some cases gdbserver cannot find the DT_DEBUG library list on its own and
+# it needs to fall back to GDB solib-svr4.c look up via symbols from the main
+# executable.
+# gdbserver :1234 ld-linux-x86-64.so.2 /bin/prog
+# gdb /bin/prog
+# (gdb) target remote ...
+
+load_lib gdbserver-support.exp
+load_lib prelink-support.exp
+
+if {[skip_gdbserver_tests] || [skip_shlib_tests]} {
+ return
+}
+
+set testfile "solib-list"
+set srclibfile ${testfile}-lib.c
+set srcfile ${testfile}-main.c
+set binlibfile ${objdir}/${subdir}/${testfile}.so
+set executable ${testfile}
+set objfile ${objdir}/${subdir}/${executable}.o
+set binfile ${objdir}/${subdir}/${executable}
+
+if { [get_compiler_info unused]
+ || [gdb_compile_shlib "${srcdir}/${subdir}/${srclibfile}" "${binlibfile}" [list debug ldflags=-Wl,-soname,${binlibfile}]] != ""
+ || [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != ""
+ || [gdb_compile "${objfile} ${binlibfile}" "${binfile}" executable {}] != "" } {
+ untested "could not compile sources"
+ return -1
+}
+
+set interp_system [section_get ${binfile} .interp]
+verbose -log "system interpreter is: $interp_system"
+
+foreach nonstop { 0 1 } { with_test_prefix "non-stop $nonstop" {
+
+ # clean_restart assumes ${objdir}/${subdir}/ which is not applicable
+ # for ${interp_system}.
+ gdb_exit
+ gdb_start
+ gdb_reinitialize_dir $srcdir/$subdir
+ gdb_load ${interp_system}
+ gdb_load_shlibs ${binfile}
+ gdb_load_shlibs ${binlibfile}
+
+ # Make sure we're disconnected, in case we're testing with an
+ # extended-remote board, therefore already connected.
+ gdb_test "disconnect" ".*"
+
+ gdb_test "set non-stop $nonstop"
+ gdb_test "set target-async $nonstop"
+
+ # It is required for the non-stop mode, GDB would try to step over
+ # _dl_debug_state breakpoint will still only ld.so loaded in gdbserver.
+ # But GDB having symbols from the main executable it would try to use
+ # displaced-stepping buffer at unmapped that time address _start.
+ gdb_test "set displaced-stepping off"
+
+ set res [gdbserver_spawn ${binfile}]
+ set gdbserver_protocol [lindex $res 0]
+ set gdbserver_gdbport [lindex $res 1]
+
+ # Load BINFILE after gdbserver_spawn (so that gdbserver executes ld.so)
+ # but before "target remote" below so that qSymbol data get already
+ # initialized from BINFILE (and not from ld.so first needing a change to
+ # BINFILE later).
+ gdb_test "file ${binfile}" {Reading symbols from .*\.\.\.done\..*} "file binfile" \
+ {(Are you sure you want to change the file|Load new symbol table from ".*")\? \(y or n\) } "y"
+
+ set test "target $gdbserver_protocol"
+ gdb_test_multiple "target $gdbserver_protocol $gdbserver_gdbport" $test {
+ -re "Remote debugging using .*\r\n$gdb_prompt " {
+ # Do not anchor end, there may be more output in non-stop mode.
+ pass $test
+ }
+ }
+
+ if $nonstop {
+ set test "non-stop interior stop"
+ gdb_test_multiple "" $test {
+ -re " #1 stopped\\.\r\n" {
+ pass $test
+ }
+ }
+ }
+
+ gdb_test "continue" "Program received signal SIGUSR1, .*"
+ gdb_test "sharedlibrary" ".*"
+ gdb_test "p libvar" " = 23"
+}}