Re: CVS commit: src/sys/ufs/ufs

[Antti Kantee <pooka%cs.hut.fi@localhost>]

On Mon Sep 28 2009 at 00:39:03 +0000, David A. Holland wrote:
> Module Name:  src
> Committed By: dholland
> Date:         Mon Sep 28 00:39:03 UTC 2009
> 
> Modified Files:
>       src/sys/ufs/ufs: ufs_lookup.c
> 
> Log Message:
> Avoid nasal demons. Code of the form
> 
>    vput(vp);
>    error = VFS_VGET(vp->v_mount, ...);
> 
> just isn't right. Because of vnode caching this *probably* never bit
> anyone, except maybe under very heavy load, but still.

Unbitten more likely because checkpath is only for rename and rename has
approximately 57 billion extra references to the vnode when calling it.
"but still".