Enable unwinding in userspace

.github/workflows/ci.yml

@@ -63,27 +63,24 @@ jobs:
 
       - uses: dtolnay/rust-toolchain@nightly
         with:
-          components: rust-src,rustfmt
+          components: clippy,rust-src,rustfmt
 
       - uses: dtolnay/rust-toolchain@master
         if: runner.os == 'macOS' && runner.arch == 'X64'
         with:
           toolchain: ${{ matrix.rust }}
           targets: x86_64-unknown-linux-musl
-          components: clippy
 
       - uses: dtolnay/rust-toolchain@master
         if: runner.os == 'macOS' && runner.arch == 'ARM64'
         with:
           toolchain: ${{ matrix.rust }}
           targets: aarch64-unknown-linux-musl
-          components: clippy
 
       - uses: dtolnay/rust-toolchain@master
         if: runner.os == 'Linux'
         with:
           toolchain: ${{ matrix.rust }}
-          components: clippy
 
       - uses: Swatinem/rust-cache@v2
 

Cargo.toml

@@ -21,12 +21,6 @@ log = { version = "0.4.22", default-features = false }
 tokio = { version = "1.40.0", default-features = false }
 which = { version = "6.0.0", default-features = false }
 
-[profile.dev]
-panic = "abort"
-
-[profile.release]
-panic = "abort"
-
 [profile.release.package.{{project-name}}-ebpf]
 debug = 2
 codegen-units = 1

test.sh

@@ -81,8 +81,18 @@ case $OS in
   # ${CRATE_NAME} and ${CRATE_NAME}-ebpf depend on ${CRATE_NAME}-common and ${CRATE_NAME} activates
   # ${CRATE_NAME}-common's aya dependency, we end up trying to compile the panic handler twice: once
   # from the bpf program, and again from std via aya.
-  cargo clippy --exclude "${CRATE_NAME}-ebpf" --all-targets --workspace -- --deny warnings
+  #
-  cargo clippy --package "${CRATE_NAME}-ebpf" --all-targets -- --deny warnings
+  # `-C panic=abort` because "unwinding panics are not supported without std";
+  # integration-ebpf contains `#[no_std]` binaries.
+  #
+  # `-Zpanic_abort_tests` because "building tests with panic=abort is not supported without
+  # `-Zpanic_abort_tests`"; Cargo does this automatically when panic=abort is set via profile
+  # but we want to preserve unwinding at runtime - here we are just running clippy so we don't
+  # care about unwinding behavior.
+  #
+  # `+nightly` because "the option `Z` is only accepted on the nightly compiler".
+  cargo +nightly clippy --exclude "${CRATE_NAME}-ebpf" --all-targets --workspace -- --deny warnings
+  cargo +nightly clippy --package "${CRATE_NAME}-ebpf" --all-targets -- --deny warnings -C panic=abort -Zpanic_abort_tests
 
   expect <<EOF
     set timeout 30        ;# Increase timeout if necessary

{{project-name}}-ebpf/.cargo/config.toml

@@ -1,12 +0,0 @@
-# We have this so that one doesn't need to manually pass
-# --target=bpfel-unknown-none -Z build-std=core when running cargo
-# check/build/doc etc.
-#
-# NB: this file gets loaded only if you run cargo from this directory, it's
-# ignored if you run from the workspace root. See
-# https://doc.rust-lang.org/cargo/reference/config.html#hierarchical-structure
-[build]
-target = ["bpfeb-unknown-none", "bpfel-unknown-none"]
-
-[unstable]
-build-std = ["core"]