babeld: Don't kill -9 on stop

Killing anything with -9 is a bad idea.  When killed this way, babeld
won't be able to properly disassociate from its neighbours, withdraw its
announced routes or remove routes from the kernel.

This got introduced in bab933d4cad8 ("babeld: Update to version 1.8.3 +
fix init") with an unrelated change.  The purpose of the change is unclear
because stopping and restarting babeld worked fine without this change.

Signed-off-by: Baptiste Jonglez <git@bitsofnetworks.org>

diff --git a/babeld/files/babeld.init b/babeld/files/babeld.init
index 7b25ef48c6b0a5cea6861824b5221ea1e11f64f2..378a8abcdb7e5e337d1e1e26ea3ad3fe9eb80cf5 100755 (executable)
--- a/babeld/files/babeld.init
+++ b/babeld/files/babeld.init
@@ -197,10 +197,6 @@ start_service() {
        procd_close_instance
 }
 
-stop_service() {
-       killall -9 babeld
-}
-
 service_triggers() {
        procd_add_reload_trigger babeld
 }