From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm1-x32a.google.com (mail-wm1-x32a.google.com [IPv6:2a00:1450:4864:20::32a]) by sourceware.org (Postfix) with ESMTPS id 4139B3938C0D for ; Mon, 27 Apr 2020 22:01:52 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 4139B3938C0D Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=embecosm.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=andrew.burgess@embecosm.com Received: by mail-wm1-x32a.google.com with SMTP id u16so630572wmc.5 for ; Mon, 27 Apr 2020 15:01:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=embecosm.com; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=DuX/7NBhZH9hK6qPIsIYd11SzBU46axN7LoE0w7wj/k=; b=KcrM+DAd8En+t4IYvpdYwsNHqXCmCO+5Jm7eA0Lm9J2yEq3/cvRvarCWtuUazkkH8e URPq4eP6HZSzAoQ9KY2sr2PdipCwIXHug8ef0EP1GlN1aQxkt26zh328rcmyVbvTuTWw pULZtNR5R3wAmBlnIDvhqAdAw7WkGhprJlEpFt/jveqAmkZXm62Yj52I7m107GC/nryM eFJMRHxusuj/9XxhzAwMUytsTF58k0L4aPGlsPcR2oacOvuCS/jZfm8vtJpgT7e0GCvd joHSBpgMmIBqjhMIFX0K2MRS7ymJ9Vx6aVILn7QDbLM59dpc8I6ung7y5gURua/u3owE 7YSA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=DuX/7NBhZH9hK6qPIsIYd11SzBU46axN7LoE0w7wj/k=; b=DrP90ajAGCFF5TTZIxdlzNCiHPMTOfI3M3QHf1Rg1ZInX3k3tXZnAVmqIwoAaeiQM4 H/q6ilt3/9WEpMpVHj2vaTtKHoN0SulQaUSJuf2wUAYqZpIPEiD6Xb+CLFITGK0u+oPx fIjVIdhOF95/wkv7dW2tBMPS8KEe7JbULLe9mz8lsB4+gCy0phbrkvrnFqMC6IIyWKFx XPO03+xaPvXZ+Xf8TLhN6Nj8kbmDbIoBU9BLKdzDFZDGSzJwOuTsXIMi4WFkEUX3JiDb Lnn1EAD8EWfiyo1eI0B5g5Ykh2wrAp+inomkDLrO6SGICEs3jzSoQvTDP1GoahFRSIK2 YIEQ== X-Gm-Message-State: AGi0PuagZCBV/7sH56eORnCuM6JnBxwbHyqXuyIhp8Y+x/eUIfg8dPkY 0rdLBOHXGECTSlxTIgSlObgRJLhUJXw= X-Google-Smtp-Source: APiQypIbHc8//0N6lOtuetdmSw4egr7KDWSNQCWRFa1/z42ZtNMKK2xi0qGJf+WYujmBvbUR0L9y8g== X-Received: by 2002:a7b:ca47:: with SMTP id m7mr928801wml.55.1588024910986; Mon, 27 Apr 2020 15:01:50 -0700 (PDT) Received: from localhost (host81-151-181-184.range81-151.btcentralplus.com. [81.151.181.184]) by smtp.gmail.com with ESMTPSA id k133sm778446wma.0.2020.04.27.15.01.50 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 27 Apr 2020 15:01:50 -0700 (PDT) From: Andrew Burgess To: gdb-patches@sourceware.org Subject: [PATCHv2 1/3] gdb/testsuite: Detect and warn if paths are used in test names Date: Mon, 27 Apr 2020 23:01:44 +0100 Message-Id: X-Mailer: git-send-email 2.25.3 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-24.1 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_SHORT, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) 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: Mon, 27 Apr 2020 22:01:56 -0000 A new library is introduced that hooks into the core of Dejagnu and detects when a test's name includes either the source or build paths. If any offending test names are detected then Dejagnu will print a new result type, '# of paths in test names'. Users should treat this result type just like other bad results types, and aim not to increase this number. Currently for a local run on my machine, I don't see any offending test names, but it is possible that different targets, or different configurations, might currently be breaking the no paths rule. In order to get this working I have needed to wrap two core Dejagnu functions, log_summary, and reset_vars. Relying on core functions that are not part of any API is always going to be risky, given the relatively slow rate of Dejagnu change this is probably OK for now, and we can possibly upstream some changes to Dejagnu that would allow this functionality to be supported in a more official way later on. Currently if the tests are run in parallel mode the new result type is not merged into the combined summary file so users will need to run in non-parallel mode to check this result. A later commit will fix this. gdb/testsuite/ChangeLog: * lib/gdb.exp: Include check-test-names.exp library. * lib/check-test-names.exp: New file. --- gdb/testsuite/ChangeLog | 5 ++ gdb/testsuite/lib/check-test-names.exp | 115 +++++++++++++++++++++++++ gdb/testsuite/lib/gdb.exp | 1 + 3 files changed, 121 insertions(+) create mode 100644 gdb/testsuite/lib/check-test-names.exp diff --git a/gdb/testsuite/lib/check-test-names.exp b/gdb/testsuite/lib/check-test-names.exp new file mode 100644 index 00000000000..702dc6ef406 --- /dev/null +++ b/gdb/testsuite/lib/check-test-names.exp @@ -0,0 +1,115 @@ +# Copyright 2020 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 . + +# This library provides some protection against the introduction of +# tests that include either the source of build paths in the test +# name. When a test includes the path in its test name it is harder +# to compare results between two runs of GDB from different trees. + +namespace eval ::CheckTestNames { + # An associative array of counts of tests that include a path in their + # test name. There are two counts, 'count', which counts occurrences + # within a single variant run, and 'total', which counts across all + # variants. + variable counts + array set counts {} + foreach nm {paths} { + set counts($nm,count) 0 + set counts($nm,total) 0 + } + + # Increment the count, and total count for TYPE. + proc inc_count { type } { + variable counts + + incr counts($type,count) + incr counts($type,total) + } + + # Check if MESSAGE contains either the source path or the build path. + # This will result in test names that can't easily be compared between + # different runs of GDB. + # + # Any offending paths in message cause PATHS_IN_TEST_NAMES to be + # incremented. + proc check { message } { + global srcdir objdir + + foreach path [list $srcdir $objdir] { + if { [ string first $path $message ] >= 0 } { + # Count each test just once. + inc_count paths + return + } + } + } + + # If COUNT is greater than zero, disply PREFIX followed by COUNT. + proc maybe_show_count { prefix count } { + if { $count > 0 } { + clone_output "$prefix$count" + } + } + + # Rename Dejagnu's log_summary procedure, and create do_log_summary to + # replace it. We arrange to have do_log_summary called later. + rename ::log_summary log_summary + proc do_log_summary { args } { + variable counts + + # If ARGS is the empty list then we don't want to pass a single + # empty string as a parameter here. + eval "CheckTestNames::log_summary $args" + + if { [llength $args] == 0 } { + set which "count" + } else { + set which [lindex $args 0] + } + + maybe_show_count "# of paths in test names\t" \ + $counts(paths,$which) + } + + # Rename Dejagnu's reset_vars procedure, and create do_reset_vars to + # replace it. We arrange to have do_reset_vars called later. + rename ::reset_vars reset_vars + proc do_reset_vars {} { + variable counts + + CheckTestNames::reset_vars + + foreach nm {paths} { + set counts($nm,count) 0 + } + } +} + +# Arrange for Dejagnu to call CheckTestNames::check for each test result. +foreach nm {pass fail xfail kfail xpass kpass unresolved untested \ + unsupported} { + set local_record_procs($nm) "CheckTestNames::check" +} + +# Create new global log_summary to replace Dejagnu's. +proc log_summary { args } { + eval "CheckTestNames::do_log_summary $args" +} + +# Create new global reset_vars to replace Dejagnu's. +proc reset_vars {} { + eval "CheckTestNames::do_reset_vars" +} + diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp index 2208f3a1a9b..aa95f6da933 100644 --- a/gdb/testsuite/lib/gdb.exp +++ b/gdb/testsuite/lib/gdb.exp @@ -29,6 +29,7 @@ load_lib libgloss.exp load_lib cache.exp load_lib gdb-utils.exp load_lib memory.exp +load_lib check-test-names.exp global GDB -- 2.25.3