cvs commit: src/sys/kern vfs_subr.c

[Pawel Jakub Dawidek]

On Thu, Dec 29, 2005 at 10:52:09PM +0000, Pawel Jakub Dawidek wrote:
+> pjd         2005-12-29 22:52:09 UTC
+> 
+>   FreeBSD src repository
+> 
+>   Modified files:
+>     sys/kern             vfs_subr.c 
+>   Log:
+>   Print a warning when we miss vinactive() call, because of race in vget().
+>   The race is very real, but conditions needed for triggering it are rather
+>   hard to meet now.
+>   When gjournal will be committed (where it is quite easy to trigger) we need
+>   to fix it.
+>   
+>   For now, verify if it is really hard to trigger.
+>   
+>   Discussed with: kan
[...]
+> +		if (usecount > 0 && vp->v_usecount == 0)
+> +			printf("vinactive() won't be called for vp=%p\n", vp);

Hmm, maybe it should be:

	if (usecount > 0 && vp->v_usecount == 0 && oweinact)

?

-- 
Pawel Jakub Dawidek                       http://www.wheel.pl
pjd at FreeBSD.org                           http://www.FreeBSD.org
FreeBSD committer                         Am I Evil? Yes, I Am!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-src/attachments/20051230/2c67ce77/attachment.bin