Commit 77cf3973 authored by Corey Minyard's avatar Corey Minyard Committed by Linus Torvalds

[PATCH] ipmi: use completions, not semaphores, in powerdown code

Don't use semaphores for IPC in the poweroff code, use completions instead.
Signed-off-by: default avatarCorey Minyard <minyard@acm.org>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 3b625943
...@@ -31,12 +31,13 @@ ...@@ -31,12 +31,13 @@
* with this program; if not, write to the Free Software Foundation, Inc., * with this program; if not, write to the Free Software Foundation, Inc.,
* 675 Mass Ave, Cambridge, MA 02139, USA. * 675 Mass Ave, Cambridge, MA 02139, USA.
*/ */
#include <asm/semaphore.h> #include <linux/config.h>
#include <linux/kdev_t.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/moduleparam.h> #include <linux/moduleparam.h>
#include <linux/proc_fs.h> #include <linux/proc_fs.h>
#include <linux/string.h> #include <linux/string.h>
#include <linux/completion.h>
#include <linux/kdev_t.h>
#include <linux/ipmi.h> #include <linux/ipmi.h>
#include <linux/ipmi_smi.h> #include <linux/ipmi_smi.h>
...@@ -89,10 +90,10 @@ static struct ipmi_recv_msg halt_recv_msg = ...@@ -89,10 +90,10 @@ static struct ipmi_recv_msg halt_recv_msg =
static void receive_handler(struct ipmi_recv_msg *recv_msg, void *handler_data) static void receive_handler(struct ipmi_recv_msg *recv_msg, void *handler_data)
{ {
struct semaphore *sem = recv_msg->user_msg_data; struct completion *comp = recv_msg->user_msg_data;
if (sem) if (comp)
up(sem); complete(comp);
} }
static struct ipmi_user_hndl ipmi_poweroff_handler = static struct ipmi_user_hndl ipmi_poweroff_handler =
...@@ -106,21 +107,21 @@ static int ipmi_request_wait_for_response(ipmi_user_t user, ...@@ -106,21 +107,21 @@ static int ipmi_request_wait_for_response(ipmi_user_t user,
struct kernel_ipmi_msg *send_msg) struct kernel_ipmi_msg *send_msg)
{ {
int rv; int rv;
struct semaphore sem; struct completion comp;
sema_init (&sem, 0); init_completion(&comp);
rv = ipmi_request_supply_msgs(user, addr, 0, send_msg, &sem, rv = ipmi_request_supply_msgs(user, addr, 0, send_msg, &comp,
&halt_smi_msg, &halt_recv_msg, 0); &halt_smi_msg, &halt_recv_msg, 0);
if (rv) if (rv)
return rv; return rv;
down (&sem); wait_for_completion(&comp);
return halt_recv_msg.msg.data[0]; return halt_recv_msg.msg.data[0];
} }
/* We are in run-to-completion mode, no semaphore is desired. */ /* We are in run-to-completion mode, no completion is desired. */
static int ipmi_request_in_rc_mode(ipmi_user_t user, static int ipmi_request_in_rc_mode(ipmi_user_t user,
struct ipmi_addr *addr, struct ipmi_addr *addr,
struct kernel_ipmi_msg *send_msg) struct kernel_ipmi_msg *send_msg)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment