-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
irqbalance: Update init script to remove duplicate spaces #23557
Conversation
Why are the checks failing? |
Missing |
Please format your commits, thanks! |
a0eeae4
to
e0c9438
Compare
Done. |
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.
Also please wrap a new line for every 70 chars in your commit message.
cc @hnyman for review |
@1715173329 - updated commit message and make use of procd_append_param for parameters, also added check for default values. There might be more options to optimize this... Please note that I've added a new debug configuration parameter, the patch for luci will follow after this PR was merged. |
I have some strange issues with irqbalance sometimes overwritin smp_affinity values for banned/ignored IRQs. The issue is reproduceable and is mitigated when I change theway how the irqbalance command line is built. The only difference between the resulting command is that there is only one space between the -t parameter and the first -i parameter value. Also see Irqbalance/irqbalance#297 Signed-off-by: Carsten Schuette <schuettecarsten@googlemail.com>
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.
lgtm, thanks
Thank you for merging, the luci patch is available at openwrt/luci#6971 |
Merged ;-) |
I have some strange issues with irqbalance sometimes overwriting smp_affinity values for banned/ignored IRQs. The issue is reproduceable and is mitigated when I change theway how the irqbalance command line is built. The only difference between the resulting command is that there is only one space between the -t parameter and the first -i parameter value.