-
Notifications
You must be signed in to change notification settings - Fork 85
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
Consolidate duplicate tc-loader scripts #9
Comments
Same thing probably goes for the configure script to detect if iproute has libbpf support. |
Yeah. IIRC, @netoptimizer expressed a preference for keeping separate configure scripts in each subdir, but they can still share code :) |
Worth mentioning that in my current PR I've extended the tc loader script somewhat to support attaching a pinned program (and gotten rid of the configure script). |
As libbpf now support TC-attaching directly, we should consider converting "pping" and "traffic-pacing-edt" to use that instead of mentioned shell script. |
Yep, converting pping to use libbpf for attaching the tc program is on my TODO list, so this issue is likely goning to resolve itself that way. |
We should probably consolidate the different copies of the TC loader scripts in subdirectories into a single version in a parent/lib directory...
The text was updated successfully, but these errors were encountered: