linux-kernel - Re: [PATCH 1/1] perf tools: Don't install libtraceevent plugins as its not anymore in the kernel sources

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] [day] [month] [year] [list]

Message-ID: <CAP-5=fVr9nuKH=su5rxWer=o3KML4bk5oEzfUp_gtJBWoF0rgw@mail.gmail.com>
Date:   Mon, 9 Jan 2023 11:40:30 -0800
From:   Ian Rogers <irogers@...gle.com>
To:     Arnaldo Carvalho de Melo <acme@...nel.org>
Cc:     Jiri Olsa <jolsa@...nel.org>,
        Adrian Hunter <adrian.hunter@...el.com>,
        Namhyung Kim <namhyung@...nel.org>,
        Steven Rostedt <rostedt@...dmis.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        linux-perf-users@...r.kernel.org
Subject: Re: [PATCH 1/1] perf tools: Don't install libtraceevent plugins as
 its not anymore in the kernel sources

On Mon, Jan 9, 2023 at 10:07 AM Arnaldo Carvalho de Melo
<acme@...nel.org> wrote:
>
> While doing 'make -C tools/perf build-test' one can notice error
> messages while trying to install libtraceevent plugins, stop doing that
> as libtraceevent isn't anymore a homie.
>
> These are the warnings dealt with:
>
>    make_install_prefix_slash_O: make install prefix=/tmp/krava/
>     failed to find: /tmp/krava/etc/bash_completion.d/perf
>     failed to find: /tmp/krava/lib64/traceevent/plugins/plugin_cfg80211.so
>     failed to find: /tmp/krava/lib64/traceevent/plugins/plugin_scsi.so
>     failed to find: /tmp/krava/lib64/traceevent/plugins/plugin_xen.so
>     failed to find: /tmp/krava/lib64/traceevent/plugins/plugin_function.so
>     failed to find: /tmp/krava/lib64/traceevent/plugins/plugin_sched_switch.so
>     failed to find: /tmp/krava/lib64/traceevent/plugins/plugin_mac80211.so
>     failed to find: /tmp/krava/lib64/traceevent/plugins/plugin_kvm.so
>     failed to find: /tmp/krava/lib64/traceevent/plugins/plugin_kmem.so
>     failed to find: /tmp/krava/lib64/traceevent/plugins/plugin_hrtimer.so
>     failed to find: /tmp/krava/lib64/traceevent/plugins/plugin_jbd2.so
>
> Fixes: 4171925aa9f3f7bf ("tools lib traceevent: Remove libtraceevent")
> Cc: Adrian Hunter <adrian.hunter@...el.com>
> Cc: Ian Rogers <irogers@...gle.com>
> Cc: Jiri Olsa <jolsa@...nel.org>
> Cc: Namhyung Kim <namhyung@...nel.org>
> Cc: Steven Rostedt <rostedt@...dmis.org>
> Link: http://lore.kernel.org/lkml/
> Signed-off-by: Arnaldo Carvalho de Melo <acme@...hat.com>

Acked-by: Ian Rogers <irogers@...gle.com>

Thanks,
Ian

> ---
>  tools/perf/Makefile.config |  8 --------
>  tools/perf/tests/make      | 12 ------------
>  2 files changed, 20 deletions(-)
>
> diff --git a/tools/perf/Makefile.config b/tools/perf/Makefile.config
> index c2504c39bdcb8992..156440ea019143f8 100644
> --- a/tools/perf/Makefile.config
> +++ b/tools/perf/Makefile.config
> @@ -1314,14 +1314,6 @@ tip_instdir_SQ = $(subst ','\'',$(tip_instdir))
>
>  export perfexec_instdir_SQ
>
> -# If we install to $(HOME) we keep the traceevent default:
> -# $(HOME)/.traceevent/plugins
> -# Otherwise we install plugins into the global $(libdir).
> -ifdef DESTDIR
> -plugindir=$(libdir)/traceevent/plugins
> -plugindir_SQ= $(subst ','\'',$(plugindir))
> -endif
> -
>  print_var = $(eval $(print_var_code)) $(info $(MSG))
>  define print_var_code
>      MSG = $(shell printf '...%40s: %s' $(1) $($(1)))
> diff --git a/tools/perf/tests/make b/tools/perf/tests/make
> index 05e818a8bbad11a4..009d6efb673ce828 100644
> --- a/tools/perf/tests/make
> +++ b/tools/perf/tests/make
> @@ -222,19 +222,7 @@ installed_files_bin := bin/perf
>  installed_files_bin += etc/bash_completion.d/perf
>  installed_files_bin += libexec/perf-core/perf-archive
>
> -installed_files_plugins := $(lib)/traceevent/plugins/plugin_cfg80211.so
> -installed_files_plugins += $(lib)/traceevent/plugins/plugin_scsi.so
> -installed_files_plugins += $(lib)/traceevent/plugins/plugin_xen.so
> -installed_files_plugins += $(lib)/traceevent/plugins/plugin_function.so
> -installed_files_plugins += $(lib)/traceevent/plugins/plugin_sched_switch.so
> -installed_files_plugins += $(lib)/traceevent/plugins/plugin_mac80211.so
> -installed_files_plugins += $(lib)/traceevent/plugins/plugin_kvm.so
> -installed_files_plugins += $(lib)/traceevent/plugins/plugin_kmem.so
> -installed_files_plugins += $(lib)/traceevent/plugins/plugin_hrtimer.so
> -installed_files_plugins += $(lib)/traceevent/plugins/plugin_jbd2.so
> -
>  installed_files_all := $(installed_files_bin)
> -installed_files_all += $(installed_files_plugins)
>
>  test_make_install       := $(call test_dest_files,$(installed_files_all))
>  test_make_install_O     := $(call test_dest_files,$(installed_files_all))
> --
> 2.39.0
>