Ah, yes, you are correct. Queued for 4.1. Thanks. -corey
On 04/22/2015 09:16 PM, Hidehiro Kawai wrote: > start_next_msg() issues a message placed in smi_info->waiting_msg > if it is non-NULL. However, sender() sets a message to > smi_info->curr_msg and NULL to smi_info->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->waiting_msg not > curr_msg. > > Signed-off-by: Hidehiro Kawai <[email protected]> > Cc: Corey Minyard <[email protected]> > Cc: [email protected] > Cc: [email protected] > --- > drivers/char/ipmi/ipmi_si_intf.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/char/ipmi/ipmi_si_intf.c > b/drivers/char/ipmi/ipmi_si_intf.c > index 5e90a18..3d49c70 100644 > --- 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 ------------------------------------------------------------------------------ One dashboard for servers and applications across Physical-Virtual-Cloud Widest out-of-the-box monitoring support with 50+ applications Performance metrics, stats and reports that give you Actionable Insights Deep dive visibility with transaction tracing using APM Insight. http://ad.doubleclick.net/ddm/clk/290420510;117567292;y _______________________________________________ Openipmi-developer mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openipmi-developer
