-
Notifications
You must be signed in to change notification settings - Fork 12
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
Allow to input multiple (IPs in command/commands) from stdin #8
Comments
@tohojo and what about this? |
Jarosław Kłopotek - INTERDUO ***@***.***> writes:
@tohojo and what about this?
That should be doable, in principle. However, you mentioned you were
using libreqos; I'm not sure that libreqos uses xdp-cpumap-tc as-is, so
you may be better off opening an issues with them?
|
Awesome!
Problems should be repaired in source. Pure xdp-cpumap-tc from gitrepo is also affected - just try to load firstly 1rule and then 4-5K enteries using XDP. See the difference. |
Jarosław Kłopotek - INTERDUO ***@***.***> writes:
> That should be doable, in principle.
Awesome!
> However, you mentioned you were
using libreqos; I'm not sure that libreqos uses xdp-cpumap-tc as-is, so
you may be better off opening an issues with them?
Problems should be repaired in source. Pure xdp-cpumap-tc from gitrepo
is also affected - just try to load firstly 1rule and then 4-5K
enteries using XDP. See the difference.
Right; well, I'm certainly not opposed to merging a pull request that
implements batching. However, I am also not planning to work on
implementing this myself :)
|
I am just good at PHP and not low level :-( If You could say how many gold You want for this feature maybe I could organise it. |
Problem:
Reloading of XDP many rules (8k+) takes too much time. At the end it cases Little packet loss.
I suggest to make:
[xdp_iphash_to_cpu_cmdline.c] to allow for multiple IPs in one command or aggregate multiple commands from file or - - batch command (like tc command).
Other way for resolving that could be to load the rules to another set and then change active set.
...or maybe there is another metod or resolving that problem?
The text was updated successfully, but these errors were encountered: