-
Notifications
You must be signed in to change notification settings - Fork 461
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
base: main
Are you sure you want to change the base?
Conversation
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>
@vbnogueira Might be interesting to you, although I do not think the p4tc back end will be much affected by these changes. |
@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: |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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!
Fixes #2407
Makes loading eBPF programs a little bit less dependent on the distribution.