On Fri, Mar 30, 2012 at 01:20:02PM +0900, nozawat wrote: > Hi Dejan > > >> Can we also do a global replace s/result/rc/? > > OK,I contribute a patch again. > I made this patch, I attach it.
Patch applied. Thanks! Dejan > Regards, > Tomo > > 2012年3月30日12:03 nozawat <noza...@gmail.com>: > > Hi Dejan > > > > Thank you for your reply. > >> This patch doesn't look right. What if the first kill(1) fails > >> (because, say, the process just exited)? And if '$result' is -ne > >> 0, then the while loop will finish anyway. > > Yes,And output an error in an origin of summons and retry it. > > because kill repaid return 0 in the case of an error as for me, I made > > modifications to return 1 and output an error. > > > >> Anyway, the terminate function looks too complicated for what it > >> does. > > As for this correction, a logic is surely complicated. > > By the way, as for this correction, kill is effective when the error > > except the case without a process occurs. The situation did not hit on > > me. > > > >> Can we also do a global replace s/result/rc/? > > OK,I contribute a patch again. > > > > 2012年3月29日22:35 Dejan Muhamedagic <de...@suse.de>: > >> Hi Tomo, > >> > >> On Tue, Mar 27, 2012 at 10:27:36AM +0900, nozawat wrote: > >>> Hi > >>> > >>> Even if kill fails in terminate, return success. > >>> When kill fails, this is the patch which I revised to return an error > >>> definitely. > >> > >> This patch doesn't look right. What if the first kill(1) fails > >> (because, say, the process just exited)? And if '$result' is -ne > >> 0, then the while loop will finish anyway. > >> > >> Anyway, the terminate function looks too complicated for what it > >> does. > >> > >> Can we also do a global replace s/result/rc/? > >> > >> Cheers, > >> > >> Dejan > >> > >>> Regards, > >>> Tomo > >> > >> > >>> _______________________________________________________ > >>> Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org > >>> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev > >>> Home Page: http://linux-ha.org/ > >> > >> _______________________________________________________ > >> Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org > >> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev > >> Home Page: http://linux-ha.org/ > _______________________________________________________ > Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev > Home Page: http://linux-ha.org/ _______________________________________________________ Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev Home Page: http://linux-ha.org/