Re: svn commit: r697357 - in /httpd/httpd/trunk: include/ modules/http/ modules/test/ server/ server/mpm/experimental/event/

2008-09-20 Thread Jim Jagielski
On Sat, Sep 20, 2008 at 11:58:09AM -, [EMAIL PROTECTED] wrote: Does anyone besides Rdiger read commit emails :-) ? No :) +else { +/* X: lol, pool allocation without a context from any thread.Yeah. Right. MPMs Suck. */ +te = malloc(sizeof(timer_event_t));

Re: svn commit: r697357 - in /httpd/httpd/trunk: include/ modules/http/ modules/test/ server/ server/mpm/experimental/event/

2008-09-20 Thread Graham Leggett
[EMAIL PROTECTED] wrote: Using this basic framework, you can return SUSPENDED from an HTTP Handler, and then register a callback that is invoked by the MPM at a later time. This initial version only supports _timers_ as callbacks, but in the future I would like to add things like wait for

Re: svn commit: r592951 - in /httpd/httpd/trunk: ./ docs/manual/mod/ include/ modules/filters/ modules/http/ server/

2008-09-20 Thread Graham Leggett
Paul Querna wrote: +1, this revision r592951, and the derivate r647263 need to be rethought. IMO is that this doesn't belong in the reqeust_rec. I'm only realizing this went in 10 months after the first commit, and 5 months after it was rewritten in r647263, a sign of my inactivity in

Re: mod_proxy race condition bug #37770

2008-09-20 Thread Ruediger Pluem
On 09/20/2008 12:21 PM, Nick Kew wrote: On 19 Sep 2008, at 23:08, Ruediger Pluem wrote: On 09/19/2008 11:24 PM, Adam Woodworth wrote: The problem with this is that it will also do this for connections to the backend that timed out. For our application, we want actual timeouts to be

Future direction of MPMs, was Re: svn commit: r697357 - in /httpd/httpd/trunk: include/ modules/http/ modules/test/ server/ server/mpm/experimental/event/

2008-09-20 Thread Paul Querna
Graham Leggett wrote: I know there are likely huge problems with this, but I would like to see how far we can push the Event MPM, figure out what to do better, if there is anything, and then really dive into the 3.0 development before ApacheCon. How difficult would this be to support in the

Re: svn commit: r697357 - in /httpd/httpd/trunk: include/ modules/http/ modules/test/ server/ server/mpm/experimental/event/

2008-09-20 Thread Paul Querna
Nick Kew wrote: On 20 Sep 2008, at 12:58, [EMAIL PROTECTED] wrote: Introduce Suspendable Requests to the Event MPM. Hmmm ... Are you sure this belongs in the MPM? The core problem is, where do we have an Event scheduling and worker thread system. The best place to keep that is the MPM

Re: svn commit: r697357 - in /httpd/httpd/trunk: include/ modules/http/ modules/test/ server/ server/mpm/experimental/event/

2008-09-20 Thread Ruediger Pluem
On 09/20/2008 01:58 PM, [EMAIL PROTECTED] wrote: Author: pquerna Date: Sat Sep 20 04:58:08 2008 New Revision: 697357 URL: http://svn.apache.org/viewvc?rev=697357view=rev Log: Introduce Suspendable Requests to the Event MPM. Modified: httpd/httpd/trunk/include/ap_mpm.h URL:

Re: svn commit: r697357 - in /httpd/httpd/trunk: include/ modules/http/ modules/test/ server/ server/mpm/experimental/event/

2008-09-20 Thread Paul Querna
Ruediger Pluem wrote: + +typedef void (ap_mpm_callback_fn_t)(void *baton); + +/* XXX: only added support in the Event MPM */ +AP_DECLARE(void) ap_mpm_register_timed_callback(apr_time_t t, +ap_mpm_callback_fn_t *cbfn, +

Re: svn commit: r697425 - in /httpd/httpd/trunk/server: mpm/experimental/event/mpm.h mpm_common.c

2008-09-20 Thread Ruediger Pluem
On 09/21/2008 12:04 AM, [EMAIL PROTECTED] wrote: Author: pquerna Date: Sat Sep 20 15:04:03 2008 New Revision: 697425 URL: http://svn.apache.org/viewvc?rev=697425view=rev Log: Add dummy ap_mpm_register_timed_callback for other MPMs besides Event. Modified: