Skip to content

fix: gate sleepable probes behind opt-in - #281

Open
swananan wants to merge 1 commit into
mainfrom
fix/gate-sleepable-probes
Open

fix: gate sleepable probes behind opt-in#281
swananan wants to merge 1 commit into
mainfrom
fix/gate-sleepable-probes

Conversation

@swananan

@swananan swananan commented Aug 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • skip sleepable uprobe helper and tail-call probes unless the mode is explicitly enabled
  • preserve the published one-argument startup API and add an explicitly named sleepable opt-in entry point
  • keep separate base and full capability caches so default startup cannot hide later opt-in detection
  • add end-to-end coverage that evicts an anonymous page with MADV_DONTNEED, proves an ordinary uprobe cannot read it, and verifies sleepable mode faults it back in
  • preserve existing source fixture line numbers used by line-level tracing tests

Testing

  • cargo fmt --all -- --check
  • cargo clippy --all-targets --all-features -- -D warnings
  • CARGO_TARGET_DIR=/tmp/ghostscope-target-gate-sleepable cargo test -p ghostscope-loader kernel_caps
  • full standard host e2e suite through the runner service (job 4a4f9d0c2d74)

Skip sleepable helper and tail-call probes unless sleepable uprobes are
enabled. Preserve the one-argument startup detection API and expose
sleepable probing through an explicitly named opt-in entry point. Keep
separate base and full capability caches so an unprobed startup result cannot
mask later opt-in detection.

Add an e2e fixture that evicts an anonymous page and verifies only a
sleepable uprobe can fault it back in and read it.
@swananan
swananan force-pushed the fix/gate-sleepable-probes branch from 042440e to eb08a41 Compare August 29, 2026 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant