From e6513c283e69cfb8f74cfc77e3e1385ce46cd48a Mon Sep 17 00:00:00 2001 From: enami Date: Mon, 6 May 2002 00:18:15 +0000 Subject: [PATCH] Stylistic change; introduce new local variable and use it instead of sprinkling different expression to test if we're pagedaemon. --- sys/miscfs/genfs/genfs_vnops.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/sys/miscfs/genfs/genfs_vnops.c b/sys/miscfs/genfs/genfs_vnops.c index 126b188c3bba..6241b9af1ce0 100644 --- a/sys/miscfs/genfs/genfs_vnops.c +++ b/sys/miscfs/genfs/genfs_vnops.c @@ -1,4 +1,4 @@ -/* $NetBSD: genfs_vnops.c,v 1.55 2002/04/26 03:57:31 enami Exp $ */ +/* $NetBSD: genfs_vnops.c,v 1.56 2002/05/06 00:18:15 enami Exp $ */ /* * Copyright (c) 1982, 1986, 1989, 1993 @@ -35,7 +35,7 @@ */ #include -__KERNEL_RCSID(0, "$NetBSD: genfs_vnops.c,v 1.55 2002/04/26 03:57:31 enami Exp $"); +__KERNEL_RCSID(0, "$NetBSD: genfs_vnops.c,v 1.56 2002/05/06 00:18:15 enami Exp $"); #include "opt_nfsserver.h" @@ -997,6 +997,7 @@ genfs_putpages(void *v) struct vm_page *pgs[n], *pg, *nextpg, *tpg, curmp, endmp; boolean_t wasclean, by_list, needs_clean, yield; boolean_t async = (flags & PGO_SYNCIO) == 0; + boolean_t pagedaemon = curproc == uvm.pagedaemon_proc; UVMHIST_FUNC("genfs_putpages"); UVMHIST_CALLED(ubchist); KASSERT(flags & (PGO_CLEANIT|PGO_FREE|PGO_DEACTIVATE)); @@ -1037,7 +1038,7 @@ genfs_putpages(void *v) * current last page. */ - freeflag = (curproc == uvm.pagedaemon_proc) ? PG_PAGEOUT : PG_RELEASED; + freeflag = pagedaemon ? PG_PAGEOUT : PG_RELEASED; curmp.uobject = uobj; curmp.offset = (voff_t)-1; curmp.flags = PG_BUSY; @@ -1087,9 +1088,9 @@ genfs_putpages(void *v) */ yield = (curproc->p_cpu->ci_schedstate.spc_flags & - SPCF_SHOULDYIELD) && curproc != uvm.pagedaemon_proc; + SPCF_SHOULDYIELD) && !pagedaemon; if (pg->flags & PG_BUSY || yield) { - KASSERT(curproc != uvm.pagedaemon_proc); + KASSERT(!pagedaemon); UVMHIST_LOG(ubchist, "busy %p", pg,0,0,0); if (by_list) { TAILQ_INSERT_BEFORE(pg, &curmp, listq); @@ -1205,7 +1206,7 @@ genfs_putpages(void *v) pmap_page_protect(tpg, VM_PROT_NONE); if (tpg->flags & PG_BUSY) { tpg->flags |= freeflag; - if (freeflag == PG_PAGEOUT) { + if (pagedaemon) { uvmexp.paging++; uvm_pagedequeue(tpg); }