From owner-svn-src-head@FreeBSD.ORG Mon Jul 29 19:59:23 2013 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id F420CBF8 for ; Mon, 29 Jul 2013 19:59:22 +0000 (UTC) (envelope-from sean_bruno@yahoo.com) Received: from nm34-vm8.bullet.mail.bf1.yahoo.com (nm34-vm8.bullet.mail.bf1.yahoo.com [72.30.239.6]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 97D4E2C7C for ; Mon, 29 Jul 2013 19:59:22 +0000 (UTC) Received: from [98.139.215.141] by nm34.bullet.mail.bf1.yahoo.com with NNFMP; 29 Jul 2013 19:59:14 -0000 Received: from [98.139.211.197] by tm12.bullet.mail.bf1.yahoo.com with NNFMP; 29 Jul 2013 19:59:14 -0000 Received: from [127.0.0.1] by smtp206.mail.bf1.yahoo.com with NNFMP; 29 Jul 2013 19:59:14 -0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yahoo.com; s=s1024; t=1375127954; bh=Ei04UKTl5robTQH1c4uGq+lkVTq02I2FYjcxOTHacyw=; h=X-Yahoo-Newman-Id:X-Yahoo-Newman-Property:X-YMail-OSG:X-Yahoo-SMTP:X-Rocket-Received:Subject:From:Reply-To:To:Cc:In-Reply-To:References:Content-Type:Date:Message-ID:Mime-Version:X-Mailer; b=mc2SSH4ZwHnvoJj+S+B4R8FhLqS4PXzfBZbslsX90lTOMbttvb8XLe9ZTqit33h8kz4H26KNNOFXK0IYPZu39habUhHyd7F8zWk3yd3r3rGyfrwYveVUTBASkdMsDIe1Ty+wtVOMOqfAzJzd2dOyu0XDHlV/MlwoAUfxXTTAbq0= X-Yahoo-Newman-Id: 289926.19393.bm@smtp206.mail.bf1.yahoo.com X-Yahoo-Newman-Property: ymail-3 X-YMail-OSG: IFwfFjUVM1n3J_ckwAWF2w6oDhazsUXNl7w7ZvlKX26HNzO QB_VPqOyGIKabh0YoM6cmwL3aL1hMMwGpK6B1vPIQYrwtJfHoUFJW9mLkZed rbjwm7XvdwrbUUe4rTFsBWrPGM6mhVWLJmlzDMoAUUW9E56s_wCsh7ThDvii 6RSd7bOnR7Ru8at99lLyzs1.JyDEQvcUz9TSaBAU_J390ZFINnknPDO5XaCL Wo88FzgGcFRUfHvGpWFqu8ozUoMyec4vWDGhiTAPr9yW.JVM5SZF9PPtQFdQ v8W2zlZ2x4Sb9p3CDDw3Nd5BkmsjNVdVCDkzXBRroY.RaMOKPXtwe8aIfZZL Yknez2svyh45zFknfYvseqK.bDQx.BpknT7L_nj6eR_kZLFov0BygkX0J4v8 ZQ4YNRIyM.JqNFbxooYCqKJa0kObWMjQ_beZ_WN7WG0kYlwmXy.ZmMDXMaec t3VgiNnSeiLzPV5sxkFeEdg6byw_IVuMNjLJeGDslFQ4KHjQpnA0arJGehOM Ogzs86YIImwr8fz4PTBTebKMfkSeruoc6za4E36OtgnjHOtHzERdzLXEI4jp QVIDy.ajFh0lh4RolxH5l3cun6o1pqR5KKksdaQivetQiHWSCIMg2q3Zx2Pl Dxt99rlQ- X-Yahoo-SMTP: u5BKR6OswBC_iZJVfGRoMkTIpc8pEA4- X-Rocket-Received: from [10.73.160.242] (sean_bruno@209.131.62.116 with ) by smtp206.mail.bf1.yahoo.com with SMTP; 29 Jul 2013 12:59:14 -0700 PDT Subject: Re: svn commit: r253708 - head/sys/dev/ipmi From: Sean Bruno To: John Baldwin In-Reply-To: <201307291054.55820.jhb@freebsd.org> References: <201307271632.r6RGWYF8046749@svn.freebsd.org> <201307291054.55820.jhb@freebsd.org> Content-Type: multipart/signed; micalg="pgp-sha1"; protocol="application/pgp-signature"; boundary="=-7uuXqBiFI4LiVSghNd9R" Date: Mon, 29 Jul 2013 12:59:12 -0700 Message-ID: <1375127952.1479.32.camel@localhost> Mime-Version: 1.0 X-Mailer: Evolution 2.32.1 FreeBSD GNOME Team Port Cc: svn-src-head@freebsd.org, Sean Bruno , src-committers@freebsd.org, svn-src-all@freebsd.org X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list Reply-To: sbruno@freebsd.org List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 29 Jul 2013 19:59:23 -0000 --=-7uuXqBiFI4LiVSghNd9R Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable 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 > >=20 > > 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 identi= fy > > function in ipmi_isa.c to attempt to probe and poke at the ISA IPMI i= nterface >=20 > They never had a common parent, even in 6.x and 7.x. >=20 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. > > =20 > > This probing appears to make Broadcom management firmware on Dell mac= hines > > crash and emit NMI EISA warnings at various times requiring power cyc= les > > of the machines to restore. >=20 > 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 memo= ry > causes side effects, then running dmidecode in userland should be hosing = your > machines as well. >=20 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. Sean --=-7uuXqBiFI4LiVSghNd9R Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.14 (FreeBSD) iQEcBAABAgAGBQJR9smMAAoJEBkJRdwI6BaHSqMH/164tPPSI0guiuAeB9W44gGj o7R6Ti9+EOP1SeDZW5wcMUFPg5VO1WvZ6oYh0e5vxp125ydr2MxkelXhx6lSpKli W9F289GSFhq7MV4I3+YCY3wJ+9yDLrqaba2e2DOw7PBdH4FYwzf8ICN+3JsTxTvV S8YGWWjM/qapZYXKLgBokdif96/HT7saj2NHAE0cl61I4WUTXxY2gSChwAEPMDcV ai+cHOPZAtkHjK1hzS6E6QkeyZi3rkEIxPD4M4x3aX11KEN+3VZqeo0/WM5Ognn1 B7sZsmvRBhPJVxXbKp0M9rzu0m0evmQ2l5v2n7uJjAHp5qd7bTWLiHjnYl0lamI= =Q9AM -----END PGP SIGNATURE----- --=-7uuXqBiFI4LiVSghNd9R--