I have Paul's version of the Event MPM patch up and running. The only glitch I saw bringing it up was a warning for two unused variables in http_core.c (patchlet below). Then I tried stressing it with SPECweb99 and saw some errors after several minutes:

[error] (24)Too many open files: apr_socket_accept: (client socket)

...and a few others with the same errno. Digging into it a bit, sockets are hanging around with netstat showing an abnormally high number of CLOSE_WAITs. We must be missing a close() somewhere. Maybe in the timeout logic? I believe that was reworked.

If no one beats me to it and day job permitting, I'll investigate further next 
week.

Greg

--- modules/http/http_core.c.orig       2004-11-17 14:15:26.704770352 -0500
+++ modules/http/http_core.c    2004-11-17 14:16:35.009770200 -0500
@@ -234,8 +234,6 @@
 static int ap_process_http_async_connection(conn_rec *c)
 {
     request_rec *r;
-    int csd_set = 0;
-    apr_socket_t *csd = NULL;
     conn_state_t *cs = c->cs;

     switch (cs->state) {



Reply via email to