From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: 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 64E8B385841D for ; Wed, 2 Mar 2022 11:57:45 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 64E8B385841D Received: from mail-wr1-f71.google.com (mail-wr1-f71.google.com [209.85.221.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-587-8dONsiIVNgOCofhLXAyofw-1; Wed, 02 Mar 2022 06:57:43 -0500 X-MC-Unique: 8dONsiIVNgOCofhLXAyofw-1 Received: by mail-wr1-f71.google.com with SMTP id f9-20020a5d58e9000000b001f0247e5e96so554665wrd.15 for ; Wed, 02 Mar 2022 03:57:43 -0800 (PST) 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:in-reply-to :references:mime-version:content-transfer-encoding; bh=EpHrxbxQ7LJ7L150J4Q/8rPtvP6W1upJnssIkHP4P/c=; b=FSyMXHh0Dcy5KsxzzUGq12zxISCKQ2tHoPAwZOuM+/YWar5bks02QtOTEp2NSfnAOP ysgPedbR8BxGocUQY+iL4BhcYMG1abX7hNjy9wOFmioRQxh1W9wtcJzgzAR+Cn8NPl4m MSzfFpSu+14+nHRpvCK7tmtp28lj2PCAnC1Gv3//VoqkM4xg5tS9vdxNwyY86zlyzk0q TAb7HmoLOg+7dKZDi3+4NSSKmFa3cXRNgGPT6T+fX7DXc+wHGcybPjCKr1TsQxai+BMO BXW59Z2VXsJXVmNfPixVkx/Y4qhLgn22j2IJrlGIIvm7i2VXZ0YerN0PpX5zjuEUOYmV P5aw== X-Gm-Message-State: AOAM533yi5x8sLrB+WF0axc+2EptG0eP4OZVl2AU2ZE2m0LP8/gx0vAT 69MSic8Oa6Pb1iw4NMDCGWOLxTarpkei4X/H3p+Usink6tTQQO9zJ876kp/+D1uQ8xiG2srtnHw u/vFssTvp1VYWABd1EVtM9Xd1uTzoVGNwxM4WpbcmVgWyNHwuWPKIVC+tpwJ8VdlmFHY9QidwiQ == X-Received: by 2002:a1c:f409:0:b0:381:500b:ad90 with SMTP id z9-20020a1cf409000000b00381500bad90mr14309729wma.42.1646222261966; Wed, 02 Mar 2022 03:57:41 -0800 (PST) X-Google-Smtp-Source: ABdhPJxT4gufwLdCCp1SIjR+sPi/+l1h9ZwNHurQ23T3y2mXXz1W7QGStkjsL6QxkJRMTTO4pY7t5A== X-Received: by 2002:a1c:f409:0:b0:381:500b:ad90 with SMTP id z9-20020a1cf409000000b00381500bad90mr14309700wma.42.1646222261507; Wed, 02 Mar 2022 03:57:41 -0800 (PST) Received: from localhost (host86-169-131-29.range86-169.btcentralplus.com. [86.169.131.29]) by smtp.gmail.com with ESMTPSA id e3-20020a5d65c3000000b001e4b3ba1423sm16673583wrw.60.2022.03.02.03.57.40 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 02 Mar 2022 03:57:41 -0800 (PST) From: Andrew Burgess To: gdb-patches@sourceware.org Subject: [PATCHv2 1/2] gdb/mi: fix regression in mi -add-inferior command Date: Wed, 2 Mar 2022 11:57:35 +0000 Message-Id: X-Mailer: git-send-email 2.25.4 In-Reply-To: References: MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="US-ASCII" X-Spam-Status: No, score=-12.2 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, KAM_SHORT, RCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H5, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_NONE, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) 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: Wed, 02 Mar 2022 11:57:47 -0000 From: Muhammad Umair Sair via Gdb-patches Prior to the multi-target support commit: commit 5b6d1e4fa4fc6827c7b3f0e99ff120dfa14d65d2 Date: Fri Jan 10 20:06:08 2020 +0000 Multi-target support When a new inferior was added using the MI -add-inferior command, the new inferior would be using the same target as all the other inferiors. This makes sense, GDB only supported a single target stack at a time. After the above commit, each inferior has its own target stack. To maintain backward compatibility, for the CLI add-inferior command, when a new inferior is added the above commit has the new inferior inherit a copy of the target stack from the current inferior. Unfortunately, this same backward compatibility is missing for the MI. This commit fixes this oversight. Now, when the -add-inferior MI command is used, the new inferior will inherit a copy of the target stack from the current inferior. --- gdb/inferior.c | 6 +- gdb/inferior.h | 7 ++ gdb/mi/mi-main.c | 4 + gdb/testsuite/gdb.mi/mi-add-inferior.exp | 122 +++++++++++++++++++++++ 4 files changed, 135 insertions(+), 4 deletions(-) create mode 100644 gdb/testsuite/gdb.mi/mi-add-inferior.exp diff --git a/gdb/inferior.c b/gdb/inferior.c index bebddb44173..7fa69c3bf52 100644 --- a/gdb/inferior.c +++ b/gdb/inferior.c @@ -757,11 +757,9 @@ add_inferior_with_spaces (void) return inf; } -/* Switch to inferior NEW_INF, a new inferior, and unless - NO_CONNECTION is true, push the process_stratum_target of ORG_INF - to NEW_INF. */ +/* See inferior.h. */ -static void +void switch_to_inferior_and_push_target (inferior *new_inf, bool no_connection, inferior *org_inf) { diff --git a/gdb/inferior.h b/gdb/inferior.h index 45de3c2d9c8..caabc2ee8d8 100644 --- a/gdb/inferior.h +++ b/gdb/inferior.h @@ -740,4 +740,11 @@ extern struct inferior *add_inferior_with_spaces (void); /* Print the current selected inferior. */ extern void print_selected_inferior (struct ui_out *uiout); +/* Switch to inferior NEW_INF, a new inferior, and unless + NO_CONNECTION is true, push the process_stratum_target of ORG_INF + to NEW_INF. */ + +extern void switch_to_inferior_and_push_target + (inferior *new_inf, bool no_connection, inferior *org_inf); + #endif /* !defined (INFERIOR_H) */ diff --git a/gdb/mi/mi-main.c b/gdb/mi/mi-main.c index 4860da7536a..2fab592d6fb 100644 --- a/gdb/mi/mi-main.c +++ b/gdb/mi/mi-main.c @@ -1708,8 +1708,12 @@ mi_cmd_add_inferior (const char *command, char **argv, int argc) if (argc != 0) error (_("-add-inferior should be passed no arguments")); + scoped_restore_current_pspace_and_thread restore_pspace_thread; + inf = add_inferior_with_spaces (); + switch_to_inferior_and_push_target (inf, false, current_inferior ()); + current_uiout->field_fmt ("inferior", "i%d", inf->num); } diff --git a/gdb/testsuite/gdb.mi/mi-add-inferior.exp b/gdb/testsuite/gdb.mi/mi-add-inferior.exp new file mode 100644 index 00000000000..3f0cd7cc06c --- /dev/null +++ b/gdb/testsuite/gdb.mi/mi-add-inferior.exp @@ -0,0 +1,122 @@ +# 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 MI '-add-inferior'. + +load_lib mi-support.exp +set MIFLAGS "-i=mi" + +standard_testfile basics.c + +if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" \ + executable {debug}] != "" } { + untested "failed to compile" + return -1 +} + +mi_clean_restart ${binfile} + +# Start execution to establish a connection. +mi_runto_main + +# Use 'info inferiors' to find the details of the current connection. +set header_line "" +set inf_line "" +gdb_test_multiple "info inferiors" "" { + + -re "^info inferiors\r\n" { + exp_continue + } + + -re "^&\[^\r\n\]+\r\n" { + exp_continue + } + + -re "~\"( Num\\s+Description\\s+Connection\[^\r\n\]+)\r\n" { + set header_line $expect_out(1,string) + exp_continue + } + + -re "^~\"(\\*\\s+1\\s+\[^\r\n\]+)\r\n" { + set inf_line $expect_out(1,string) + exp_continue + } + + -re "^\\^done\r\n" { + exp_continue + } + + -re "^$mi_gdb_prompt$" { + gdb_assert { [string length "$header_line"] > 0 } + gdb_assert { [string length "$inf_line"] > 0 } + pass $gdb_test_name + } +} + +# Now extract the string that represents the connection, and convert +# it into a regexp. +set idx [string first "Connection" "${header_line}"] +gdb_assert { $idx > -1 } +set inf_line [string range "${inf_line}" $idx end] +regexp "^(${decimal} \\(\[^)\]+\\))" $inf_line conn_info +set conn_pattern [string_to_regexp "${conn_info}"] + +# Now add a new inferior, this should use the connection of the +# current inferior. +mi_gdb_test "-add-inferior" \ + [multi_line "=thread-group-added,id=\"\[^\"\]+\"" \ + "~\"\\\[New inferior 2\\\]\\\\n\"" \ + "\~\"Added inferior 2 on connection ${conn_pattern}\\\\n\"" \ + "\\^done,inferior=\"\[^\"\]+\"" ] \ + "mi add inferior" + +# Now run 'info inferiors' again to check that the currently selected +# inferior has not changed. +set saw_current_inferior false +set saw_new_inferior false +gdb_test_multiple "info inferiors" \ + "info inferiors, after new inferior was created" { + + -re "^info inferiors\r\n" { + exp_continue + } + + -re "^&\[^\r\n\]+\r\n" { + exp_continue + } + + -re "~\"\\s+Num\\s+Description\\s+Connection\[^\r\n\]+\r\n" { + exp_continue + } + + -re "^~\"\\*\\s+1\\s+\[^\r\n\]+\\s+${conn_pattern}\\s+\[^\r\n\]+\r\n" { + set saw_current_inferior true + exp_continue + } + + -re "^~\"\\s+2\\s+\[^\r\n\]+\\s+${conn_pattern}\\s+\[^\r\n\]+\r\n" { + set saw_new_inferior true + exp_continue + } + + -re "^\\^done\r\n" { + exp_continue + } + + -re "^$mi_gdb_prompt$" { + gdb_assert { $saw_current_inferior && $saw_new_inferior } + pass $gdb_test_name + } + } -- 2.25.4