]> granicus.if.org Git - apache/commitdiff
document yet another worker MPM glitch
authorJeff Trawick <trawick@apache.org>
Fri, 22 Mar 2002 03:43:22 +0000 (03:43 +0000)
committerJeff Trawick <trawick@apache.org>
Fri, 22 Mar 2002 03:43:22 +0000 (03:43 +0000)
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@94117 13f79535-47bb-0310-9956-ffa450edef68

server/mpm/worker/worker.c

index b3b3b1d6b0d4d3613b2c085de42b479072a5ed07..dbc1ae8ec9c124b43bb2138b0d10791bb94a165a 100644 (file)
@@ -926,6 +926,14 @@ static void * APR_THREAD_FUNC start_threads(apr_thread_t *thd, void *dummy)
     my_info->pid = my_child_num;
     my_info->tid = i;
     my_info->sd = 0;
+
+    /* XXX we shouldn't create the listener thread until we have at least
+     *     one worker thread...  for now I'll blame this bug for some very
+     *     rare hung connections I've seen during restart testing
+     *     (I've also seen cases where a child process starts but is
+     *     never able to take over worker thread slots, so the theory
+     *     does make sense.)  Jeff
+     */
     rv = apr_thread_create(&ts->listener, thread_attr, listener_thread,
                            my_info, pchild);
     if (rv != APR_SUCCESS) {