ipmi: Fix a problem that messages are not issued in run_to_completion mode

Hidehiro Kawai · cminyard · commit 9f8127048ab8 · 2015-05-05T19:33:49.000-05:00
start_next_msg() issues a message placed in smi_info-&gt;waiting_msg
if it is non-NULL.  However, sender() sets a message to
smi_info-&gt;curr_msg and NULL to smi_info-&gt;waiting_msg in the context
of run_to_completion mode.  As the result, it leads an infinite
loop by waiting the completion of unissued message when leaving
dying message after kernel panic.

sender() should set the message to smi_info-&gt;waiting_msg not
curr_msg.

Signed-off-by: Hidehiro Kawai &lt;hidehiro.kawai.ez@hitachi.com&gt;
Signed-off-by: Corey Minyard &lt;cminyard@mvista.com&gt;
diff --git a/drivers/char/ipmi/ipmi_si_intf.c b/drivers/char/ipmi/ipmi_si_intf.c
@@ -942,8 +942,7 @@ static void sender(void                *send_info,
 		 * If we are running to completion, start it and run
 		 * transactions until everything is clear.
 		 */
-		smi_info->curr_msg = msg;
-		smi_info->waiting_msg = NULL;
+		smi_info->waiting_msg = msg;
 
 		/*
 		 * Run to completion means we are single-threaded, no
