From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Wed, 20 Jul 2011 18:18:25 +0000 (-0400)
Subject: Windows: cm_daemonCheckOfflineVol fix
X-Git-Tag: upstream/1.6.0^2~73
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=545479dea3e9fc0f3d2b3537aa148c2caf02589a;p=packages%2Fo%2Fopenafs.git

Windows: cm_daemonCheckOfflineVol fix

When computing whether or not to perform an offline volume
check it is critical that the 'lastBusyVolCheck' variable
be assigned the current time instead of 'lastVolCheck'.
By setting the wrong variable a new offline volume check is
performed every 10 seconds which is undesireable.

Reviewed-on: http://gerrit.openafs.org/5054
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
(cherry picked from 0783b8f3821f59004146b4c34a53189ada178d70)

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

diff --git a/src/WINNT/afsd/cm_daemon.c b/src/WINNT/afsd/cm_daemon.c
index dbe81baf2..1a158bd9e 100644
--- a/src/WINNT/afsd/cm_daemon.c
+++ b/src/WINNT/afsd/cm_daemon.c
@@ -548,7 +548,7 @@ void cm_Daemon(long parm)
         if ((bAddrChangeCheck || now > lastBusyVolCheck + cm_daemonCheckOfflineVolInterval) &&
             daemon_ShutdownFlag == 0 &&
             powerStateSuspended == 0) {
-            lastVolCheck = now;
+            lastBusyVolCheck = now;
             cm_CheckOfflineVolumes();
             if (daemon_ShutdownFlag == 1)
                 break;