svn commit: r253708 - head/sys/dev/ipmi
sean_bruno at yahoo.com
Mon Jul 29 19:59:23 UTC 2013
On Mon, 2013-07-29 at 10:54 -0400, John Baldwin wrote:
> On Saturday, July 27, 2013 12:32:34 pm Sean Bruno wrote:
> > Author: sbruno
> > Date: Sat Jul 27 16:32:34 2013
> > New Revision: 253708
> > URL: http://svnweb.freebsd.org/changeset/base/253708
> > Log:
> > At some point after stable/7 the ACPI and ISA interfaces to the IPMI controller
> > no longer have the parent in the device tree. This causes the identify
> > function in ipmi_isa.c to attempt to probe and poke at the ISA IPMI interface
> They never had a common parent, even in 6.x and 7.x.
The identify function in isa_ipmi.c shows that there is already an
ipmi(4) device attached (ACPI) version and aborts on 7.x. in 9.x and
higher (not testing on 8.x) the identify function does not see an
attached ipmi interface and attempts to create /dev/ipmi1
Am I just confused on the bus relationship here?
We've gone over this a couple of times in different emails on different
lists. I've just never sat down and walked through the code. If you
see a better way to keep ipmi(4) from erroneously attaching to the ISA
interface, let me know.
> > Move the check for ipmi_attached out of the ipmi_isa_attach function and into
> > the ipmi_isa_identify function. Remove the check of the device tree for
> > ipmi devices attached.
> > This probing appears to make Broadcom management firmware on Dell machines
> > crash and emit NMI EISA warnings at various times requiring power cycles
> > of the machines to restore.
> This makes no sense. All you are doing is skipping ipmi_smbios_identify()
> which just looks at the SMBIOS table in RAM. It doesn't try to probe the
> BMC at all (no register accesses, etc.). If just reading a table in memory
> causes side effects, then running dmidecode in userland should be hosing your
> machines as well.
Probably right. I'm not exactly sure what is making the Broadcom
firmware fall over and die. But when I see the console emitting "NMI
EISA" error messages, this ususally requires a full reboot as the
network interface has crashed. Hopefully, I can dig up more "fact" soon
as testing continues.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 488 bytes
Desc: This is a digitally signed message part
More information about the svn-src-head