>Number: 151586
>Category: kern
>Synopsis: [patch] [ipmi] Incorrect firmware version printed at boot time
>Confidential: no
>Severity: non-critical
>Priority: low
>Responsible: freebsd-bugs
>State: open
>Quarter:
>Keywords:
>Date-Required:
>Class: sw-bug
>Submitter-Id: current-users
>Arrival-Date: Tue Oct 19 21:50:07 UTC 2010
>Closed-Date:
>Last-Modified:
>Originator: Andrew Boyer
>Release: head
>Organization:
Avere Systems
>Environment:
N/A
>Description:
The ipmi driver prints out some status information when the driver loads:
ipmi0: <IPMI System Interface> on isa0
ipmi0: KCS mode found at io 0xca2 alignment 0x1 on isa
ipmi0: IPMI device rev. 1, firmware rev. 1.2, version 2.0
ipmi0: Number of channels 8
I noticed that the 'firmware rev' is not correct. In this case, the rev should
be 1.60 (hex) or 1.96 (dec).
The wrong byte is used in the printf statement. See IPMI 2.0 spec 1.0 Table
20-2 on page 240.
>How-To-Repeat:
Compare the printed firmware rev to the installed firmware.
>Fix:
--- a/src/sys/dev/ipmi/ipmi.c
+++ b/src/sys/dev/ipmi/ipmi.c
@@ -726,7 +726,7 @@ ipmi_startup(void *arg)
device_printf(dev, "IPMI device rev. %d, firmware rev. %d.%d, "
"version %d.%d\n",
req->ir_reply[1] & 0x0f,
- req->ir_reply[2] & 0x0f, req->ir_reply[4],
+ req->ir_reply[2] & 0x0f, req->ir_reply[3],
req->ir_reply[4] & 0x0f, req->ir_reply[4] >> 4);
ipmi_free_request(req);
>Release-Note:
>Audit-Trail:
>Unformatted:
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "[email protected]"
