From: wessels <>
Date: Fri, 21 Jul 2000 12:06:22 +0000 (+0000)
Subject: DW:
X-Git-Tag: SQUID_3_0_PRE1~1885
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=284ac611b348e4f8f32fb2a504fdc74f897f96c0;p=thirdparty%2Fsquid.git

DW:
 - Once again wasted time tracking down a hit ratio problem, eventually
   re-learning that storeDirSelectSwapDirLeastLoad() could return -1
   causing a swapout to fail.  This patch logs those cases to store.log.
---

diff --git a/src/enums.h b/src/enums.h
index 7415a46983..92f77e728f 100644
--- a/src/enums.h
+++ b/src/enums.h
@@ -1,6 +1,6 @@
 
 /*
- * $Id: enums.h,v 1.169 2000/07/18 06:16:41 wessels Exp $
+ * $Id: enums.h,v 1.170 2000/07/21 06:06:22 wessels Exp $
  *
  *
  * SQUID Internet Object Cache  http://squid.nlanr.net/Squid/
@@ -636,7 +636,8 @@ enum {
     STORE_LOG_CREATE,
     STORE_LOG_SWAPIN,
     STORE_LOG_SWAPOUT,
-    STORE_LOG_RELEASE
+    STORE_LOG_RELEASE,
+    STORE_LOG_SWAPOUTFAIL
 };
 
 typedef enum {
diff --git a/src/store_log.cc b/src/store_log.cc
index 76c6e0df28..e29e46f9f5 100644
--- a/src/store_log.cc
+++ b/src/store_log.cc
@@ -1,6 +1,6 @@
 
 /*
- * $Id: store_log.cc,v 1.17 2000/06/25 22:28:43 wessels Exp $
+ * $Id: store_log.cc,v 1.18 2000/07/21 06:06:22 wessels Exp $
  *
  * DEBUG: section 20    Storage Manager Logging Functions
  * AUTHOR: Duane Wessels
@@ -40,7 +40,8 @@ static char *storeLogTags[] =
     "CREATE",
     "SWAPIN",
     "SWAPOUT",
-    "RELEASE"
+    "RELEASE",
+    "SO_FAIL",
 };
 
 static Logfile *storelog = NULL;
diff --git a/src/store_swapout.cc b/src/store_swapout.cc
index ae508c25eb..9019802e97 100644
--- a/src/store_swapout.cc
+++ b/src/store_swapout.cc
@@ -1,6 +1,6 @@
 
 /*
- * $Id: store_swapout.cc,v 1.73 2000/06/27 22:06:05 hno Exp $
+ * $Id: store_swapout.cc,v 1.74 2000/07/21 06:06:22 wessels Exp $
  *
  * DEBUG: section 20    Storage Manager Swapout Functions
  * AUTHOR: Duane Wessels
@@ -69,6 +69,7 @@ storeSwapOutStart(StoreEntry * e)
 	e->swap_status = SWAPOUT_NONE;
 	cbdataFree(c);
 	xfree(buf);
+	storeLog(STORE_LOG_SWAPOUTFAIL, e);
 	return;
     }
     storeLockObject(e);		/* Don't lock until after create, or the replacement