> -----Ursprüngliche Nachricht----- > Von: [EMAIL PROTECTED] > Auftrag von Vinicius Petrucci > Gesendet: Mittwoch, 5. September 2007 18:09 > An: [email protected] > Betreff: Re: AW: mod_proxy_balancer > > > Well, debugging a little bit more... I realize that the condition > (worker is already initialized?) in proxy_util.c(1672) is always > bypassed, which is the same condition used by the Rüdiger's patch. > > But, in proxy_util.c(1706) the same condition is rechecked (the > comment says "Possible if proxy is using scoreboard to hold shared > stats").
This is the reason why my patch does not work. Attached two proposed solutions: worker_init_patch_plus_r572937.diff: A simple and nice solution, but IMHO it is not backportable as it changes the API. worker_init_patch_plus_r572937_back.diff: A more complex and nasty solution that IMHO is backportable. Both patches require r572937 to be applied first. Can you please give both patches a try? Other comments welcome as well. Regards Rüdiger > > Attached is the patch applied and the error_log with debug > information. > > Vinicius > > On 05/09/07, Vinicius Petrucci <[EMAIL PROTECTED]> wrote: > > Well... I will give another try... Indeed, modifying proxy_utils.c > > might not be a nice solution... > > > > On 05/09/07, Plüm, Rüdiger, VF-Group > <[EMAIL PROTECTED]> wrote: > > > > > > > > > > -----Ursprüngliche Nachricht----- > > > > Von: Jim Jagielski > > > > Gesendet: Mittwoch, 5. September 2007 15:13 > > > > An: [email protected] > > > > Betreff: Re: AW: mod_proxy_balancer > > > > > > > > > > > > > > > Like you said, logically it doesn't make sense that the > > > > 2 patches work different (yours doesn't and Vinicius' > > > > does)... So I was thinking a different logic test for > > > > Vinicius to try, to see if its something local on > > > > his end... > > > > > > Ahh Ok. Now I get it. Thanks. > > > > > > Regards > > > > > > Rüdiger > > > > > > > > > > > > -- > > Vinicius Tavares Petrucci > > home page: http://www.ic.uff.br/~vpetrucci > > > > > -- > Vinicius Tavares Petrucci > home page: http://www.ic.uff.br/~vpetrucci >
worker_init_patch_plus_r572937.diff
Description: worker_init_patch_plus_r572937.diff
worker_init_patch_plus_r572937_back.diff
Description: worker_init_patch_plus_r572937_back.diff
