From owner-cvs-all Sun Dec 30 10:29:41 2001 Delivered-To: cvs-all@freebsd.org Received: from mout0.freenet.de (mout0.freenet.de [194.97.50.131]) by hub.freebsd.org (Postfix) with ESMTP id D1D3237B417; Sun, 30 Dec 2001 10:29:29 -0800 (PST) Received: from [194.97.50.138] (helo=mx0.freenet.de) by mout0.freenet.de with esmtp (Exim 3.33 #3) id 16Kki8-0007LL-00; Sun, 30 Dec 2001 19:29:28 +0100 Received: from aa81c.pppool.de ([213.6.168.28] helo=Magelan.Leidinger.net) by mx0.freenet.de with esmtp (Exim 3.33 #3) id 16Kki7-0001d6-00; Sun, 30 Dec 2001 19:29:28 +0100 Received: from Leidinger.net (netchild@localhost [127.0.0.1]) by Magelan.Leidinger.net (8.11.6/8.11.6) with ESMTP id fBUIDR298344; Sun, 30 Dec 2001 19:13:28 +0100 (CET) (envelope-from netchild@Leidinger.net) Message-Id: <200112301813.fBUIDR298344@Magelan.Leidinger.net> Date: Sun, 30 Dec 2001 19:13:26 +0100 (CET) From: Alexander Leidinger Subject: Re: cvs commit: src/sys/i4b/capi capi_l4if.c src/sys/i4b/capi/iavc iavc_isa.c iavc_pci.c src/sys/i4b/driver i4b_ing.c i4b_ipr.c i4b_ispppsubr.c i4b_rbch.c i4b_trace.c src/sys/i4b/layer1/ifpi i4b_ifpi_pci.c src/sys/i4b/ To: gj@FreeBSD.ORG Cc: cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG In-Reply-To: <200112300927.fBU9RSP39484@freefall.freebsd.org> MIME-Version: 1.0 Content-Type: TEXT/plain; charset=US-ASCII Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On 30 Dez, Gary Jennejohn wrote: > gj 2001/12/30 01:27:28 PST > > Modified files: > sys/i4b/capi capi_l4if.c > sys/i4b/capi/iavc iavc_isa.c iavc_pci.c > sys/i4b/driver i4b_ing.c i4b_ipr.c i4b_ispppsubr.c > i4b_rbch.c i4b_trace.c > sys/i4b/layer1/ifpi i4b_ifpi_pci.c > sys/i4b/layer1/ifpi2 i4b_ifpi2_pci.c > sys/i4b/layer1/ifpnp i4b_ifpnp_avm.c > sys/i4b/layer1/ihfc i4b_ihfc_drv.c > sys/i4b/layer1/isic i4b_bchan.c > sys/i4b/layer1/iwic i4b_iwic_bchan.c > sys/i4b/layer2 i4b_l2.c > sys/i4b/layer4 i4b_i4bdrv.c > Log: > Protect mtx_init() invocations with mtx_intialized() checks to avoid a > reported panic. > > Submitted by: Alexander Leidinger (partly) > MFC after: 4 weeks Do you really need it in the other files too? I just got the panics in layer1 (with my "Fritz! Classic"), that's the reason I just did the layer1 fixes (they are enough for me to not get a kernel panic again). In a perfect world, mtx_initialized() isn't needed, but therefore you have to rewrite the attach and probe handling of the drivers (at least this is what I got told by jhb(?)... "porting it to newbus" or something like this... if I remember corectly). mtx_initialized() is just a work around (because it didn't seems someone changes the handling in the near future). Bye, Alexander. -- Speak softly and carry a cellular phone. http://www.Leidinger.net Alexander @ Leidinger.net GPG fingerprint = C518 BC70 E67F 143F BE91 3365 79E2 9C60 B006 3FE7 To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message