Author: kib
Date: Fri Apr 16 08:32:08 2010
New Revision: 206699
URL: http://svn.freebsd.org/changeset/base/206699

Log:
  MFC r206546:
  Remove XXX comment. Add another comment, describing why f_vnode assignment
  is useful.

Modified:
  stable/8/sys/kern/vfs_syscalls.c
Directory Properties:
  stable/8/sys/   (props changed)
  stable/8/sys/amd64/include/xen/   (props changed)
  stable/8/sys/cddl/contrib/opensolaris/   (props changed)
  stable/8/sys/contrib/dev/acpica/   (props changed)
  stable/8/sys/contrib/pf/   (props changed)
  stable/8/sys/dev/xen/xenpci/   (props changed)

Modified: stable/8/sys/kern/vfs_syscalls.c
==============================================================================
--- stable/8/sys/kern/vfs_syscalls.c    Fri Apr 16 07:02:28 2010        
(r206698)
+++ stable/8/sys/kern/vfs_syscalls.c    Fri Apr 16 08:32:08 2010        
(r206699)
@@ -1124,7 +1124,12 @@ kern_openat(struct thread *td, int fd, c
        NDFREE(&nd, NDF_ONLY_PNBUF);
        vp = nd.ni_vp;
 
-       fp->f_vnode = vp;       /* XXX Does devfs need this? */
+       /*
+        * Store the vnode, for any f_type. Typically, the vnode use
+        * count is decremented by direct call to vn_closefile() for
+        * files that switched type in the cdevsw fdopen() method.
+        */
+       fp->f_vnode = vp;
        /*
         * If the file wasn't claimed by devfs bind it to the normal
         * vnode operations here.
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to