linux-perf-users.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v1 00/11] Libdw/dwarf build clean up
@ 2024-09-24 16:04 Ian Rogers
  2024-09-24 16:04 ` [PATCH v1 01/11] perf build: Rename NO_DWARF to NO_LIBDW Ian Rogers
                   ` (11 more replies)
  0 siblings, 12 replies; 51+ messages in thread
From: Ian Rogers @ 2024-09-24 16:04 UTC (permalink / raw)
  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, Guilherme Amadio,
	Changbin Du, Steinar H. Gunderson, Masami Hiramatsu (Google),
	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, linux-perf-users,
	Yang Jihong

There patches are 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.

In the Makefile code 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.

Ian Rogers (11):
  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
  perf build: Rename PERF_HAVE_DWARF_REGS to PERF_HAVE_LIBDW_REGS

 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                    | 57 ++++++-------------
 tools/perf/Makefile.perf                      |  2 +-
 tools/perf/arch/arm/Makefile                  |  4 +-
 tools/perf/arch/arm/util/Build                |  2 +-
 tools/perf/arch/arm64/Makefile                |  4 +-
 tools/perf/arch/arm64/util/Build              |  2 +-
 tools/perf/arch/csky/Makefile                 |  4 +-
 tools/perf/arch/csky/util/Build               |  2 +-
 tools/perf/arch/loongarch/Makefile            |  4 +-
 tools/perf/arch/loongarch/util/Build          |  2 +-
 tools/perf/arch/mips/Makefile                 |  4 +-
 tools/perf/arch/mips/util/Build               |  2 +-
 tools/perf/arch/powerpc/Makefile              |  4 +-
 .../perf/arch/powerpc/annotate/instructions.c |  4 +-
 tools/perf/arch/powerpc/util/Build            |  4 +-
 tools/perf/arch/riscv/Makefile                |  4 +-
 tools/perf/arch/riscv/util/Build              |  2 +-
 tools/perf/arch/s390/Makefile                 |  4 +-
 tools/perf/arch/s390/util/Build               |  2 +-
 tools/perf/arch/sh/Makefile                   |  4 +-
 tools/perf/arch/sh/util/Build                 |  2 +-
 tools/perf/arch/sparc/Makefile                |  4 +-
 tools/perf/arch/sparc/util/Build              |  2 +-
 tools/perf/arch/x86/Makefile                  |  4 +-
 tools/perf/arch/x86/annotate/instructions.c   |  2 +-
 tools/perf/arch/x86/util/Build                |  2 +-
 tools/perf/arch/xtensa/Makefile               |  4 +-
 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, 168 insertions(+), 279 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

-- 
2.46.0.792.g87dc391469-goog


^ permalink raw reply	[flat|nested] 51+ messages in thread

end of thread, other threads:[~2024-10-04 19:23 UTC | newest]

Thread overview: 51+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-09-24 16:04 [PATCH v1 00/11] Libdw/dwarf build clean up Ian Rogers
2024-09-24 16:04 ` [PATCH v1 01/11] perf build: Rename NO_DWARF to NO_LIBDW Ian Rogers
2024-09-26  0:26   ` Namhyung Kim
2024-09-26 15:02     ` Ian Rogers
2024-09-26 19:28       ` Namhyung Kim
2024-09-29  2:01         ` Masami Hiramatsu
2024-09-29  2:01   ` Masami Hiramatsu
2024-09-24 16:04 ` [PATCH v1 02/11] perf build: Remove defined but never used variable Ian Rogers
2024-09-29  2:02   ` Masami Hiramatsu
2024-09-24 16:04 ` [PATCH v1 03/11] perf build: Rename test-dwarf to test-libdw Ian Rogers
2024-09-26  0:28   ` Namhyung Kim
2024-09-26 15:37     ` Ian Rogers
2024-09-26 18:23       ` Namhyung Kim
2024-09-29  2:06   ` Masami Hiramatsu
2024-09-24 16:04 ` [PATCH v1 04/11] perf build: Combine libdw-dwarf-unwind into libdw feature tests Ian Rogers
2024-09-24 16:04 ` [PATCH v1 05/11] perf build: Combine test-dwarf-getlocations into test-libdw Ian Rogers
2024-09-24 16:04 ` [PATCH v1 06/11] perf build: Combine test-dwarf-getcfi " Ian Rogers
2024-09-24 16:04 ` [PATCH v1 07/11] perf probe: Move elfutils support check to libdw check Ian Rogers
2024-09-26  0:29   ` Namhyung Kim
2024-09-26 15:08     ` Ian Rogers
2024-09-26 19:35       ` Namhyung Kim
2024-09-24 16:04 ` [PATCH v1 08/11] perf libdw: Remove unnecessary defines Ian Rogers
2024-09-29  2:10   ` Masami Hiramatsu
2024-09-24 16:04 ` [PATCH v1 09/11] perf build: Rename HAVE_DWARF_SUPPORT to HAVE_LIBDW_SUPPORT Ian Rogers
2024-09-26  0:34   ` Namhyung Kim
2024-09-26 15:10     ` Ian Rogers
2024-09-26 19:36       ` Namhyung Kim
2024-09-24 16:04 ` [PATCH v1 10/11] perf build: Rename CONFIG_DWARF to CONFIG_LIBDW Ian Rogers
2024-09-24 16:04 ` [PATCH v1 11/11] perf build: Rename PERF_HAVE_DWARF_REGS to PERF_HAVE_LIBDW_REGS Ian Rogers
2024-09-26  3:27   ` Namhyung Kim
2024-09-26 12:47     ` Ian Rogers
2024-09-26 19:39       ` Namhyung Kim
2024-09-26 19:55         ` Ian Rogers
2024-09-27 17:16           ` Namhyung Kim
2024-09-27 18:15             ` Ian Rogers
2024-09-29  2:35               ` Masami Hiramatsu
2024-10-01  4:02                 ` Ian Rogers
2024-10-01 23:09                   ` Namhyung Kim
2024-10-01 23:17                     ` Ian Rogers
2024-10-01 23:28                       ` Masami Hiramatsu
2024-10-02  1:31                         ` Ian Rogers
2024-10-02 13:56                           ` Masami Hiramatsu
2024-10-02 14:27                             ` Ian Rogers
2024-10-03 22:48                               ` Namhyung Kim
2024-10-04  0:58                                 ` Ian Rogers
2024-10-04  5:12                                   ` Namhyung Kim
2024-10-04 14:45                                     ` Masami Hiramatsu
2024-10-04 15:15                                       ` Ian Rogers
2024-10-04 19:23                                         ` Namhyung Kim
2024-10-04 14:32                                   ` Masami Hiramatsu
2024-09-24 19:44 ` [PATCH v1 00/11] Libdw/dwarf build clean up Leo Yan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).