cvs commit: src/sys/dev/fxp if_fxp.c if_fxpvar.h

Sam Leffler sam at errno.com
Fri Apr 25 08:50:32 PDT 2003


> For developers, note that the locking in the code path only protects the
> various fxp routines (fxp_start, fxp_intr, fxp_tick, ...) and is not
> intended to serialize access to ANY external structures.  This is how it
> should be.  Please do not copy the exact approach taken here for a little
> while until ifnet locking is finished as there may need to be some changes
> made to this model.

This doesn't make much sense to me.  I've locked numerous chunks of code and
used a totally different approach: synchronize access to data structures,
not code paths.  Perhaps you and Jeffrey Hsu need to have a private
discussion...

    Sam



More information about the cvs-src mailing list