cvs commit: src/sys/fs/nullfs null_vnops.c

Jeff Roberson jroberson at chesapeake.net
Tue Mar 15 05:28:11 PST 2005


On Tue, 15 Mar 2005, Jeff Roberson wrote:

> jeff        2005-03-15 11:28:45 UTC
>
>   FreeBSD src repository
>
>   Modified files:
>     sys/fs/nullfs        null_vnops.c
>   Log:
>    - We have to transfer lockers after reseting our vnlock pointer.

Upon further review, this is wrong, but the code that was there before was
wrong too.  The problem is that if we transferlockers we may transfer
threads that are legitimately waiting on the ufs lock rather than the null
lock.  I believe the solution is to reset our lock, but allow callers to
naturally return out of the ufs lock routine when they're ready.  The
vnode reference count will prevent anything from going away while we're
stuck in ufs's locking function.

>
>   Sponsored by:   Isilon Systems, Inc.
>
>   Revision  Changes    Path
>   1.84      +5 -0      src/sys/fs/nullfs/null_vnops.c
>


More information about the cvs-src mailing list