ipmi: Avoid touching possible corrupted lists in the panic context
authorHidehiro Kawai <hidehiro.kawai.ez@hitachi.com>
Mon, 27 Jul 2015 05:55:16 +0000 (14:55 +0900)
committerCorey Minyard <cminyard@mvista.com>
Thu, 3 Sep 2015 20:02:29 +0000 (15:02 -0500)
commit06e5e345fea8df24b1d935f98741343df4cab664
tree7ba5bd37d180c796b1518199c057af065d9450c0
parent82802f968bd3118af04eaeb3814c21d9813be527
ipmi: Avoid touching possible corrupted lists in the panic context

When processing queued messages in the panic context, IPMI driver
tries to do it without any locking to avoid deadlocks.  However,
this means we can touch a corrupted list if the kernel panicked
while manipulating the list.  Fortunately, current `add-tail and
del-from-head' style implementation won't touch the corrupted part,
but it is inherently risky.

To get rid of the risk, this patch re-initializes the message lists
on panic if the related spinlock has already been acquired.  As the
result, we may lose queued messages, but it's not so painful.
Dropping messages on the received message list is also less
problematic because no one can respond the received messages.

Signed-off-by: Hidehiro Kawai <hidehiro.kawai.ez@hitachi.com>
Fixed a comment typo.

Signed-off-by: Corey Minyard <cminyard@mvista.com>
drivers/char/ipmi/ipmi_msghandler.c