From patchwork Thu Mar 7 13:28:41 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Metzger, Markus T" X-Patchwork-Id: 86928 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 11EAA385771A for ; Thu, 7 Mar 2024 13:29:58 +0000 (GMT) X-Original-To: gdb-patches@sourceware.org Delivered-To: gdb-patches@sourceware.org Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.15]) by sourceware.org (Postfix) with ESMTPS id 208133857833 for ; Thu, 7 Mar 2024 13:28:55 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 208133857833 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=intel.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org 208133857833 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=192.198.163.15 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1709818137; cv=none; b=PxEWRA9th6ObuGv/MyYTGnwrL41RbB/oSkuveNXxkLa4ZkJTL6lC2yx8t1emQcI9RutHcGXBbtOp1XK9DqPa91vpigEb+Q0agW9UIHy7DPV+4NiObS/bhMuyD/UIqBKJ7PIWxYI4LZjovAUj9gp+Jx+gfuSoIVQAmeLiUZeIu0k= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1709818137; c=relaxed/simple; bh=GyjHhpRcC4HGF6czK/JPt1upnjxDSin9KKGgQx/giyE=; h=DKIM-Signature:From:To:Subject:Date:Message-Id:MIME-Version; b=Mi3x787urYTTwtj2bNCEnLWU9sQT6XFxOr4LEYP7kfOCrDnz17AS66um5qzvYmbHZr+w74V8TQjDw8NDkYepUiuzLbnGZQ22fB3Ie6Y8QsFanPoFyXiici12Q8WlDbA8YEuXPF1yiCSVE+hQbcSpT0/+UU5JbpjunzuRaIz4wVo= ARC-Authentication-Results: i=1; server2.sourceware.org DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1709818135; x=1741354135; h=from:to:subject:date:message-id:in-reply-to:references: mime-version:content-transfer-encoding; bh=GyjHhpRcC4HGF6czK/JPt1upnjxDSin9KKGgQx/giyE=; b=XARGFRISL0PP4kcdg6TT8HC3xo8gltOso+pmE2eR5JVLFH/z8YrV2sfg dXnAyKCRHlwpbP3+1xH/345d1ChK1Ut+d7+mdPX0CMnGkcnGM5Zi/Gv1H sEExhWrTS7OS2iXDg89m7AT2rylsIfaMX57AyveD7ksEvc8J+FcYsQHft JLr8505ctRjiI18qD6xUNsE649xcRXKRe2AZseFhGuSLyAGx72ZtLEVSw dL3D8u44wfUx3WhDyYX+IpVcIR7Chjp8xXvkbsy/y5Da02aooWoAsRk0C 8URyJUSFfvIrwXqN0gEJGyMXSSfv2bCCUEK91pdypJpOjAqI/XNKkYcW9 Q==; X-IronPort-AV: E=McAfee;i="6600,9927,11005"; a="4654414" X-IronPort-AV: E=Sophos;i="6.07,211,1708416000"; d="scan'208";a="4654414" Received: from fmviesa007.fm.intel.com ([10.60.135.147]) by fmvoesa109.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Mar 2024 05:28:55 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.07,211,1708416000"; d="scan'208";a="10010146" Received: from gkldtt-dev-004.igk.intel.com (HELO localhost) ([10.123.221.202]) by fmviesa007-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Mar 2024 05:28:54 -0800 From: Markus Metzger To: gdb-patches@sourceware.org Subject: [PATCH 2/6] gdb, btrace: simplify gdb.btrace/multi-inferior.exp Date: Thu, 7 Mar 2024 13:28:41 +0000 Message-Id: <20240307132845.2909415-3-markus.t.metzger@intel.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20240307132845.2909415-1-markus.t.metzger@intel.com> References: <20240307132845.2909415-1-markus.t.metzger@intel.com> MIME-Version: 1.0 X-Spam-Status: No, score=-10.0 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, 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.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 We don't really need three inferiors to test multi-inferior recording. We don't really need to start recording on the second inferior first. We don't really need to check info record before starting recording. If we were recording, there would be output, causing a fail. This just complicates the test when there is something to debug. --- gdb/testsuite/gdb.btrace/multi-inferior.exp | 24 +++------------------ 1 file changed, 3 insertions(+), 21 deletions(-) diff --git a/gdb/testsuite/gdb.btrace/multi-inferior.exp b/gdb/testsuite/gdb.btrace/multi-inferior.exp index 6996b182e65..174d38364a4 100644 --- a/gdb/testsuite/gdb.btrace/multi-inferior.exp +++ b/gdb/testsuite/gdb.btrace/multi-inferior.exp @@ -37,6 +37,8 @@ with_test_prefix "inferior 1" { if ![runto_main] { return -1 } + + gdb_test_no_output "record btrace" } with_test_prefix "inferior 2" { @@ -48,25 +50,5 @@ with_test_prefix "inferior 2" { return -1 } - gdb_test_no_output "record btrace" "record btrace" -} - -with_test_prefix "inferior 1" { - gdb_test "inferior 1" "Switching to inferior 1.*" - - gdb_test "info record" "No recording is currently active\\." - gdb_test_no_output "record btrace" "record btrace" -} - -with_test_prefix "inferior 3" { - gdb_test "add-inferior -exec ${host_binfile}" "Added inferior 3.*" \ - "add third inferior" - gdb_test "inferior 3" "Switching to inferior 3.*" - - if ![runto_main] { - return -1 - } - - gdb_test "info record" "No recording is currently active\\." - gdb_test_no_output "record btrace" "record btrace" + gdb_test_no_output "record btrace" }