From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Wed, 20 Oct 2010 23:29:32 +0000 (-0400)
Subject: Windows: optimize cm_BkgDaemon()
X-Git-Tag: upstream/1.8.0_pre1^2~4641
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=49e57b8c3b5e13f7ce48c6be5677827d970df5fa;p=packages%2Fo%2Fopenafs.git

Windows: optimize cm_BkgDaemon()

In cm_BkgDaemon test for CM_SCACHEFLAG_DATASTORING before
cm_ServerAvailable() in order to avoid unnecessary work.

LICENSE MIT

Change-Id: I75f88212a18d765790c9435de17e3b0fa735a7ab
Reviewed-on: http://gerrit.openafs.org/3020
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
---

diff --git a/src/WINNT/afsd/cm_daemon.c b/src/WINNT/afsd/cm_daemon.c
index 3c765cccd..f8284332d 100644
--- a/src/WINNT/afsd/cm_daemon.c
+++ b/src/WINNT/afsd/cm_daemon.c
@@ -114,8 +114,8 @@ void cm_BkgDaemon(void * parm)
         /* we found a request */
         for (rp = cm_bkgListEndp; rp; rp = (cm_bkgRequest_t *) osi_QPrev(&rp->q))
 	{
-	    if (cm_ServerAvailable(&rp->scp->fid, rp->userp) && 
-                !(rp->scp->flags & CM_SCACHEFLAG_DATASTORING))
+	    if (!(rp->scp->flags & CM_SCACHEFLAG_DATASTORING) &&
+                cm_ServerAvailable(&rp->scp->fid, rp->userp))
 		break;
 	}
 	if (rp == NULL) {