[src/thorpej-devvp]: src/sys/kern Clarify the somewhat confusing if() stateme...

[fvdl <fvdl%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/f8e1b132973d
branches:  thorpej-devvp
changeset: 514615:f8e1b132973d
user:      fvdl <fvdl%NetBSD.org@localhost>
date:      Fri Sep 21 15:00:12 2001 +0000

description:
Clarify the somewhat confusing if() statement in checkalias with a comment.

diffstat:

 sys/kern/vfs_subr.c |  15 ++++++++++++++-
 1 files changed, 14 insertions(+), 1 deletions(-)

diffs (29 lines):

diff -r c3e3c95bfaad -r f8e1b132973d sys/kern/vfs_subr.c
--- a/sys/kern/vfs_subr.c       Fri Sep 21 12:08:30 2001 +0000
+++ b/sys/kern/vfs_subr.c       Fri Sep 21 15:00:12 2001 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: vfs_subr.c,v 1.156.2.2 2001/09/18 19:13:54 fvdl Exp $  */
+/*     $NetBSD: vfs_subr.c,v 1.156.2.3 2001/09/21 15:00:12 fvdl Exp $  */
 
 /*-
  * Copyright (c) 1997, 1998 The NetBSD Foundation, Inc.
@@ -1113,6 +1113,19 @@
                }
                break;
        }
+       /*
+        * This if() statement seems a little confusing. Here is what
+        * it means:
+        *
+        * Create a new vnode, if it is a cloning device, if no vnode for
+        * this device was found, or if a dead vnode for this device was
+        * found. Do not create aliases for block devices (unless it was
+        * explicitly marked as cloning).
+        *
+        * If a (possibly dead) vnode was found, add the new vnode to
+        * the hash chain that contains all vnodes describing the same
+        * device.
+        */
        if (vp == NULL || vp->v_tag != VT_NON || vp->v_type != VBLK ||
            iscloningvnode(vp)) {
                MALLOC(nvp->v_specinfo, struct specinfo *,