tor 2009-07-16 klockan 23:15 +1200 skrev Amos Jeffries: > Right, following up on that I had a hunch and discovered that n_active > is already performing this duty. The bug is therefore in my earlier fix > using n_running as a base instead of n_active. > > How does this new attached patch look to you two?
Looks fine to me, assuming rotate still starts a new set of helpers (wasn't clear to me). Will fix the "too many helpers have crashed, need to start new ones" case, and leave the rotate & reconfigure case alone. With this we can also start new helpers more or less immediately (at some seconds delay just in case) when seeing them crash... Regards Henrik