cvs commit: src/sys/emulation/linux/i386/linprocfs linprocfs.h

Matthew Dillon dillon at apollo.backplane.com
Thu Feb 8 09:20:27 PST 2007


:> >DragonFly src repository
:> >
:> >  Modified files:
:> >    sys/emulation/linux/i386/linprocfs linprocfs.h 
:> >  Log:
:> >  add prototype for linprocfs_init()
:> 
:> Is this function used at all (I remember vaguely that it isn't)?
:
:(I didn't notice that my reply didn't make it to commits@ ...)
:No.  The driver happens to work without calling linprocfs_init() because
:all it does is to call lwkt_token_init(), which only zero'es the token,
:which is already zero'ed when it was loaded.  Besides that,
:lwkt_token_uninit() is called from the bottom of linprocfs_exit(), which is
:wrong, because it's at_exit()'s callback function and can be called
:more than once.  Yeah, I know lwkt_token_uninit() is a no-op anyway, so
:yanking lwkt_token_{,un}init() practically solves the problem, but it
:doesn't look right...
:
:Cheers.
    
    I did that in the far past at a time when I thought tokens would be
    used more then they area.  I wanted an uninit function that could be
    used for debugging and tracking tokens.  But I never really needed
    to debug them at that level so it wound up being unused.

					-Matt
					Matthew Dillon 
					<dillon at backplane.com>





More information about the Commits mailing list