diff options
author | Martin Wilck <martin.wilck@ts.fujitsu.com> | 2010-06-29 18:05:31 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-06-29 18:29:31 -0400 |
commit | 8d1f66dc9b4f80a1441bc1c33efa98aca99e8813 (patch) | |
tree | 0876484c740e90568f9f0300f012e674622679c9 | |
parent | 5c0c165490e763eddcbb0b6c8c41ab5cc11ddea0 (diff) |
ipmi: set schedule_timeout_wait() value back to one
Fix a regression introduced by ae74e823cb7d ("ipmi: add parameter to limit
CPU usage in kipmid").
Some systems were seeing CPU usage go up dramatically with the recent
changes to try to reduce timer usage in the IPMI driver. This was traced
down to schedule_timeout_interruptible(1) being changed to
schedule_timeout_interruptbile(0). Revert that part of the change.
Addresses https://bugzilla.kernel.org/show_bug.cgi?id=16147
Reported-by: Thomas Jarosch <thomas.jarosch@intra2net.com>
Signed-off-by: Corey Minyard <cminyard@mvista.com>
Tested-by: Thomas Jarosch <thomas.jarosch@intra2net.com>
Cc: <stable@kernel.org> [2.6.34.x]
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | drivers/char/ipmi/ipmi_si_intf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/char/ipmi/ipmi_si_intf.c b/drivers/char/ipmi/ipmi_si_intf.c index 311f85b67cc1..094bdc355b1f 100644 --- a/drivers/char/ipmi/ipmi_si_intf.c +++ b/drivers/char/ipmi/ipmi_si_intf.c | |||
@@ -1024,7 +1024,7 @@ static int ipmi_thread(void *data) | |||
1024 | else if (smi_result == SI_SM_IDLE) | 1024 | else if (smi_result == SI_SM_IDLE) |
1025 | schedule_timeout_interruptible(100); | 1025 | schedule_timeout_interruptible(100); |
1026 | else | 1026 | else |
1027 | schedule_timeout_interruptible(0); | 1027 | schedule_timeout_interruptible(1); |
1028 | } | 1028 | } |
1029 | return 0; | 1029 | return 0; |
1030 | } | 1030 | } |