From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm1-f46.google.com (mail-wm1-f46.google.com [209.85.128.46]) by sourceware.org (Postfix) with ESMTPS id EA38F384D1BE for ; Fri, 24 Jun 2022 14:44:05 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org EA38F384D1BE Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=palves.net Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-wm1-f46.google.com with SMTP id o16-20020a05600c379000b003a02eaea815so2423009wmr.0 for ; Fri, 24 Jun 2022 07:44:05 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:subject:date:message-id:mime-version :content-transfer-encoding; bh=SDY0FjRmysO610+Fbizjuvg3I+Z82GQAQpQnhXM+VZc=; b=KoHtuJZ6ISwR1gZHGLyCsU7Qetc7yhbZwrHKbw/QEhiiPUi8TY0hlTcwwdFaCUuxxG MBfcTx2VqpxGrNO4Fit+fY/qAPoFW67wtVlEFYnIQPILtBFST0x+J/idgxShUK6A2hRt HEnkqy3tsj4YFMMWnwrNTFcX8WC/Y1DaV4Sw/VJT3HHQKczxd9RiZXCSVimhb6RaZ0FV ynhxR/+ilXyJNwIWrD6g5fbEalNzmSN0JZ4fGDu8WXc7J02IbTftB7iRh2lWuN44MmY0 dnAAwx7zcWpJoYSiFssKLBq88ZXaMfCMYERIFsGvqCDQ5PKkIPTo9ci2NqhHDRfR1pIr B66A== X-Gm-Message-State: AJIora91v09jMxnGcLFlO6TS3HYR5EUfWrG9x/oSZkzlyqEYME9v8T/c Y33ptw416tYvksSByAPPeqBl39efNfE= X-Google-Smtp-Source: AGRyM1sUk214/onBc3AZU+tUIi/peniKWjjXXk8eZu0Hh4j/flU45uPrxieukEy7pby8duWTH3tWww== X-Received: by 2002:a05:600c:1991:b0:39c:88ba:2869 with SMTP id t17-20020a05600c199100b0039c88ba2869mr4045764wmq.14.1656081844134; Fri, 24 Jun 2022 07:44:04 -0700 (PDT) Received: from localhost ([2001:8a0:f924:2600:209d:85e2:409e:8726]) by smtp.gmail.com with ESMTPSA id m34-20020a05600c3b2200b003a03ae64f57sm2912955wms.8.2022.06.24.07.44.02 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 24 Jun 2022 07:44:02 -0700 (PDT) From: Pedro Alves To: gdb-patches@sourceware.org Subject: [PATCH] Always emit =thread-exited notifications, even if silent Date: Fri, 24 Jun 2022 15:44:02 +0100 Message-Id: <20220624144402.1189751-1-pedro@palves.net> X-Mailer: git-send-email 2.36.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-9.9 required=5.0 tests=BAYES_00, FREEMAIL_FORGED_FROMDOMAIN, FREEMAIL_FROM, GIT_PATCH_0, HEADER_FROM_DIFFERENT_DOMAINS, KAM_DMARC_STATUS, KAM_SHORT, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2, SPF_HELO_NONE, SPF_PASS, 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-List-Received-Date: Fri, 24 Jun 2022 14:44:08 -0000 [Note: the testcased added by this commit depends on https://sourceware.org/pipermail/gdb-patches/2022-June/190259.html, otherwise GDB just crashes when detaching the core] Currently, in MI, =thread-created are always emitted, like: =thread-group-started,id="i1",pid="195680" =thread-created,id="1",group-id="i1" ... but on teardown, if the target uses exit_inferior_silent, then you'll only see the inferior exit notification (thread-group-exited), no notification for threads. The core target is one of the few targets that use exit_inferior_silent. Here's an example session: -target-select core $corefile =thread-group-started,id="i1",pid="195680" =thread-created,id="1",group-id="i1" ... ^connected,frame=.... (gdb) -target-detach =thread-group-exited,id="i1" ^done (gdb) This imbalance of emitting =thread-created but then not =thread-exited seems off to me. (And, it complicates changes I want to do to centralize emitting thread exit notifications for the CLI, which is why I'm looking at this.) And then, since most other targets use exit_inferior instead of exit_inferior_silent, MI is already emitting =thread-exited notifications when tearing down an inferior, for most targets. This commit makes MI always emit the =thread-exited notifications, even for exit_inferior_silent. Afterwards, when debugging a core, MI outputs: (gdb) -target-detach =thread-exited,id="1",group-id="i1" << new line =thread-group-exited,id="i1" ^done (gdb) Surprisingly, there's no MI testcase debugging a core file. This commit adds the first. Change-Id: I5100501a46f07b6bbad3e04d120c2562a51c93a4 --- gdb/mi/mi-interp.c | 3 -- gdb/testsuite/gdb.mi/mi-corefile.c | 27 +++++++++++ gdb/testsuite/gdb.mi/mi-corefile.exp | 69 ++++++++++++++++++++++++++++ 3 files changed, 96 insertions(+), 3 deletions(-) create mode 100644 gdb/testsuite/gdb.mi/mi-corefile.c create mode 100644 gdb/testsuite/gdb.mi/mi-corefile.exp diff --git a/gdb/mi/mi-interp.c b/gdb/mi/mi-interp.c index 8d6e0334a90..937a3d9d6f5 100644 --- a/gdb/mi/mi-interp.c +++ b/gdb/mi/mi-interp.c @@ -358,9 +358,6 @@ mi_new_thread (struct thread_info *t) static void mi_thread_exit (struct thread_info *t, int silent) { - if (silent) - return; - SWITCH_THRU_ALL_UIS () { struct mi_interp *mi = as_mi_interp (top_level_interpreter ()); diff --git a/gdb/testsuite/gdb.mi/mi-corefile.c b/gdb/testsuite/gdb.mi/mi-corefile.c new file mode 100644 index 00000000000..58e6f35a61a --- /dev/null +++ b/gdb/testsuite/gdb.mi/mi-corefile.c @@ -0,0 +1,27 @@ +/* Copyright 2022 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 . */ + +/* Simple little program that just generates a core dump. */ + +#include + +int +main (int argc, char **argv) +{ + abort (); + return 0; +} diff --git a/gdb/testsuite/gdb.mi/mi-corefile.exp b/gdb/testsuite/gdb.mi/mi-corefile.exp new file mode 100644 index 00000000000..d8ca6b3b34f --- /dev/null +++ b/gdb/testsuite/gdb.mi/mi-corefile.exp @@ -0,0 +1,69 @@ +# Copyright 2022 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 loading and unloading a core file. + +load_lib mi-support.exp +set MIFLAGS "-i=mi" + +if ![isnative] then { + return +} + +standard_testfile + +if {[build_executable $testfile.exp $testfile $srcfile debug] == -1} { + untested "failed to compile" + return -1 +} + +set corefile [core_find $binfile {}] +if {$corefile == ""} { + return 0 +} + +if [mi_gdb_start] { + return +} + +# Test loading and unloading a core file. + +proc test_core_file {} { + mi_delete_breakpoints + mi_gdb_reinitialize_dir $::srcdir/$::subdir + mi_gdb_load ${::binfile} + + # This also tests that MI emits paired =thread-created and + # =thread-exited notifications when loading and unloading the + # core. At some point, GDB emitted =thread-created notifications, + # but not =thread-exited notifications. + + mi_gdb_test \ + "-target-select core $::corefile" \ + [multi_line \ + "=thread-group-started,id=\"i1\",.*" \ + "=thread-created,id=\"1\",group-id=\"i1\"" \ + ".*\\^connected,frame=.*"] \ + "-target-select core \$corefile" + + mi_gdb_test \ + "-target-detach" \ + [multi_line \ + "=thread-exited,id=\"1\",group-id=\"i1\"" \ + "=thread-group-exited,id=\"i1\"" \ + ".*\\^done"] +} + +test_core_file base-commit: 9704b8b4bc58f4f464961cca97d362fd33740ce8 -- 2.36.0