From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Wed, 08 May 2024 11:59:16 +0200 Received: from metis.whiteo.stw.pengutronix.de ([2a0a:edc0:2:b01:1d::104]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1s4e4y-002kGp-13 for lore@lore.pengutronix.de; Wed, 08 May 2024 11:59:16 +0200 Received: from localhost ([127.0.0.1] helo=metis.whiteo.stw.pengutronix.de) by metis.whiteo.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1s4e4x-0004Ru-Jb; Wed, 08 May 2024 11:59:15 +0200 Received: from drehscheibe.grey.stw.pengutronix.de ([2a0a:edc0:0:c01:1d::a2]) by metis.whiteo.stw.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1s4e4l-0004Rh-4V; Wed, 08 May 2024 11:59:03 +0200 Received: from [2a0a:edc0:2:b01:1d::c5] (helo=pty.whiteo.stw.pengutronix.de) by drehscheibe.grey.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1s4e4k-000FZs-OD; Wed, 08 May 2024 11:59:02 +0200 Received: from mtr by pty.whiteo.stw.pengutronix.de with local (Exim 4.96) (envelope-from ) id 1s4e4k-000aC8-28; Wed, 08 May 2024 11:59:02 +0200 Date: Wed, 8 May 2024 11:59:02 +0200 From: Michael Tretter To: ptxdist@pengutronix.de, Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= Message-ID: References: <20240508090845.2143026-1-u.kleine-koenig@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20240508090845.2143026-1-u.kleine-koenig@pengutronix.de> X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-Accept-Language: de,en X-Accept-Content-Type: text/plain Subject: Re: [ptxdist] [PATCH] libtraceevent: new package X-BeenThere: ptxdist@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Sender: "ptxdist" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: ptxdist-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.whiteo.stw.pengutronix.de); SAEximRunCond expanded to false On Wed, 08 May 2024 11:08:45 +0200, Uwe Kleine-König wrote: > This is required for perf (PTXCONF_KERNEL_TOOL_PERF) to work with > tracepoints and enable the subcommands kmem, kwork, lock, sched and > timechart. The perf recipe is adapted accordingly to make use of it. Nice. Thanks for the patch. Maybe the addition of libtraceevent and the change to use it in perf could be separate patches. > > Signed-off-by: Uwe Kleine-König > --- > rules/kernel.in | 1 + > rules/kernel.make | 1 - > rules/libtraceevent.in | 8 +++++ > rules/libtraceevent.make | 75 ++++++++++++++++++++++++++++++++++++++++ > 4 files changed, 84 insertions(+), 1 deletion(-) > create mode 100644 rules/libtraceevent.in > create mode 100644 rules/libtraceevent.make > > diff --git a/rules/kernel.in b/rules/kernel.in > index 817481577..862d52598 100644 > --- a/rules/kernel.in > +++ b/rules/kernel.in > @@ -6,6 +6,7 @@ menuconfig KERNEL > select LIBELF_LIBDW if KERNEL_TOOL_PERF > select SLANG if KERNEL_TOOL_PERF > select BINUTILS if KERNEL_TOOL_PERF > + select LIBTRACEEVENT if KERNEL_TOOL_PERF > select LIBUNWIND if KERNEL_TOOL_PERF > > if KERNEL > diff --git a/rules/kernel.make b/rules/kernel.make > index d82aa27fb..0fe4f1e1b 100644 > --- a/rules/kernel.make > +++ b/rules/kernel.make > @@ -223,7 +223,6 @@ KERNEL_TOOL_PERF_OPTS := \ > NO_LIBBIONIC=1 \ > NO_LIBCRYPTO=1 \ > NO_LIBDW_DWARF_UNWIND= \ > - NO_LIBTRACEEVENT=1 \ > NO_PERF_READ_VDSO32=1 \ > NO_PERF_READ_VDSOX32=1 \ > NO_ZLIB= \ > diff --git a/rules/libtraceevent.in b/rules/libtraceevent.in > new file mode 100644 > index 000000000..de1c7675f > --- /dev/null > +++ b/rules/libtraceevent.in > @@ -0,0 +1,8 @@ > +## SECTION=project_specific The section is wrong: Maybe SECTION=system_libraries > + > +config LIBTRACEEVENT > + bool > + prompt "libtraceevent" > + select HOST_MESON > + help > + FIXME Add a help text: Maybe "Linux kernel trace event library" > diff --git a/rules/libtraceevent.make b/rules/libtraceevent.make > new file mode 100644 > index 000000000..6017a983c > --- /dev/null > +++ b/rules/libtraceevent.make > @@ -0,0 +1,75 @@ > +# -*-makefile-*- > +# > +# Copyright (C) 2024 by Uwe Kleine-Koenig > +# > +# For further information about the PTXdist project and license conditions > +# see the README file. > +# > + > +# > +# We provide this package > +# > +PACKAGES-$(PTXCONF_LIBTRACEEVENT) += libtraceevent > + > +# > +# Paths and names > +# > +LIBTRACEEVENT_VERSION := 1.8.2 > +LIBTRACEEVENT_MD5 := 671645965e835ef4236b96851fb889c9 > +LIBTRACEEVENT := libtraceevent-$(LIBTRACEEVENT_VERSION) > +LIBTRACEEVENT_SUFFIX := tar.xz > +LIBTRACEEVENT_URL := https://git.kernel.org/pub/scm/libs/libtrace/libtraceevent.git;tag=libtraceevent-$(LIBTRACEEVENT_VERSION) > +LIBTRACEEVENT_SOURCE := $(SRCDIR)/$(LIBTRACEEVENT).$(LIBTRACEEVENT_SUFFIX) > +LIBTRACEEVENT_DIR := $(BUILDDIR)/$(LIBTRACEEVENT) > +LIBTRACEEVENT_LICENSE := LGPL-2.1 + GPL-2.0 LIBTRACEEVENT_LICENSE := LGPL-2.1 AND GPL-2.0 Maybe set LIBTRACEEVENT_LICENSE_FILES, too. > + > +# ---------------------------------------------------------------------------- > +# Get > +# ---------------------------------------------------------------------------- > + > +#$(LIBTRACEEVENT_SOURCE): > +# @$(call targetinfo) > +# @$(call get, LIBTRACEEVENT) Drop the empty get stage. > + > +# ---------------------------------------------------------------------------- > +# Prepare > +# ---------------------------------------------------------------------------- > + > +#LIBTRACEEVENT_CONF_ENV := $(CROSS_ENV) > + > +# > +# meson > +# > +LIBTRACEEVENT_CONF_TOOL := meson > +LIBTRACEEVENT_CONF_OPT := \ > + $(CROSS_MESON_USR) \ > + -Ddoc=false The list isn't complete. Use the scripts/configure_helper.py to explicitly set all options. > + > +# ---------------------------------------------------------------------------- > +# Target-Install > +# ---------------------------------------------------------------------------- > + > +$(STATEDIR)/libtraceevent.targetinstall: > + @$(call targetinfo) > + > + @$(call install_init, libtraceevent) > + @$(call install_fixup, libtraceevent, PRIORITY, optional) > + @$(call install_fixup, libtraceevent, SECTION, base) > + @$(call install_fixup, libtraceevent, AUTHOR, "Uwe Kleine-König ") > + @$(call install_fixup, libtraceevent, DESCRIPTION, missing) > + > + @$(call install_lib, libtraceevent, 0, 0, 0644, libtraceevent) > + > + @$(call install_finish, libtraceevent) > + > + @$(call touch) > + > +# ---------------------------------------------------------------------------- > +# Clean > +# ---------------------------------------------------------------------------- > + > +#$(STATEDIR)/libtraceevent.clean: > +# @$(call targetinfo) > +# @$(call clean_pkg, LIBTRACEEVENT) Drop the empty clean stage. Michael > + > +# vim: syntax=make > -- > 2.39.2 > > >