-
Notifications
You must be signed in to change notification settings - Fork 64
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
Reintroduce hfsc-based shapers #114
Comments
wow lol |
xlfjn <notifications@github.com> writes:
It seems that sqm-scripts used to have hfsc shapers until 39eb7a0,
when it was removed due to its incompatibility with fq_codel at that
time. This kernel issue has since been resolved, as recorded in
a67824b.
It may be worthwhile to re-add these shapers as described in the
original removal commit 39eb7a0, "This commit can be reverted at a
newer kernel." It would help users test how hfsc performs in their
environment.
Not necessarily opposed to this; but I'm not sure if it's just a matter
of reverting that commit, as we've changed how we call sqm_start since
those scripts were removed...
|
Same goes also for never seeing an actual benefit to hfsc support in the first place. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It seems that sqm-scripts used to have hfsc shapers until 39eb7a0, when it was removed due to its incompatibility with fq_codel at that time. This kernel issue has since been resolved, as recorded in a67824b.
It may be worthwhile to re-add these shapers as described in the original removal commit 39eb7a0, "This commit can be reverted at a newer kernel." It would help users test how hfsc performs in their environment.
The text was updated successfully, but these errors were encountered: