[PATCH 1/1] tools build: Don't assume libtracefs-devel is always available

[PATCH 1/1] tools build: Don't assume libtracefs-devel is always available

[Arnaldo Carvalho de Melo]

perf doesn't use libtracefs and so it doesn't make sense to assume it is
always available when building test-all.bin, defeating the feature check
speedup it provides.

The other tools/build/ users such as rtla, rv, etc call $(feature_check
libtracefs) to check its availability instead of using the test-all.bin
mechanism, stopping the build and asking for libtracefs-devel to be
installed.

Noticed after applying Ian Roger's removal unused libtracefs
configuration in tools/perf/.

Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Gabriele Monaco <gmonaco@redhat.com>
Cc: Ian Rogers <irogers@google.com>
Cc: James Clark <james.clark@linaro.org>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: Tomas Glozar <tglozar@redhat.com>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 tools/build/feature/test-all.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/tools/build/feature/test-all.c b/tools/build/feature/test-all.c
index e1847db6f8e63750..2df593593b6ec15e 100644
--- a/tools/build/feature/test-all.c
+++ b/tools/build/feature/test-all.c
@@ -150,10 +150,6 @@
 # include "test-libtraceevent.c"
 #undef main
 
-#define main main_test_libtracefs
-# include "test-libtracefs.c"
-#undef main
-
 int main(int argc, char *argv[])
 {
 	main_test_libpython();
@@ -187,7 +183,6 @@ int main(int argc, char *argv[])
 	main_test_reallocarray();
 	main_test_libzstd();
 	main_test_libtraceevent();
-	main_test_libtracefs();
 
 	return 0;
 }
-- 
2.49.0

Re: [PATCH 1/1] tools build: Don't assume libtracefs-devel is always available

[Tomas Glozar]

čt 2. 10. 2025 v 15:05 odesílatel Arnaldo Carvalho de Melo
<acme@kernel.org> napsal:
>
> perf doesn't use libtracefs and so it doesn't make sense to assume it is
> always available when building test-all.bin, defeating the feature check
> speedup it provides.
>
> The other tools/build/ users such as rtla, rv, etc call $(feature_check
> libtracefs) to check its availability instead of using the test-all.bin
> mechanism, stopping the build and asking for libtracefs-devel to be
> installed.
>
> Noticed after applying Ian Roger's removal unused libtracefs
> configuration in tools/perf/.
>
> Cc: Adrian Hunter <adrian.hunter@intel.com>
> Cc: Gabriele Monaco <gmonaco@redhat.com>
> Cc: Ian Rogers <irogers@google.com>
> Cc: James Clark <james.clark@linaro.org>
> Cc: Jiri Olsa <jolsa@kernel.org>
> Cc: Kan Liang <kan.liang@linux.intel.com>
> Cc: Namhyung Kim <namhyung@kernel.org>
> Cc: Steven Rostedt <rostedt@goodmis.org>
> Cc: Tomas Glozar <tglozar@redhat.com>
> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

Acked-by: Tomas Glozar <tglozar@redhat.com>

