[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAP-5=fX-Hy-YLD_P_73Z2+7PfPnSU=0e5yaDgoeZ3gw_getaYw@mail.gmail.com>
Date: Thu, 29 May 2025 12:31:59 -0700
From: Ian Rogers <irogers@...gle.com>
To: Yuzhuo Jing <yuzhuo@...gle.com>
Cc: Peter Zijlstra <peterz@...radead.org>, Ingo Molnar <mingo@...hat.com>,
Arnaldo Carvalho de Melo <acme@...nel.org>, Namhyung Kim <namhyung@...nel.org>,
Mark Rutland <mark.rutland@....com>,
Alexander Shishkin <alexander.shishkin@...ux.intel.com>, Jiri Olsa <jolsa@...nel.org>,
Adrian Hunter <adrian.hunter@...el.com>, Liang Kan <kan.liang@...ux.intel.com>,
Nathan Chancellor <nathan@...nel.org>, Nick Desaulniers <nick.desaulniers+lkml@...il.com>,
Bill Wendling <morbo@...gle.com>, Justin Stitt <justinstitt@...gle.com>,
"Steven Rostedt (Google)" <rostedt@...dmis.org>, James Clark <james.clark@...aro.org>,
Tomas Glozar <tglozar@...hat.com>, Leo Yan <leo.yan@....com>,
Guilherme Amadio <amadio@...too.org>, Yang Jihong <yangjihong@...edance.com>,
"Masami Hiramatsu (Google)" <mhiramat@...nel.org>, Adhemerval Zanella <adhemerval.zanella@...aro.org>,
Wei Yang <richard.weiyang@...il.com>, Ard Biesheuvel <ardb@...nel.org>,
"Mike Rapoport (Microsoft)" <rppt@...nel.org>, Athira Rajeev <atrajeev@...ux.vnet.ibm.com>,
Kajol Jain <kjain@...ux.ibm.com>, Aditya Gupta <adityag@...ux.ibm.com>,
Charlie Jenkins <charlie@...osinc.com>, "Steinar H. Gunderson" <sesse@...gle.com>,
"Dr. David Alan Gilbert" <linux@...blig.org>, Herbert Xu <herbert@...dor.apana.org.au>,
Jeff Johnson <jeff.johnson@....qualcomm.com>, Al Viro <viro@...iv.linux.org.uk>,
linux-kernel@...r.kernel.org, linux-perf-users@...r.kernel.org,
llvm@...ts.linux.dev
Subject: Re: [PATCH v1 0/4] perf: Remove libcrypto dependency
On Wed, May 21, 2025 at 3:54 PM Yuzhuo Jing <yuzhuo@...gle.com> wrote:
>
> Currently, genelf.c is the only file in the perf tool that depends on
> libcrypto (e.g. openssl), which only uses it to calculate a SHA1/MD5
> Build ID. This patch series pulls in the SHA1 implementation from the
> kernel tree, and removes the libcrypto dependency from perf. This also
> switches the default Build ID calculation method from MD5 to the more
> commonly used SHA1.
>
> Yuzhuo Jing (4):
> perf utils: Add support functions for sha1 utils
> perf tools: Add sha1 utils
> perf genelf: Remove libcrypto dependency and use sha1 utils
> tools: Remove libcrypto dependency
Tested-by: Ian Rogers <irogers@...gle.com>
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/include/linux/bitops.h | 10 ++
> tools/include/linux/compiler.h | 17 ++++
> tools/include/linux/string.h | 22 +++++
> tools/perf/Documentation/perf-check.txt | 1 -
> tools/perf/Makefile.config | 13 ---
> tools/perf/Makefile.perf | 3 -
> tools/perf/builtin-check.c | 1 -
> tools/perf/tests/make | 4 +-
> tools/perf/util/Build | 2 +
> tools/perf/util/genelf.c | 72 ++------------
> tools/perf/util/sha1.c | 122 ++++++++++++++++++++++++
> tools/perf/util/sha1.h | 41 ++++++++
> tools/perf/util/sha1_base.h | 103 ++++++++++++++++++++
> tools/perf/util/sha1_generic.c | 49 ++++++++++
> 18 files changed, 373 insertions(+), 123 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
> create mode 100644 tools/perf/util/sha1_base.h
> create mode 100644 tools/perf/util/sha1_generic.c
>
> --
> 2.49.0.1164.gab81da1b16-goog
>
Powered by blists - more mailing lists