Re: [PATCH v3 0/4] perf: Remove libcrypto dependency
From: Ian Rogers
Date: Wed Jun 25 2025 - 17:34:08 EST
On Wed, Jun 25, 2025 at 1:24 PM Eric Biggers <ebiggers@xxxxxxxxxx> wrote:
>
> This is a reworked version of
> https://lore.kernel.org/all/20250521225307.743726-1-yuzhuo@xxxxxxxxxx/.
> I've changed it to add a new minimal SHA-1 implementation, instead of
> trying to pull in the kernel's SHA-1 implementation which is not really
> designed to be extracted into userspace programs. I also added a test.
>
> Changed in v3:
> - Rebased onto perf-tools-next
> - Removed unnecessary parentheses in array indices in sha1.c
> - Updated my email address. I've started consistently using my
> kernel.org email address.
>
> Eric Biggers (2):
> perf build: enable -fno-strict-aliasing
> perf util: add a basic SHA-1 implementation
>
> Yuzhuo Jing (2):
> perf genelf: Remove libcrypto dependency and use built-in sha1()
> tools: Remove libcrypto dependency
I'm trying to make it so that the -fno-strict-aliasing is unnecessary
but we can remove that when the changes land.
Reviewed-by: Ian Rogers <irogers@xxxxxxxxxx>
Thanks!
Ian
> tools/build/Makefile.feature | 2 -
> tools/build/feature/Makefile | 4 -
> tools/build/feature/test-all.c | 5 --
> tools/build/feature/test-libcrypto.c | 25 -------
> tools/perf/Documentation/perf-check.txt | 1 -
> tools/perf/Makefile.config | 17 +----
> tools/perf/Makefile.perf | 3 -
> tools/perf/builtin-check.c | 1 -
> tools/perf/tests/make | 4 +-
> tools/perf/tests/util.c | 45 +++++++++++-
> tools/perf/util/Build | 1 +
> tools/perf/util/genelf.c | 85 +---------------------
> tools/perf/util/sha1.c | 97 +++++++++++++++++++++++++
> tools/perf/util/sha1.h | 6 ++
> 14 files changed, 156 insertions(+), 140 deletions(-)
> delete mode 100644 tools/build/feature/test-libcrypto.c
> create mode 100644 tools/perf/util/sha1.c
> create mode 100644 tools/perf/util/sha1.h
>
>
> base-commit: 9c9f4a27eb1096beb650f312a1ce996a9960b56c
> --
> 2.50.0
>