Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for multiple levels of bpf2bpf calls #4047

Merged
merged 3 commits into from
Nov 27, 2024

Conversation

dthaler
Copy link
Collaborator

@dthaler dthaler commented Nov 25, 2024

Description

  • Update verifier, which includes the fix to make bpf2bpf calls work with multiple stack levels
  • Update ebpf-for-windows to deal with minor API breaking changes in the verifier
  • Remove redundant crab_verifier_wrapper.hpp and just use ebpf_verifier_wrapper.hpp
  • Update bindmonitor_bpf2bpf.c sample to test with multiple stack levels
  • Add bindmonitor_bpf2bpf.c as a bpf2c test case

Fixes #3388

Testing

This PR includes test changes.

Documentation

No impact.

Installation

No impact.

@dthaler dthaler changed the title Update verifier to latest Add support for multiple levels of bpf2bpf calls Nov 25, 2024
* Update verifier, which includes the fix to make bpf2bpf calls work with multiple stack levels
* Update ebpf-for-windows to deal with minor API breaking changes in the verifier
* Remove redundant crab_verifier_wrapper.hpp and just use ebpf_verifier_wrapper.hpp
* Update bindmonitor_bpf2bpf.c sample to test with multiple stack levels
* Add bindmonitor_bpf2bpf.c as a bpf2c test case

Signed-off-by: Dave Thaler <[email protected]>
Signed-off-by: Dave Thaler <[email protected]>
@dthaler dthaler requested a review from shpalani November 25, 2024 21:28
@dthaler dthaler enabled auto-merge November 27, 2024 01:44
@dthaler dthaler added this pull request to the merge queue Nov 27, 2024
Merged via the queue into microsoft:main with commit 99ebdec Nov 27, 2024
89 checks passed
@dthaler dthaler deleted the bpf2bpf-multistack branch November 27, 2024 03:10
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.

bpf2c should support bpf2bpf function calls
3 participants