mirror of
git://git.openembedded.org/meta-openembedded
synced 2026-01-01 13:58:06 +00:00
bpftrace: Upgrade to 0.24.0
Fix build with clang-21 and libc++ Signed-off-by: Khem Raj <raj.khem@gmail.com>
This commit is contained in:
parent
4e0a5e434c
commit
8783b260a4
|
|
@ -0,0 +1,39 @@
|
|||
From ead4dc45e35fc6c3770ef4021720e0e6a5b60674 Mon Sep 17 00:00:00 2001
|
||||
From: Khem Raj <raj.khem@gmail.com>
|
||||
Date: Sun, 31 Aug 2025 15:23:27 -0700
|
||||
Subject: [PATCH 1/3] Include missing c++ runtime headers
|
||||
|
||||
These headers are required and found to fail builds
|
||||
when using llvm/libc++ instead of libstdc++ on linux
|
||||
|
||||
Upstream-Status: Submitted [https://github.com/bpftrace/bpftrace/pull/4526]
|
||||
Signed-off-by: Khem Raj <raj.khem@gmail.com>
|
||||
---
|
||||
src/util/bpf_names.cpp | 1 +
|
||||
tests/opaque.cpp | 1 +
|
||||
2 files changed, 2 insertions(+)
|
||||
|
||||
diff --git a/src/util/bpf_names.cpp b/src/util/bpf_names.cpp
|
||||
index 68525bfc..ab8f6f2f 100644
|
||||
--- a/src/util/bpf_names.cpp
|
||||
+++ b/src/util/bpf_names.cpp
|
||||
@@ -1,6 +1,7 @@
|
||||
#include <algorithm>
|
||||
#include <iomanip>
|
||||
#include <iostream>
|
||||
+#include <sstream> // for std::ostringstream
|
||||
|
||||
#include "util/bpf_names.h"
|
||||
|
||||
diff --git a/tests/opaque.cpp b/tests/opaque.cpp
|
||||
index a030a05d..034888d3 100644
|
||||
--- a/tests/opaque.cpp
|
||||
+++ b/tests/opaque.cpp
|
||||
@@ -1,6 +1,7 @@
|
||||
#include "util/opaque.h"
|
||||
#include "gtest/gtest.h"
|
||||
#include <cstring>
|
||||
+#include <numbers> // for std::numbers
|
||||
#include <string>
|
||||
#include <vector>
|
||||
|
||||
|
|
@ -0,0 +1,24 @@
|
|||
From 7bd8ec690ad587e7f180bcd061a6205b28d86260 Mon Sep 17 00:00:00 2001
|
||||
From: Khem Raj <raj.khem@gmail.com>
|
||||
Date: Sun, 31 Aug 2025 09:50:35 -0700
|
||||
Subject: [PATCH] cmake: Raise max supported clang version to clang-21
|
||||
|
||||
Upstream-Status: Pending
|
||||
Signed-off-by: Khem Raj <raj.khem@gmail.com>
|
||||
---
|
||||
CMakeLists.txt | 2 +-
|
||||
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||
|
||||
diff --git a/CMakeLists.txt b/CMakeLists.txt
|
||||
index 025fa738..a0f16dd1 100644
|
||||
--- a/CMakeLists.txt
|
||||
+++ b/CMakeLists.txt
|
||||
@@ -172,7 +172,7 @@ if(CMAKE_BUILD_TYPE STREQUAL "Debug")
|
||||
# releases.
|
||||
set(MAX_LLVM_MAJOR 999)
|
||||
else()
|
||||
- set(MAX_LLVM_MAJOR 20)
|
||||
+ set(MAX_LLVM_MAJOR 21)
|
||||
endif()
|
||||
|
||||
if((${LLVM_VERSION_MAJOR} VERSION_LESS ${MIN_LLVM_MAJOR}) OR (${LLVM_VERSION_MAJOR} VERSION_GREATER ${MAX_LLVM_MAJOR}))
|
||||
|
|
@ -12,7 +12,7 @@ Upstream-Status: Pending
|
|||
|
||||
--- a/CMakeLists.txt
|
||||
+++ b/CMakeLists.txt
|
||||
@@ -100,16 +100,20 @@ include_directories(SYSTEM ${LIBCEREAL_I
|
||||
@@ -108,16 +108,21 @@ include_directories(SYSTEM ${LIBCEREAL_I
|
||||
|
||||
find_package(BISON REQUIRED)
|
||||
find_package(FLEX REQUIRED)
|
||||
|
|
@ -20,15 +20,16 @@ Upstream-Status: Pending
|
|||
+# avoid buildpaths in generated #line statements and allow to pass --file-prefix-map=OLD=NEW
|
||||
+set(BISON_FLAGS "${BISON_FLAGS} -l")
|
||||
+set(FLEX_FLAGS "${FLEX_FLAGS} -L")
|
||||
+
|
||||
# `parser_class_name` is deprecated and generates warnings in bison >= 3.3.
|
||||
# But `api.parser.class` is not supported in bison < 3.3. So we must inject
|
||||
# the %define based on the bison version here.
|
||||
if(${BISON_VERSION} VERSION_GREATER_EQUAL 3.3)
|
||||
- set(BISON_FLAGS "-Dapi.parser.class={Parser}")
|
||||
+ set(BISON_FLAGS "${BISON_FLAGS} -Dapi.parser.class={Parser}")
|
||||
- set(BISON_FLAGS "-Dapi.parser.class={Parser} -Wcounterexamples")
|
||||
+ set(BISON_FLAGS "${BISON_FLAGS} -Dapi.parser.class={Parser} -Wcounterexamples")
|
||||
else()
|
||||
- set(BISON_FLAGS "-Dparser_class_name={Parser}")
|
||||
+ set(BISON_FLAGS "${BISON_FLAGS} -Dparser_class_name={Parser}")
|
||||
- set(BISON_FLAGS "-Dparser_class_name={Parser} -Wcounterexamples")
|
||||
+ set(BISON_FLAGS "${BISON_FLAGS} -Dparser_class_name={Parser} -Wcounterexamples")
|
||||
endif()
|
||||
bison_target(bison_parser src/parser.yy ${CMAKE_BINARY_DIR}/parser.tab.cc COMPILE_FLAGS ${BISON_FLAGS} VERBOSE)
|
||||
-flex_target(flex_lexer src/lexer.l ${CMAKE_BINARY_DIR}/lex.yy.cc)
|
||||
|
|
|
|||
|
|
@ -0,0 +1,54 @@
|
|||
From 381047c14dfbc3b89a5e87404cb7cf886f10c119 Mon Sep 17 00:00:00 2001
|
||||
From: Khem Raj <raj.khem@gmail.com>
|
||||
Date: Sun, 31 Aug 2025 15:26:36 -0700
|
||||
Subject: [PATCH 2/3] chrono: Fix build when using libc++ casting ns to
|
||||
system_clock::duration
|
||||
MIME-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
|
||||
libc++ from LLVM/Clang 21 release requires that time_point::operator+=
|
||||
receive the exact duration type of the time_point. On many Linux configs,
|
||||
system_clock::duration is microseconds, so doing:
|
||||
|
||||
t += std::chrono::nanoseconds(...);
|
||||
|
||||
fails with:
|
||||
error: no viable overloaded '+='
|
||||
note: candidate function not viable: no known conversion from
|
||||
'duration<..., nano>' to 'const duration<..., micro>' for 1st argument
|
||||
|
||||
Cast the nanoseconds to system_clock::duration via duration_cast before
|
||||
adding them. This builds with Clang 21 + libc++ and remains compatible
|
||||
with libstdc++. Semantics are unchanged except for truncation to the
|
||||
clock’s native resolution (which already applies).
|
||||
|
||||
No functional change intended.
|
||||
|
||||
Upstream-Status: Submitted [https://github.com/bpftrace/bpftrace/pull/4526]
|
||||
|
||||
Signed-off-by: Khem Raj <raj.khem@gmail.com>
|
||||
---
|
||||
src/bpftrace.cpp | 6 ++++--
|
||||
1 file changed, 4 insertions(+), 2 deletions(-)
|
||||
|
||||
diff --git a/src/bpftrace.cpp b/src/bpftrace.cpp
|
||||
index a49a3efe..485267e0 100644
|
||||
--- a/src/bpftrace.cpp
|
||||
+++ b/src/bpftrace.cpp
|
||||
@@ -992,11 +992,13 @@ std::chrono::time_point<std::chrono::system_clock> BPFtrace::resolve_timestamp(
|
||||
<< "Cannot resolve timestamp due to failed boot time calculation";
|
||||
} else {
|
||||
t += std::chrono::seconds(boottime_->tv_sec);
|
||||
- t += std::chrono::nanoseconds(boottime_->tv_nsec);
|
||||
+ t += std::chrono::duration_cast<std::chrono::system_clock::duration>(
|
||||
+ std::chrono::nanoseconds(boottime_->tv_nsec));
|
||||
}
|
||||
}
|
||||
|
||||
- t += std::chrono::nanoseconds(nsecs);
|
||||
+ t += std::chrono::duration_cast<std::chrono::system_clock::duration>(
|
||||
+ std::chrono::nanoseconds(nsecs));
|
||||
return t;
|
||||
}
|
||||
|
||||
|
|
@ -0,0 +1,78 @@
|
|||
From 9771348249981680c2b893a435099673e79997c4 Mon Sep 17 00:00:00 2001
|
||||
From: Khem Raj <raj.khem@gmail.com>
|
||||
Date: Sun, 31 Aug 2025 15:29:26 -0700
|
||||
Subject: [PATCH 3/3] ast: Adapt to Clang/LLVM 21 DiagnosticOptions API
|
||||
MIME-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
|
||||
Clang 21 removed intrusive ref-counting from DiagnosticOptions and
|
||||
switched consumers to take a DiagnosticOptions&. Building bpftrace
|
||||
with clang-21/libc++ failed with errors like:
|
||||
|
||||
IntrusiveRefCntPtr.h:163: error: no member named 'Retain' in
|
||||
'clang::DiagnosticOptions'
|
||||
Diagnostic.h:578: no known conversion from
|
||||
'IntrusiveRefCntPtr<clang::DiagnosticOptions>' to
|
||||
'DiagnosticOptions&'
|
||||
|
||||
Update the frontend glue:
|
||||
- For LLVM >= 21, construct a real DiagnosticOptions object and pass
|
||||
it by reference to TextDiagnosticPrinter and DiagnosticsEngine
|
||||
(no IntrusiveRefCntPtr). Keep it alive via shared_ptr to satisfy
|
||||
DiagnosticsEngine’s reference lifetime.
|
||||
- Replace ci.setInvocation(inv) with
|
||||
ci.getInvocation() = *inv; which is stable across modern Clang.
|
||||
|
||||
Retain the old code path for LLVM < 21 via #if guards.
|
||||
|
||||
This fixes builds with clang-21/libc++ while keeping compatibility
|
||||
with older LLVM releases.
|
||||
|
||||
No functional change intended.
|
||||
|
||||
Upstream-Status: Submitted [https://github.com/bpftrace/bpftrace/pull/4526]
|
||||
|
||||
Signed-off-by: Khem Raj <raj.khem@gmail.com>
|
||||
---
|
||||
src/ast/passes/clang_build.cpp | 15 +++++++++++++--
|
||||
1 file changed, 13 insertions(+), 2 deletions(-)
|
||||
|
||||
diff --git a/src/ast/passes/clang_build.cpp b/src/ast/passes/clang_build.cpp
|
||||
index fa5b2add..3debc350 100644
|
||||
--- a/src/ast/passes/clang_build.cpp
|
||||
+++ b/src/ast/passes/clang_build.cpp
|
||||
@@ -78,12 +78,21 @@ static Result<> build(CompileContext &ctx,
|
||||
// a string, which we can then capture and associate with the import.
|
||||
std::string errstr;
|
||||
llvm::raw_string_ostream err(errstr);
|
||||
+#if LLVM_VERSION_MAJOR < 21
|
||||
auto diagOpts = llvm::makeIntrusiveRefCnt<clang::DiagnosticOptions>();
|
||||
auto diags = std::make_unique<clang::DiagnosticsEngine>(
|
||||
llvm::makeIntrusiveRefCnt<clang::DiagnosticIDs>(),
|
||||
diagOpts,
|
||||
new clang::TextDiagnosticPrinter(err, diagOpts.get()));
|
||||
-
|
||||
+#else
|
||||
+ // Clang 21: DiagnosticOptions is NOT intrusive-refcounted anymore.
|
||||
+ // Keep it alive for the program lifetime (or store it on a longer-lived object).
|
||||
+ static std::shared_ptr<clang::DiagnosticOptions> diagOpts =
|
||||
+ std::make_shared<clang::DiagnosticOptions>();
|
||||
+ llvm::IntrusiveRefCntPtr<clang::DiagnosticIDs> diagID(new clang::DiagnosticIDs());
|
||||
+ auto client = std::make_unique<clang::TextDiagnosticPrinter>(err, *diagOpts);
|
||||
+ auto diags = std::make_unique<clang::DiagnosticsEngine>(diagID, *diagOpts, client.release());
|
||||
+#endif
|
||||
// We create a temporary memfd that we can use to store the output,
|
||||
// since the ClangDriver API is framed in terms of filenames. Perhaps
|
||||
// we could use the internals here, but that carries other risks.
|
||||
@@ -122,7 +131,9 @@ static Result<> build(CompileContext &ctx,
|
||||
inv->getCodeGenOpts().DebugColumnInfo = true;
|
||||
|
||||
clang::CompilerInstance ci;
|
||||
- ci.setInvocation(inv);
|
||||
+ // Cross-version friendly: assign into the existing invocation
|
||||
+ // (works across modern Clang majors, including 21)
|
||||
+ ci.getInvocation() = *inv;
|
||||
ci.setDiagnostics(diags.release());
|
||||
ci.setFileManager(new clang::FileManager(clang::FileSystemOptions(), vfs));
|
||||
ci.createSourceManager(ci.getFileManager());
|
||||
|
|
@ -7,6 +7,7 @@ DEPENDS += "bison-native \
|
|||
flex-native \
|
||||
gzip-native \
|
||||
elfutils \
|
||||
bpftool-native \
|
||||
bcc \
|
||||
systemtap \
|
||||
libcereal \
|
||||
|
|
@ -16,11 +17,15 @@ DEPENDS += "${@bb.utils.contains('PTEST_ENABLED', '1', 'pahole-native llvm-nativ
|
|||
|
||||
RDEPENDS:${PN} += "bash python3 xz"
|
||||
|
||||
SRC_URI = "git://github.com/iovisor/bpftrace;branch=release/0.23.x;protocol=https \
|
||||
SRC_URI = "git://github.com/iovisor/bpftrace;branch=master;protocol=https \
|
||||
file://run-ptest \
|
||||
file://0001-cmake-Raise-max-supported-clang-version-to-clang-21.patch \
|
||||
file://0002-CMakeLists.txt-allow-to-set-BISON_FLAGS-like-l.patch \
|
||||
file://0001-Include-missing-c-runtime-headers.patch \
|
||||
file://0002-chrono-Fix-build-when-using-libc-casting-ns-to-syste.patch \
|
||||
file://0003-ast-Adapt-to-Clang-LLVM-21-DiagnosticOptions-API.patch \
|
||||
"
|
||||
SRCREV = "e48ccc66c9971515648b63699bc4b1490c388d85"
|
||||
SRCREV = "3b78184eed28501ab4bbb55e45c4172538999da1"
|
||||
|
||||
inherit bash-completion cmake ptest pkgconfig
|
||||
|
||||
Loading…
Reference in New Issue
Block a user