Agreed. I didn't see his patch, when I did the fix (and didn't check again when posting the fix). Amusing we did it at the same time. I'm fine with either patch as the loop shouldn't be triggered in general, nor do I think it would really help ...
-brian an intel employee On Mon, May 1, 2017 at 4:15 PM, Martin Kelly <[email protected]> wrote: > Looks like this is addressing the same problem as Jan's patch from earlier > today. I think the differences are: > > - kill vs start-stop-daemon > > - Jan removed the retry loop while this patch does not > > > On 05/01/2017 02:59 PM, brian avery wrote: > >> The current initscript was lacking a -USR2 signal in the stop section. >> This resulted in /etc/init.d/tcf-agent stop; taking roughly 10 seconds >> then failing to kill the tcf-agent process. This patch brings us in line >> with how upstream stops the agent. >> >> Fixes [YOCTO #10858] >> >> Signed-off-by: brian avery <[email protected]> >> --- >> meta/recipes-devtools/tcf-agent/tcf-agent/tcf-agent.init | 3 +-- >> 1 file changed, 1 insertion(+), 2 deletions(-) >> >> diff --git a/meta/recipes-devtools/tcf-agent/tcf-agent/tcf-agent.init >> b/meta/recipes-devtools/tcf-agent/tcf-agent/tcf-agent.init >> index 6303280..13497cc 100755 >> --- a/meta/recipes-devtools/tcf-agent/tcf-agent/tcf-agent.init >> +++ b/meta/recipes-devtools/tcf-agent/tcf-agent/tcf-agent.init >> @@ -35,7 +35,7 @@ case "$1" in >> echo -n "Stopping $DAEMON_NAME: " >> count=0 >> while [ -n "`/bin/pidof $DAEMON_PATH`" -a $count -lt 10 ] ; do >> - killproc $DAEMON_PATH >& /dev/null >> + kill -USR2 `pidofproc $DAEMON_PATH` >& /dev/null >> sleep 1 >> RETVAL=$? >> if [ $RETVAL != 0 -o -n "`/bin/pidof $DAEMON_PATH`" ] ; then >> @@ -72,4 +72,3 @@ case "$1" in >> esac >> >> exit $RETVAL >> - >> >> -- > _______________________________________________ > Openembedded-core mailing list > [email protected] > http://lists.openembedded.org/mailman/listinfo/openembedded-core >
-- _______________________________________________ Openembedded-core mailing list [email protected] http://lists.openembedded.org/mailman/listinfo/openembedded-core
