Skip to content

Support 64bit limit for police actions #1081

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.

Sign up for GitHub

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

Jump to bottom
Open
jfogh-akamai opened this issue May 7, 2025 · 1 comment · May be fixed by #1084
Open

Support 64bit limit for police actions #1081

jfogh-akamai opened this issue May 7, 2025 · 1 comment · May be fixed by #1084

Comments

@jfogh-akamai
Copy link

The issue

The library currently only supports 32 bit values for "rate" and "peak rate" in police actions.

Implementation notes

The fields are stored in the struct nl.TcPolice, which only supports 32 bits. The protocol supports separate 64bit attributes, which can be appended after the nl.TcPolice instance.

This change is completely analogous to the commit "implemented handling of uint64 for HTB class rate/ceil
".

NB:
I expect to have a PR ready within a few days.
@aboch
Copy link
Collaborator

aboch commented May 7, 2025

Thank you

@jfogh-akamai jfogh-akamai linked a pull request May 9, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Support 64 bit limit for police actions jfogh-akamai/netlink
2 participants
@aboch @jfogh-akamai