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

From: Henrik Nordstrom <henrik_at_henriknordstrom.net>
Date: Thu, 16 Jul 2009 23:30:39 +0200

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
Received on Thu Jul 16 2009 - 21:30:45 MDT

This archive was generated by hypermail 2.2.0 : Fri Jul 17 2009 - 12:00:05 MDT