On Jan 5, 2008 7:45 PM, William A. Rowe, Jr. <[EMAIL PROTECTED]> wrote:
> Tom Donovan wrote:
> > Yes, the crash dissappears.
> >
> > I built with MS Visual Studio 8 on win2k and tested on win2k, winxp, and
> > vista.
> >
> >   ab -n 100000 -c 100 http://localhost/large_file.html
> >
> > Where large_file.html is 200kb.  No continued memory growth observed
> > (after the first few seconds), so I don't think it is leaking allocators.
>
> Fantastic.
>
> Code drift between the Win9x case (now pretty much irrelevant) and WinNT
> disabling acceptex really is unacceptable now, but I don't want to touch
> the 2.0/2.2 code again if we can avoid it.  Best solution is a refactoring
> which I might find a weekend for in the next month or so.
>
> I'm going to propose we dump DisableWin32AcceptEx syntax entirely in 2.4,
> and instead use our new friendly syntax;
>
> AcceptFilter http ex-data (current acceptex, plus some data)
> AcceptFilter https ex     (current acceptex, but no data)
>
> AcceptFilter http none    (vanilla accept)
>

Isn't AcceptFilter http ex the default now?
Wouldn't :
AcceptFilter http none (current acceptex, no data)
AcceptFilter http data (current acceptex + data)
AcceptFilter http no-ex (vanilla accept)

be better?

Just my 2 cents

> Does this sound like the right approach before I start hacking anything?
>
> Bill
>



-- 
~Jorge

Reply via email to