From patchwork Thu Oct 17 00:13:43 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ian Rogers X-Patchwork-Id: 13839127 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id DF3E3D2F7D7 for ; Thu, 17 Oct 2024 00:29:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Type:To:From:Subject :Mime-Version:Message-Id:Date:Reply-To:Cc:Content-Transfer-Encoding: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Owner; bh=EiSvPeoRUp8BxyR9oFd1+sHW5VYvYw8ni5QPRWGYfRA=; b=cSbh4EsrLV/VsXnzQfcFtwgY7G if8YKSIQe/QL6WlrJosajrKvetTEa7LLLHbMF639gstGbNg2a83ETjhzgBPSLA48IBgXWDAHx0Q+T 9OTY/7LyESIr42L3GnMOHsCk9Qnu8CqvUctEhJgxkTylfMAddb8JhcqaFeowKtwBR+qARzBHR5kJR /320BwY6Myy50/nGuj2VCjyJaVmNNW5bAkfK39CY4YwBbzkRP40f2Wf3XuxDaqBrLgAXeQQ1Mx5N4 w+w9dLYT/x2fagZcTRL9/OkxMOe0YAwdyAOm8Rgci0emmHUHduZABX8Vaejb51UWRlznYEJTp9U4g VgsdK5Kg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t1EOW-0000000DLD2-48TV; Thu, 17 Oct 2024 00:29:36 +0000 Received: from mail-yw1-x114a.google.com ([2607:f8b0:4864:20::114a]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1t1E9U-0000000DI8v-3u4Y for linux-arm-kernel@lists.infradead.org; Thu, 17 Oct 2024 00:14:06 +0000 Received: by mail-yw1-x114a.google.com with SMTP id 00721157ae682-6dbbeee08f0so21800927b3.0 for ; Wed, 16 Oct 2024 17:14:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1729124043; x=1729728843; darn=lists.infradead.org; h=to:from:subject:mime-version:message-id:date:from:to:cc:subject :date:message-id:reply-to; bh=EiSvPeoRUp8BxyR9oFd1+sHW5VYvYw8ni5QPRWGYfRA=; b=hn0he7JzRcJUolx7Lr00N+dUaOg3eMqkGTCdS0P8E6a4sXdumKjafwheUHfpVWCDfP Zp06fC7gBK7V0oDDqqy5fGg0GqbL9kglOYmBoJfrZD/q2NPB5KAj4Pm+wgilpwZIDj6G syVZ1ThwwoRadtxjkpy4FLNYkQcH63/ETiuBVf9n+Ltr3yH+3TDyq2kCX7pGTCOxYF1f B0AgCL1swnrSo5tTxkYUsPFr8OG8cdVBKcEDCoJ5TMw3ILi7tRz1brBIkp+eKQm19/bZ aNSKIxElV/QgYgPtNyBYlXR+wBYX7j+MQFUpFk0rMbreZfZ6guRVx3T+wk+NgtSy0o2a 380Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1729124043; x=1729728843; h=to:from:subject:mime-version:message-id:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=EiSvPeoRUp8BxyR9oFd1+sHW5VYvYw8ni5QPRWGYfRA=; b=fheKbSBajkPqkJZmWnoWtgDSYFSN9ada+iY0NNc2gHtYburTSE5pH6CXuOvsFUv5H5 ukPyqLlw0SNpdo4oFKWJQ6UsyX7gp7nXTRo1S9+qL3N8h4A43rYxrnnKbtS+eqZ8Rhbc SanYB/WZSIvPdpHgWNnf2dsDZB+hMdnjVnFqbKtpgza0Ybs8cutxkIkihZYSFNycH81v xwMX7CpGQReFn8c3oxfgBRS3kCD9JWGCGuOrNfHjJ2HVxV2ivpgmQEfkwTtOvEiYJ90O XzF/mQldabYQ0Q2endAFI7/w5wXenJFKzDF/Ki9XeJyn8rMug5aHnY6fGUImWNgZYFFF jAnA== X-Forwarded-Encrypted: i=1; AJvYcCXUbyWdGIPzd7zGQSb7et7gvp1sW7zFP04VyuqtKrP0JS/Mkln95E1OrrjHCotrJzAIwXndNokt/R7h64hQZpvq@lists.infradead.org X-Gm-Message-State: AOJu0Yxq7VtulJIfatr9A2+7MU/CQBYJ9xnUc3W6LIfWxcupisIDzpyE hX+3WRp4AvOE3d3Z/awuOsTwvLHwWTtBSAqlRoqxN+thfu6PT+4g8dtnSjJbFe0sNO5FnoUHkLR mnSR8Ew== X-Google-Smtp-Source: AGHT+IGWAA1ezUi0zImlaYNN3LkMp/FqrMVYh+LBbCtzIRBEEa/71ZWVK8WYNSeDn0R7vnfNGuT6qxuKOTd6 X-Received: from irogers.svl.corp.google.com ([2620:15c:2c5:11:a00a:f237:9bc0:79c]) (user=irogers job=sendgmr) by 2002:a25:664c:0:b0:e29:7454:e773 with SMTP id 3f1490d57ef6-e2b9d02921cmr1374276.5.1729124043150; Wed, 16 Oct 2024 17:14:03 -0700 (PDT) Date: Wed, 16 Oct 2024 17:13:43 -0700 Message-Id: <20241017001354.56973-1-irogers@google.com> Mime-Version: 1.0 X-Mailer: git-send-email 2.47.0.105.g07ac214952-goog Subject: [PATCH v3 00/11] Libdw/dwarf build clean up From: Ian Rogers To: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Namhyung Kim , Mark Rutland , Alexander Shishkin , Jiri Olsa , Ian Rogers , Adrian Hunter , Kan Liang , John Garry , Will Deacon , James Clark , Mike Leach , Leo Yan , Guo Ren , Paul Walmsley , Palmer Dabbelt , Albert Ou , Nick Terrell , "Masami Hiramatsu (Google)" , Changbin Du , Guilherme Amadio , Yang Jihong , Aditya Gupta , Athira Rajeev , Masahiro Yamada , Bibo Mao , Huacai Chen , Kajol Jain , Atish Patra , Shenlin Liang , Anup Patel , Oliver Upton , "Steinar H. Gunderson" , "Dr. David Alan Gilbert" , Chen Pei , Dima Kogan , Przemek Kitszel , "David S. Miller" , Alexander Lobakin , linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-csky@vger.kernel.org, linux-riscv@lists.infradead.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241016_171405_000331_2CB0CA3C X-CRM114-Status: GOOD ( 18.86 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org These patches were originally on top of: https://lore.kernel.org/lkml/20240924003720.617258-1-irogers@google.com/ where it was pointed out that a lot of the libdw conditional compilation was due to features that have now been standard for 10 or 15 years. The patches remove the conditional compilation assuming the features are in libdw where the feature test is expanded to check there pressence. The patch series is now on top of: https://lore.kernel.org/lkml/20241016235622.52166-1-irogers@google.com/ In the Makefile code, and for `perf record --call-graph`, dwarf tends to mean unwind or libdw support for dwarf things. To make it clearer when dwarf really just means libdw numerous build variables and defines are renamed. There is some tech debt in the changes as perf check still reports the values using the old name and for features that are no longer tested. This can be cleanup for another day. v3: Move PERF_HAVE_DWARF_REGS cleanup to follow up patch series. Add build fix from patch series these changes are on top of. v2: Instead of renaming PERF_HAVE_DWARF_REGS to PERF_HAVE_LIBDW_REGS, remove it. Ian Rogers (11): perf build: Fix LIBDW_DIR perf build: Rename NO_DWARF to NO_LIBDW perf build: Remove defined but never used variable perf build: Rename test-dwarf to test-libdw perf build: Combine libdw-dwarf-unwind into libdw feature tests perf build: Combine test-dwarf-getlocations into test-libdw perf build: Combine test-dwarf-getcfi into test-libdw perf probe: Move elfutils support check to libdw check perf libdw: Remove unnecessary defines perf build: Rename HAVE_DWARF_SUPPORT to HAVE_LIBDW_SUPPORT perf build: Rename CONFIG_DWARF to CONFIG_LIBDW tools/build/Makefile.feature | 11 +--- tools/build/feature/Makefile | 24 ++------ tools/build/feature/test-all.c | 16 +----- tools/build/feature/test-dwarf.c | 11 ---- tools/build/feature/test-dwarf_getcfi.c | 9 --- tools/build/feature/test-dwarf_getlocations.c | 13 ----- tools/build/feature/test-libdw-dwarf-unwind.c | 14 ----- tools/build/feature/test-libdw.c | 56 +++++++++++++++++++ tools/perf/Documentation/perf-check.txt | 6 +- tools/perf/Makefile.config | 47 +++++----------- tools/perf/Makefile.perf | 2 +- tools/perf/arch/arm/Makefile | 2 +- tools/perf/arch/arm/util/Build | 2 +- tools/perf/arch/arm64/Makefile | 2 +- tools/perf/arch/arm64/util/Build | 2 +- tools/perf/arch/csky/Makefile | 2 +- tools/perf/arch/csky/util/Build | 2 +- tools/perf/arch/loongarch/Makefile | 2 +- tools/perf/arch/loongarch/util/Build | 2 +- tools/perf/arch/mips/Makefile | 2 +- tools/perf/arch/mips/util/Build | 2 +- tools/perf/arch/powerpc/Makefile | 2 +- .../perf/arch/powerpc/annotate/instructions.c | 4 +- tools/perf/arch/powerpc/util/Build | 4 +- tools/perf/arch/riscv/Makefile | 2 +- tools/perf/arch/riscv/util/Build | 2 +- tools/perf/arch/s390/Makefile | 2 +- tools/perf/arch/s390/util/Build | 2 +- tools/perf/arch/sh/Makefile | 2 +- tools/perf/arch/sh/util/Build | 2 +- tools/perf/arch/sparc/Makefile | 2 +- tools/perf/arch/sparc/util/Build | 2 +- tools/perf/arch/x86/Makefile | 2 +- tools/perf/arch/x86/annotate/instructions.c | 2 +- tools/perf/arch/x86/util/Build | 2 +- tools/perf/arch/xtensa/Makefile | 2 +- tools/perf/arch/xtensa/util/Build | 2 +- tools/perf/builtin-annotate.c | 2 +- tools/perf/builtin-check.c | 6 +- tools/perf/builtin-probe.c | 14 ++--- tools/perf/builtin-report.c | 4 +- tools/perf/util/Build | 12 ++-- tools/perf/util/annotate-data.h | 8 +-- tools/perf/util/debuginfo.h | 6 +- tools/perf/util/disasm.c | 4 +- tools/perf/util/disasm.h | 4 +- tools/perf/util/dwarf-aux.c | 6 -- tools/perf/util/dwarf-aux.h | 54 ------------------ tools/perf/util/genelf.c | 4 +- tools/perf/util/genelf.h | 2 +- tools/perf/util/include/dwarf-regs.h | 6 +- tools/perf/util/probe-event.c | 4 +- tools/perf/util/probe-finder.c | 6 -- tools/perf/util/probe-finder.h | 6 +- 54 files changed, 154 insertions(+), 259 deletions(-) delete mode 100644 tools/build/feature/test-dwarf.c delete mode 100644 tools/build/feature/test-dwarf_getcfi.c delete mode 100644 tools/build/feature/test-dwarf_getlocations.c delete mode 100644 tools/build/feature/test-libdw-dwarf-unwind.c create mode 100644 tools/build/feature/test-libdw.c Acked-by: Namhyung Kim