Bug 837048 - irqbalance ONESHOT configuration not implemented properly
Summary: irqbalance ONESHOT configuration not implemented properly
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: irqbalance
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Petr Holasek
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-07-02 15:22 UTC by Andrew J. Schorr
Modified: 2016-10-04 04:08 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-09-17 23:54:36 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Andrew J. Schorr 2012-07-02 15:22:13 UTC
Description of problem:  When irqbalance was converted to systemd, the ONESHOT
configuration variable was not converted properly.  In /etc/sysconfig/irqbalance,
it suggests setting ONESHOT to yes to enable that feature.  But
the /lib/systemd/system/irqbalance.service file simply passes this argument
to irqbalance, which will not understand an argument of "yes".  


Version-Release number of selected component (if applicable):
irqbalance-1.0.3

How reproducible: Try setting ONESHOT to yes in /etc/sysconfig/irqbalance


Steps to Reproduce:
1. Look at /etc/sysconfig/irqbalance and /lib/systemd/system/irqbalance.service and observe that ONESHOT is not implemented properly.
2.
3.
  
Actual results:


Expected results:


Additional info:  The simplest fix is probably to edit /etc/sysconfig/irqbalance to suggest setting ONESHOT to --oneshot if that behavior
is desired.

Comment 1 Neil Horman 2012-07-04 17:17:31 UTC
yup, Andrew is correct.  That should definately be fixed.  I'd recommend however, instead of using the --oneshot mechanism, we convert the sysconfig file to set IRQBALANCE_ONESHOT, and just remove the $ONESHOT from the unit file.

Comment 2 Andrew J. Schorr 2012-07-04 17:31:30 UTC
Hi Neil,

Since it may be desirable to pass other arguments to irqbalance,
such as --powerthresh=1 or --shouldban=/usr/sbin/irqban.sh, would it make
sense simply to have IRQBALANCE_ARGS in the sysconfig and systemd files?

Also, I think IRQ_AFFINITY_MASK is obsolete.

Thanks,
Andrew

Comment 3 Neil Horman 2012-07-04 23:28:51 UTC
Yes, it would make sense to have an IRQBALANCE_ARGS environment variable that we can use to pass args to the daemon, I'll add that upstream, and petr can pull it in here (right after I finish with the banscript work, which I hope should be tomorrow).  

And yes, IRQ_AFFINTY_MASK is obsolete.  Its been removed from upstream for some time now, and should be removed in fedora.  Petr, can you square that up please?

Comment 4 Petr Holasek 2012-07-09 17:46:27 UTC
Ok, I will sync it with upstream in the next release during this week.

Comment 5 Fedora Update System 2012-08-23 14:12:23 UTC
irqbalance-1.0.3-6.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/irqbalance-1.0.3-6.fc18

Comment 6 Fedora Update System 2012-08-23 15:37:56 UTC
Package irqbalance-1.0.3-6.fc18:
* should fix your issue,
* was pushed to the Fedora 18 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing irqbalance-1.0.3-6.fc18'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-12570/irqbalance-1.0.3-6.fc18
then log in and leave karma (feedback).

Comment 7 Fedora Update System 2012-08-29 15:06:48 UTC
irqbalance-1.0.3-7.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/irqbalance-1.0.3-7.fc18

Comment 8 Fedora Update System 2012-09-03 11:26:35 UTC
irqbalance-1.0.3-8.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/irqbalance-1.0.3-8.fc18

Comment 9 Fedora Update System 2012-09-17 23:54:36 UTC
irqbalance-1.0.3-8.fc18 has been pushed to the Fedora 18 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.