Date: Thu, 16 Aug 2018 09:11:01 -0700 From: Ravi Pokala <rpokala@freebsd.org> To: Doug Ambrisko <ambrisko@FreeBSD.org>, <src-committers@freebsd.org>, <svn-src-all@freebsd.org>, <svn-src-head@freebsd.org> Subject: Re: svn commit: r337913 - in head/sys: dev/ipmi modules/ipmi Message-ID: <F78BBCB2-B021-4767-8280-6DD91F5C189D@panasas.com> In-Reply-To: <201808161559.w7GFx3pg061595@repo.freebsd.org> References: <201808161559.w7GFx3pg061595@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
The confluence of these two issues is that on amd64 ipmi.ko loads, but does= n't attach to the BMC or create /dev/ipmi, and that the kernel panics when t= rying to unload. Right? If so, that's exactly the issue I hit on Monday, but didn't get a chance to= triage. Thanks! -Ravi =EF=BB=BF-----Original Message----- From: <owner-src-committers@freebsd.org> on behalf of Doug Ambrisko <ambris= ko@FreeBSD.org> Date: 2018-08-16, Thursday at 08:59 To: <src-committers@freebsd.org>, <svn-src-all@freebsd.org>, <svn-src-head@= freebsd.org> Subject: svn commit: r337913 - in head/sys: dev/ipmi modules/ipmi Author: ambrisko Date: Thu Aug 16 15:59:02 2018 New Revision: 337913 URL: https://svnweb.freebsd.org/changeset/base/337913 Log: Fix a module Makefile error on amd64 so the IPMI HW interfaces are built. When the module is being unloaded and no HW interfaces were created don't clean up. This was exposed by the amd64 module build issue. Modified: head/sys/dev/ipmi/ipmi.c head/sys/modules/ipmi/Makefile Modified: head/sys/dev/ipmi/ipmi.c =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D --- head/sys/dev/ipmi/ipmi.c Thu Aug 16 15:54:12 2018 (r337912) +++ head/sys/dev/ipmi/ipmi.c Thu Aug 16 15:59:02 2018 (r337913) @@ -1053,6 +1053,8 @@ ipmi_unload(void *arg) int count; int i; =20 + if (ipmi_devclass =3D=3D NULL) + return; if (devclass_get_devices(ipmi_devclass, &devs, &count) !=3D 0) return; for (i =3D 0; i < count; i++) Modified: head/sys/modules/ipmi/Makefile =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D --- head/sys/modules/ipmi/Makefile Thu Aug 16 15:54:12 2018 (r337912) +++ head/sys/modules/ipmi/Makefile Thu Aug 16 15:59:02 2018 (r337913) @@ -7,7 +7,7 @@ KMOD=3D ipmi SRCS=3D ipmi.c SRCS+=3D bus_if.h device_if.h -.if ${MACHINE_CPUARCH} =3D=3D "i386" || ${MACHINE_CPUARCH} =3D=3D "i386" +.if ${MACHINE_CPUARCH} =3D=3D "i386" || ${MACHINE_CPUARCH} =3D=3D "amd64" SRCS+=3D ipmi_kcs.c ipmi_smic.c ipmi_smbios.c ipmi_ssif.c SRCS+=3D ipmi_acpi.c ipmi_isa.c ipmi_pci.c ipmi_smbus.c SRCS+=3D opt_acpi.h
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?F78BBCB2-B021-4767-8280-6DD91F5C189D>