- Remove an old comment; vnodes have been working without Giant for

   years now.
 - Clarify the locking required for VI_DOOMED in preparation for
   simplifications to vget() and vn_lock().

1 files changed, 3 insertions, 3 deletions

diff --git a/sys/sys/vnode.h b/sys/sys/vnode.h
index ba23b18..feeac69 100644
--- a/sys/sys/vnode.h
+++ b/sys/sys/vnode.h
@@ -92,9 +92,6 @@ struct vpollinfo {
  *	4) vget with LK_INTERLOCK and check for ENOENT, or
  *	5) Check for DOOMED if the vnode lock is not required.
  *	6) Perform your operation, then vput().
- *
- * XXX Not all fields are locked yet and some fields that are marked are not
- * locked consistently.  This is a work in progress.  Requires Giant!
  */
 
 #if defined(_KERNEL) || defined(_KVM_VNODE)
@@ -230,6 +227,9 @@ struct xvnode {
  * Vnode flags.
  *	VI flags are protected by interlock and live in v_iflag
  *	VV flags are protected by the vnode lock and live in v_vflag
+ *
+ *	VI_DOOMED is doubly protected by the interlock and vnode lock.  Both
+ *	are required for writing but the status may be checked with either.
  */
 #define	VI_MOUNT	0x0020	/* Mount in progress */
 #define	VI_AGE		0x0040	/* Insert vnode at head of free list */