linux-kernel - Re: [PATCH] perf test: Use existing config value for objdump path

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAM9d7cj=nF_Vq7dPkOGvYd1G0nFOffiCoyZQ=hx1hco-0_ZMBw@mail.gmail.com>
Date:   Mon, 20 Nov 2023 12:48:38 -0800
From:   Namhyung Kim <namhyung@...nel.org>
To:     James Clark <james.clark@....com>
Cc:     linux-perf-users@...r.kernel.org, irogers@...gle.com,
        acme@...nel.org, Peter Zijlstra <peterz@...radead.org>,
        Ingo Molnar <mingo@...hat.com>,
        Mark Rutland <mark.rutland@....com>,
        Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
        Jiri Olsa <jolsa@...nel.org>,
        Adrian Hunter <adrian.hunter@...el.com>,
        Yonghong Song <yhs@...com>, Fangrui Song <maskray@...gle.com>,
        Kan Liang <kan.liang@...ux.intel.com>,
        Yang Jihong <yangjihong1@...wei.com>,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] perf test: Use existing config value for objdump path

Hello,

On Mon, Nov 13, 2023 at 2:23 AM James Clark <james.clark@....com> wrote:
>
> There is already an existing config value for changing the objdump path,
> so instead of having two values that do the same thing, make perf test
> use annotate.objdump as well.
>
> Signed-off-by: James Clark <james.clark@....com>

Acked-by: Namhyung Kim <namhyung@...nel.org>

Thanks,
Namhyung

> ---
>  tools/perf/Documentation/perf-config.txt | 8 ++------
>  tools/perf/tests/builtin-test.c          | 2 +-
>  2 files changed, 3 insertions(+), 7 deletions(-)
>
> diff --git a/tools/perf/Documentation/perf-config.txt b/tools/perf/Documentation/perf-config.txt
> index 16398babd1ef..379f9d7a8ab1 100644
> --- a/tools/perf/Documentation/perf-config.txt
> +++ b/tools/perf/Documentation/perf-config.txt
> @@ -251,7 +251,8 @@ annotate.*::
>                 addr2line binary to use for file names and line numbers.
>
>         annotate.objdump::
> -               objdump binary to use for disassembly and annotations.
> +               objdump binary to use for disassembly and annotations,
> +               including in the 'perf test' command.
>
>         annotate.disassembler_style::
>                 Use this to change the default disassembler style to some other value
> @@ -722,11 +723,6 @@ session-<NAME>.*::
>                 Defines new record session for daemon. The value is record's
>                 command line without the 'record' keyword.
>
> -test.*::
> -
> -       test.objdump::
> -               objdump binary to use for disassembly and annotations.
> -
>  SEE ALSO
>  --------
>  linkperf:perf[1]
> diff --git a/tools/perf/tests/builtin-test.c b/tools/perf/tests/builtin-test.c
> index 113e92119e1d..b8c21e81a021 100644
> --- a/tools/perf/tests/builtin-test.c
> +++ b/tools/perf/tests/builtin-test.c
> @@ -518,7 +518,7 @@ static int run_workload(const char *work, int argc, const char **argv)
>  static int perf_test__config(const char *var, const char *value,
>                              void *data __maybe_unused)
>  {
> -       if (!strcmp(var, "test.objdump"))
> +       if (!strcmp(var, "annotate.objdump"))
>                 test_objdump_path = value;
>
>         return 0;
> --
> 2.34.1
>