Make syscall policy explicit; rename deny to blocklist#38
Merged
congwang-mk merged 5 commits intomainfrom May 6, 2026
Merged
Conversation
Signed-off-by: Cong Wang <cwang@multikernel.io>
Signed-off-by: Cong Wang <cwang@multikernel.io>
Signed-off-by: Cong Wang <cwang@multikernel.io>
Signed-off-by: Cong Wang <cwang@multikernel.io>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
confine()API rename: drops the legacy spelling that mismatched the rest of the public API. (643097d)policy.allow_syscalls/policy.deny_syscallspair with a single explicitSyscallPolicy { None, Allow(_), Deny(_) }enum so it's impossible for a policy to be in an underspecified middle state. (a5bc946)Allow(_)arm was never wired into the BPF emitter — drop it rather than ship a knob that silently does nothing. (783350a)DEFAULT_DENY_SYSCALLS→DEFAULT_BLOCKLIST_SYSCALLS,SyscallPolicy::Deny→SyscallPolicy::Blocklist,deny_syscall_numbers→blocklist_syscall_numbers, builder methoddeny_syscalls→block_syscalls. Prose-level mentions of "deny JEQs" / "kernel-deny" stay since those describe the BPF concept, not the renamed list. (540aebe)Rebased onto current
main(the Handler-trait reshape from #36 plus its post-merge cleanup). Touches Rust core, FFI, CLI, Python, README, anddocs/extension-handlers.md.Test plan
cargo build --workspace --testscleancargo test --workspace --lib— 258 passingcargo test --workspace --test integration extra— 14 passing (full Handler-trait integration coverage)pyteston the newpolicy.py/_profile.py/_sdk.pyshape