Date: Thu, 16 Aug 2018 11:23:31 -0500 From: Justin Hibbits <chmeeedalf@gmail.com> To: Ravi Pokala <rpokala@freebsd.org> Cc: 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: <20180816112331.2d6d638f@ralga.knownspace> In-Reply-To: <F78BBCB2-B021-4767-8280-6DD91F5C189D@panasas.com> References: <201808161559.w7GFx3pg061595@repo.freebsd.org> <F78BBCB2-B021-4767-8280-6DD91F5C189D@panasas.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Woops, my fault way back when I made powerpc64's IPMI OPAL buildable as a module. Can't believe I had i386 twice in there. - Justin On Thu, 16 Aug 2018 09:11:01 -0700 Ravi Pokala <rpokala@freebsd.org> wrote: > The confluence of these two issues is that on amd64 ipmi.ko loads, > but doesn't attach to the BMC or create /dev/ipmi, and that the > kernel panics when trying to unload. Right? >=20 > If so, that's exactly the issue I hit on Monday, but didn't get a > chance to triage. Thanks! >=20 > -Ravi >=20 > =EF=BB=BF-----Original Message----- > From: <owner-src-committers@freebsd.org> on behalf of Doug Ambrisko > <ambrisko@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 >=20 > Author: ambrisko > Date: Thu Aug 16 15:59:02 2018 > New Revision: 337913 > URL: https://svnweb.freebsd.org/changeset/base/337913 >=20 > 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. >=20 > Modified: > head/sys/dev/ipmi/ipmi.c > head/sys/modules/ipmi/Makefile >=20 > 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++) >=20 > 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 >=20 >=20 >=20 >=20
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20180816112331.2d6d638f>