Perhaps it would also make sense to clarify in a comment that
test-all.c is to speed up perf build rather than rv, rtla etc. (which
don't use most of the dependencies listed there)?

Tomas

Re: [PATCH 1/1] tools build: Don't assume libtracefs-devel is always available

[Ian Rogers]

On Thu, Oct 2, 2025 at 7:13 AM Tomas Glozar <tglozar@redhat.com> wrote:
>
> čt 2. 10. 2025 v 15:05 odesílatel Arnaldo Carvalho de Melo
> <acme@kernel.org> napsal:
> >
> > perf doesn't use libtracefs and so it doesn't make sense to assume it is
> > always available when building test-all.bin, defeating the feature check
> > speedup it provides.
> >
> > The other tools/build/ users such as rtla, rv, etc call $(feature_check
> > libtracefs) to check its availability instead of using the test-all.bin
> > mechanism, stopping the build and asking for libtracefs-devel to be
> > installed.
> >
> > Noticed after applying Ian Roger's removal unused libtracefs
> > configuration in tools/perf/.
> >
> > Cc: Adrian Hunter <adrian.hunter@intel.com>
> > Cc: Gabriele Monaco <gmonaco@redhat.com>
> > Cc: Ian Rogers <irogers@google.com>
> > Cc: James Clark <james.clark@linaro.org>
> > Cc: Jiri Olsa <jolsa@kernel.org>
> > Cc: Kan Liang <kan.liang@linux.intel.com>
> > Cc: Namhyung Kim <namhyung@kernel.org>
> > Cc: Steven Rostedt <rostedt@goodmis.org>
> > Cc: Tomas Glozar <tglozar@redhat.com>
> > Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
>
> Acked-by: Tomas Glozar <tglozar@redhat.com>
>
> Perhaps it would also make sense to clarify in a comment that
> test-all.c is to speed up perf build rather than rv, rtla etc. (which
> don't use most of the dependencies listed there)?

There's a big comment in Makefile.feature:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/tools/build/Makefile.feature#n24
I think that FEATURE_TESTS_BASIC tests are covered by test-all.c so I
think this change needs to remove libtracefs from FEATURE_TESTS_BASIC
otherwise  $(feature-libtracefs) will evaluate to 1 even though no
libtracefs testing has been performed. We can't just remove the
tracefs feature test as there are dependencies in rv, rtla, etc.

Thanks,
Ian

Re: [PATCH 1/1] tools build: Don't assume libtracefs-devel is always available

[Arnaldo Carvalho de Melo]

On Thu, Oct 02, 2025 at 08:21:08AM -0700, Ian Rogers wrote:
> On Thu, Oct 2, 2025 at 7:13 AM Tomas Glozar <tglozar@redhat.com> wrote:
> >
> > čt 2. 10. 2025 v 15:05 odesílatel Arnaldo Carvalho de Melo
> > <acme@kernel.org> napsal:
> > >
> > > perf doesn't use libtracefs and so it doesn't make sense to assume it is
> > > always available when building test-all.bin, defeating the feature check
> > > speedup it provides.
> > >
> > > The other tools/build/ users such as rtla, rv, etc call $(feature_check
> > > libtracefs) to check its availability instead of using the test-all.bin
> > > mechanism, stopping the build and asking for libtracefs-devel to be
> > > installed.
> > >
> > > Noticed after applying Ian Roger's removal unused libtracefs
> > > configuration in tools/perf/.
> > >
> > > Cc: Adrian Hunter <adrian.hunter@intel.com>
> > > Cc: Gabriele Monaco <gmonaco@redhat.com>
> > > Cc: Ian Rogers <irogers@google.com>
> > > Cc: James Clark <james.clark@linaro.org>
> > > Cc: Jiri Olsa <jolsa@kernel.org>
> > > Cc: Kan Liang <kan.liang@linux.intel.com>
> > > Cc: Namhyung Kim <namhyung@kernel.org>
> > > Cc: Steven Rostedt <rostedt@goodmis.org>
> > > Cc: Tomas Glozar <tglozar@redhat.com>
> > > Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> >
> > Acked-by: Tomas Glozar <tglozar@redhat.com>
> >
> > Perhaps it would also make sense to clarify in a comment that
> > test-all.c is to speed up perf build rather than rv, rtla etc. (which
> > don't use most of the dependencies listed there)?
> 
> There's a big comment in Makefile.feature:
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/tools/build/Makefile.feature#n24
> I think that FEATURE_TESTS_BASIC tests are covered by test-all.c so I
> think this change needs to remove libtracefs from FEATURE_TESTS_BASIC
> otherwise  $(feature-libtracefs) will evaluate to 1 even though no
> libtracefs testing has been performed.

Done, thanks for catching this.

> We can't just remove the tracefs feature test as there are
> dependencies in rv, rtla, etc.

And those, as noted, don't rely on test-all.bin to detect the presence
of libtracefs:

⬢ [acme@toolbx perf-tools-next]$ git grep feature.*libtracefs tools/
tools/tracing/latency/Makefile.config:$(call feature_check,libtracefs)
tools/tracing/latency/Makefile.config:ifeq ($(feature-libtracefs), 1)
tools/tracing/rtla/Makefile.config:$(call feature_check,libtracefs)
tools/tracing/rtla/Makefile.config:ifeq ($(feature-libtracefs), 1)
tools/verification/rv/Makefile.config:$(call feature_check,libtracefs)
tools/verification/rv/Makefile.config:ifeq ($(feature-libtracefs), 1)
⬢ [acme@toolbx perf-tools-next]$

Can I have your Reviewed-by or Acked-by?

Thanks,

- Arnaldo