Date: Sun, 30 Dec 2001 19:13:26 +0100 (CET) From: Alexander Leidinger <Alexander@Leidinger.net> To: gj@FreeBSD.ORG Cc: cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG 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/ Message-ID: <200112301813.fBUIDR298344@Magelan.Leidinger.net> In-Reply-To: <200112300927.fBU9RSP39484@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
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 <Alexander@Leidinger.net> (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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200112301813.fBUIDR298344>