httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeff Trawick <traw...@gmail.com>
Subject Re: svn commit: r159470 - in httpd/httpd/branches/2.0.x: CHANGES STATUS include/mpm_common.h server/mpm/worker/worker.c server/mpm_common.c
Date Thu, 31 Mar 2005 14:18:54 GMT
On Wed, 30 Mar 2005 12:16:03 -0600, William A. Rowe, Jr.
<wrowe@rowe-clan.net> wrote:
> At 03:42 AM 3/30/2005, trawick@apache.org wrote:
> >New Revision: 159470
> >URL: http://svn.apache.org/viewcvs?view=rev&rev=159470
> >
> >  *) Unix MPMs: Shut down the server more quickly when child processes are
> >     slow to exit.  [Joe Orton, Jeff Trawick]
> 
> I have one concern with this backport.  Have you considered
> cgi processes which are database-bound or have other lengthly
> cleanup operations?  Some CGI processes may take a whole minute
> or so to flush out the transaction they conducted.  Does this
> patch make it _more_ likely that the cleanups would be prematurely
> terminated?

As Joe said: CGI processes are not affected.  (On that topic, somebody
from HP had some queries about stranded CGI processes recently and we
discussed a method for cleaning those up.  But I never saw a patch ;) 
It seemed clear that mod_cgid at least needed to do something to clean
those up before it went away.  I think your concern would be relevant
to that type of change.

Going further in the direction of trivial details: This patch didn't
change the intended sequence very much but it did avoid some major
bogosity with the prior use of a sliding timer.  The timer could grow
large enough (30 or more seconds) so that even once the children had
exited, the parent would not wake up to check again for some time.

Mime
View raw message