Re: [PATCH v3 0/2] perf test: Add test for java symbol

[Arnaldo Carvalho de Melo <acme@kernel.org>]

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