Received: by 2002:a05:6358:4e97:b0:b3:742d:4702 with SMTP id ce23csp2130476rwb; Fri, 12 Aug 2022 12:37:43 -0700 (PDT) X-Google-Smtp-Source: AA6agR7Ca535br2JyPE19f+BYLXnpeGzP59gISRR6dTlbYDaix1PoNqsLEM1W0PrRdApa53KCEjH X-Received: by 2002:a17:902:e5d1:b0:16f:15a1:6dbc with SMTP id u17-20020a170902e5d100b0016f15a16dbcmr5468404plf.73.1660333063364; Fri, 12 Aug 2022 12:37:43 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1660333063; cv=none; d=google.com; s=arc-20160816; b=Y+VZf5TGrD7y6gQE+Y9N359tRTofgwjmjp377M1O78MjgvHl9s2e+TgRnL5gkoig1W P9eA6b15JBYbfRzmk/vjaJqq5350ycxESgkhKDaQckWgqnzeAj6vX/ABT0KaCqRTU0e5 fbEt6EZFVQ1P/rq0ZL5sCJRF2OibK5918IhtYx0aftmxDo7S3XGxnfIra5qMaBwuyY3M aracqgz4dn0TOlFkh2WnsxJ2BcNtSftyjpRJZpamz8LgPEQViCfu1knph+ArNUKff8pe D6dIof2d/JLwAKQIBvMnbdKbwUxDS+NOkwP6EFf9NqByb1wgVfuzXuUTbesuzT+kLNAN yzdg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=PH4gyK8tBNYBMS+14jHjTw8v11WUHuYQpPdmVCXltqk=; b=npuLMeRczicaJSMqA+VfnPKeosaHOWwwXuH7wdsjiRVyZtVIdUppZ5R9d9BBmLoggc Dify8WQaNCbAOqRfAI5TKeN+y9BZ+rmQ/9I16+UHr9FiskFP39+gMcKrkamWYEfcEUrN 1D4CHlNLBzfise587bijiAgbfhMmPK5NIqvgFKSqtrzjwWJ9d4HBD08Cm2NXUR+1Wm1x QA/A9Kplkf+FqsXK73+AlismyPM8q83YZ7JOvlR19w74H2jQLPHeyEmxE0om7Fcu14qd RDfCdwt+I34i40hEU0Akb7S1j2wtKs0Gq+u2p/tzdCwlDdvl5pJucSgGhWLozF2mmVCP qG2A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b="ndnmwBh/"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id a9-20020a170902b58900b0016a2bfe5c7csi3053196pls.172.2022.08.12.12.37.31; Fri, 12 Aug 2022 12:37:43 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b="ndnmwBh/"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237243AbiHLTC6 (ORCPT + 99 others); Fri, 12 Aug 2022 15:02:58 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35798 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229760AbiHLTC5 (ORCPT ); Fri, 12 Aug 2022 15:02:57 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9780A2E7; Fri, 12 Aug 2022 12:02:56 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 4B023B82532; Fri, 12 Aug 2022 19:02:55 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8D1C7C433D6; Fri, 12 Aug 2022 19:02:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1660330973; bh=MFK5pQXQAeFIYVQLPuNMr/kHAjpScCwzUYnAX6y1KHQ=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=ndnmwBh/iSJtROTMRlvmxthd0lkRV7s0BHBl9gElmDzMPEhz7iPhRcG+Rbuagb+Zu PQMdG2/i4hEsz64twY0DneGiUnXamiSiB9mQCtA80wPpCRFfttRzEQnGHObCIajsGf 3jICwJM9X2rTECKcsu0xiDla5T0A55hvbkAcCQv/0bOmA6KCv2uU4pKLZEqkmWpLil LRcQ9JcR0qQnI9pSK1VKZaD7Rr2lwhb/y7jqKmKWzQZpPuVaqqKybE6oi7nWAySSGx z8lW5P9c8PSHM+J0QMguugicnyzDaQLfsRV5kT+UKeGtgVhGIdPGPrlC3wXLQEFxDg be3OlWnxpyfCQ== Received: by quaco.ghostprotocols.net (Postfix, from userid 1000) id CAF584035A; Fri, 12 Aug 2022 16:02:50 -0300 (-03) Date: Fri, 12 Aug 2022 16:02:50 -0300 From: Arnaldo Carvalho de Melo To: Leo Yan , carsten.haitzler@foss.arm.com Cc: linux-kernel@vger.kernel.org, coresight@lists.linaro.org, suzuki.poulose@arm.com, mathieu.poirier@linaro.org, mike.leach@linaro.org, linux-perf-users@vger.kernel.org Subject: Re: [PATCH v7 00/14] perf: test: Add trace data quality tests for CoreSight Message-ID: References: <20220812121641.336465-1-carsten.haitzler@foss.arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220812121641.336465-1-carsten.haitzler@foss.arm.com> X-Url: http://acmel.wordpress.com X-Spam-Status: No, score=-7.7 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI, SPF_HELO_NONE,SPF_PASS,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 lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Em Fri, Aug 12, 2022 at 01:16:27PM +0100, carsten.haitzler@foss.arm.com escreveu: > From: Carsten Haitzler > > This is a prelude to adding more tests to shell tests and in order to > support putting those tests into subdirectories, I need to change the > test code that scans/finds and runs them. > > To support subdirs I have to recurse so it's time to refactor the code to > allow this and centralize the shell script finding into one location and > only one single scan that builds a list of all the found tests in memory > instead of it being duplicated in 3 places. > > This code also optimizes things like knowing the max width of desciption > strings (as we can do that while we scan instead of a whole new pass > of opening files). It also more cleanly filters scripts to see only > *.sh files thus skipping random other files in directories like *~ > backup files, other random junk/data files that may appear and the > scripts must be executable to make the cut (this ensures the script > lib dir is not seen as scripts to run). This avoids perf test running > previous older versions of test scripts that are editor backup files > as well as skipping perf.data files that may appear and so on. > > Signed-off-by: Carsten Haitzler Leo, can you please check if your previous Reviewed-by tags stand for this new version? Thanks, - Arnaldo > > Carsten Haitzler (14): > perf test: Refactor shell tests allowing subdirs > perf test: Add CoreSight shell lib shared code for future tests > perf test: Add build infra for perf test tools for CoreSight tests > perf test: Add asm pureloop test tool > perf test: Add asm pureloop test shell script > perf test: Add git ignore for perf data generated by the CoreSight > tests > perf test: Add memcpy thread test tool > perf test: Add memcpy thread test shell script > perf test: Add thread loop test tool > perf test: Add thread loop test shell scripts > perf test: Add unroll thread test tool > perf test: Add unroll thread test shell script > perf test: Add git ignore for tmp and output files of CoreSight tests > perf test: Add relevant documentation about CoreSight testing > > .../trace/coresight/coresight-perf.rst | 158 +++++++++++++ > MAINTAINERS | 1 + > tools/perf/.gitignore | 6 +- > .../perf/Documentation/perf-arm-coresight.txt | 5 + > tools/perf/Makefile.config | 2 + > tools/perf/Makefile.perf | 17 +- > tools/perf/tests/Build | 1 + > tools/perf/tests/builtin-test-list.c | 207 ++++++++++++++++++ > tools/perf/tests/builtin-test-list.h | 12 + > tools/perf/tests/builtin-test.c | 152 ++----------- > tools/perf/tests/shell/coresight/Makefile | 30 +++ > .../tests/shell/coresight/Makefile.miniconfig | 14 ++ > .../tests/shell/coresight/asm_pure_loop.sh | 18 ++ > .../shell/coresight/asm_pure_loop/.gitignore | 1 + > .../shell/coresight/asm_pure_loop/Makefile | 34 +++ > .../coresight/asm_pure_loop/asm_pure_loop.S | 28 +++ > .../shell/coresight/memcpy_thread/.gitignore | 1 + > .../shell/coresight/memcpy_thread/Makefile | 33 +++ > .../coresight/memcpy_thread/memcpy_thread.c | 79 +++++++ > .../shell/coresight/memcpy_thread_16k_10.sh | 18 ++ > .../shell/coresight/thread_loop/.gitignore | 1 + > .../shell/coresight/thread_loop/Makefile | 33 +++ > .../shell/coresight/thread_loop/thread_loop.c | 86 ++++++++ > .../coresight/thread_loop_check_tid_10.sh | 19 ++ > .../coresight/thread_loop_check_tid_2.sh | 19 ++ > .../coresight/unroll_loop_thread/.gitignore | 1 + > .../coresight/unroll_loop_thread/Makefile | 33 +++ > .../unroll_loop_thread/unroll_loop_thread.c | 74 +++++++ > .../shell/coresight/unroll_loop_thread_10.sh | 18 ++ > tools/perf/tests/shell/lib/coresight.sh | 132 +++++++++++ > 30 files changed, 1094 insertions(+), 139 deletions(-) > create mode 100644 Documentation/trace/coresight/coresight-perf.rst > create mode 100644 tools/perf/Documentation/perf-arm-coresight.txt > create mode 100644 tools/perf/tests/builtin-test-list.c > create mode 100644 tools/perf/tests/builtin-test-list.h > create mode 100644 tools/perf/tests/shell/coresight/Makefile > create mode 100644 tools/perf/tests/shell/coresight/Makefile.miniconfig > create mode 100755 tools/perf/tests/shell/coresight/asm_pure_loop.sh > create mode 100644 tools/perf/tests/shell/coresight/asm_pure_loop/.gitignore > create mode 100644 tools/perf/tests/shell/coresight/asm_pure_loop/Makefile > create mode 100644 tools/perf/tests/shell/coresight/asm_pure_loop/asm_pure_loop.S > create mode 100644 tools/perf/tests/shell/coresight/memcpy_thread/.gitignore > create mode 100644 tools/perf/tests/shell/coresight/memcpy_thread/Makefile > create mode 100644 tools/perf/tests/shell/coresight/memcpy_thread/memcpy_thread.c > create mode 100755 tools/perf/tests/shell/coresight/memcpy_thread_16k_10.sh > create mode 100644 tools/perf/tests/shell/coresight/thread_loop/.gitignore > create mode 100644 tools/perf/tests/shell/coresight/thread_loop/Makefile > create mode 100644 tools/perf/tests/shell/coresight/thread_loop/thread_loop.c > create mode 100755 tools/perf/tests/shell/coresight/thread_loop_check_tid_10.sh > create mode 100755 tools/perf/tests/shell/coresight/thread_loop_check_tid_2.sh > create mode 100644 tools/perf/tests/shell/coresight/unroll_loop_thread/.gitignore > create mode 100644 tools/perf/tests/shell/coresight/unroll_loop_thread/Makefile > create mode 100644 tools/perf/tests/shell/coresight/unroll_loop_thread/unroll_loop_thread.c > create mode 100755 tools/perf/tests/shell/coresight/unroll_loop_thread_10.sh > create mode 100644 tools/perf/tests/shell/lib/coresight.sh > > -- > 2.32.0 -- - Arnaldo