Jean McCormack wrote: > Susan Sohn wrote: >> I would definitely add a comment explaining why ctrun is being used here. >> Also, just to double check, are we still killing the right process at >> 186? > Sue, > > That was a good find. You were correct in that the pid used to kill the > process at 186 was > no longer correct. I have updated the code review. > http://cr.opensolaris.org/~jeanm/slim_8399_release
Jean, Looks ok now. Sue >> On 05/06/09 14:53, Jean McCormack wrote: >>> >>> I need reviewers and authorization from either Dave or Sanjay to push >>> for the following >>> >>> CR: http://defect.opensolaris.org/bz/show_bug.cgi?id=8399 >>> >>> Webrev: http://cr.opensolaris.org/~jeanm/slim_8399_release/ >>> This has fixed a subset of Andre's hangs. There appear to be two >>> issues causing the hangs. >>> This solves the one manifested by multiple delete/disable services in >>> rapid order where the >>> final service is finally deleted. >>> >>> There is another hang caused by a delete quickly following a create >>> that is caused by the >>> smf service not quite getting to the online state before you delete >>> the service. That is not >>> addressed by this push. I would like to propose pushing the fix and >>> opening a bug to address >>> this second issue. This new bug would then be release noted. >>> >>> Jean >>> _______________________________________________ >>> caiman-discuss mailing list >>> caiman-discuss at opensolaris.org >>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss >> >
