From patchwork Wed Oct 9 22:07:42 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Aaron Merey X-Patchwork-Id: 98610 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 2AC80385E836 for ; Wed, 9 Oct 2024 22:08:27 +0000 (GMT) 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 ESMTP id F3C583858417 for ; Wed, 9 Oct 2024 22:07:58 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org F3C583858417 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=redhat.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=redhat.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org F3C583858417 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=170.10.129.124 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1728511681; cv=none; b=qlrROVBC39icXnRpgzMDlXQyPoUvp9rnb1XHSBYvY4bv94/wGTFmsyhM8vvZoHLdJgsH6MbnKtZhMiy0BqDdtdvEd4mlYEHgP4S1RlOg/7RRq4xlrnwWDvcCCwsmueyjhUvu9cV3cd+rk45Gx7LrMgKSDhWEIdXunAzFQi+xAFg= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1728511681; c=relaxed/simple; bh=A0FheFjJj7NvQKJqZRwmmTYLTwiiavBs8Sl8wT++Hfk=; h=DKIM-Signature:From:To:Subject:Date:Message-ID:MIME-Version; b=Yjsu7IqaPu5hVOd7qLg9LXQ2v+MPKrs7p6trYhQTjoBvOuQGc/ypnSAQzCJZkR9HP8g0cKpTvKDIQ5P+wLpgHBA4UUxyGgJ7XPD7qo9PysZowQB91mxYxzjiL0iIYbQ6amHlRBSzo6VuAHkcFzMQGzJyJsfKQD67umXUXoFOw8A= ARC-Authentication-Results: i=1; server2.sourceware.org DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1728511678; 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: content-transfer-encoding:content-transfer-encoding; bh=kFEITptln7JIj8a5/+AiO6YPS30OdMNBQe3rR+7y5ws=; b=CEUOw/+KcElBM+cJPJqU+CHb6fuQb6+GW3n6XEvQJmcxZ+b1E8LEHDHEcVVNVvfJjYO+ew 7HgO2zgD8Koy3wsXWcSzqgAnTYTjWENjJ6riSW7fiXVk7Lr7C29/ndUiV4zh0yXdDuWipD nJI1E9aApILWeX8nOmV9IDQS7XXO6X4= Received: from mx-prod-mc-04.mail-002.prod.us-west-2.aws.redhat.com (ec2-54-186-198-63.us-west-2.compute.amazonaws.com [54.186.198.63]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-453-MkNOHSlnMJqZrxYucGGtow-1; Wed, 09 Oct 2024 18:07:55 -0400 X-MC-Unique: MkNOHSlnMJqZrxYucGGtow-1 Received: from mx-prod-int-01.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-01.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-04.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 7F13A19560A3; Wed, 9 Oct 2024 22:07:54 +0000 (UTC) Received: from fedora.redhat.com (unknown [10.22.64.3]) by mx-prod-int-01.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTP id 274C1300018D; Wed, 9 Oct 2024 22:07:52 +0000 (UTC) From: Aaron Merey To: gdb-patches@sourceware.org Cc: Aaron Merey , =?utf-8?q?Martin_Li=C5=A1ka?= , Tom de Vries Subject: [PATCH v5] gdb/debuginfod: Ctrl-C asks to cancel further downloads Date: Wed, 9 Oct 2024 18:07:42 -0400 Message-ID: <20241009220742.160505-1-amerey@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.4.1 on 10.30.177.4 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com X-Spam-Status: No, score=-11.7 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, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_NONE, TXREP 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.30 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: gdb-patches-bounces~patchwork=sourceware.org@sourceware.org Currently gdb only allows for debuginfod downloads to be cancelled one at a time via Ctrl-C. This can be a burden if one wishes to cancel a large series of downloads. Additionally there can also be ambiguity between whether Ctrl-C during a download was intended to cancel a download or interrupt the inferior. This patch addresses these issues by adding a new debuginfod setting and changing the behavior of Ctrl-C during a download. Add a new command "set debuginfod cancel one/all/ask", where: - "one" means Ctrl-C cancels one download, - "all" means Ctrl-C cancels all further downloads, and - "ask" means Ctrl-C asks whether to cancel all further downloads. A "yes" implies "set debuginfod cancel all", and a "no" implies "set debuginfod cancel one", so the question is only asked once. Note that the behaviour as it was before this patch is equivalent to "set debuginfod cancel one". Instead, the new default is "set debuginfod cancel ask". Note that cancelling all further downloads implies "set debuginfod enabled off". A single Ctrl-C during downloading now sets the quit_flag and proceeds with all downloads. If the inferior has the terminal, then a second Ctrl-C during downloading triggers a query asking whether to cancel the download or interrupt the inferior. If the user wishes to cancel the download then the setting of 'set debuginfod cancel' determines whether one or all downloads are cancelled. In the case of "set debuginfod cancel ask", there will be another query at this point asking whether to cancel one or all downloads. If the inferior does not have the terminal, then a second Ctrl-C during downloading simply cancels the download according to the setting of "set debuginfod cancel". In this case there is no query asking whether to interrupt the inferior or cancel a download. Example session where inferior has terminal: (gdb) run [...] Downloading separate debug info for /lib64/libxyz.so [### ]^C^C Cancel the current download? If no, then Ctrl-C will be sent to the target process. ([y] or n) y Cancelling download of separate debug info for /lib64/libxyz.so... Cancel further downloading for this session? (y or [n]) n Downloading separate debug info for /lib64/libabcd.so Example session where inferior does not have terminal: (gdb) run [...] Downloading separate debug info for /lib64/libxyz.so [### ]^C^C Cancelling download of separate debug info for /lib64/libxyz.so... Cancel further downloading for this session? (y or [n]) y Debuginfod has been disabled. To re-enable use the 'set debuginfod enabled' command. Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=29582 Suggested-By: Martin Liška Co-Authored-By: Tom de Vries --- Eli has already approved the documentation parts of the patch. This patch has previously been posted [1]. I'm reposting here in a separate thread to make it more clear what the current up-to-date version of this patch is. Aaron [1] https://sourceware.org/pipermail/gdb-patches/2024-June/209896.html gdb/debuginfod-support.c | 174 +++++++++++++++++++++++++++++++++++++-- gdb/doc/gdb.texinfo | 21 +++++ 2 files changed, 189 insertions(+), 6 deletions(-) diff --git a/gdb/debuginfod-support.c b/gdb/debuginfod-support.c index 841b6f2078c..28cedf6b7a8 100644 --- a/gdb/debuginfod-support.c +++ b/gdb/debuginfod-support.c @@ -37,6 +37,8 @@ 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"; +static const char debuginfod_one[] = "one"; +static const char debuginfod_all[] = "all"; static const char *debuginfod_enabled_enum[] = { @@ -46,6 +48,20 @@ static const char *debuginfod_enabled_enum[] = nullptr }; +/* Valid values for set debuginfod cancel command. */ + +static const char *debuginfod_cancel_enum[] = +{ + debuginfod_one, + debuginfod_all, + debuginfod_ask, + nullptr +}; + +/* Value of debuginfod cancellation mode. */ + +static const char *debuginfod_cancel = debuginfod_ask; + static const char *debuginfod_enabled = #if defined(HAVE_LIBDEBUGINFOD) debuginfod_ask; @@ -108,9 +124,17 @@ debuginfod_section_query (const unsigned char *build_id, struct user_data { user_data (const char *desc, const char *fname) - : desc (desc), fname (fname) - { } + : pass_quit_flag (false), desc (desc), fname (fname) + { + /* Record whether the current inferior owns the terminal. */ + if (!target_terminal::is_ours ()) + inf_had_term = true; + else + inf_had_term = false; + } + bool pass_quit_flag; + bool inf_had_term; const char * const desc; const char * const fname; ui_out::progress_update progress; @@ -155,10 +179,84 @@ progressfn (debuginfod_client *c, long cur, long total) if (check_quit_flag ()) { ui_file *outstream = get_unbuffered (gdb_stdout); - gdb_printf (outstream, _("Cancelling download of %s %s...\n"), - data->desc, styled_fname.c_str ()); - return 1; - } + + /* If a single Ctrl-C occurs during downloading, let it propagate to the + target. If more than one Ctrl-C occurs, ask whether to cancel the + current download or interrupt the target. If the download is + cancelled, the setting of debuginfod_cancel will determine whether + the current download is cancelled or debuginfod is disabled. */ + if (!data->pass_quit_flag) + data->pass_quit_flag = true; + else + { + int resp = 1; + bool extra_nl = false; + + if (data->inf_had_term) + { + /* If Ctrl-C occurs during the following prompts, catch the + exception to prevent unsafe early returns to gdb's main + event loop. During these prompts, Ctrl-C is equivalent to + answering 'y'. */ + try + { + resp = yquery (_("Cancel the current download?\nIf no, " + "then Ctrl-C will be sent to the target " + "process. ")); + } + catch (const gdb_exception &) + { + /* If the query doesn't complete, then we need an additional + newline to get "Cancelling download of..." printed on a + separate line. */ + extra_nl = true; + } + } + if (resp) + { + if (extra_nl) + { + gdb_printf (outstream, "\n"); + extra_nl = false; + } + + gdb_printf (outstream, _("Cancelling download of %s %s...\n"), + data->desc, styled_fname.c_str ()); + if (debuginfod_cancel == debuginfod_ask) + { + try + { + resp = nquery + (_("Cancel further downloading for this session? ")); + } + catch (const gdb_exception &) + { + resp = 1; + extra_nl = true; + } + + if (resp) + debuginfod_cancel = debuginfod_all; + else + debuginfod_cancel = debuginfod_one; + } + if (debuginfod_cancel == debuginfod_all) + { + if (extra_nl) + gdb_printf (outstream, "\n"); + + gdb_printf (outstream, + _("Interrupted -- debuginfod will be disabled.\n" + "To re-enable use the 'set debuginfod " + "enabled' command.\n")); + debuginfod_enabled = debuginfod_off; + } + + data->pass_quit_flag = false; + return 1; + } + } + } if (debuginfod_verbose == 0) return 0; @@ -334,6 +432,11 @@ debuginfod_source_query (const unsigned char *build_id, build_id_len, srcpath, &dname)); + if (data.pass_quit_flag) + set_quit_flag (); + if (data.inf_had_term && term_state.has_value ()) + target_terminal::inferior (); + debuginfod_set_user_data (c, nullptr); } @@ -377,6 +480,11 @@ debuginfod_debuginfo_query (const unsigned char *build_id, fd = scoped_fd (debuginfod_find_debuginfo (c, build_id, build_id_len, &dname)); + if (data.pass_quit_flag) + set_quit_flag (); + if (data.inf_had_term && term_state.has_value ()) + target_terminal::inferior (); + debuginfod_set_user_data (c, nullptr); } @@ -420,6 +528,11 @@ debuginfod_exec_query (const unsigned char *build_id, fd = scoped_fd (debuginfod_find_executable (c, build_id, build_id_len, &dname)); + if (data.pass_quit_flag) + set_quit_flag (); + if (data.inf_had_term && term_state.has_value ()) + target_terminal::inferior (); + debuginfod_set_user_data (c, nullptr); } @@ -468,6 +581,11 @@ debuginfod_section_query (const unsigned char *build_id, fd = scoped_fd (debuginfod_find_section (c, build_id, build_id_len, section_name, &dname)); + if (data.pass_quit_flag) + set_quit_flag (); + if (data.inf_had_term && term_state.has_value ()) + target_terminal::inferior (); + debuginfod_set_user_data (c, nullptr); } @@ -516,6 +634,33 @@ show_debuginfod_enabled (ui_file *file, int from_tty, cmd_list_element *cmd, "\"%s\".\n"), debuginfod_enabled); } +/* Set callback for "set debuginfod cancel". */ + +static void +set_debuginfod_cancel (const char *value) +{ + debuginfod_cancel = value; +} + +/* Get callback for "set debuginfod cancel". */ + +static const char * +get_debuginfod_cancel () +{ + return debuginfod_cancel; +} + +/* Show callback for "set debuginfod cancel". */ + +static void +show_debuginfod_cancel (ui_file *file, int from_tty, cmd_list_element *cmd, + const char *value) +{ + gdb_printf (file, + _("Debuginfod cancellation mode is currently set to " + "\"%s\".\n"), debuginfod_cancel); +} + /* Set callback for "set debuginfod urls". */ static void @@ -620,6 +765,23 @@ When set to \"ask\", prompt whether to enable or disable debuginfod." ), &set_debuginfod_prefix_list, &show_debuginfod_prefix_list); + add_setshow_enum_cmd ("cancel", class_run, debuginfod_cancel_enum, + _("Set Ctrl-C behaviour for debuginfod."), + _("Show Ctrl-C behaviour for debuginfod."), + _("\ +When set to \'one\', pressing Ctrl-C twice cancels a single \ +download.\nWhen set to \'all\', pressing Ctrl-C twice cancels all further downloads.\n\ +When set to \'ask\', pressing Ctrl-C twice asks what to do.\nA single Ctrl-C during \ +downloading is passed to the target process being debugged.\nA second Ctrl-C \ +during downloading may raise a prompt asking whether to cancel the download or \ +send Ctrl-C to the target.\nIf the download is cancelled, then no Ctrl-C is \ +sent to the target."), + set_debuginfod_cancel, + get_debuginfod_cancel, + show_debuginfod_cancel, + &set_debuginfod_prefix_list, + &show_debuginfod_prefix_list); + /* set/show debuginfod urls */ add_setshow_string_noescape_cmd ("urls", class_run, _("\ Set the list of debuginfod server URLs."), _("\ diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo index 46ca62ec0c3..4a2f2f16b33 100644 --- a/gdb/doc/gdb.texinfo +++ b/gdb/doc/gdb.texinfo @@ -50502,6 +50502,27 @@ is set to @code{ask} for interactive sessions. Display whether @code{debuginfod enabled} is set to @code{on}, @code{off} or @code{ask}. +@kindex set debuginfod cancel +@anchor{set debuginfod cancel} +@item set debuginfod cancel +@itemx set debuginfod cancel one +@cindex debuginfod @kbd{Ctrl-C} behaviour +Pressing @kbd{Ctrl-C} twice during downloading will cancel the current +download. + +@item set debuginfod cancel all +Pressing @kbd{Ctrl-C} twice during downloading will cancel this and all +further downloads. + +@item set debuginfod cancel ask +Pressing @kbd{Ctrl-C} twice during downloading will cancel the current +download and prompt whether to cancel all further downloads. By default, +@code{debuginfod cancel} is set to @code{ask} for interactive sessions. + +@kindex show debuginfod cancel +@item show debuginfod cancel +Display the current setting of @code{debuginfod cancel}. + @kindex set debuginfod urls @cindex configure debuginfod URLs @item set debuginfod urls