[src/trunk]: src/sys/miscfs/procfs Procfs_lookup() does not lookup directory ...

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

details:   https://anonhg.NetBSD.org/src/rev/7ff78838291c
branches:  trunk
changeset: 755517:7ff78838291c
user:      hannken <hannken%NetBSD.org@localhost>
date:      Tue Jun 08 08:24:16 2010 +0000

description:
Procfs_lookup() does not lookup directory descriptors in the fd/
subdirectory.  There is no need for recursive vnode locking here.

Ok: Christos Zoulas <christos%netbsd.org@localhost>

diffstat:

 sys/miscfs/procfs/procfs_vnops.c |  7 +++----
 1 files changed, 3 insertions(+), 4 deletions(-)

diffs (28 lines):

diff -r 6762255b27de -r 7ff78838291c sys/miscfs/procfs/procfs_vnops.c
--- a/sys/miscfs/procfs/procfs_vnops.c  Tue Jun 08 08:20:37 2010 +0000
+++ b/sys/miscfs/procfs/procfs_vnops.c  Tue Jun 08 08:24:16 2010 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: procfs_vnops.c,v 1.177 2010/01/08 11:35:11 pooka Exp $ */
+/*     $NetBSD: procfs_vnops.c,v 1.178 2010/06/08 08:24:16 hannken Exp $       */
 
 /*-
  * Copyright (c) 2006, 2007, 2008 The NetBSD Foundation, Inc.
@@ -105,7 +105,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: procfs_vnops.c,v 1.177 2010/01/08 11:35:11 pooka Exp $");
+__KERNEL_RCSID(0, "$NetBSD: procfs_vnops.c,v 1.178 2010/06/08 08:24:16 hannken Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -1153,8 +1153,7 @@
                        vref(fvp);
                        closef(fp);
                        procfs_proc_unlock(p);
-                       vn_lock(fvp, LK_EXCLUSIVE | LK_RETRY |
-                           (p == curproc ? LK_CANRECURSE : 0));
+                       vn_lock(fvp, LK_EXCLUSIVE | LK_RETRY);
                        *vpp = fvp;
                        return 0;
                }