step E.4(android): replace full Oboe compile with minimal C++ smoke file
Some checks failed
Mirror to GitHub / mirror (push) Failing after 40s
Build Release Binaries / build-amd64 (push) Failing after 3m44s

Bisection for the __init_tcb+4 crash that Step E introduced: drop the
full Oboe C++ build (200+ files, hundreds of KB of code) and replace
it with ONE tiny cpp/cpp_smoke.cpp that exercises the libc++ features
Oboe uses — std::atomic, std::mutex, std::thread — via an
extern "C" wzp_cpp_smoke() function that's exported but NEVER called
from Rust.

Still compiled with cpp_link_stdlib("c++_shared"), same as Oboe.
libc++_shared.so still copied into gen/android jniLibs. But no Oboe
headers, no Oboe source files, no -llog / -lOpenSLES links.

Hypothesis: if cpp_smoke.cpp alone reproduces the __init_tcb crash,
the trigger is "any libc++_shared link that references
std::thread/std::mutex" and Oboe is not the specific culprit. If it
launches cleanly, Oboe itself (its size, its static constructors, or
a specific header) is responsible — and we then bisect Oboe's
source tree.

fetch_oboe() and add_cpp_files_recursive() are retained in build.rs
with #[allow(dead_code)] so re-enabling the full Oboe compile is a
one-line edit once we've identified what's safe to include.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Siavash Sameni
2026-04-09 15:39:30 +04:00
parent d216dcc7a3
commit aa240c6d83
2 changed files with 53 additions and 45 deletions

View File

