From patchwork Fri Feb 7 19:27:10 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Aaron Merey X-Patchwork-Id: 37746 Received: (qmail 59839 invoked by alias); 7 Feb 2020 19:27:36 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 59830 invoked by uid 89); 7 Feb 2020 19:27:35 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-18.2 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_BADIPHTTP, KAM_SHORT, RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.1 spammy=UD:disasm.c, disasm.c, disasmc X-HELO: us-smtp-1.mimecast.com Received: from us-smtp-delivery-1.mimecast.com (HELO us-smtp-1.mimecast.com) (207.211.31.120) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 07 Feb 2020 19:27:27 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1581103645; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type; bh=sH7rpQO3bPcEAP/EcxUnlux8czEKtCpBIkgv5AyjUBI=; b=JFWvHcr8+arLsZCI9rv/ByLNYqmomKVSMCGFFmhcC7MYO+POOq0bnkBj/tbEpm5NC9G07c AHuWTedEDsQKDALMaNkrRcLBWK8zoRjirfLwqY6BMEldLtz9xD1fyU3K8ALy5Z12K0R8j9 xIRYonSh+26abtBKn71C2hermcWqqCk= Received: from mail-vk1-f200.google.com (mail-vk1-f200.google.com [209.85.221.200]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-39-5oDj8VI7MZOsxEcxiLKqSg-1; Fri, 07 Feb 2020 14:27:22 -0500 Received: by mail-vk1-f200.google.com with SMTP id u7so283227vkb.1 for ; Fri, 07 Feb 2020 11:27:21 -0800 (PST) MIME-Version: 1.0 From: Aaron Merey Date: Fri, 7 Feb 2020 14:27:10 -0500 Message-ID: Subject: [PATCH v3] Add debuginfod support to GDB To: gdb-patches@sourceware.org Cc: Tom Tromey , Christian Biesinger X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com The previous revision of this patch had pkg.m4 included in gdb/aclocal.m4. In order to avoid any concerns regarding the copyright of pkg.m4 I've removed it and slightly modified the configure-time handling of debuginfod support. Aaron From 73706e462b6a540e156c40008986fce10d2dc0f9 Mon Sep 17 00:00:00 2001 From: Aaron Merey Date: Thu, 6 Feb 2020 14:05:05 -0500 Subject: [PATCH] Add debuginfod support to GDB debuginfod is a lightweight web service that indexes ELF/DWARF debugging resources by build-id and serves them over HTTP. This patch enables GDB to query debuginfod servers for separate debug files and source code when it is otherwise not able to find them. GDB can be built with debuginfod using the --with-debuginfod configure option. This requires that libdebuginfod be installed and found at configure time. debuginfod is packaged with elfutils, starting with version 0.178. For more information see https://sourceware.org/elfutils/. Tested on x86_64 Fedora 31. gdb/ChangeLog: 2020-02-07 Aaron Merey * Makefile.in: Handle optional debuginfod support. * NEWS: Update. * README: Add --with-debuginfod summary. * aclocal.m4: Regenerate. * config.in: Regenerate. * configure: Regenerate. * configure.ac: Handle optional debuginfod support. * debuginfod-support.c: debuginfod helper functions. * debuginfod-support.h: Ditto. * doc/gdb.texinfo: Add --with-debuginfod to configure options summary. * dwarf2read.c (dwarf2_get_dwz_file): Query debuginfod servers when a dwz file cannot be found. * elfread.c (elf_symfile_read): Query debuginfod servers when a debuginfo file cannot be found. * source.c (open_source_file): Query debuginfod servers when a source file cannot be found. * top.c (print_gdb_configuration): Include --{with,without}-debuginfod in the output. gdb/testsuite/ChangeLog: 2020-02-07 Aaron Merey * gdb.debuginfod: New directory for debuginfod tests. * gdb.debuginfod/main.c: New test file. * gdb.debuginfod/fetch_src_and_symbols.exp: New tests. --- gdb/Makefile.in | 7 +- gdb/NEWS | 14 ++ gdb/README | 9 + gdb/config.in | 3 + gdb/configure | 95 ++++++- gdb/configure.ac | 34 ++- gdb/debuginfod-support.c | 102 ++++++++ gdb/debuginfod-support.h | 43 ++++ gdb/doc/gdb.texinfo | 8 + gdb/dwarf2read.c | 25 ++ gdb/elfread.c | 33 ++- gdb/source.c | 44 ++++ .../gdb.debuginfod/fetch_src_and_symbols.exp | 232 ++++++++++++++++++ gdb/testsuite/gdb.debuginfod/main.c | 25 ++ gdb/top.c | 10 + 15 files changed, 677 insertions(+), 7 deletions(-) create mode 100644 gdb/debuginfod-support.c create mode 100644 gdb/debuginfod-support.h create mode 100644 gdb/testsuite/gdb.debuginfod/fetch_src_and_symbols.exp create mode 100644 gdb/testsuite/gdb.debuginfod/main.c diff --git a/gdb/Makefile.in b/gdb/Makefile.in index 45d1586e85..e31493df9e 100644 --- a/gdb/Makefile.in +++ b/gdb/Makefile.in @@ -591,7 +591,8 @@ INTERNAL_CFLAGS_BASE = \ $(GDB_CFLAGS) $(OPCODES_CFLAGS) $(READLINE_CFLAGS) $(ZLIBINC) \ $(BFD_CFLAGS) $(INCLUDE_CFLAGS) $(LIBDECNUMBER_CFLAGS) \ $(INTL_CFLAGS) $(INCGNU) $(INCSUPPORT) $(ENABLE_CFLAGS) \ - $(INTERNAL_CPPFLAGS) $(SRCHIGH_CFLAGS) $(TOP_CFLAGS) $(PTHREAD_CFLAGS) + $(INTERNAL_CPPFLAGS) $(SRCHIGH_CFLAGS) $(TOP_CFLAGS) $(PTHREAD_CFLAGS) \ + @DEBUGINFOD_CFLAGS@ INTERNAL_WARN_CFLAGS = $(INTERNAL_CFLAGS_BASE) $(GDB_WARN_CFLAGS) INTERNAL_CFLAGS = $(INTERNAL_WARN_CFLAGS) $(GDB_WERROR_CFLAGS) @@ -616,7 +617,8 @@ CLIBS = $(SIM) $(READLINE) $(OPCODES) $(BFD) $(LIBCTF) $(ZLIB) \ @LIBS@ @GUILE_LIBS@ @PYTHON_LIBS@ \ $(LIBEXPAT) $(LIBLZMA) $(LIBBABELTRACE) $(LIBIPT) \ $(WIN32LIBS) $(LIBGNU) $(LIBICONV) \ - $(LIBMPFR) $(SRCHIGH_LIBS) $(LIBXXHASH) $(PTHREAD_LIBS) + $(LIBMPFR) $(SRCHIGH_LIBS) $(LIBXXHASH) $(PTHREAD_LIBS) \ + @DEBUGINFOD_LIBS@ CDEPS = $(NAT_CDEPS) $(SIM) $(BFD) $(READLINE_DEPS) $(LIBCTF) \ $(OPCODES) $(INTL_DEPS) $(LIBIBERTY) $(CONFIG_DEPS) $(LIBGNU) \ $(LIBSUPPORT) @@ -990,6 +992,7 @@ COMMON_SFILES = \ dbxread.c \ dcache.c \ debug.c \ + debuginfod-support.c \ dictionary.c \ disasm.c \ disasm-selftests.c \ diff --git a/gdb/NEWS b/gdb/NEWS index d4e2e70f38..2f9e7756b0 100644 --- a/gdb/NEWS +++ b/gdb/NEWS @@ -3,6 +3,20 @@ *** Changes since GDB 9 +* GDB now supports debuginfod, an HTTP server for distributing ELF/DWARF + debugging information as well as source code. + + When built with debuginfod, GDB can automatically query debuginfod + servers for the separate debug files and source code of the executable + being debugged. + + To build GDB with debuginfod, pass --with-debuginfod to configure (this + requires libdebuginfod, the debuginfod client library). + + debuginfod is distributed with elfutils, starting with version 0.178. + + You can get the latest version from https://sourceware.org/elfutils. + * Debugging MS-Windows processes now sets $_exitsignal when the inferior is terminated by a signal, instead of setting $_exitcode. diff --git a/gdb/README b/gdb/README index be7fdcb65d..840cf1f85c 100644 --- a/gdb/README +++ b/gdb/README @@ -432,6 +432,15 @@ more obscure GDB `configure' options are not listed here. Use the curses library instead of the termcap library, for text-mode terminal operations. +`--with-debuginfod' + Build GDB with libdebuginfod, the debuginfod client library. Used + to automatically fetch source files and separate debug files from + debuginfod servers using the associated executable's build ID. + Enabled by default if libdebuginfod is installed and found at + configure time. debuginfod is packaged with elfutils, starting + with version 0.178. You can get the latest version from + 'https://sourceware.org/elfutils/'. + `--with-libunwind-ia64' Use the libunwind library for unwinding function call stack on ia64 target platforms. diff --git a/gdb/config.in b/gdb/config.in index cb886ba8e1..5939b8e436 100644 --- a/gdb/config.in +++ b/gdb/config.in @@ -227,6 +227,9 @@ /* Define if you have the babeltrace library. */ #undef HAVE_LIBBABELTRACE +/* Define to 1 if debuginfod is enabled. */ +#undef HAVE_LIBDEBUGINFOD + /* Define if you have the expat library. */ #undef HAVE_LIBEXPAT diff --git a/gdb/configure b/gdb/configure index 72ffad8d37..cdacc1dd8e 100755 --- a/gdb/configure +++ b/gdb/configure @@ -719,7 +719,6 @@ GUILE_LIBS GUILE_CPPFLAGS GUILD_TARGET_FLAG GUILD -pkg_config_prog_path HAVE_PYTHON_FALSE HAVE_PYTHON_TRUE PYTHON_LIBS @@ -758,6 +757,9 @@ REPORT_BUGS_TEXI REPORT_BUGS_TO PKGVERSION CODESIGN_CERT +DEBUGINFOD_CFLAGS +DEBUGINFOD_LIBS +pkg_config_prog_path HAVE_NATIVE_GCORE_TARGET TARGET_OBS subdirs @@ -869,6 +871,7 @@ enable_64_bit_bfd enable_gdbmi enable_tui enable_gdbtk +with_debuginfod with_libunwind_ia64 with_curses enable_profiling @@ -1598,6 +1601,8 @@ Optional Packages: [--with-auto-load-dir] --without-auto-load-safe-path do not restrict auto-loaded files locations + --with-debuginfod Enable debuginfo lookups with debuginfod + (auto/yes/no) --with-libunwind-ia64 use libunwind frame unwinding for ia64 targets --with-curses use the curses library instead of the termcap library @@ -6824,8 +6829,94 @@ $as_echo "$as_me: WARNING: gdbtk isn't supported on $host; disabling" >&2;} enable_gdbtk=no ;; esac -# Libunwind support for ia64. +# Handle optional debuginfod support + +# Check whether --with-debuginfod was given. +if test "${with_debuginfod+set}" = set; then : + withval=$with_debuginfod; +else + with_debuginfod=auto +fi +{ $as_echo "$as_me:${as_lineno-$LINENO}: checking whether to use debuginfod" >&5 +$as_echo_n "checking whether to use debuginfod... " >&6; } +{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $with_debuginfod" >&5 +$as_echo "$with_debuginfod" >&6; } + +if test "x$with_debuginfod" != xno; then + # Extract the first word of "pkg-config", so it can be a program name with args. +set dummy pkg-config; ac_word=$2 +{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for $ac_word" >&5 +$as_echo_n "checking for $ac_word... " >&6; } +if ${ac_cv_path_pkg_config_prog_path+:} false; then : + $as_echo_n "(cached) " >&6 +else + case $pkg_config_prog_path in + [\\/]* | ?:[\\/]*) + ac_cv_path_pkg_config_prog_path="$pkg_config_prog_path" # Let the user override the test with a path. + ;; + *) + as_save_IFS=$IFS; IFS=$PATH_SEPARATOR +for as_dir in $PATH +do + IFS=$as_save_IFS + test -z "$as_dir" && as_dir=. + for ac_exec_ext in '' $ac_executable_extensions; do + if as_fn_executable_p "$as_dir/$ac_word$ac_exec_ext"; then + ac_cv_path_pkg_config_prog_path="$as_dir/$ac_word$ac_exec_ext" + $as_echo "$as_me:${as_lineno-$LINENO}: found $as_dir/$ac_word$ac_exec_ext" >&5 + break 2 + fi +done + done +IFS=$as_save_IFS + + test -z "$ac_cv_path_pkg_config_prog_path" && ac_cv_path_pkg_config_prog_path="missing" + ;; +esac +fi +pkg_config_prog_path=$ac_cv_path_pkg_config_prog_path +if test -n "$pkg_config_prog_path"; then + { $as_echo "$as_me:${as_lineno-$LINENO}: result: $pkg_config_prog_path" >&5 +$as_echo "$pkg_config_prog_path" >&6; } +else + { $as_echo "$as_me:${as_lineno-$LINENO}: result: no" >&5 +$as_echo "no" >&6; } +fi + + + if test "${pkg_config_prog_path}" != "missing"; then + ${pkg_config_prog_path} --exists --atleast-version=0.178 libdebuginfod + if test $? = 0; then + DEBUGINFOD_LIBS=`${pkg_config_prog_path} --libs libdebuginfod` + + DEBUGINFOD_CFLAGS=`${pkg_config_prog_path} --cflags libdebuginfod` + + +$as_echo "#define HAVE_LIBDEBUGINFOD 1" >>confdefs.h + + else + if test "x$with_debuginfod" = xyes; then + as_fn_error $? "\"--with-debuginfod was given, but libdebuginfod is missing or unusable.\"" "$LINENO" 5 + else + { $as_echo "$as_me:${as_lineno-$LINENO}: WARNING: libdebuginfod is missing or unusable; some features may be unavailable." >&5 +$as_echo "$as_me: WARNING: libdebuginfod is missing or unusable; some features may be unavailable." >&2;} + fi + fi + else + if test "x$with_debuginfod" = xyes; then + as_fn_error $? "pkg-config missing or unusable; cannot find libdebuginfod" "$LINENO" 5 + else + { $as_echo "$as_me:${as_lineno-$LINENO}: WARNING: pkg-config missing or unusable; cannot find libdebuginfod" >&5 +$as_echo "$as_me: WARNING: pkg-config missing or unusable; cannot find libdebuginfod" >&2;} + fi + fi +else + { $as_echo "$as_me:${as_lineno-$LINENO}: WARNING: debuginfod support disabled; some features may be unavailable." >&5 +$as_echo "$as_me: WARNING: debuginfod support disabled; some features may be unavailable." >&2;} +fi + +# Libunwind support for ia64. # Check whether --with-libunwind-ia64 was given. if test "${with_libunwind_ia64+set}" = set; then : diff --git a/gdb/configure.ac b/gdb/configure.ac index 0ca169101b..e9b3133940 100644 --- a/gdb/configure.ac +++ b/gdb/configure.ac @@ -323,8 +323,40 @@ case $host_os in enable_gdbtk=no ;; esac -# Libunwind support for ia64. +# Handle optional debuginfod support +AC_ARG_WITH([debuginfod], + AC_HELP_STRING([--with-debuginfod], [Enable debuginfo lookups with debuginfod (auto/yes/no)]), + [], [with_debuginfod=auto]) +AC_MSG_CHECKING([whether to use debuginfod]) +AC_MSG_RESULT([$with_debuginfod]) + +if test "x$with_debuginfod" != xno; then + AC_PATH_PROG(pkg_config_prog_path, pkg-config, missing) + if test "${pkg_config_prog_path}" != "missing"; then + ${pkg_config_prog_path} --exists --atleast-version=0.178 libdebuginfod + if test $? = 0; then + AC_SUBST([DEBUGINFOD_LIBS], [`${pkg_config_prog_path} --libs libdebuginfod`]) + AC_SUBST([DEBUGINFOD_CFLAGS], [`${pkg_config_prog_path} --cflags libdebuginfod`]) + AC_DEFINE([HAVE_LIBDEBUGINFOD], [1], [Define to 1 if debuginfod is enabled.]) + else + if test "x$with_debuginfod" = xyes; then + AC_MSG_ERROR(["--with-debuginfod was given, but libdebuginfod is missing or unusable."]) + else + AC_MSG_WARN([libdebuginfod is missing or unusable; some features may be unavailable.]) + fi + fi + else + if test "x$with_debuginfod" = xyes; then + AC_MSG_ERROR([--with-debuginfod was given, but pkg-config missing or unusable; cannot find libdebuginfod]) + else + AC_MSG_WARN([pkg-config missing or unusable; debuginfod support disabled.]) + fi + fi +else + AC_MSG_WARN([debuginfod support disabled; some features may be unavailable.]) +fi +# Libunwind support for ia64. AC_ARG_WITH(libunwind-ia64, AS_HELP_STRING([--with-libunwind-ia64], [use libunwind frame unwinding for ia64 targets]),, diff --git a/gdb/debuginfod-support.c b/gdb/debuginfod-support.c new file mode 100644 index 0000000000..04b55ca1d5 --- /dev/null +++ b/gdb/debuginfod-support.c @@ -0,0 +1,102 @@ +/* debuginfod utilities for GDB. + Copyright (C) 2020 Free Software Foundation, Inc. + + This file is part of GDB. + + 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 . */ + +#include +#include "defs.h" +#include "debuginfod-support.h" + +#ifndef HAVE_LIBDEBUGINFOD +int +debuginfod_source_query (const unsigned char *build_id __attribute__((unused)), + int build_id_len __attribute__((unused)), + const char *srcpath __attribute__((unused)), + char **filename __attribute__((unused))) +{ + return -ENOSYS; +} + +int +debuginfod_debuginfo_query (const unsigned char *build_id __attribute__((unused)), + int build_id_len __attribute__((unused)), + char **filename __attribute__((unused))) +{ + return -ENOSYS; +} +#else +#include + +static int +progressfn (debuginfod_client *c, + long a __attribute__((unused)), + long b __attribute__((unused))) +{ + return check_quit_flag (); +} + +static debuginfod_client * +debuginfod_init () +{ + debuginfod_client *c = debuginfod_begin (); + + if (c != nullptr) + debuginfod_set_progressfn (c, progressfn); + + return c; +} + +/* See debuginfod-support.h */ + +int +debuginfod_source_query (const unsigned char *build_id, + int build_id_len, + const char *srcpath, + char **filename) +{ + debuginfod_client *c = debuginfod_init (); + + if (c == nullptr) + return -ENOMEM; + + int fd = debuginfod_find_source (c, + build_id, + build_id_len, + srcpath, + filename); + debuginfod_end (c); + + return fd; +} + +/* See debuginfod-support.h */ + +int +debuginfod_debuginfo_query (const unsigned char *build_id, + int build_id_len, + char **filename) +{ + debuginfod_client *c = debuginfod_init (); + + if (c == nullptr) + return -ENOMEM; + + int fd = debuginfod_find_debuginfo (c, build_id, build_id_len, filename); + debuginfod_end (c); + + return fd; +} +#endif diff --git a/gdb/debuginfod-support.h b/gdb/debuginfod-support.h new file mode 100644 index 0000000000..e3fdeb4526 --- /dev/null +++ b/gdb/debuginfod-support.h @@ -0,0 +1,43 @@ +/* debuginfod utilities for GDB. + Copyright (C) 2020 Free Software Foundation, Inc. + + This file is part of GDB. + + 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 . */ + +#ifndef DEBUGINFOD_SUPPORT_H +#define DEBUGINFOD_SUPPORT_H + +/* Query debuginfod servers for a source file associated with an + an executable with build_id. src_path should be the source + file's absolute path that includes the compilation directory of + the CU associated with the source file. If the file is + successfully retrieved, its path on the local machine is stored + at filename. The caller should free() this value. If GDB is not + built with debuginfod, this function returns -ENOSYS. */ + +extern int debuginfod_source_query (const unsigned char *build_id, + int build_id_len, + const char *src_path, + char **filename); + +/* Query debuginfod servers for a debuginfo file with build_id. If the + file is successfully retrieved, its path on the local machine is + stored at filename. The caller should free() this value. If GDB + is not built with debuginfod, this function returns -ENOSYS. */ + +extern int debuginfod_debuginfo_query (const unsigned char *build_id, + int build_id_len, + char **filename); +#endif /* DEBUGINFOD_SUPPORT_H */ diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo index a2866a2833..9d2922be34 100644 --- a/gdb/doc/gdb.texinfo +++ b/gdb/doc/gdb.texinfo @@ -37794,6 +37794,14 @@ supported). Use the curses library instead of the termcap library, for text-mode terminal operations. +@item --with-debuginfod +Build @value{GDBN} with libdebuginfod, the debuginfod client library. +Used to automatically fetch source files and separate debug files from +debuginfod servers using the associated executable's build ID. Enabled +by default if libdebuginfod is installed and found at configure time. +debuginfod is packaged with elfutils, starting with version 0.178. You +can get the latest version from `https://sourceware.org/elfutils/'. + @item --with-libunwind-ia64 Use the libunwind library for unwinding function call stack on ia64 target platforms. See http://www.nongnu.org/libunwind/index.html for diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c index dafe01d94a..4121789571 100644 --- a/gdb/dwarf2read.c +++ b/gdb/dwarf2read.c @@ -77,6 +77,9 @@ #include "gdbsupport/selftest.h" #include "rust-lang.h" #include "gdbsupport/pathstuff.h" +#if HAVE_LIBDEBUGINFOD +#include "debuginfod-support.h" +#endif /* When == 1, print basic high level tracing messages. When > 1, be more verbose. @@ -2746,6 +2749,28 @@ dwarf2_get_dwz_file (struct dwarf2_per_objfile *dwarf2_per_objfile) if (dwz_bfd == NULL) dwz_bfd = build_id_to_debug_bfd (buildid_len, buildid); +#if HAVE_LIBDEBUGINFOD + if (dwz_bfd == nullptr) + { + char *alt_filename; + scoped_fd fd (debuginfod_debuginfo_query (buildid, + buildid_len, + &alt_filename)); + + if (fd.get () >= 0) + { + /* File successfully retrieved from server. */ + dwz_bfd = gdb_bfd_open (alt_filename, gnutarget, -1); + + if (dwz_bfd != nullptr + && !build_id_verify (dwz_bfd.get (), buildid_len, buildid)) + dwz_bfd.reset (nullptr); + + xfree (alt_filename); + } + } +#endif /* HAVE_LIBDEBUGINFOD */ + if (dwz_bfd == NULL) error (_("could not find '.gnu_debugaltlink' file for %s"), objfile_name (dwarf2_per_objfile->objfile)); diff --git a/gdb/elfread.c b/gdb/elfread.c index 453bca527e..84496258df 100644 --- a/gdb/elfread.c +++ b/gdb/elfread.c @@ -49,6 +49,10 @@ #include "mdebugread.h" #include "ctfread.h" #include "gdbsupport/gdb_string_view.h" +#include "gdbsupport/scoped_fd.h" +#if HAVE_LIBDEBUGINFOD +#include "debuginfod-support.h" +#endif /* Forward declarations. */ extern const struct sym_fns elf_sym_fns_gdb_index; @@ -1316,8 +1320,33 @@ elf_symfile_read (struct objfile *objfile, symfile_add_flags symfile_flags) symbol_file_add_separate (debug_bfd.get (), debugfile.c_str (), symfile_flags, objfile); } - else - has_dwarf2 = false; + else + { + has_dwarf2 = false; + +#if HAVE_LIBDEBUGINFOD + const struct bfd_build_id *build_id = build_id_bfd_get (objfile->obfd); + + if (build_id != nullptr) + { + char *symfile_path; + scoped_fd fd (debuginfod_debuginfo_query (build_id->data, + build_id->size, + &symfile_path)); + + if (fd.get () >= 0) + { + /* File successfully retrieved from server. */ + gdb_bfd_ref_ptr debug_bfd (symfile_bfd_open (symfile_path)); + + symbol_file_add_separate (debug_bfd.get (), symfile_path, + symfile_flags, objfile); + xfree (symfile_path); + has_dwarf2 = true; + } + } +#endif /* HAVE_LIBDEBUGINFOD */ + } } /* Read the CTF section only if there is no DWARF info. */ diff --git a/gdb/source.c b/gdb/source.c index 327e9c1229..0513ce6655 100644 --- a/gdb/source.c +++ b/gdb/source.c @@ -48,6 +48,10 @@ #include "source-cache.h" #include "cli/cli-style.h" #include "observable.h" +#include "build-id.h" +#ifdef HAVE_LIBDEBUGINFOD +#include "debuginfod-support.h" +#endif #define OPEN_MODE (O_RDONLY | O_BINARY) #define FDOPEN_MODE FOPEN_RB @@ -1153,6 +1157,46 @@ open_source_file (struct symtab *s) s->fullname = NULL; scoped_fd fd = find_and_open_source (s->filename, SYMTAB_DIRNAME (s), &fullname); + +#if HAVE_LIBDEBUGINFOD + if (fd.get () < 0) + { + if (SYMTAB_COMPUNIT (s) != nullptr) + { + const objfile *ofp = COMPUNIT_OBJFILE (SYMTAB_COMPUNIT (s)); + + std::string srcpath; + if (IS_DIR_SEPARATOR (s->filename[0])) + srcpath = s->filename; + else + { + srcpath = SYMTAB_DIRNAME (s); + srcpath += SLASH_STRING; + srcpath += s->filename; + } + + const struct bfd_build_id *build_id = build_id_bfd_get (ofp->obfd); + + if (build_id != nullptr) + { + /* Query debuginfod for the source file. */ + char *filename; + scoped_fd src_fd (debuginfod_source_query (build_id->data, + build_id->size, + srcpath.c_str (), + &filename)); + + if (src_fd.get () >= 0) + fullname.reset (filename); + + s->fullname = fullname.release (); + return src_fd; + + } + } + } +#endif /* HAVE_LIBDEBUGINFOD */ + s->fullname = fullname.release (); return fd; } diff --git a/gdb/testsuite/gdb.debuginfod/fetch_src_and_symbols.exp b/gdb/testsuite/gdb.debuginfod/fetch_src_and_symbols.exp new file mode 100644 index 0000000000..a4042ac93a --- /dev/null +++ b/gdb/testsuite/gdb.debuginfod/fetch_src_and_symbols.exp @@ -0,0 +1,232 @@ +# Copyright 2020 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 . + +# test debuginfod functionality + +set test "debuginfod" + +standard_testfile main.c + +load_lib dwarf.exp + +if { [which debuginfod] == 0 } { + untested "$test (cannot find debuginfod)" + return -1 +} + +if { [which curl] == 0 } { + untested "$test (cannot find curl)" + return -1 +} + +# skip testing if gdb was not configured with debuginfod +if { [string first "with-debuginfod" [exec $GDB --configuration]] == -1 } { + untested "$test (gdb not configured with debuginfod)" + return -1 +} + +set cache [file join [standard_output_file {}] ".client_cache"] +set db [file join [standard_output_file {}] ".debuginfod.db"] + +# make sure there isn't an old client cache or db lying around +file delete -force $cache +file delete -force $db + +# make a copy source file that we can move around +if { [catch {file copy -force ${srcdir}/${subdir}/${srcfile} \ + [standard_output_file tmp-${srcfile}]}] != 0 } { + error "$test (create temporary file)" + return -1 +} + +set sourcetmp [standard_output_file tmp-${srcfile}] +set outputdir [standard_output_file {}] + +if { [gdb_compile "$sourcetmp" "$binfile" executable {debug}] != "" } { + fail "$test (compile)" + return -1 +} + +# Find an unused port +set port [exec sh -c "while true; do PORT=`expr '(' \$RANDOM % 1000 ')' + 9000`; ss -atn | fgrep \":\$PORT\" || break; done; echo \$PORT"] + +setenv DEBUGINFOD_URLS "" +setenv DEBUGINFOD_TIMEOUT 30 +setenv DEBUGINFOD_CACHE_PATH $cache + +# test that gdb cannot find source without debuginfod +gdb_start +gdb_load $binfile +gdb_test_no_output "set substitute-path $outputdir /dev/null" +gdb_test "list" ".*No such file or directory.*" +gdb_exit + +# strip symbols into separate file and move it so gdb cannot find it without debuginfod +if { [gdb_gnu_strip_debug $binfile ""] != 0 } { + fail "$test (strip debuginfo)" + return -1 +} + +set debugdir [file join [standard_output_file {}] "debug"] +set debuginfo [file join [standard_output_file {}] "fetch_src_and_symbols.debug"] + +file mkdir $debugdir +file rename -force $debuginfo $debugdir + +# test that gdb cannot find symbols without debuginfod +gdb_start +gdb_load $binfile +gdb_test "file" ".*No symbol file.*" +gdb_exit + +# Write some assembly that just has a .gnu_debugaltlink section. +# Copied from testsuite/gdb.dwarf2/dwzbuildid.exp. +proc write_just_debugaltlink {filename dwzname buildid} { + set asm_file [standard_output_file $filename] + + Dwarf::assemble $asm_file { + upvar dwzname dwzname + upvar buildid buildid + + gnu_debugaltlink $dwzname $buildid + + # Only the DWARF reader checks .gnu_debugaltlink, so make sure + # there is a bit of DWARF in here. + cu {} { + compile_unit {{language @DW_LANG_C}} { + } + } + } +} + +# Write some DWARF that also sets the buildid. +# Copied from testsuite/gdb.dwarf2/dwzbuildid.exp. +proc write_dwarf_file {filename buildid {value 99}} { + set asm_file [standard_output_file $filename] + + Dwarf::assemble $asm_file { + declare_labels int_label int_label2 + + upvar buildid buildid + upvar value value + + build_id $buildid + + cu {} { + compile_unit {{language @DW_LANG_C}} { + int_label2: base_type { + {name int} + {byte_size 4 sdata} + {encoding @DW_ATE_signed} + } + + constant { + {name the_int} + {type :$int_label2} + {const_value $value data1} + } + } + } + } +} + +set buildid "01234567890abcdef0123456" + +write_just_debugaltlink ${binfile}_has_altlink.S ${binfile}_dwz.o $buildid +write_dwarf_file ${binfile}_dwz.S $buildid + +if {[gdb_compile ${binfile}_has_altlink.S ${binfile}_alt.o object nodebug] != ""} { + fail "$test (compile main with altlink)" + return -1 +} + +if {[gdb_compile ${binfile}_dwz.S ${binfile}_dwz.o object nodebug] != ""} { + fail "$test (compile altlink)" + return -1 +} + +file rename -force ${binfile}_dwz.o $debugdir + +# Test that gdb cannot find dwz without debuginfod. +gdb_start +gdb_test "file ${binfile}_alt.o" ".*could not find '.gnu_debugaltlink'.*" +gdb_exit + +set debuginfod_pid 0 + +# Kill the server if we abort early +proc sigint_handler {} { + global debuginfod_pid + + if { $debuginfod_pid != 0 } { + catch {exec kill -INT $debuginfod_pid} + } + + exit +} + +trap sigint_handler INT + +# Start up debuginfod +set debuginfod_pid [exec debuginfod -d $db -p $port -F $debugdir 2>/dev/null &] + +if { !$debuginfod_pid } { + fail "$test (server init)" + return -1 +} + +set metrics [list "ready 1" \ + "thread_work_total{role=\"traverse\"} 1" \ + "thread_work_pending{role=\"scan\"} 0" \ + "thread_busy{role=\"scan\"} 0"] + +# Check server metrics to confirm init has completed. +foreach m $metrics { + set timelim 20 + while { $timelim != 0 } { + sleep 0.5 + catch {exec curl -s http://127.0.0.1:$port/metrics} got + + if { [regexp $m $got] } { + break + } + + incr timelim -1 + } + + if { $timelim == 0 } { + fail "$test (server init timeout)" + catch {exec kill -INT $debuginfod_pid} + return -1 + } +} + +# Point the client to the server +setenv DEBUGINFOD_URLS http://127.0.0.1:$port + +# gdb should now find the symbol and source files +gdb_start +gdb_load $binfile +gdb_test_no_output "set substitute-path $outputdir /dev/null" +gdb_test "br main" "Breakpoint 1 at.*file.*" +gdb_test "l" ".*This program is distributed in the hope.*" +gdb_exit + +# gdb should now find the debugaltlink file +gdb_start +gdb_test "file ${binfile}_alt.o" ".*Reading symbols from ${binfile}_alt.o\.\.\.\[\r\n\]" +gdb_exit + +catch {exec kill -INT $debuginfod_pid} diff --git a/gdb/testsuite/gdb.debuginfod/main.c b/gdb/testsuite/gdb.debuginfod/main.c new file mode 100644 index 0000000000..73abaf58b1 --- /dev/null +++ b/gdb/testsuite/gdb.debuginfod/main.c @@ -0,0 +1,25 @@ +/* This testcase is part of GDB, the GNU debugger. + + Copyright 2020 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 . */ + +/* Dummy main function. */ + +int +main() +{ + asm ("main_label: .globl main_label"); + return 0; +} diff --git a/gdb/top.c b/gdb/top.c index f702af9acd..1a98ae198c 100644 --- a/gdb/top.c +++ b/gdb/top.c @@ -1528,6 +1528,16 @@ This GDB was configured as follows:\n\ ")); #endif +#if HAVE_LIBDEBUGINFOD + fprintf_filtered (stream, _("\ + --with-debuginfod\n\ +")); +#else + fprintf_filtered (stream, _("\ + --without-debuginfod\n\ +")); +#endif + #if HAVE_GUILE fprintf_filtered (stream, _("\ --with-guile\n\ -- 2.24.1