Re: [PATCH] Bug 2680: ** helper errors after -k rotate

From: Adrian Chadd <adrian_at_squid-cache.org>
Date: Thu, 16 Jul 2009 10:27:39 +0800

NOte that winbind has a hard coded limit that is by default very low.

Opening 2n ntlm_auth helpers may make things blow up in horrible ways.

Adrian

2009/7/16 Robert Collins <robertc_at_robertcollins.net>:
> On Thu, 2009-07-16 at 14:08 +1200, Amos Jeffries wrote:
>>
>> Both reconfigure and helper recovery use startHelpers() where the
>> limit
>> needs to take place.
>> The DOS bug fix broke *rotate* (reconfigure has an async step added by
>> Alex
>> that prevents it being a problem).
>
> s/rotate/reconfigure then :) In my mind one is a subset of the other.
>
>> > If someone is running hundreds of helpers on openwrt/olpc then
>> things
>> > are broken already :). I'd really suggest that such environments
>> > pipeline through a single helper rather than many concurrent
>> helpers.
>> > Such platorms are single core and you'll get better usage of memory
>> > doing many requests in a single helper than one request each to many
>> > helpers.
>>
>> lol, NTLM concurrent? try it!
>
> I did. IIRC the winbindd is fully capable of handling multiple
> overlapping requests, and each NTLM helper is *solely* a thunk layer
> between squid's format and the winbindd *state*.
>
> ASCII art time, 3 requests:
> Multiple helpers:
>       /------1-helper------\
> squid-*-------2-helper-------* winbindd [state1, state2, state3]
>       \------3-helper------/
> One helper:
> squid-*------1-helper-------* winbindd [state1, state2, state3]
>
> -Rob
>
>
Received on Thu Jul 16 2009 - 02:27:45 MDT

This archive was generated by hypermail 2.2.0 : Thu Jul 16 2009 - 12:00:05 MDT