From mboxrd@z Thu Jan 1 00:00:00 1970 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 2DB33101C8; Sun, 29 Sep 2024 02:02:01 +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=1727575321; cv=none; b=dGMvS8UkfEOBcyYEQ7fZOw9xbfgVLauB8R8EHQCVaMgLWnRLQcOTm7dqG/JENKuoaX9BOf7gW8eMMPT76HuHQRt28il3ni4qfK73bAmIFrE8+me8NmItr8jD1fuVpBjKID5T2bXtirSsooF2EHquH673yIo8JsiBDmx+Rn/GzcU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1727575321; c=relaxed/simple; bh=xq0w9pry4t4F4+Anm2I9iKI2f8yg1nmPvpfL9KTINsA=; h=Date:From:To:Cc:Subject:Message-Id:In-Reply-To:References: Mime-Version:Content-Type; b=pcdP+FrubNvvToU5M727IjbCEHyzkKnk0IaQ4xdPy0xDVFUpVO5rnd623FlPOUdOUffb88C38vbOj2kPI6ETulnSQ0GiMFbqx6coQ7YOrv/WmK6IL5Ogdwjce+j8Uzb7VUGlcWBBoFXlj3luodXACmd7wkKl9XZ1jVrzp2/PqOs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=IW8HkFcG; 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="IW8HkFcG" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 19425C4CEC9; Sun, 29 Sep 2024 02:01:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1727575321; bh=xq0w9pry4t4F4+Anm2I9iKI2f8yg1nmPvpfL9KTINsA=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=IW8HkFcGNPMFanHo9WLf6ndqwsALy3vO7S99juiza78H8Rq83quV+bajgWPn5kS9R uxgbyg8OjykIkNsfyQs5l1aj+6eKUdizZz5Tongt3WkZ1N1HuCu7zHATzteF+Atvis TP9hz0FdAmYJz88O0o9qNyVTRgwX0vksol6NoOsHgG6y9KEZk37hXbp5xbo940v3tr SKI5J/9UDhpNTj40s9E/LPudS18uDCCvUjYuCPjn3Tp4h/UmVPqblblZcdEv8S2AKG o1Hk/2x4hrckaQCdsXOaU9qV3rDO1c2WTXp5jJyYv2/bOeS6eyufGR9aUtrcQg9SCE W7jywSBdOu+MA== Date: Sun, 29 Sep 2024 11:01:49 +0900 From: Masami Hiramatsu (Google) To: Ian Rogers Cc: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Namhyung Kim , Mark Rutland , Alexander Shishkin , Jiri Olsa , Adrian Hunter , Kan Liang , John Garry , Will Deacon , James Clark , Mike Leach , Leo Yan , Guo Ren , Paul Walmsley , Palmer Dabbelt , Albert Ou , Nick Terrell , Guilherme Amadio , Changbin Du , "Steinar H. Gunderson" , Aditya Gupta , Athira Rajeev , Masahiro Yamada , Huacai Chen , Bibo Mao , Kajol Jain , Anup Patel , Shenlin Liang , Atish Patra , Oliver Upton , Chen Pei , Dima Kogan , Alexander Lobakin , "David S. Miller" , Przemek Kitszel , linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, Yang Jihong Subject: Re: [PATCH v1 01/11] perf build: Rename NO_DWARF to NO_LIBDW Message-Id: <20240929110149.d0b15305217af9b2a25b84a9@kernel.org> In-Reply-To: <20240924160418.1391100-2-irogers@google.com> References: <20240924160418.1391100-1-irogers@google.com> <20240924160418.1391100-2-irogers@google.com> X-Mailer: Sylpheed 3.7.0 (GTK+ 2.24.33; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Tue, 24 Sep 2024 09:04:08 -0700 Ian Rogers wrote: > NO_DWARF could mean more than NO_LIBDW support, in particular no > libunwind support. Rename to be more intention revealing. > Looks good to me. Reviewed-by: Masami Hiramatsu (Google) Thanks, > Signed-off-by: Ian Rogers > --- > tools/perf/Makefile.config | 16 ++++++++-------- > tools/perf/Makefile.perf | 2 +- > tools/perf/arch/arm/Makefile | 2 +- > tools/perf/arch/arm64/Makefile | 2 +- > tools/perf/arch/csky/Makefile | 2 +- > tools/perf/arch/loongarch/Makefile | 2 +- > tools/perf/arch/mips/Makefile | 2 +- > tools/perf/arch/powerpc/Makefile | 2 +- > tools/perf/arch/riscv/Makefile | 2 +- > tools/perf/arch/s390/Makefile | 2 +- > tools/perf/arch/sh/Makefile | 2 +- > tools/perf/arch/sparc/Makefile | 2 +- > tools/perf/arch/x86/Makefile | 2 +- > tools/perf/arch/xtensa/Makefile | 2 +- > tools/perf/builtin-probe.c | 2 +- > 15 files changed, 22 insertions(+), 22 deletions(-) > > diff --git a/tools/perf/Makefile.config b/tools/perf/Makefile.config > index 5e26d3a91b36..55a39211496d 100644 > --- a/tools/perf/Makefile.config > +++ b/tools/perf/Makefile.config > @@ -429,7 +429,7 @@ ifeq ($(feature-file-handle), 1) > endif > > ifdef NO_LIBELF > - NO_DWARF := 1 > + NO_LIBDW := 1 > NO_LIBUNWIND := 1 > NO_LIBDW_DWARF_UNWIND := 1 > NO_LIBBPF := 1 > @@ -471,9 +471,9 @@ else > endif > endif > ifneq ($(feature-dwarf), 1) > - ifndef NO_DWARF > + ifndef NO_LIBDW > $(warning No libdw.h found or old libdw.h found or elfutils is older than 0.138, disables dwarf support. Please install new elfutils-devel/libdw-dev) > - NO_DWARF := 1 > + NO_LIBDW := 1 > endif > else > ifneq ($(feature-dwarf_getlocations), 1) > @@ -496,7 +496,7 @@ ifeq ($(feature-libaio), 1) > endif > endif > > -ifdef NO_DWARF > +ifdef NO_LIBDW > NO_LIBDW_DWARF_UNWIND := 1 > endif > > @@ -574,17 +574,17 @@ ifndef NO_LIBELF > endif > endif > > - ifndef NO_DWARF > + ifndef NO_LIBDW > ifeq ($(origin PERF_HAVE_DWARF_REGS), undefined) > $(warning DWARF register mappings have not been defined for architecture $(SRCARCH), DWARF support disabled) > - NO_DWARF := 1 > + NO_LIBDW := 1 > else > CFLAGS += -DHAVE_DWARF_SUPPORT $(LIBDW_CFLAGS) > LDFLAGS += $(LIBDW_LDFLAGS) > EXTLIBS += ${DWARFLIBS} > $(call detected,CONFIG_DWARF) > endif # PERF_HAVE_DWARF_REGS > - endif # NO_DWARF > + endif # NO_LIBDW > > ifndef NO_LIBBPF > ifeq ($(feature-bpf), 1) > @@ -633,7 +633,7 @@ ifdef PERF_HAVE_JITDUMP > endif > > ifeq ($(SRCARCH),powerpc) > - ifndef NO_DWARF > + ifndef NO_LIBDW > CFLAGS += -DHAVE_SKIP_CALLCHAIN_IDX > endif > endif > diff --git a/tools/perf/Makefile.perf b/tools/perf/Makefile.perf > index 9dd2e8d3f3c9..a144bfaf8aeb 100644 > --- a/tools/perf/Makefile.perf > +++ b/tools/perf/Makefile.perf > @@ -40,7 +40,7 @@ include ../scripts/utilities.mak > # > # Define EXTRA_PERFLIBS to pass extra libraries to PERFLIBS. > # > -# Define NO_DWARF if you do not want debug-info analysis feature at all. > +# Define NO_LIBDW if you do not want debug-info analysis feature at all. > # > # Define WERROR=0 to disable treating any warnings as errors. > # > diff --git a/tools/perf/arch/arm/Makefile b/tools/perf/arch/arm/Makefile > index 1d88fdab13bf..9b164d379548 100644 > --- a/tools/perf/arch/arm/Makefile > +++ b/tools/perf/arch/arm/Makefile > @@ -1,5 +1,5 @@ > # SPDX-License-Identifier: GPL-2.0-only > -ifndef NO_DWARF > +ifndef NO_LIBDW > PERF_HAVE_DWARF_REGS := 1 > endif > PERF_HAVE_JITDUMP := 1 > diff --git a/tools/perf/arch/arm64/Makefile b/tools/perf/arch/arm64/Makefile > index 5735ed4479bb..8a5ffbfe809f 100644 > --- a/tools/perf/arch/arm64/Makefile > +++ b/tools/perf/arch/arm64/Makefile > @@ -1,5 +1,5 @@ > # SPDX-License-Identifier: GPL-2.0 > -ifndef NO_DWARF > +ifndef NO_LIBDW > PERF_HAVE_DWARF_REGS := 1 > endif > PERF_HAVE_JITDUMP := 1 > diff --git a/tools/perf/arch/csky/Makefile b/tools/perf/arch/csky/Makefile > index 88c08eed9c7b..119b06a64bed 100644 > --- a/tools/perf/arch/csky/Makefile > +++ b/tools/perf/arch/csky/Makefile > @@ -1,4 +1,4 @@ > # SPDX-License-Identifier: GPL-2.0-only > -ifndef NO_DWARF > +ifndef NO_LIBDW > PERF_HAVE_DWARF_REGS := 1 > endif > diff --git a/tools/perf/arch/loongarch/Makefile b/tools/perf/arch/loongarch/Makefile > index c89d6bb6b184..1cc5eb01f32b 100644 > --- a/tools/perf/arch/loongarch/Makefile > +++ b/tools/perf/arch/loongarch/Makefile > @@ -1,5 +1,5 @@ > # SPDX-License-Identifier: GPL-2.0 > -ifndef NO_DWARF > +ifndef NO_LIBDW > PERF_HAVE_DWARF_REGS := 1 > endif > PERF_HAVE_ARCH_REGS_QUERY_REGISTER_OFFSET := 1 > diff --git a/tools/perf/arch/mips/Makefile b/tools/perf/arch/mips/Makefile > index cd0b011b3be5..733f7b76f52d 100644 > --- a/tools/perf/arch/mips/Makefile > +++ b/tools/perf/arch/mips/Makefile > @@ -1,5 +1,5 @@ > # SPDX-License-Identifier: GPL-2.0 > -ifndef NO_DWARF > +ifndef NO_LIBDW > PERF_HAVE_DWARF_REGS := 1 > endif > > diff --git a/tools/perf/arch/powerpc/Makefile b/tools/perf/arch/powerpc/Makefile > index bf6d323574f6..7672d555f6cd 100644 > --- a/tools/perf/arch/powerpc/Makefile > +++ b/tools/perf/arch/powerpc/Makefile > @@ -1,5 +1,5 @@ > # SPDX-License-Identifier: GPL-2.0 > -ifndef NO_DWARF > +ifndef NO_LIBDW > PERF_HAVE_DWARF_REGS := 1 > endif > > diff --git a/tools/perf/arch/riscv/Makefile b/tools/perf/arch/riscv/Makefile > index 90c3c476a242..4664a78a1afd 100644 > --- a/tools/perf/arch/riscv/Makefile > +++ b/tools/perf/arch/riscv/Makefile > @@ -1,4 +1,4 @@ > -ifndef NO_DWARF > +ifndef NO_LIBDW > PERF_HAVE_DWARF_REGS := 1 > endif > PERF_HAVE_ARCH_REGS_QUERY_REGISTER_OFFSET := 1 > diff --git a/tools/perf/arch/s390/Makefile b/tools/perf/arch/s390/Makefile > index 56994e63b43a..3f66e2ede3f7 100644 > --- a/tools/perf/arch/s390/Makefile > +++ b/tools/perf/arch/s390/Makefile > @@ -1,5 +1,5 @@ > # SPDX-License-Identifier: GPL-2.0-only > -ifndef NO_DWARF > +ifndef NO_LIBDW > PERF_HAVE_DWARF_REGS := 1 > endif > HAVE_KVM_STAT_SUPPORT := 1 > diff --git a/tools/perf/arch/sh/Makefile b/tools/perf/arch/sh/Makefile > index 88c08eed9c7b..119b06a64bed 100644 > --- a/tools/perf/arch/sh/Makefile > +++ b/tools/perf/arch/sh/Makefile > @@ -1,4 +1,4 @@ > # SPDX-License-Identifier: GPL-2.0-only > -ifndef NO_DWARF > +ifndef NO_LIBDW > PERF_HAVE_DWARF_REGS := 1 > endif > diff --git a/tools/perf/arch/sparc/Makefile b/tools/perf/arch/sparc/Makefile > index 4031db72ba71..7741184894c8 100644 > --- a/tools/perf/arch/sparc/Makefile > +++ b/tools/perf/arch/sparc/Makefile > @@ -1,5 +1,5 @@ > # SPDX-License-Identifier: GPL-2.0-only > -ifndef NO_DWARF > +ifndef NO_LIBDW > PERF_HAVE_DWARF_REGS := 1 > endif > > diff --git a/tools/perf/arch/x86/Makefile b/tools/perf/arch/x86/Makefile > index 67b4969a6738..9aa58acb5564 100644 > --- a/tools/perf/arch/x86/Makefile > +++ b/tools/perf/arch/x86/Makefile > @@ -1,5 +1,5 @@ > # SPDX-License-Identifier: GPL-2.0 > -ifndef NO_DWARF > +ifndef NO_LIBDW > PERF_HAVE_DWARF_REGS := 1 > endif > HAVE_KVM_STAT_SUPPORT := 1 > diff --git a/tools/perf/arch/xtensa/Makefile b/tools/perf/arch/xtensa/Makefile > index 88c08eed9c7b..119b06a64bed 100644 > --- a/tools/perf/arch/xtensa/Makefile > +++ b/tools/perf/arch/xtensa/Makefile > @@ -1,4 +1,4 @@ > # SPDX-License-Identifier: GPL-2.0-only > -ifndef NO_DWARF > +ifndef NO_LIBDW > PERF_HAVE_DWARF_REGS := 1 > endif > diff --git a/tools/perf/builtin-probe.c b/tools/perf/builtin-probe.c > index 003a3bcebfdf..91672bb3047c 100644 > --- a/tools/perf/builtin-probe.c > +++ b/tools/perf/builtin-probe.c > @@ -616,7 +616,7 @@ __cmd_probe(int argc, const char **argv) > set_option_flag(options, 'L', "line", PARSE_OPT_EXCLUSIVE); > set_option_flag(options, 'V', "vars", PARSE_OPT_EXCLUSIVE); > #else > -# define set_nobuild(s, l, c) set_option_nobuild(options, s, l, "NO_DWARF=1", c) > +# define set_nobuild(s, l, c) set_option_nobuild(options, s, l, "NO_LIBDW=1", c) > set_nobuild('L', "line", false); > set_nobuild('V', "vars", false); > set_nobuild('\0', "externs", false); > -- > 2.46.0.792.g87dc391469-goog > > -- Masami Hiramatsu (Google)