cvs commit: src/sys/pci if_dc.c if_pcn.c if_rl.c if_sf.c if_sis.c if_sk.c if_ste.c if_ti.c if_tl.c if_vr.c if_wb.c if_xl.c

[M. Warner Losh]

In message: <Pine.BSF.4.21.0303311628380.8123-100000 at root.org>
            Nate Lawson <nate at root.org> writes:
: On Mon, 31 Mar 2003, Andrew Gallatin wrote:
: > Nate Lawson [nate at root.org] wrote:
: > > On Mon, 31 Mar 2003, Nate Lawson wrote:
: > > >   Modified files:
: > > >     sys/pci              if_dc.c if_pcn.c if_rl.c if_sf.c if_sis.c 
: > > >                          if_sk.c if_ste.c if_ti.c if_tl.c if_vr.c 
: > > >                          if_wb.c if_xl.c 
: > > >   Log:
: > > >   Clean up locking and resource management for pci/if_*
: > > 
: > > This is a rather large patch so if you begin to experience problems
: > > attaching or detaching (say on module load/unload) a previously-working
: > > card, please let me know.
: > 
: > Very nice.
: > 
: > Now that the locking is right, do you plan to enable INTR_MPSAFE on
: > these?
: 
: The locking is only right on attach.  There is still more locking work to
: be done on the data path.

Many of the drivers get the locking almost right, but there may still
be issues with the watchdog timeout routines and lock ordering...
That's the only corner of most of these drivers that I've seen major
locking problems in...

Warner