[src/trunk]: src/sys/fs/puffs The pageflush_selector gets a vnode with v_inte...

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

details:   https://anonhg.NetBSD.org/src/rev/de64b2105a6b
branches:  trunk
changeset: 329473:de64b2105a6b
user:      hannken <hannken%NetBSD.org@localhost>
date:      Sun May 25 17:43:47 2014 +0000

description:
The pageflush_selector gets a vnode with v_interlock held.
Remove the mutex_enter()/mutex_exit() and simplify.

Hi christos...

diffstat:

 sys/fs/puffs/puffs_vfsops.c |  11 +++--------
 1 files changed, 3 insertions(+), 8 deletions(-)

diffs (33 lines):

diff -r 3676c3e0b0b3 -r de64b2105a6b sys/fs/puffs/puffs_vfsops.c
--- a/sys/fs/puffs/puffs_vfsops.c       Sun May 25 16:31:51 2014 +0000
+++ b/sys/fs/puffs/puffs_vfsops.c       Sun May 25 17:43:47 2014 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: puffs_vfsops.c,v 1.111 2014/05/24 16:34:03 christos Exp $      */
+/*     $NetBSD: puffs_vfsops.c,v 1.112 2014/05/25 17:43:47 hannken Exp $       */
 
 /*
  * Copyright (c) 2005, 2006  Antti Kantee.  All Rights Reserved.
@@ -30,7 +30,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: puffs_vfsops.c,v 1.111 2014/05/24 16:34:03 christos Exp $");
+__KERNEL_RCSID(0, "$NetBSD: puffs_vfsops.c,v 1.112 2014/05/25 17:43:47 hannken Exp $");
 
 #include <sys/param.h>
 #include <sys/kernel.h>
@@ -513,13 +513,8 @@
 static bool
 pageflush_selector(void *cl, struct vnode *vp)
 {
-       bool rv;
 
-       mutex_enter(vp->v_interlock);
-       rv = vp->v_type == VREG && !UVM_OBJ_IS_CLEAN(&vp->v_uobj);
-       mutex_exit(vp->v_interlock);
-
-       return rv;
+       return vp->v_type == VREG && !UVM_OBJ_IS_CLEAN(&vp->v_uobj);
 }
 
 static int