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;
Date:   Mon, 26 Sep 2022 20:56:36 +0100
From:   Arnaldo Carvalho de Melo <acme@kernel.org>
To:     Leo Yan <leo.yan@linaro.org>
Cc:     Jiri Olsa <jolsa@kernel.org>, Namhyung Kim <namhyung@kernel.org>,
        Ian Rogers <irogers@google.com>,
        Peter Zijlstra <peterz@infradead.org>,
        Ingo Molnar <mingo@redhat.com>,
        Mark Rutland <mark.rutland@arm.com>,
        Alexander Shishkin <alexander.shishkin@linux.intel.com>,
        linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org
Subject: Re: [PATCH v3 0/2] perf test: Add test for java symbol
Message-ID: <YzID9Hrt5BOKiX1T@kernel.org>
References: <20220925025835.70364-1-leo.yan@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220925025835.70364-1-leo.yan@linaro.org>
Precedence: bulk

Em Sun, Sep 25, 2022 at 10:58:33AM +0800, Leo Yan escreveu:
> This patch set is to add a test for java symbol.
> 
> To allow a shell script to know the installed lib path, we need to
> export the environment variable "PREFIX" in the C code.  We use the
> first patch for this purpose.  The second patch introduces the java
> symbol testing.
> 
> The test has been verified on x86_64 machine with perf/core branch with
> the latest commit 62e64c9d2fd1 ("perf test: Add basic core_wide
> expression test").


Thanks, applied locally.

- Arnaldo


> Changes from v2:
> - Skipped test if jshell is not existed (Namhyung);
> - Fixed a typo for s/if/elif/ (Namhyung).
> 
> Changes from v1:
> - Added Ian's suggested tag;
> - Added the search folder for the system lib libperf-jvmti.so;
> - Skip the test if fail to find jvmti lib.
> 
> 
> Leo Yan (2):
>   perf subcmd: Set environment variable "PREFIX"
>   perf test: Introduce script for java symbol testing
> 
>  tools/lib/subcmd/exec-cmd.c                |  3 +
>  tools/perf/tests/shell/test_java_symbol.sh | 75 ++++++++++++++++++++++
>  2 files changed, 78 insertions(+)
>  create mode 100755 tools/perf/tests/shell/test_java_symbol.sh
> 
> -- 
> 2.34.1

-- 

- Arnaldo