On 01/22/10 08:41 AM, Sean Wilcox wrote: > Sean Wilcox wrote: >> Webrev located at : >> >> http://cr.opensolaris.org/~swilcox/6919271_webrev.0/ >> >> Thanks, >> > 6919271 - wait services are not getting timers reset on a clean exit > > Logging in and out of a console can trigger the service to go into > maintenance mode, with the current 6219078/6311392 fix. > > This is because the timers are not being reset via the stop contract > action of a wait style process. The better place to reset the timers for > a wait style service is in stop_instance() as the fault_counters are > reset. This removes the dependency on the contract action occurring, and > the stop_instance() will be handled. >
This looks right to me. Sorry for not catching in the previous review. liane