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

Use bpftool instead of iproute2 to load eBPF and XDP maps. #5183

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

fruffy
Copy link
Collaborator

@fruffy fruffy commented Mar 18, 2025

Fixes #2407

Makes loading eBPF programs a little bit less dependent on the distribution.

@fruffy fruffy added ebpf Topics related to the eBPF back end p4tc Topics related to the P4-TC back end. On PRs, also triggers p4tc CI tests to run. ebpf-psa Topics related to the eBPF PSA back end labels Mar 18, 2025
@fruffy fruffy marked this pull request as ready for review March 19, 2025 13:49
fruffy added 4 commits March 19, 2025 09:51
Signed-off-by: fruffy <fruffy@nyu.edu>
Signed-off-by: fruffy <fruffy@nyu.edu>
This reverts commit 46fde69.

Signed-off-by: fruffy <fruffy@nyu.edu>
Signed-off-by: fruffy <fruffy@nyu.edu>
@fruffy fruffy requested a review from tatry March 19, 2025 16:44
@fruffy
Copy link
Collaborator Author

fruffy commented Mar 19, 2025

@vbnogueira Might be interesting to you, although I do not think the p4tc back end will be much affected by these changes.

@fruffy fruffy requested a review from jafingerhut March 26, 2025 16:31
@fruffy
Copy link
Collaborator Author

fruffy commented Mar 26, 2025

@jafingerhut Unlikely to find any stakeholders with approval rights on this PR so I am asking for a review from you :).

@@ -145,6 +154,44 @@ def _attach_filters(self, bridge, proc):
return result
return testutils.SUCCESS

# def _attach_filters(self, bridge: Bridge, proc: subprocess.Popen) -> int:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you think this commented-out function is likely to be useful in the future?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, looks like this is the wrong branch... making this draft again. Losing track of all the PRs. Apologize for the premature ping!

@fruffy fruffy marked this pull request as draft March 26, 2025 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ebpf Topics related to the eBPF back end ebpf-psa Topics related to the eBPF PSA back end p4tc Topics related to the P4-TC back end. On PRs, also triggers p4tc CI tests to run.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use libbpf instead of iproute2 to load eBPF programs
2 participants