lkml.org 
[lkml]   [2022]   [Nov]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v2 08/14] tools lib perf: Add missing install headers
On Wed, Nov 9, 2022 at 10:50 AM Ian Rogers <irogers@google.com> wrote:
>
> Headers necessary for the perf build. Note, internal headers are also
> installed as these are necessary for the build.

Yeah, it's sad we are using those internal headers in perf.
Ideally libperf provides callbacks to associate private data
to each public data structure (e.g. evsel, evlist, etc). And
external users just use public APIs only.

But that would be a major change.

Thanks,
Namhyung


>
> Signed-off-by: Ian Rogers <irogers@google.com>
> ---
> tools/lib/perf/Makefile | 10 +++++++++-
> 1 file changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/tools/lib/perf/Makefile b/tools/lib/perf/Makefile
> index 21df023a2103..1badc0a04676 100644
> --- a/tools/lib/perf/Makefile
> +++ b/tools/lib/perf/Makefile
> @@ -189,13 +189,21 @@ install_lib: libs
>
> install_headers:
> $(call QUIET_INSTALL, headers) \
> + $(call do_install,include/perf/bpf_perf.h,$(prefix)/include/perf,644); \
> $(call do_install,include/perf/core.h,$(prefix)/include/perf,644); \
> $(call do_install,include/perf/cpumap.h,$(prefix)/include/perf,644); \
> $(call do_install,include/perf/threadmap.h,$(prefix)/include/perf,644); \
> $(call do_install,include/perf/evlist.h,$(prefix)/include/perf,644); \
> $(call do_install,include/perf/evsel.h,$(prefix)/include/perf,644); \
> $(call do_install,include/perf/event.h,$(prefix)/include/perf,644); \
> - $(call do_install,include/perf/mmap.h,$(prefix)/include/perf,644);
> + $(call do_install,include/perf/mmap.h,$(prefix)/include/perf,644); \
> + $(call do_install,include/internal/cpumap.h,$(prefix)/include/internal,644); \
> + $(call do_install,include/internal/evlist.h,$(prefix)/include/internal,644); \
> + $(call do_install,include/internal/evsel.h,$(prefix)/include/internal,644); \
> + $(call do_install,include/internal/lib.h,$(prefix)/include/internal,644); \
> + $(call do_install,include/internal/mmap.h,$(prefix)/include/internal,644); \
> + $(call do_install,include/internal/threadmap.h,$(prefix)/include/internal,644); \
> + $(call do_install,include/internal/xyarray.h,$(prefix)/include/internal,644);
>
> install_pkgconfig: $(LIBPERF_PC)
> $(call QUIET_INSTALL, $(LIBPERF_PC)) \
> --
> 2.38.1.431.g37b22c650d-goog
>

\
 
 \ /
  Last update: 2022-11-09 21:13    [W:0.101 / U:1.204 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site