On Mon Jul 26 2010 at 15:22:17 +0000, Juergen Hannken-Illjes wrote: > Module Name: src > Committed By: hannken > Date: Mon Jul 26 15:22:16 UTC 2010 > > Modified Files: > src/sys/kern: vfs_subr.c > > Log Message: > When both vget() and vrelel() call vn_lock() we know VI_XLOCK is clear. > No need to use LK_INTERLOCK or LK_RETRY here.
I'd like the commit message to include the essential bit about them examining the return value and not blindly plowing on after vn_lock(). Purely from the commit message I get the impression that this introduced a race condition.