cvs commit: src/sys/dev/fxp if_fxp.c if_fxpvar.h
Poul-Henning Kamp
phk at phk.freebsd.dk
Wed Apr 30 02:10:32 PDT 2003
In message <20030430093931.V31027 at beagle.fokus.fraunhofer.de>, Harti Brandt wri
tes:
>On Tue, 29 Apr 2003, M. Warner Losh wrote:
>
>MWL>In message: <Pine.BSF.4.21.0304291101001.75697-100000 at root.org>
>
>MWL>: > 4) Return immediately if gone is true in fxp_ioctl
>MWL>: > 5) Return immediately if gone is true in fxp_intr
>MWL>:
>MWL>: Not sure this approach is necessary.
>MWL>
>MWL>I am. Otherwise ioctl panics with recursive locks when the card is
>MWL>detached. A simple kld_unload if_fxp would provoke these races,
>MWL>including the recursive lock panic.
>
>You can replace the pointers to ioctl and start with pointers to dummy
>functions. In fact, an if_dead(ifp) function which does just this would
>be nice to have.
isn't that what if_disc.c does already ?
--
Poul-Henning Kamp | UNIX since Zilog Zeus 3.20
phk at FreeBSD.ORG | TCP/IP since RFC 956
FreeBSD committer | BSD since 4.3-tahoe
Never attribute to malice what can adequately be explained by incompetence.
More information about the cvs-src
mailing list