From: Jeff Trawick <trawick@apache.org>
Date: Sat, 9 Apr 2011 16:19:33 +0000 (+0000)
Subject: WinNT MPM: Improve robustness under heavy load.
X-Git-Tag: 2.3.12~63
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=e5c9d39f76d4fab5600eee12c9a08e7d6f4c8f38;p=thirdparty%2Fapache%2Fhttpd.git

WinNT MPM: Improve robustness under heavy load.


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1090621 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/CHANGES b/CHANGES
index 917523fe6df..6e9b68ea96a 100644
--- a/CHANGES
+++ b/CHANGES
@@ -2,6 +2,8 @@
 
 Changes with Apache 2.3.12
 
+  *) WinNT MPM: Improve robustness under heavy load.  [Jeff Trawick]
+
   *) MinGW build improvements.  PR 49535.  [John Vandenberg 
      <jayvdb gmail.com>, Jeff Trawick]
 
diff --git a/server/mpm/winnt/child.c b/server/mpm/winnt/child.c
index 2eee477b929..3ef137c1962 100644
--- a/server/mpm/winnt/child.c
+++ b/server/mpm/winnt/child.c
@@ -137,11 +137,12 @@ static void mpm_recycle_completion_context(winnt_conn_ctx_t *context)
     }
 }
 
-static winnt_conn_ctx_t *mpm_get_completion_context(void)
+static winnt_conn_ctx_t *mpm_get_completion_context(int *timeout)
 {
     apr_status_t rv;
     winnt_conn_ctx_t *context = NULL;
 
+    *timeout = 0;
     while (1) {
         /* Grab a context off the queue */
         apr_thread_mutex_lock(qlock);
@@ -186,6 +187,7 @@ static winnt_conn_ctx_t *mpm_get_completion_context(void)
                         ap_log_error(APLOG_MARK, APLOG_DEBUG, 0, ap_server_conf,
                                      "mpm_get_completion_context: Failed to get a "
                                      "free context within 1 second");
+                        *timeout = 1;
                     }
                     else {
                         /* should be the unexpected, generic WAIT_FAILED */
@@ -358,15 +360,19 @@ reinit: /* target of data or connect upon too many AcceptEx failures */
 
     while (!shutdown_in_progress) {
         if (!context) {
-            context = mpm_get_completion_context();
+            int timeout;
+
+            context = mpm_get_completion_context(&timeout);
             if (!context) {
-                /* Hopefully a temporary condition in the provider? */
-                ++err_count;
-                if (err_count > MAX_ACCEPTEX_ERR_COUNT) {
-                    ap_log_error(APLOG_MARK, APLOG_CRIT, 0, ap_server_conf,
-                                 "winnt_accept: Too many failures grabbing a "
-                                 "connection ctx.  Aborting.");
-                    break;
+                if (!timeout) {
+                    /* Hopefully a temporary condition in the provider? */
+                    ++err_count;
+                    if (err_count > MAX_ACCEPTEX_ERR_COUNT) {
+                        ap_log_error(APLOG_MARK, APLOG_CRIT, 0, ap_server_conf,
+                                     "winnt_accept: Too many failures grabbing a "
+                                     "connection ctx.  Aborting.");
+                        break;
+                    }
                 }
                 Sleep(100);
                 continue;