@@ -37,48 +37,23 @@ fn build_android_native() {
.file("cpp/getauxval_fix.c") .file("cpp/getauxval_fix.c")
.compile("getauxval_fix"); .compile("getauxval_fix");
// ─── Step E: full Oboe C++ bridge ────────────────────────────────────── // ─── Step E.4: minimal C++ smoke file instead of full Oboe ─────────────
// Clones google/oboe@1.8.1 into OUT_DIR and compiles the bridge + all // The full Oboe compile (Step E, commit 4250f1b) triggered the
// Oboe source files as a single static library. NOT yet called from // __init_tcb+4 crash at launch — even without any FFI call from Rust.
// Rust — this step only verifies that the C++ compile + link path // Bisection jump: replace the 200+ Oboe source files with ONE tiny
// doesn't regress the known-good build. Same approach as the legacy // cpp/cpp_smoke.cpp that uses the same libc++ features Oboe does
// crates/wzp-android/build.rs, copied verbatim below. // (std::atomic, std::mutex, std::thread), still linked via
println!("cargo:rerun-if-changed=cpp/oboe_bridge.cpp"); // libc++_shared. If this crashes too, the trigger is just "any C++
println!("cargo:rerun-if-changed=cpp/oboe_bridge.h"); // link that references libc++ threads/mutexes". If it passes, Oboe
println!("cargo:rerun-if-changed=cpp/oboe_stub.cpp"); // itself (size, specific headers, static ctors) is the culprit.
println!("cargo:rerun-if-changed=cpp/cpp_smoke.cpp");
match fetch_oboe() {
Some(oboe_path) => {
println!("cargo:warning=Building with Oboe from {:?}", oboe_path);
let mut build = cc::Build::new();
build
.cpp(true)
.std("c++17")
// Shared libc++ — static pulls broken libc stubs that crash
// in .so libraries (getauxval, __init_tcb, pthread_create).
// Google's official NDK guidance.
.cpp_link_stdlib(Some("c++_shared"))
.include("cpp")
.include(oboe_path.join("include"))
.include(oboe_path.join("src"))
.define("WZP_HAS_OBOE", None)
.file("cpp/oboe_bridge.cpp");
add_cpp_files_recursive(&mut build, &oboe_path.join("src"));
build.compile("oboe_bridge");
}
None => {
println!("cargo:warning=Oboe not found, building with stub");
cc::Build::new() cc::Build::new()
.cpp(true) .cpp(true)
.std("c++17") .std("c++17")
// Shared libc++ — same linkage Oboe uses.
.cpp_link_stdlib(Some("c++_shared")) .cpp_link_stdlib(Some("c++_shared"))
.file("cpp/oboe_stub.cpp") .file("cpp/cpp_smoke.cpp")
.include("cpp") .compile("wzp_cpp_smoke");
.compile("oboe_bridge");
}
}
// Copy libc++_shared.so next to libwzp_desktop_lib.so in the Tauri // Copy libc++_shared.so next to libwzp_desktop_lib.so in the Tauri
// jniLibs directory so the dynamic linker can resolve it at runtime. // jniLibs directory so the dynamic linker can resolve it at runtime.
@@ -105,9 +80,10 @@ fn build_android_native() {
} }
} }
// Oboe requires Android log + OpenSLES backends // Step E.4 drops the Oboe-specific -llog / -lOpenSLES link requirements
println!("cargo:rustc-link-lib=log"); // since cpp_smoke.cpp doesn't call into Android's logging or audio HAL.
println!("cargo:rustc-link-lib=OpenSLES"); // Keep libc++_shared.so in jniLibs (copied above) because the smoke
// file still dynamically links against libc++.
} }
fn target_os_abi() -> Option<(&'static str, &'static str)> { fn target_os_abi() -> Option<(&'static str, &'static str)> {
@@ -126,6 +102,7 @@ fn target_os_abi() -> Option<(&'static str, &'static str)> {
} }
/// Recursively add all .cpp files from a directory to a cc::Build. /// Recursively add all .cpp files from a directory to a cc::Build.
#[allow(dead_code)] // re-enabled when Step E.x restores the full Oboe compile
fn add_cpp_files_recursive(build: &mut cc::Build, dir: &std::path::Path) { fn add_cpp_files_recursive(build: &mut cc::Build, dir: &std::path::Path) {
if !dir.is_dir() { if !dir.is_dir() {
return; return;
@@ -142,6 +119,7 @@ fn add_cpp_files_recursive(build: &mut cc::Build, dir: &std::path::Path) {
} }
/// Try to find or fetch Oboe headers + source (v1.8.1). /// Try to find or fetch Oboe headers + source (v1.8.1).
#[allow(dead_code)] // re-enabled when Step E.x restores the full Oboe compile
fn fetch_oboe() -> Option<PathBuf> { fn fetch_oboe() -> Option<PathBuf> {
let out_dir = PathBuf::from(std::env::var("OUT_DIR").unwrap()); let out_dir = PathBuf::from(std::env::var("OUT_DIR").unwrap());
let oboe_dir = out_dir.join("oboe"); let oboe_dir = out_dir.join("oboe");

View File

@@ -0,0 +1,30 @@
// cpp_smoke.cpp — minimal C++ test that exercises the libc++_shared
// features Oboe uses (std::thread, std::mutex, std::atomic) without being
// Oboe itself.
//
// Built via cc::Build::new().cpp(true).cpp_link_stdlib("c++_shared") and
// replaces the full Oboe bridge compile during the Step E bisection of
// the __init_tcb+4 crash. The function is `extern "C"` and exported so
// the linker can't dead-code-eliminate it — the std::thread /
// std::lock_guard / std::atomic::fetch_add uses pull in libc++'s
// bindings to bionic pthread, matching what Oboe would force.
//
// The function is NEVER called from Rust. If we crash anyway, the trigger
// is just *linking* this code in. If it launches cleanly, Oboe itself
// (size, static ctors, specific headers) is the culprit.
#include <atomic>
#include <mutex>
#include <thread>
namespace {
std::atomic<int> g_counter{0};
std::mutex g_mutex;
}
extern "C" int wzp_cpp_smoke(void) {
std::lock_guard<std::mutex> lock(g_mutex);
std::thread t([]() { g_counter.fetch_add(1); });
t.join();
return g_counter.load();
}