2005-12-09 18:03:08

by Matt Domsch

[permalink] [raw]
Subject: [PATCH 2.6] ipmi: panic generator ID

The IPMI specifcation says the generator ID is 0x20, but that is for
bits 7-1. Bit 0 is set to specify it is a software event. The
correct value is 0x41. Without this fix, panic events written into
the System Event Log appear to come from an "unknown" generator,
rather than from the kernel.

Signed-off-by: Jordan Hargrave <[email protected]>
Signed-off-by: Matt Domsch <[email protected]>
Ack'd-by: Corey Minyard <[email protected]>


--
Matt Domsch
Software Architect
Dell Linux Solutions linux.dell.com & http://www.dell.com/linux
Linux on Dell mailing lists @ http://lists.us.dell.com

--- linux-2.6/drivers/char/ipmi/ipmi_msghandler.c Tue Dec 6 12:53:19 2005
+++ linux-2.6/drivers/char/ipmi/ipmi_msghandler.c Fri Dec 9 10:27:42 2005
@@ -2986,7 +2986,7 @@ static void send_panic_events(char *str)
msg.cmd = 2; /* Platform event command. */
msg.data = data;
msg.data_len = 8;
- data[0] = 0x21; /* Kernel generator ID, IPMI table 5-4 */
+ data[0] = 0x41; /* Kernel generator ID, IPMI table 5-4 */
data[1] = 0x03; /* This is for IPMI 1.0. */
data[2] = 0x20; /* OS Critical Stop, IPMI table 36-3 */
data[4] = 0x6f; /* Sensor specific, IPMI table 36-1 */


2005-12-09 18:31:40

by Corey Minyard

[permalink] [raw]
Subject: Re: [PATCH 2.6] ipmi: panic generator ID

Yes, this is correct, I can't shift bits :).

Thanks, Jordon and Matt.

-Corey

Matt Domsch wrote:

>The IPMI specifcation says the generator ID is 0x20, but that is for
>bits 7-1. Bit 0 is set to specify it is a software event. The
>correct value is 0x41. Without this fix, panic events written into
>the System Event Log appear to come from an "unknown" generator,
>rather than from the kernel.
>
>Signed-off-by: Jordan Hargrave <[email protected]>
>Signed-off-by: Matt Domsch <[email protected]>
>Ack'd-by: Corey Minyard <[email protected]>
>
>
>
>