From patchwork Thu Jun 1 22:35:04 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Aaron Merey X-Patchwork-Id: 70480 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 41DB33858404 for ; Thu, 1 Jun 2023 22:36:38 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 41DB33858404 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1685658998; bh=jbPwus2vL4cwV2wE0ePXyXAyhDjmS/eaJGhoSXRzNy4=; h=To:Cc:Subject:Date:In-Reply-To:References:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From:Reply-To:From; b=tAQldKq5F1GgOCEwI/QesZZBmyAqgQSgB73jxb/kuvV5+NaykIokhYtVfUUTucn8+ 7Ccvo1jyWdFodJ2bJ0HNjDgQVmokzxNoQCTS3J4iNFl2yG5da6crinEjZy1KC4sInd PUbaunaMYnKDYONqUDfGXZXEXLBAip2TWNqRwlf8= X-Original-To: gdb-patches@sourceware.org Delivered-To: gdb-patches@sourceware.org Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by sourceware.org (Postfix) with ESMTPS id A09B73858C50 for ; Thu, 1 Jun 2023 22:35:28 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org A09B73858C50 Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-639-LnNLxA0EMtqDjnK0c_EQJQ-1; Thu, 01 Jun 2023 18:35:25 -0400 X-MC-Unique: LnNLxA0EMtqDjnK0c_EQJQ-1 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.rdu2.redhat.com [10.11.54.8]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id E2794185A78B; Thu, 1 Jun 2023 22:35:24 +0000 (UTC) Received: from localhost.localdomain (unknown [10.22.32.18]) by smtp.corp.redhat.com (Postfix) with ESMTP id 93160C154D7; Thu, 1 Jun 2023 22:35:24 +0000 (UTC) To: eliz@gnu.org Cc: aburgess@redhat.com, gdb-patches@sourceware.org, Aaron Merey Subject: [PATCH 2/6 v2] gdb: Add command 'maint set/show debuginfod download-sections' Date: Thu, 1 Jun 2023 18:35:04 -0400 Message-Id: <20230601223504.107197-1-amerey@redhat.com> In-Reply-To: <83bkhzemlr.fsf@gnu.org> References: <83bkhzemlr.fsf@gnu.org> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.8 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com X-Spam-Status: No, score=-11.3 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_NONE, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-Patchwork-Original-From: Aaron Merey via Gdb-patches From: Aaron Merey Reply-To: Aaron Merey Errors-To: gdb-patches-bounces+patchwork=sourceware.org@sourceware.org Sender: "Gdb-patches" Thanks Eli, I incorporated your suggestions. Aaron Reviewed-By: Eli Zaretskii --- v1: https://sourceware.org/pipermail/gdb-patches/2023-February/197461.html v2 removes 'set debuginfod enabled lazy' and replaces it with 'maint set debuginfod download-sections'. Commit message: This setting controls whether GDB may attempt to download ELF/DWARF sections from debuginfod servers. This setting is enabled by default if gdb is built with debuginfod section download support (requires libdebuginfod 0.188). Also update debuginfod command help text and gdb.texinfo with information on section downloading and the new command. Reviewed-By: Eli Zaretskii --- gdb/debuginfod-support.c | 65 ++++++++++++++++++++++++++++++++++++++-- gdb/doc/gdb.texinfo | 18 +++++++++-- 2 files changed, 78 insertions(+), 5 deletions(-) diff --git a/gdb/debuginfod-support.c b/gdb/debuginfod-support.c index 8be43a91dcc..66b58e8b673 100644 --- a/gdb/debuginfod-support.c +++ b/gdb/debuginfod-support.c @@ -31,6 +31,10 @@ static cmd_list_element *set_debuginfod_prefix_list; static cmd_list_element *show_debuginfod_prefix_list; +/* maint set/show debuginfod commands. */ +static cmd_list_element *maint_set_debuginfod_cmdlist; +static cmd_list_element *maint_show_debuginfod_cmdlist; + static const char debuginfod_on[] = "on"; static const char debuginfod_off[] = "off"; static const char debuginfod_ask[] = "ask"; @@ -50,6 +54,13 @@ static const char *debuginfod_enabled = debuginfod_off; #endif +static bool debuginfod_download_sections = +#if defined(HAVE_LIBDEBUGINFOD_FIND_SECTION) + true; +#else + false; +#endif + static unsigned int debuginfod_verbose = 1; #ifndef HAVE_LIBDEBUGINFOD @@ -424,7 +435,7 @@ debuginfod_section_query (const unsigned char *build_id, return scoped_fd (-ENOSYS); #else - if (!debuginfod_is_enabled ()) + if (!debuginfod_download_sections || !debuginfod_is_enabled ()) return scoped_fd (-ENOSYS); debuginfod_client *c = get_debuginfod_client (); @@ -550,6 +561,31 @@ show_debuginfod_verbose_command (ui_file *file, int from_tty, value); } +/* Set callback for "maint set debuginfod download-sections". */ + +static void +maint_set_debuginfod_download_sections (bool value) +{ +#if !defined(HAVE_LIBDEBUGINFOD_FIND_SECTION) + if (value) + { + error (_("Support for section downloading is not compiled into GDB. " \ +"Defaulting to \"off\".")); + return; + } +#endif + + debuginfod_download_sections = value; +} + +/* Get callback for "maint set debuginfod download-sections". */ + +static bool +maint_get_debuginfod_download_sections () +{ + return debuginfod_download_sections; +} + /* Register debuginfod commands. */ void _initialize_debuginfod (); @@ -568,8 +604,10 @@ _initialize_debuginfod () _("Set whether to use debuginfod."), _("Show whether to use debuginfod."), _("\ -When on, enable the use of debuginfod to download missing debug info and\n\ -source files."), +When set to \"on\", enable the use of debuginfod to download missing\n\ +debug info and source files. GDB may also download components of debug\n\ +info instead of entire files. \"off\" disables the use of debuginfod.\n\ +When set to \"ask\", prompt whether to enable or disable debuginfod." ), set_debuginfod_enabled, get_debuginfod_enabled, show_debuginfod_enabled, @@ -600,4 +638,25 @@ query.\nTo disable, set to zero. Verbose output is displayed by default."), show_debuginfod_verbose_command, &set_debuginfod_prefix_list, &show_debuginfod_prefix_list); + + /* maint set/show debuginfod */ + add_setshow_prefix_cmd ("debuginfod", class_maintenance, + _("Set debuginfod specific variables."), + _("Show debuginfod specific variables."), + &maint_set_debuginfod_cmdlist, + &maint_show_debuginfod_cmdlist, + &maintenance_set_cmdlist, &maintenance_show_cmdlist); + + /* maint set/show debuginfod download-sections */ + add_setshow_boolean_cmd ("download-sections", class_maintenance, _("\ +Set whether debuginfod may download individual ELF/DWARF sections."), _("\ +Show whether debuginfod may download individual ELF/DWARF sections."), _("\ +When enabled, debuginfod may attempt to download individual ELF/DWARF \ +sections from debug info files.\nIf disabled, only whole debug info files \ +may be downloaded."), + maint_set_debuginfod_download_sections, + maint_get_debuginfod_download_sections, + nullptr, + &maint_set_debuginfod_cmdlist, + &maint_show_debuginfod_cmdlist); } diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo index fc55c4e7b43..c717e5a5b23 100644 --- a/gdb/doc/gdb.texinfo +++ b/gdb/doc/gdb.texinfo @@ -40872,6 +40872,16 @@ Create a core file? (y or n) @kbd{n} (@value{GDBP}) @end smallexample +@kindex maint set debuginfod +@kindex maint show debuginfod +@item maint set debuginfod download-sections +@itemx maint set debuginfod download-sections @r{[}on|off@r{]} +@itemx maint show debuginfod download-sections +Controls whether @value{GDBN} will attempt to download individual +ELF/DWARF sections from @code{debuginfod}. If disabled, only +whole debug info files will be downloaded; this could result +in @value{GDBN} downloading larger amounts of data. + @cindex @value{GDBN} internal error @cindex internal errors, control of @value{GDBN} behavior @cindex demangler crashes @@ -49382,8 +49392,12 @@ regarding @code{debuginfod}. @item set debuginfod enabled @itemx set debuginfod enabled on @cindex enable debuginfod -@value{GDBN} will attempt to query @code{debuginfod} servers when missing debug -info or source files. +@value{GDBN} may query @code{debuginfod} servers for missing debug info and +source files. @value{GDBN} may also download individual ELF/DWARF sections +such as @code{.gdb_index} to help reduce the total amount of data downloaded +from @code{debuginfod} servers; this can be controlled by @w{@code{maint +set debuginfod download-sections}} (@pxref{Maintenance Commands, maint set +debuginfod download-sections}). @item set debuginfod enabled off @value{GDBN} will not attempt to query @code{debuginfod} servers when missing