* amd/srvr_nfs.c (start_nfs_pings): don't set FSF_PINGING if
authorIon Badulescu <ib42@cs.columbia.edu>
Sun, 29 Dec 2002 01:51:25 +0000 (01:51 +0000)
committerIon Badulescu <ib42@cs.columbia.edu>
Sun, 29 Dec 2002 01:51:25 +0000 (01:51 +0000)
pings are disabled

ChangeLog
amd/srvr_nfs.c

index 5d4a36cc833b3124ae065a17f03395de5fb29266..59785971c64ae8ea50b152761e0cfa3ec14e3d87 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2002-12-28  Ion Badulescu  <ionut@moisil.badula.org>
 
+       * amd/srvr_nfs.c (start_nfs_pings): don't set FSF_PINGING if
+       pings are disabled
+
        * libamu/xdr_func.c (xdr_ftype): use a local enum_t variable
        instead of casting the pointer to (enum_t *)
        (xdr_mountstat3): ditto
index e96bb697eb4ead9574f146792aa6ea8058c8e46a..448aeafff9a4e4bad0bd5c5ffbdaa695b12d65bf 100644 (file)
@@ -37,7 +37,7 @@
  * SUCH DAMAGE.
  *
  *
- * $Id: srvr_nfs.c,v 1.22 2002/12/27 22:43:53 ezk Exp $
+ * $Id: srvr_nfs.c,v 1.23 2002/12/29 01:51:26 ib42 Exp $
  *
  */
 
@@ -577,19 +577,20 @@ nfs_srvr_port(fserver *fs, u_short *port, voidp wchan)
 static void
 start_nfs_pings(fserver *fs, int pingval)
 {
-  if (!(fs->fs_flags & FSF_PINGING)) {
-    fs->fs_flags |= FSF_PINGING;
-    if (fs->fs_cid)
-      untimeout(fs->fs_cid);
-    if (pingval < 0) {
-      srvrlog(fs, "wired up (pings disabled)");
-      fs->fs_flags |= FSF_VALID;
-      fs->fs_flags &= ~FSF_DOWN;
-    } else {
-      nfs_keepalive(fs);
-    }
-  } else {
+  if (fs->fs_flags & FSF_PINGING) {
     dlog("Already running pings to %s", fs->fs_host);
+    return;
+  }
+
+  if (fs->fs_cid)
+    untimeout(fs->fs_cid);
+  if (pingval < 0) {
+    srvrlog(fs, "wired up (pings disabled)");
+    fs->fs_flags |= FSF_VALID;
+    fs->fs_flags &= ~FSF_DOWN;
+  } else {
+    fs->fs_flags |= FSF_PINGING;
+    nfs_keepalive(fs);
   }
 }