Date: Tue, 29 Apr 2003 11:55:24 -0600 (MDT) From: "M. Warner Losh" <imp@bsdimp.com> To: gallatin@cs.duke.edu Cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/fxp if_fxp.c if_fxpvar.h Message-ID: <20030429.115524.21927823.imp@bsdimp.com> In-Reply-To: <20030429133708.A84234@grasshopper.cs.duke.edu> References: <200304290545.h3T5j99Y076513@repoman.freebsd.org> <20030429133708.A84234@grasshopper.cs.duke.edu>
next in thread | previous in thread | raw e-mail | index | archive | help
In message: <20030429133708.A84234@grasshopper.cs.duke.edu>
Andrew Gallatin <gallatin@cs.duke.edu> writes:
: What is this deadlock with bus_teardown_intr? Could we possibly fix
: this some other way than by adding (mostly) useless code to the
: critical path?
I can remove it if it really annoys you that much. The locking code
itself will swamp the extra load/compare that happens. It will put
the race back into the code, however. This likely means that some
higher level of locking is necessary so that we can make sure that the
interrupts can't happen once detach starts.
Warner
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030429.115524.21927823.imp>