From nobody Wed Feb 11 05:02:10 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 021DC3CF75; Fri, 15 Mar 2024 14:25:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710512723; cv=none; b=rjRRtvm8sjrA/ES1973HnsUwv19wYE+EqpG72WdsR/SeC0pyTT2+RWQl0RQsTRtoCpys+hjMNQTFCy4daBRWU+LiKONGoECFgpO1+dnnrN6KunkpPdJwZ04tAxSjl00eZDMKRelOImJvuWJ1L0t9HcQznBRDntFS7rg4l3yTC2o= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710512723; c=relaxed/simple; bh=Y2yOUwbxFvC5z66QLU97vVn0Nc6CFjizJgTW4LLWiG4=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=eJEyu7rs2slWNycg3xs8zjhFF5Yfd1QD86mm39AAe/diBBiaywgDzf02t0HcN03CQNQ6B/kUDVUp5xKzujLVbJAZKi5KGGzxKLZnUX1yfkbt6btszCn5YXn2jdpfM1OGnaB6tTdP7CasjDoiv97VFSs3ul3ModY2fPKRXiXAaSA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=P/zcyySd; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="P/zcyySd" Received: by smtp.kernel.org (Postfix) with ESMTPSA id C74D4C43390; Fri, 15 Mar 2024 14:25:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1710512722; bh=Y2yOUwbxFvC5z66QLU97vVn0Nc6CFjizJgTW4LLWiG4=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=P/zcyySd4ZXP+0mwWwK/zuHU0PVLlWH6NChYOKl5YOjlUPHVzLBFpJI7c33pD8F7P 9fpBZmInv3FixXLOOEl+f8FRM4fbvdB73xaVqheYjHP4Vtu/ei69NXOfbAJl+dFBJ5 qjAYLhFIFPnoKkDpnzq1TztVNECfo+gFSiFlaPeiY4ZA1mZ0aDWfqEdRYGvbLNKztG 7RHXi+MAtF09lc1xS4cav3oJh5LZEQmlpeo/jybogJ0Gd7l3Gp0RsIGBSGZtSDnXka xvTnvk6/4dxlt+PBQ/yWC6zqYcOl/qzEpHA55PCeq4wFQ1XkOqXNy6K/s6tnKFs6Nf MlLWT04THnJcw== From: Daniel Bristot de Oliveira To: Linus Torvalds , Steven Rostedt , Arnaldo Carvalho de Melo Cc: Daniel Bristot de Oliveira , Josh Poimboeuf , John Kacur , Namhyung Kim , Jiri Olsa , linux-perf-users@vger.kernel.org, linux-trace-kernel@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH V2 1/3] tools/tracing: Use tools/build makefiles on latency-collector Date: Fri, 15 Mar 2024 15:24:59 +0100 Message-ID: X-Mailer: git-send-email 2.44.0 In-Reply-To: References: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Use tools/build/ makefiles to build latency-collector, inheriting the benefits of it. For example: Before this patch, a missing tracefs/traceevents headers will result in fail like this: Suggested-by: Linus Torvalds -------------------------------- %< ------------------------------- ~/linux/tools/tracing/latency $ make cc -Wall -Wextra -g -O2 -o latency-collector latency-collector.c -lpthread latency-collector.c:26:10: fatal error: tracefs.h: No such file or directory 26 | #include | ^~~~~~~~~~~ compilation terminated. make: *** [Makefile:14: latency-collector] Error 1 -------------------------------- %< ------------------------------- Which is not that helpful. After this change it reports: -------------------------------- %< ------------------------------- ~/linux/tools/tracing/latency# make Auto-detecting system features: ... libtraceevent: [ OFF ] ... libtracefs: [ OFF ] libtraceevent is missing. Please install libtraceevent-dev/libtraceevent-de= vel libtracefs is missing. Please install libtracefs-dev/libtracefs-devel Makefile.config:29: *** Please, check the errors above.. Stop. -------------------------------- %< ------------------------------- This type of output is common across other tools in tools/ like perf and objtool. Suggested-by: Linus Torvalds Signed-off-by: Daniel Bristot de Oliveira --- tools/tracing/latency/.gitignore | 5 +- tools/tracing/latency/Build | 1 + tools/tracing/latency/Makefile | 105 +++++++++++++++++++++----- tools/tracing/latency/Makefile.config | 30 ++++++++ 4 files changed, 122 insertions(+), 19 deletions(-) create mode 100644 tools/tracing/latency/Build create mode 100644 tools/tracing/latency/Makefile.config diff --git a/tools/tracing/latency/.gitignore b/tools/tracing/latency/.giti= gnore index 0863960761e7..2bb8e60f7fdd 100644 --- a/tools/tracing/latency/.gitignore +++ b/tools/tracing/latency/.gitignore @@ -1,2 +1,5 @@ -# SPDX-License-Identifier: GPL-2.0 +# SPDX-License-Identifier: GPL-2.0-only latency-collector +fixdep +feature +FEATURE-DUMP diff --git a/tools/tracing/latency/Build b/tools/tracing/latency/Build new file mode 100644 index 000000000000..0ce65ea72bf9 --- /dev/null +++ b/tools/tracing/latency/Build @@ -0,0 +1 @@ +latency-collector-y +=3D latency-collector.o diff --git a/tools/tracing/latency/Makefile b/tools/tracing/latency/Makefile index 40c4ddaf8be1..6518b03e05c7 100644 --- a/tools/tracing/latency/Makefile +++ b/tools/tracing/latency/Makefile @@ -1,24 +1,93 @@ -# SPDX-License-Identifier: GPL-2.0 -# Makefile for vm tools -# -VAR_CFLAGS :=3D $(shell pkg-config --cflags libtracefs 2>/dev/null) -VAR_LDLIBS :=3D $(shell pkg-config --libs libtracefs 2>/dev/null) +# SPDX-License-Identifier: GPL-2.0-only =20 -TARGETS =3D latency-collector -CFLAGS =3D -Wall -Wextra -g -O2 $(VAR_CFLAGS) -LDFLAGS =3D -lpthread $(VAR_LDLIBS) +ifeq ($(srctree),) + srctree :=3D $(patsubst %/,%,$(dir $(CURDIR))) + srctree :=3D $(patsubst %/,%,$(dir $(srctree))) + srctree :=3D $(patsubst %/,%,$(dir $(srctree))) +endif =20 -all: $(TARGETS) +include $(srctree)/tools/scripts/Makefile.include =20 -%: %.c - $(CC) $(CFLAGS) -o $@ $< $(LDFLAGS) +# O is an alias for OUTPUT +OUTPUT :=3D $(O) =20 -clean: - $(RM) latency-collector +ifeq ($(OUTPUT),) + OUTPUT :=3D $(CURDIR) +else + # subdir is used by the ../Makefile in $(call descend,) + ifneq ($(subdir),) + OUTPUT :=3D $(OUTPUT)/$(subdir) + endif +endif + +ifneq ($(patsubst %/,,$(lastword $(OUTPUT))),) + OUTPUT :=3D $(OUTPUT)/ +endif + +LATENCY-COLLECTOR :=3D $(OUTPUT)latency-collector +LATENCY-COLLECTOR_IN :=3D $(LATENCY-COLLECTOR)-in.o + +export CC :=3D gcc +export LD :=3D ld +export AR :=3D ar +export PKG_CONFIG :=3D pkg-config + +FEATURE_TESTS :=3D libtraceevent +FEATURE_TESTS +=3D libtracefs +FEATURE_DISPLAY :=3D libtraceevent +FEATURE_DISPLAY +=3D libtracefs + +ifeq ($(V),1) + Q =3D +else + Q =3D @ +endif + +all: $(LATENCY-COLLECTOR) + +include $(srctree)/tools/build/Makefile.include + +# check for dependencies only on required targets +NON_CONFIG_TARGETS :=3D clean install =20 -prefix ?=3D /usr/local -sbindir ?=3D ${prefix}/sbin +config :=3D 1 +ifdef MAKECMDGOALS +ifeq ($(filter-out $(NON_CONFIG_TARGETS),$(MAKECMDGOALS)),) + config :=3D 0 +endif +endif =20 -install: all - install -d $(DESTDIR)$(sbindir) - install -m 755 -p $(TARGETS) $(DESTDIR)$(sbindir) +ifeq ($(config),1) + include $(srctree)/tools/build/Makefile.feature + include Makefile.config +endif + +CFLAGS +=3D $(INCLUDES) $(LIB_INCLUDES) + +export CFLAGS OUTPUT srctree + +$(LATENCY-COLLECTOR): $(LATENCY-COLLECTOR_IN) + $(QUIET_LINK)$(CC) $(LDFLAGS) -o $(LATENCY-COLLECTOR) $(LATENCY-COLLECTOR= _IN) $(EXTLIBS) + +latency-collector.%: fixdep FORCE + make -f $(srctree)/tools/build/Makefile.build dir=3D. $@ + +$(LATENCY-COLLECTOR_IN): fixdep FORCE + make $(build)=3Dlatency-collector + +INSTALL :=3D install +MKDIR :=3D mkdir +STRIP :=3D strip +BINDIR :=3D /usr/bin + +install: + @$(MKDIR) -p $(DESTDIR)$(BINDIR) + $(call QUIET_INSTALL,latency-collector)$(INSTALL) $(LATENCY-COLLECTOR) -m= 755 $(DESTDIR)$(BINDIR) + @$(STRIP) $(DESTDIR)$(BINDIR)/latency-collector + +clean: + $(call QUIET_CLEAN, latency-collector) + $(Q)find . -name '*.o' -delete -o -name '\.*.cmd' -delete -o -name '\.*.d= ' -delete + $(Q)@rm -f latency-collector fixdep FEATURE-DUMP + $(Q)rm -rf feature +.PHONY: FORCE clean install diff --git a/tools/tracing/latency/Makefile.config b/tools/tracing/latency/= Makefile.config new file mode 100644 index 000000000000..b25e531a1f95 --- /dev/null +++ b/tools/tracing/latency/Makefile.config @@ -0,0 +1,30 @@ +# SPDX-License-Identifier: GPL-2.0-only + +STOP_ERROR :=3D + +define lib_setup + $(eval EXTLIBS +=3D -l$(1)) + $(eval LIB_INCLUDES +=3D $(shell sh -c "$(PKG_CONFIG) --cflags lib$(1)")) +endef + +$(call feature_check,libtraceevent) +ifeq ($(feature-libtraceevent), 1) + $(call detected,CONFIG_LIBTRACEEVENT) + $(call lib_setup,traceevent) +else + STOP_ERROR :=3D 1 + $(info libtraceevent is missing. Please install libtraceevent-dev/libtra= ceevent-devel) +endif + +$(call feature_check,libtracefs) +ifeq ($(feature-libtracefs), 1) + $(call detected,CONFIG_LIBTRACEFS) + $(call lib_setup,tracefs) +else + STOP_ERROR :=3D 1 + $(info libtracefs is missing. Please install libtracefs-dev/libtracefs-d= evel) +endif + +ifeq ($(STOP_ERROR),1) + $(error Please, check the errors above.) +endif --=20 2.44.0 From nobody Wed Feb 11 05:02:10 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 493983D55E; Fri, 15 Mar 2024 14:25:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710512726; cv=none; b=TyksSd/zhaIV0iB95a06ZxenyO4dv3mzgHN8MpBybKzBye8yTrRXYqdwchw6kSwVvVxs03ORkKEWn0xsx4MunvbMyz1J0P5KuxGda5hdJoYWV2OG6gyX0Bjy4Kt6pdcQIsuPbeqqcKfz5p+tmTHV1kIjb3XiE/+ktsds96xsUiU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710512726; c=relaxed/simple; bh=j5Dwu+kxAMxlgA6FxiP6Idn8v3HEC3UVpWaiUnQoytc=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=GLIOWZhbIa1/2FQlproDKLsMxLFeKv+dCAuPYf4+gV6ymfTxns4wBCDhLpHFZxcZWTfuwEAZjTgaR2RNbICdtG9ZytXMPN/CyAq/EPTI59U6rwlFWSMgyjr1/Jynn7EP4ZS/NyHV/KvOSEhSXP/7/3PAYaV4y1CYdFZ2dKyPryc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=hJHVzQ/y; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="hJHVzQ/y" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 0F95FC433F1; Fri, 15 Mar 2024 14:25:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1710512725; bh=j5Dwu+kxAMxlgA6FxiP6Idn8v3HEC3UVpWaiUnQoytc=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=hJHVzQ/yqjlsp84od61PKUVlnpBMkSj2VUzNsHMtG4WXz8xyth5jNPpn0Qri9YD5g j4/FRT8eRyh9vw6sCRRHUm56+1eoDQjToOiARFMnCSo6BS4dbkX68e9hpaOB36s4xi i0O8Epk2/CoIjHmwxKjT4BPYPAvPHufNIawA1YlKq9UTr7f2HIDQs1Whij0nVVXGMD hItpi8IYj+VNJ8ZVoY7RNjSpZVc4joOUh28X5YeeTf3p7cezpqsv/Cnd00jcTX4hPj gXVtZrfIOoKO4V1yMrmhUv7Dn5fgPoz5V3tN+CD8KEJBGuCwDAoN3bOsNLtWTQ2Z5Q ouLNCo2oR1t8A== From: Daniel Bristot de Oliveira To: Linus Torvalds , Steven Rostedt , Arnaldo Carvalho de Melo Cc: Daniel Bristot de Oliveira , Josh Poimboeuf , John Kacur , Namhyung Kim , Jiri Olsa , linux-perf-users@vger.kernel.org, linux-trace-kernel@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH V2 2/3] tools/rtla: Use tools/build makefiles to build rtla Date: Fri, 15 Mar 2024 15:25:00 +0100 Message-ID: <7f4ac90c527edd8a508386c9a9151b0ce979951b.1710512430.git.bristot@kernel.org> X-Mailer: git-send-email 2.44.0 In-Reply-To: References: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Use tools/build/ makefiles to build rtla, inheriting the benefits of it. For example, having a proper way to handle dependencies. rtla is built using perf infra-structure when building inside the kernel tree. At this point, rtla diverges from perf in two points: Documentation and tarball generation/build. At the documentation level, rtla is one step ahead, placing the documentation at Documentation/tools/rtla/, using the same build tools as kernel documentation. The idea is to move perf documentation to the same scheme and then share the same makefiles. rtla has a tarball target that the (old) RHEL8 uses. The tarball was kept using a simple standalone makefile for compatibility. The standalone makefile shares most of the code, e.g., flags, with regular buildings. The tarball method was set as deprecated. If necessary, we can make a rtla tarball like perf, which includes the entire tools/build. But this would also require changes in the user side (the directory structure changes, and probably the deps to build the package). Inspired on perf and objtool. Suggested-by: Linus Torvalds Signed-off-by: Daniel Bristot de Oliveira --- tools/tracing/rtla/.gitignore | 7 +- tools/tracing/rtla/Build | 1 + tools/tracing/rtla/Makefile | 217 +++++++++---------------- tools/tracing/rtla/Makefile.config | 47 ++++++ tools/tracing/rtla/Makefile.rtla | 80 +++++++++ tools/tracing/rtla/Makefile.standalone | 26 +++ tools/tracing/rtla/src/Build | 11 ++ 7 files changed, 244 insertions(+), 145 deletions(-) create mode 100644 tools/tracing/rtla/Build create mode 100644 tools/tracing/rtla/Makefile.config create mode 100644 tools/tracing/rtla/Makefile.rtla create mode 100644 tools/tracing/rtla/Makefile.standalone create mode 100644 tools/tracing/rtla/src/Build diff --git a/tools/tracing/rtla/.gitignore b/tools/tracing/rtla/.gitignore index e9df32419b2b..293f0dbb0ca2 100644 --- a/tools/tracing/rtla/.gitignore +++ b/tools/tracing/rtla/.gitignore @@ -1 +1,6 @@ -/rtla +# SPDX-License-Identifier: GPL-2.0-only +rtla +rtla-static +fixdep +feature +FEATURE-DUMP diff --git a/tools/tracing/rtla/Build b/tools/tracing/rtla/Build new file mode 100644 index 000000000000..6c9d5b36a315 --- /dev/null +++ b/tools/tracing/rtla/Build @@ -0,0 +1 @@ +rtla-y +=3D src/ diff --git a/tools/tracing/rtla/Makefile b/tools/tracing/rtla/Makefile index afd18c678ff5..b5878be36125 100644 --- a/tools/tracing/rtla/Makefile +++ b/tools/tracing/rtla/Makefile @@ -1,157 +1,86 @@ -NAME :=3D rtla -# Follow the kernel version -VERSION :=3D $(shell cat VERSION 2> /dev/null || make -sC ../../.. kernelv= ersion | grep -v make) - -# From libtracefs: -# Makefiles suck: This macro sets a default value of $(2) for the -# variable named by $(1), unless the variable has been set by -# environment or command line. This is necessary for CC and AR -# because make sets default values, so the simpler ?=3D approach -# won't work as expected. -define allow-override - $(if $(or $(findstring environment,$(origin $(1))),\ - $(findstring command line,$(origin $(1)))),,\ - $(eval $(1) =3D $(2))) -endef - -# Allow setting CC and AR, or setting CROSS_COMPILE as a prefix. -$(call allow-override,CC,$(CROSS_COMPILE)gcc) -$(call allow-override,AR,$(CROSS_COMPILE)ar) -$(call allow-override,STRIP,$(CROSS_COMPILE)strip) -$(call allow-override,PKG_CONFIG,pkg-config) -$(call allow-override,LD_SO_CONF_PATH,/etc/ld.so.conf.d/) -$(call allow-override,LDCONFIG,ldconfig) - -INSTALL =3D install -MKDIR =3D mkdir -FOPTS :=3D -flto=3Dauto -ffat-lto-objects -fexceptions -fstack-protector-s= trong \ - -fasynchronous-unwind-tables -fstack-clash-protection -WOPTS :=3D -Wall -Werror=3Dformat-security -Wp,-D_FORTIFY_SOURCE=3D2 -Wp,= -D_GLIBCXX_ASSERTIONS -Wno-maybe-uninitialized - -ifeq ($(CC),clang) - FOPTS :=3D $(filter-out -ffat-lto-objects, $(FOPTS)) - WOPTS :=3D $(filter-out -Wno-maybe-uninitialized, $(WOPTS)) +# SPDX-License-Identifier: GPL-2.0-only + +ifeq ($(srctree),) + srctree :=3D $(patsubst %/,%,$(dir $(CURDIR))) + srctree :=3D $(patsubst %/,%,$(dir $(srctree))) + srctree :=3D $(patsubst %/,%,$(dir $(srctree))) endif =20 -TRACEFS_HEADERS :=3D $$($(PKG_CONFIG) --cflags libtracefs) - -CFLAGS :=3D -O -g -DVERSION=3D\"$(VERSION)\" $(FOPTS) $(MOPTS) $(WOPTS) $(= TRACEFS_HEADERS) $(EXTRA_CFLAGS) -LDFLAGS :=3D -flto=3Dauto -ggdb $(EXTRA_LDFLAGS) -LIBS :=3D $$($(PKG_CONFIG) --libs libtracefs) - -SRC :=3D $(wildcard src/*.c) -HDR :=3D $(wildcard src/*.h) -OBJ :=3D $(SRC:.c=3D.o) -DIRS :=3D src -FILES :=3D Makefile README.txt -CEXT :=3D bz2 -TARBALL :=3D $(NAME)-$(VERSION).tar.$(CEXT) -TAROPTS :=3D -cvjf $(TARBALL) -BINDIR :=3D /usr/bin -DATADIR :=3D /usr/share -DOCDIR :=3D $(DATADIR)/doc -MANDIR :=3D $(DATADIR)/man -LICDIR :=3D $(DATADIR)/licenses -SRCTREE :=3D $(or $(BUILD_SRC),$(CURDIR)) - -# If running from the tarball, man pages are stored in the Documentation -# dir. If running from the kernel source, man pages are stored in -# Documentation/tools/rtla/. -ifneq ($(wildcard Documentation/.*),) -DOCSRC =3D Documentation/ +include $(srctree)/tools/scripts/Makefile.include + +# O is an alias for OUTPUT +OUTPUT :=3D $(O) + +ifeq ($(OUTPUT),) + OUTPUT :=3D $(CURDIR) else -DOCSRC =3D $(SRCTREE)/../../../Documentation/tools/rtla/ + # subdir is used by the ../Makefile in $(call descend,) + ifneq ($(subdir),) + OUTPUT :=3D $(OUTPUT)/$(subdir) + endif +endif + +ifneq ($(patsubst %/,,$(lastword $(OUTPUT))),) + OUTPUT :=3D $(OUTPUT)/ endif =20 -LIBTRACEEVENT_MIN_VERSION =3D 1.5 -LIBTRACEFS_MIN_VERSION =3D 1.3 +RTLA :=3D $(OUTPUT)rtla +RTLA_IN :=3D $(RTLA)-in.o =20 -.PHONY: all warnings show_warnings -all: warnings rtla +VERSION :=3D $(shell sh -c "make -sC ../../.. kernelversion | grep -v mak= e") +DOCSRC :=3D ../../../Documentation/tools/rtla/ =20 -TEST_LIBTRACEEVENT =3D $(shell sh -c "$(PKG_CONFIG) --atleast-version $(LI= BTRACEEVENT_MIN_VERSION) libtraceevent > /dev/null 2>&1 || echo n") -ifeq ("$(TEST_LIBTRACEEVENT)", "n") -WARNINGS =3D show_warnings -MISSING_LIBS +=3D echo "** libtraceevent version $(LIBTRACEEVENT_MIN_VER= SION) or higher"; -MISSING_PACKAGES +=3D "libtraceevent-devel" -MISSING_SOURCE +=3D echo "** https://git.kernel.org/pub/scm/libs/libtrace= /libtraceevent.git/ "; +FEATURE_TESTS :=3D libtraceevent +FEATURE_TESTS +=3D libtracefs +FEATURE_DISPLAY :=3D libtraceevent +FEATURE_DISPLAY +=3D libtracefs + +ifeq ($(V),1) + Q =3D +else + Q =3D @ endif =20 -TEST_LIBTRACEFS =3D $(shell sh -c "$(PKG_CONFIG) --atleast-version $(LIBTR= ACEFS_MIN_VERSION) libtracefs > /dev/null 2>&1 || echo n") -ifeq ("$(TEST_LIBTRACEFS)", "n") -WARNINGS =3D show_warnings -MISSING_LIBS +=3D echo "** libtracefs version $(LIBTRACEFS_MIN_VERSION) = or higher"; -MISSING_PACKAGES +=3D "libtracefs-devel" -MISSING_SOURCE +=3D echo "** https://git.kernel.org/pub/scm/libs/libtrace= /libtracefs.git/ "; +all: $(RTLA) + +include $(srctree)/tools/build/Makefile.include +include Makefile.rtla + +# check for dependencies only on required targets +NON_CONFIG_TARGETS :=3D clean install tarball doc doc_clean doc_install + +config :=3D 1 +ifdef MAKECMDGOALS +ifeq ($(filter-out $(NON_CONFIG_TARGETS),$(MAKECMDGOALS)),) + config :=3D 0 +endif endif =20 -define show_dependencies - @echo "********************************************"; \ - echo "** NOTICE: Failed build dependencies"; \ - echo "**"; \ - echo "** Required Libraries:"; \ - $(MISSING_LIBS) \ - echo "**"; \ - echo "** Consider installing the latest libtracefs from your"; \ - echo "** distribution, e.g., 'dnf install $(MISSING_PACKAGES)' on Fedora,= "; \ - echo "** or from source:"; \ - echo "**"; \ - $(MISSING_SOURCE) \ - echo "**"; \ - echo "********************************************" -endef - -show_warnings: - $(call show_dependencies); - -ifneq ("$(WARNINGS)", "") -ERROR_OUT =3D $(error Please add the necessary dependencies) - -warnings: $(WARNINGS) - $(ERROR_OUT) +ifeq ($(config),1) + include $(srctree)/tools/build/Makefile.feature + include Makefile.config endif =20 -rtla: $(OBJ) - $(CC) -o rtla $(LDFLAGS) $(OBJ) $(LIBS) - -static: $(OBJ) - $(CC) -o rtla-static $(LDFLAGS) --static $(OBJ) $(LIBS) -lpthread -ldl - -.PHONY: install -install: doc_install - $(MKDIR) -p $(DESTDIR)$(BINDIR) - $(INSTALL) rtla -m 755 $(DESTDIR)$(BINDIR) - $(STRIP) $(DESTDIR)$(BINDIR)/rtla - @test ! -f $(DESTDIR)$(BINDIR)/osnoise || rm $(DESTDIR)$(BINDIR)/osnoise - ln -s rtla $(DESTDIR)$(BINDIR)/osnoise - @test ! -f $(DESTDIR)$(BINDIR)/hwnoise || rm $(DESTDIR)$(BINDIR)/hwnoise - ln -s rtla $(DESTDIR)$(BINDIR)/hwnoise - @test ! -f $(DESTDIR)$(BINDIR)/timerlat || rm $(DESTDIR)$(BINDIR)/timerlat - ln -s rtla $(DESTDIR)$(BINDIR)/timerlat - -.PHONY: clean tarball -clean: doc_clean - @test ! -f rtla || rm rtla - @test ! -f rtla-static || rm rtla-static - @test ! -f src/rtla.o || rm src/rtla.o - @test ! -f $(TARBALL) || rm -f $(TARBALL) - @rm -rf *~ $(OBJ) *.tar.$(CEXT) - -tarball: clean - rm -rf $(NAME)-$(VERSION) && mkdir $(NAME)-$(VERSION) - echo $(VERSION) > $(NAME)-$(VERSION)/VERSION - cp -r $(DIRS) $(FILES) $(NAME)-$(VERSION) - mkdir $(NAME)-$(VERSION)/Documentation/ - cp -rp $(SRCTREE)/../../../Documentation/tools/rtla/* $(NAME)-$(VERSION)/= Documentation/ - tar $(TAROPTS) --exclude=3D'*~' $(NAME)-$(VERSION) - rm -rf $(NAME)-$(VERSION) - -.PHONY: doc doc_clean doc_install -doc: - $(MAKE) -C $(DOCSRC) - -doc_clean: - $(MAKE) -C $(DOCSRC) clean - -doc_install: - $(MAKE) -C $(DOCSRC) install +CFLAGS +=3D $(INCLUDES) $(LIB_INCLUDES) + +export CFLAGS OUTPUT srctree + +$(RTLA): $(RTLA_IN) + $(QUIET_LINK)$(CC) $(LDFLAGS) -o $(RTLA) $(RTLA_IN) $(EXTLIBS) + +static: $(RTLA_IN) + $(eval LDFLAGS +=3D -static) + $(QUIET_LINK)$(CC) -static $(LDFLAGS) -o $(RTLA)-static $(RTLA_IN) $(EXT= LIBS) + +rtla.%: fixdep FORCE + make -f $(srctree)/tools/build/Makefile.build dir=3D. $@ + +$(RTLA_IN): fixdep FORCE + make $(build)=3Drtla + +clean: doc_clean fixdep-clean + $(call QUIET_CLEAN, rtla) + $(Q)find . -name '*.o' -delete -o -name '\.*.cmd' -delete -o -name '\.*.d= ' -delete + $(Q)rm -f rtla rtla-static fixdep FEATURE-DUMP rtla-* + $(Q)rm -rf feature +.PHONY: FORCE clean diff --git a/tools/tracing/rtla/Makefile.config b/tools/tracing/rtla/Makefi= le.config new file mode 100644 index 000000000000..6d4ba77847b6 --- /dev/null +++ b/tools/tracing/rtla/Makefile.config @@ -0,0 +1,47 @@ +# SPDX-License-Identifier: GPL-2.0-only + +STOP_ERROR :=3D + +LIBTRACEEVENT_MIN_VERSION =3D 1.5 +LIBTRACEFS_MIN_VERSION =3D 1.3 + +define lib_setup + $(eval LIB_INCLUDES +=3D $(shell sh -c "$(PKG_CONFIG) --cflags lib$(1)")) + $(eval EXTLIBS +=3D -l$(1)) +endef + +$(call feature_check,libtraceevent) +ifeq ($(feature-libtraceevent), 1) + $(call detected,CONFIG_LIBTRACEEVENT) + + TEST =3D $(shell sh -c "$(PKG_CONFIG) --atleast-version $(LIBTRACEEVENT_= MIN_VERSION) libtraceevent > /dev/null 2>&1 && echo y || echo n") + ifeq ($(TEST),n) + $(info libtraceevent version is too low, it must be at least $(LIBTRAC= EEVENT_MIN_VERSION)) + STOP_ERROR :=3D 1 + endif + + $(call lib_setup,traceevent) +else + STOP_ERROR :=3D 1 + $(info libtraceevent is missing. Please install libtraceevent-dev/libtra= ceevent-devel) +endif + +$(call feature_check,libtracefs) +ifeq ($(feature-libtracefs), 1) + $(call detected,CONFIG_LIBTRACEFS) + + TEST =3D $(shell sh -c "$(PKG_CONFIG) --atleast-version $(LIBTRACEFS_MIN= _VERSION) libtracefs > /dev/null 2>&1 && echo y || echo n") + ifeq ($(TEST),n) + $(info libtracefs version is too low, it must be at least $(LIBTRACEFS= _MIN_VERSION)) + STOP_ERROR :=3D 1 + endif + + $(call lib_setup,tracefs) +else + STOP_ERROR :=3D 1 + $(info libtracefs is missing. Please install libtracefs-dev/libtracefs-d= evel) +endif + +ifeq ($(STOP_ERROR),1) + $(error Please, check the errors above.) +endif diff --git a/tools/tracing/rtla/Makefile.rtla b/tools/tracing/rtla/Makefile= .rtla new file mode 100644 index 000000000000..3ff0b8970896 --- /dev/null +++ b/tools/tracing/rtla/Makefile.rtla @@ -0,0 +1,80 @@ +# SPDX-License-Identifier: GPL-2.0-only + +define allow-override + $(if $(or $(findstring environment,$(origin $(1))),\ + $(findstring command line,$(origin $(1)))),,\ + $(eval $(1) =3D $(2))) +endef + +# Allow setting CC and AR, or setting CROSS_COMPILE as a prefix. +$(call allow-override,CC,$(CROSS_COMPILE)gcc) +$(call allow-override,AR,$(CROSS_COMPILE)ar) +$(call allow-override,STRIP,$(CROSS_COMPILE)strip) +$(call allow-override,PKG_CONFIG,pkg-config) +$(call allow-override,LD_SO_CONF_PATH,/etc/ld.so.conf.d/) +$(call allow-override,LDCONFIG,ldconfig) +export CC AR STRIP PKG_CONFIG LD_SO_CONF_PATH LDCONFIG + +FOPTS :=3D -flto=3Dauto -ffat-lto-objects -fexceptions -fstack-protector-= strong \ + -fasynchronous-unwind-tables -fstack-clash-protection +WOPTS :=3D -O -Wall -Werror=3Dformat-security -Wp,-D_FORTIFY_SOURCE=3D2 \ + -Wp,-D_GLIBCXX_ASSERTIONS -Wno-maybe-uninitialized + +ifeq ($(CC),clang) + FOPTS :=3D $(filter-out -flto=3Dauto -ffat-lto-objects, $(FOPTS)) + WOPTS :=3D $(filter-out -Wno-maybe-uninitialized, $(WOPTS)) +endif + +CFLAGS :=3D -g -DVERSION=3D\"$(VERSION)\" $(FOPTS) $(WOPTS) $(CFLAGS) +LDFLAGS :=3D -ggdb $(LDFLAGS) + +RM :=3D rm -rf +LN :=3D ln -s +INSTALL :=3D install +MKDIR :=3D mkdir +STRIP :=3D strip +BINDIR :=3D /usr/bin + +.PHONY: install +install: doc_install + @$(MKDIR) -p $(DESTDIR)$(BINDIR) + $(call QUIET_INSTALL,rtla)$(INSTALL) rtla -m 755 $(DESTDIR)$(BINDIR) + @$(STRIP) $(DESTDIR)$(BINDIR)/rtla + @test ! -f $(DESTDIR)$(BINDIR)/osnoise || $(RM) $(DESTDIR)$(BINDIR)/osnoi= se + @$(LN) rtla $(DESTDIR)$(BINDIR)/osnoise + @test ! -f $(DESTDIR)$(BINDIR)/hwnoise || $(RM) $(DESTDIR)$(BINDIR)/hwnoi= se + @$(LN) -s rtla $(DESTDIR)$(BINDIR)/hwnoise + @test ! -f $(DESTDIR)$(BINDIR)/timerlat || $(RM) $(DESTDIR)$(BINDIR)/time= rlat + @$(LN) -s rtla $(DESTDIR)$(BINDIR)/timerlat + +.PHONY: doc doc_clean doc_install +doc: + $(MAKE) -C $(DOCSRC) + +doc_clean: + $(MAKE) -C $(DOCSRC) clean + +doc_install: + $(MAKE) -C $(DOCSRC) install + +# This section is neesary for the tarball, when the tarball +# support is removed, we can delete these entries. +NAME :=3D rtla +DIRS :=3D src +FILES :=3D Makefile README.txt +CEXT :=3D bz2 +TARBALL :=3D $(NAME)-$(VERSION).tar.$(CEXT) +TAROPTS :=3D -cvjf $(TARBALL) +SRCTREE :=3D $(or $(BUILD_SRC),$(CURDIR)) + +tarball: clean + $(RM) $(NAME)-$(VERSION) && $(MKDIR) $(NAME)-$(VERSION) + echo $(VERSION) > $(NAME)-$(VERSION)/VERSION + cp -r $(DIRS) $(FILES) $(NAME)-$(VERSION) + $(MKDIR) $(NAME)-$(VERSION)/Documentation/ + cp -rp $(SRCTREE)/../../../Documentation/tools/$(NAME)/* $(NAME)-$(VERSIO= N)/Documentation/ + cp Makefile.standalone $(NAME)-$(VERSION)/Makefile + cp Makefile.$(NAME) $(NAME)-$(VERSION)/ + tar $(TAROPTS) --exclude=3D'*~' $(NAME)-$(VERSION) + $(RM) $(NAME)-$(VERSION) +.PHONY: tarball diff --git a/tools/tracing/rtla/Makefile.standalone b/tools/tracing/rtla/Ma= kefile.standalone new file mode 100644 index 000000000000..86d07cb52fa5 --- /dev/null +++ b/tools/tracing/rtla/Makefile.standalone @@ -0,0 +1,26 @@ +# SPDX-License-Identifier: GPL-2.0-only + +VERSION :=3D $(shell cat VERSION) +CFLAGS +=3D $$($(PKG_CONFIG) --cflags libtracefs) +EXTLIBS +=3D $$($(PKG_CONFIG) --libs libtracefs) + +rtla: + +include Makefile.rtla + +SRC :=3D $(wildcard src/*.c) +HDR :=3D $(wildcard src/*.h) +OBJ :=3D $(SRC:.c=3D.o) +DOCSRC :=3D Documentation/ + +rtla: $(OBJ) + $(CC) -o rtla $(LDFLAGS) $(OBJ) $(LIBS) $(EXTLIBS) + $(info This is a deprecated method to compile RTLA, please compile from L= inux kernel source) + +.PHONY: clean tarball +clean: doc_clean + @test ! -f rtla || rm rtla + @test ! -f rtla-static || rm rtla-static + @test ! -f src/rtla.o || rm src/rtla.o + @test ! -f $(TARBALL) || rm -f $(TARBALL) + @rm -rf *~ $(OBJ) *.tar.$(CEXT) diff --git a/tools/tracing/rtla/src/Build b/tools/tracing/rtla/src/Build new file mode 100644 index 000000000000..dbed9e31829b --- /dev/null +++ b/tools/tracing/rtla/src/Build @@ -0,0 +1,11 @@ +rtla-y +=3D trace.o +rtla-y +=3D utils.o +rtla-y +=3D osnoise.o +rtla-y +=3D osnoise_top.o +rtla-y +=3D osnoise_hist.o +rtla-y +=3D timerlat.o +rtla-y +=3D timerlat_top.o +rtla-y +=3D timerlat_hist.o +rtla-y +=3D timerlat_u.o +rtla-y +=3D timerlat_aa.o +rtla-y +=3D rtla.o --=20 2.44.0 From nobody Wed Feb 11 05:02:10 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 706B43BBF8; Fri, 15 Mar 2024 14:25:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710512729; cv=none; b=s5KVzNaOrO0OKnc40SQNVRjtKhKv0CAoorPgiVKeadmg04RYLs56nycgSm/Or0j9nl50i+X12/4asBD0bxrGYsV/AcHmevkQ4SJWPo2KOQmNoB4u2QDFmBC+Vrc6/3UApWexdZEsXn3/y2ApBRov0QkbxuQTY0NAvor+HdvCrj4= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710512729; c=relaxed/simple; bh=spCdXPyAduE0DcQHiqfxpNxoQQ+cPRYp9CjjQg2iIFQ=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=s+V148nWIJDOVTsutfYOdPgaa6PUQi0zj/PkBrc7bQw6oIKgMbEgD/VcCq8aYk/KgisS1Y1Ztbbq21bORemAs4XPEjG27ek9iMoCcfoWZ1Y42EnLcTOBqHujhfUw4bmxggZKv7w9QNruVqWZbKVkPBS4N57P/MaDl6KAUe/Nbic= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=tflolLaA; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="tflolLaA" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 474EBC43399; Fri, 15 Mar 2024 14:25:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1710512729; bh=spCdXPyAduE0DcQHiqfxpNxoQQ+cPRYp9CjjQg2iIFQ=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=tflolLaA1xVDxlY5Y7plAbuV/exgzYI0PW+xkEX5nwJ7nXtTtQroapsaCjUHnsGgo ZsAcoVrYwpsGP2oOex4xkMZpos/0fj74HrmwPf43bcTTOLb9oSGFFeaEY3KbjWhoVA 3yoLZHF1J7PeJGazkTL/XL5t/YGwXMm2RL8h0qpEu5X8vxMj5yVQnkVCy5zItbxnmi z/hqjVEZ+rmMkydb3cYV2RxvhSRkUhP9YyfmnzJJNsZ3gzzFH4DcSy9T3cIgJ9eDbt DpESvey8gWlRSF4ag235p9BVqf9HwvlkxVUh7jl6epWlvuc3wPMr0pr5tp09CTw7Yk 5lCFLZKqx19CA== From: Daniel Bristot de Oliveira To: Linus Torvalds , Steven Rostedt , Arnaldo Carvalho de Melo Cc: Daniel Bristot de Oliveira , Josh Poimboeuf , John Kacur , Namhyung Kim , Jiri Olsa , linux-perf-users@vger.kernel.org, linux-trace-kernel@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH V2 3/3] tools/verification: Use tools/build makefiles on rv Date: Fri, 15 Mar 2024 15:25:01 +0100 Message-ID: X-Mailer: git-send-email 2.44.0 In-Reply-To: References: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Use tools/build/ makefiles to build rv, inheriting the benefits of it. For example, having a proper way to handle dependencies. Suggested-by: Linus Torvalds Signed-off-by: Daniel Bristot de Oliveira --- tools/verification/rv/.gitignore | 6 + tools/verification/rv/Build | 1 + tools/verification/rv/Makefile | 207 +++++++++----------------- tools/verification/rv/Makefile.config | 47 ++++++ tools/verification/rv/Makefile.rv | 51 +++++++ tools/verification/rv/src/Build | 4 + 6 files changed, 183 insertions(+), 133 deletions(-) create mode 100644 tools/verification/rv/.gitignore create mode 100644 tools/verification/rv/Build create mode 100644 tools/verification/rv/Makefile.config create mode 100644 tools/verification/rv/Makefile.rv create mode 100644 tools/verification/rv/src/Build diff --git a/tools/verification/rv/.gitignore b/tools/verification/rv/.giti= gnore new file mode 100644 index 000000000000..34a486585a34 --- /dev/null +++ b/tools/verification/rv/.gitignore @@ -0,0 +1,6 @@ +# SPDX-License-Identifier: GPL-2.0-only +rv +rv-static +fixdep +feature +FEATURE-DUMP diff --git a/tools/verification/rv/Build b/tools/verification/rv/Build new file mode 100644 index 000000000000..a44c22349d4b --- /dev/null +++ b/tools/verification/rv/Build @@ -0,0 +1 @@ +rv-y +=3D src/ diff --git a/tools/verification/rv/Makefile b/tools/verification/rv/Makefile index 485f8aeddbe0..411d62b3d8eb 100644 --- a/tools/verification/rv/Makefile +++ b/tools/verification/rv/Makefile @@ -1,146 +1,87 @@ -NAME :=3D rv -# Follow the kernel version -VERSION :=3D $(shell cat VERSION 2> /dev/null || make -sC ../../.. kernelv= ersion | grep -v make) - -# From libtracefs: -# Makefiles suck: This macro sets a default value of $(2) for the -# variable named by $(1), unless the variable has been set by -# environment or command line. This is necessary for CC and AR -# because make sets default values, so the simpler ?=3D approach -# won't work as expected. -define allow-override - $(if $(or $(findstring environment,$(origin $(1))),\ - $(findstring command line,$(origin $(1)))),,\ - $(eval $(1) =3D $(2))) -endef - -# Allow setting CC and AR, or setting CROSS_COMPILE as a prefix. -$(call allow-override,CC,$(CROSS_COMPILE)gcc) -$(call allow-override,AR,$(CROSS_COMPILE)ar) -$(call allow-override,STRIP,$(CROSS_COMPILE)strip) -$(call allow-override,PKG_CONFIG,pkg-config) -$(call allow-override,LD_SO_CONF_PATH,/etc/ld.so.conf.d/) -$(call allow-override,LDCONFIG,ldconfig) - -INSTALL =3D install -MKDIR =3D mkdir -FOPTS :=3D -flto=3Dauto -ffat-lto-objects -fexceptions -fstack-protector-s= trong \ - -fasynchronous-unwind-tables -fstack-clash-protection -WOPTS :=3D -Wall -Werror=3Dformat-security -Wp,-D_FORTIFY_SOURCE=3D2 -Wp,= -D_GLIBCXX_ASSERTIONS -Wno-maybe-uninitialized - -ifeq ($(CC),clang) - FOPTS :=3D $(filter-out -ffat-lto-objects, $(FOPTS)) - WOPTS :=3D $(filter-out -Wno-maybe-uninitialized, $(WOPTS)) +# SPDX-License-Identifier: GPL-2.0-only + +ifeq ($(srctree),) + srctree :=3D $(patsubst %/,%,$(dir $(CURDIR))) + srctree :=3D $(patsubst %/,%,$(dir $(srctree))) + srctree :=3D $(patsubst %/,%,$(dir $(srctree))) endif =20 -TRACEFS_HEADERS :=3D $$($(PKG_CONFIG) --cflags libtracefs) - -CFLAGS :=3D -O -g -DVERSION=3D\"$(VERSION)\" $(FOPTS) $(MOPTS) $(WOPTS) $(= TRACEFS_HEADERS) $(EXTRA_CFLAGS) -I include -LDFLAGS :=3D -flto=3Dauto -ggdb $(EXTRA_LDFLAGS) -LIBS :=3D $$($(PKG_CONFIG) --libs libtracefs) - -SRC :=3D $(wildcard src/*.c) -HDR :=3D $(wildcard src/*.h) -OBJ :=3D $(SRC:.c=3D.o) -DIRS :=3D src -FILES :=3D Makefile README.txt -CEXT :=3D bz2 -TARBALL :=3D $(NAME)-$(VERSION).tar.$(CEXT) -TAROPTS :=3D -cvjf $(TARBALL) -BINDIR :=3D /usr/bin -DATADIR :=3D /usr/share -DOCDIR :=3D $(DATADIR)/doc -MANDIR :=3D $(DATADIR)/man -LICDIR :=3D $(DATADIR)/licenses -SRCTREE :=3D $(or $(BUILD_SRC),$(CURDIR)) - -# If running from the tarball, man pages are stored in the Documentation -# dir. If running from the kernel source, man pages are stored in -# Documentation/tools/rv/. -ifneq ($(wildcard Documentation/.*),) -DOCSRC =3D Documentation/ +include $(srctree)/tools/scripts/Makefile.include + +# O is an alias for OUTPUT +OUTPUT :=3D $(O) + +ifeq ($(OUTPUT),) + OUTPUT :=3D $(CURDIR)/ else -DOCSRC =3D $(SRCTREE)/../../../Documentation/tools/rv/ + # subdir is used by the ../Makefile in $(call descend,) + ifneq ($(subdir),) + OUTPUT :=3D $(OUTPUT)/$(subdir) + endif +endif + +ifneq ($(patsubst %/,,$(lastword $(OUTPUT))),) + OUTPUT :=3D $(OUTPUT)/ endif =20 -LIBTRACEEVENT_MIN_VERSION =3D 1.5 -LIBTRACEFS_MIN_VERSION =3D 1.3 +RV :=3D $(OUTPUT)rv +RV_IN :=3D $(RV)-in.o =20 -.PHONY: all warnings show_warnings -all: warnings rv +VERSION :=3D $(shell sh -c "make -sC ../../.. kernelversion | grep -v mak= e") +DOCSRC :=3D ../../../Documentation/tools/rv/ =20 -TEST_LIBTRACEEVENT =3D $(shell sh -c "$(PKG_CONFIG) --atleast-version $(LI= BTRACEEVENT_MIN_VERSION) libtraceevent > /dev/null 2>&1 || echo n") -ifeq ("$(TEST_LIBTRACEEVENT)", "n") -WARNINGS =3D show_warnings -MISSING_LIBS +=3D echo "** libtraceevent version $(LIBTRACEEVENT_MIN_VER= SION) or higher"; -MISSING_PACKAGES +=3D "libtraceevent-devel" -MISSING_SOURCE +=3D echo "** https://git.kernel.org/pub/scm/libs/libtrace= /libtraceevent.git/ "; +FEATURE_TESTS :=3D libtraceevent +FEATURE_TESTS +=3D libtracefs +FEATURE_DISPLAY :=3D libtraceevent +FEATURE_DISPLAY +=3D libtracefs + +ifeq ($(V),1) + Q =3D +else + Q =3D @ endif =20 -TEST_LIBTRACEFS =3D $(shell sh -c "$(PKG_CONFIG) --atleast-version $(LIBTR= ACEFS_MIN_VERSION) libtracefs > /dev/null 2>&1 || echo n") -ifeq ("$(TEST_LIBTRACEFS)", "n") -WARNINGS =3D show_warnings -MISSING_LIBS +=3D echo "** libtracefs version $(LIBTRACEFS_MIN_VERSION) = or higher"; -MISSING_PACKAGES +=3D "libtracefs-devel" -MISSING_SOURCE +=3D echo "** https://git.kernel.org/pub/scm/libs/libtrace= /libtracefs.git/ "; +all: $(RV) + +include $(srctree)/tools/build/Makefile.include +include Makefile.rv + +# check for dependencies only on required targets +NON_CONFIG_TARGETS :=3D clean install doc doc_clean doc_install + +config :=3D 1 +ifdef MAKECMDGOALS + ifeq ($(filter-out $(NON_CONFIG_TARGETS),$(MAKECMDGOALS)),) + config :=3D 0 +endif endif =20 -define show_dependencies - @echo "********************************************"; \ - echo "** NOTICE: Failed build dependencies"; \ - echo "**"; \ - echo "** Required Libraries:"; \ - $(MISSING_LIBS) \ - echo "**"; \ - echo "** Consider installing the latest libtracefs from your"; \ - echo "** distribution, e.g., 'dnf install $(MISSING_PACKAGES)' on Fedora,= "; \ - echo "** or from source:"; \ - echo "**"; \ - $(MISSING_SOURCE) \ - echo "**"; \ - echo "********************************************" -endef - -show_warnings: - $(call show_dependencies); - -ifneq ("$(WARNINGS)", "") -ERROR_OUT =3D $(error Please add the necessary dependencies) - -warnings: $(WARNINGS) - $(ERROR_OUT) +ifeq ($(config),1) + include $(srctree)/tools/build/Makefile.feature + include Makefile.config endif =20 -rv: $(OBJ) - $(CC) -o rv $(LDFLAGS) $(OBJ) $(LIBS) - -.PHONY: install -install: doc_install - $(MKDIR) -p $(DESTDIR)$(BINDIR) - $(INSTALL) rv -m 755 $(DESTDIR)$(BINDIR) - $(STRIP) $(DESTDIR)$(BINDIR)/rv - -.PHONY: clean tarball -clean: doc_clean - @test ! -f rv || rm rv - @test ! -f $(TARBALL) || rm -f $(TARBALL) - @rm -rf *~ $(OBJ) *.tar.$(CEXT) - -tarball: clean - rm -rf $(NAME)-$(VERSION) && mkdir $(NAME)-$(VERSION) - echo $(VERSION) > $(NAME)-$(VERSION)/VERSION - cp -r $(DIRS) $(FILES) $(NAME)-$(VERSION) - mkdir $(NAME)-$(VERSION)/Documentation/ - cp -rp $(SRCTREE)/../../../Documentation/tools/rv/* $(NAME)-$(VERSION)/Do= cumentation/ - tar $(TAROPTS) --exclude=3D'*~' $(NAME)-$(VERSION) - rm -rf $(NAME)-$(VERSION) - -.PHONY: doc doc_clean doc_install -doc: - $(MAKE) -C $(DOCSRC) - -doc_clean: - $(MAKE) -C $(DOCSRC) clean - -doc_install: - $(MAKE) -C $(DOCSRC) install +CFLAGS +=3D $(INCLUDES) $(LIB_INCLUDES) + +export CFLAGS OUTPUT srctree + +$(RV): $(RV_IN) + $(QUIET_LINK)$(CC) $(LDFLAGS) -o $(RV) $(RV_IN) $(EXTLIBS) + +static: $(RV_IN) + $(eval LDFLAGS +=3D -static) + $(QUIET_LINK)$(CC) $(LDFLAGS) -o $(RV)-static $(RV_IN) $(EXTLIBS) + +rv.%: fixdep FORCE + make -f $(srctree)/tools/build/Makefile.build dir=3D. $@ + +$(RV_IN): fixdep FORCE + make $(build)=3Drv + +clean: doc_clean fixdep-clean + $(call QUIET_CLEAN, rv) + $(Q)find . -name '*.o' -delete -o -name '\.*.cmd' -delete -o -name '\.*.d= ' -delete + $(Q)rm -f rv rv-static fixdep FEATURE-DUMP rv-* + $(Q)rm -rf feature + +.PHONY: FORCE clean diff --git a/tools/verification/rv/Makefile.config b/tools/verification/rv/= Makefile.config new file mode 100644 index 000000000000..6d4ba77847b6 --- /dev/null +++ b/tools/verification/rv/Makefile.config @@ -0,0 +1,47 @@ +# SPDX-License-Identifier: GPL-2.0-only + +STOP_ERROR :=3D + +LIBTRACEEVENT_MIN_VERSION =3D 1.5 +LIBTRACEFS_MIN_VERSION =3D 1.3 + +define lib_setup + $(eval LIB_INCLUDES +=3D $(shell sh -c "$(PKG_CONFIG) --cflags lib$(1)")) + $(eval EXTLIBS +=3D -l$(1)) +endef + +$(call feature_check,libtraceevent) +ifeq ($(feature-libtraceevent), 1) + $(call detected,CONFIG_LIBTRACEEVENT) + + TEST =3D $(shell sh -c "$(PKG_CONFIG) --atleast-version $(LIBTRACEEVENT_= MIN_VERSION) libtraceevent > /dev/null 2>&1 && echo y || echo n") + ifeq ($(TEST),n) + $(info libtraceevent version is too low, it must be at least $(LIBTRAC= EEVENT_MIN_VERSION)) + STOP_ERROR :=3D 1 + endif + + $(call lib_setup,traceevent) +else + STOP_ERROR :=3D 1 + $(info libtraceevent is missing. Please install libtraceevent-dev/libtra= ceevent-devel) +endif + +$(call feature_check,libtracefs) +ifeq ($(feature-libtracefs), 1) + $(call detected,CONFIG_LIBTRACEFS) + + TEST =3D $(shell sh -c "$(PKG_CONFIG) --atleast-version $(LIBTRACEFS_MIN= _VERSION) libtracefs > /dev/null 2>&1 && echo y || echo n") + ifeq ($(TEST),n) + $(info libtracefs version is too low, it must be at least $(LIBTRACEFS= _MIN_VERSION)) + STOP_ERROR :=3D 1 + endif + + $(call lib_setup,tracefs) +else + STOP_ERROR :=3D 1 + $(info libtracefs is missing. Please install libtracefs-dev/libtracefs-d= evel) +endif + +ifeq ($(STOP_ERROR),1) + $(error Please, check the errors above.) +endif diff --git a/tools/verification/rv/Makefile.rv b/tools/verification/rv/Make= file.rv new file mode 100644 index 000000000000..161baa29eb86 --- /dev/null +++ b/tools/verification/rv/Makefile.rv @@ -0,0 +1,51 @@ +# SPDX-License-Identifier: GPL-2.0-only + +define allow-override + $(if $(or $(findstring environment,$(origin $(1))),\ + $(findstring command line,$(origin $(1)))),,\ + $(eval $(1) =3D $(2))) +endef + +# Allow setting CC and AR, or setting CROSS_COMPILE as a prefix. +$(call allow-override,CC,$(CROSS_COMPILE)gcc) +$(call allow-override,AR,$(CROSS_COMPILE)ar) +$(call allow-override,STRIP,$(CROSS_COMPILE)strip) +$(call allow-override,PKG_CONFIG,pkg-config) +$(call allow-override,LD_SO_CONF_PATH,/etc/ld.so.conf.d/) +$(call allow-override,LDCONFIG,ldconfig) +export CC AR STRIP PKG_CONFIG LD_SO_CONF_PATH LDCONFIG + +FOPTS :=3D-flto=3Dauto -ffat-lto-objects -fexceptions -fstack-protector-s= trong \ + -fasynchronous-unwind-tables -fstack-clash-protection +WOPTS :=3D -O -Wall -Werror=3Dformat-security -Wp,-D_FORTIFY_SOURCE=3D2 \ + -Wp,-D_GLIBCXX_ASSERTIONS -Wno-maybe-uninitialized + +ifeq ($(CC),clang) + FOPTS :=3D $(filter-out -flto=3Dauto -ffat-lto-objects, $(FOPTS)) + WOPTS :=3D $(filter-out -Wno-maybe-uninitialized, $(WOPTS)) +endif + +INCLUDE :=3D -Iinclude/ +CFLAGS :=3D -g -DVERSION=3D\"$(VERSION)\" $(FOPTS) $(WOPTS) $(EXTRA_CFLAG= S) $(INCLUDE) +LDFLAGS :=3D -ggdb $(EXTRA_LDFLAGS) + +INSTALL :=3D install +MKDIR :=3D mkdir +STRIP :=3D strip +BINDIR :=3D /usr/bin + +.PHONY: install +install: doc_install + $(Q)$(MKDIR) -p $(DESTDIR)$(BINDIR) + $(call QUIET_INSTALL,rv)$(INSTALL) $(OUTPUT)rv -m 755 $(DESTDIR)$(BINDIR) + $(Q)@$(STRIP) $(DESTDIR)$(BINDIR)/rv + +.PHONY: doc doc_clean doc_install +doc: + $(MAKE) -C $(DOCSRC) + +doc_clean: + $(MAKE) -C $(DOCSRC) clean + +doc_install: + $(MAKE) -C $(DOCSRC) install diff --git a/tools/verification/rv/src/Build b/tools/verification/rv/src/Bu= ild new file mode 100644 index 000000000000..d781983c1a79 --- /dev/null +++ b/tools/verification/rv/src/Build @@ -0,0 +1,4 @@ +rv-y +=3D trace.o +rv-y +=3D utils.o +rv-y +=3D in_kernel.o +rv-y +=3D rv.o --=20 2.44.0