rtapi_app: start command / do not autostart

This gives rtapi_app a deterministic behaivour. No other changes needed
exept implementing start / stop in realtime due to all apps run:
realtime start at startup
realtime stop at exit

Author: hdiethelm

scripts/realtime.in

@@ -122,8 +122,10 @@ CheckStatus(){
     case $RTPREFIX in
     uspace)
         if [ -z "$($PS -o stat= -o comm= -C rtapi_app | $GREP -v '^Z')" ]; then
+            echo Not running
             exit 1
         else
+            echo Running
             exit 0
         fi ;;
     *)
@@ -168,14 +170,20 @@ CheckMem(){
 
 Load(){
     CheckKernel
-    for MOD in $MODULES_LOAD ; do
-	if ! [ -d "/sys/module/$(basename "$MOD" .ko)" ]; then
-		$INSMOD "$MOD" || return $?
-	fi
-    done
-    if [ "$DEBUG" != "" ] && [ -w /proc/rtapi/debug ] ; then
-        echo "$DEBUG" > /proc/rtapi/debug
-    fi
+    case $RTPREFIX in
+    uspace)
+        rtapi_app start
+    ;;
+    (*rtai*)
+        for MOD in $MODULES_LOAD ; do
+        if ! [ -d "/sys/module/$(basename "$MOD" .ko)" ]; then
+            $INSMOD "$MOD" || return $?
+        fi
+        done
+        if [ "$DEBUG" != "" ] && [ -w /proc/rtapi/debug ] ; then
+            echo "$DEBUG" > /proc/rtapi/debug
+        fi
+    esac
 }
 
 CheckLoaded(){

src/rtapi/uspace_rtapi_main.cc

@@ -753,7 +753,10 @@ static int handle_command(const std::vector<std::string> &args) {
     if (args.size() == 0) {
         return 0;
     }
-    if (args.size() == 1 && args[0] == "exit") {
+    if (args.size() == 1 && args[0] == "start") {
+        rtapi_print_msg(RTAPI_MSG_ERR, "rtapi_app: start received while running\n");
+        return 0;
+    } else if (args.size() == 1 && args[0] == "exit") {
         force_exit = 1;
         return 0;
     } else if (args.size() >= 2 && args[0] == "load") {
@@ -771,7 +774,7 @@ static int handle_command(const std::vector<std::string> &args) {
     } else if (args.size() == 1 && args[0] == "getrt") {
         return do_getrt_cmd();
     } else {
-        rtapi_print_msg(RTAPI_MSG_ERR, "Unrecognized command starting with %s\n", args[0].c_str());
+        rtapi_print_msg(RTAPI_MSG_ERR, "rtapi_app: unrecognized command starting with %s\n", args[0].c_str());
         return -1;
     }
 }
@@ -830,12 +833,12 @@ static bool master_process_socket_command(int fd) {
         }
         close(fd1);
     }
-    return !force_exit && instance_count > 0;
+    return !force_exit;
 }
 
 static pthread_t main_thread{};
 
-static int master(int fd, const std::vector<std::string> &args) {
+static int master(int fd) {
     main_thread = pthread_self();
     int result;
     if ((result = pthread_create(&queue_thread, nullptr, &queue_function, nullptr)) != 0) {
@@ -846,16 +849,8 @@ static int master(int fd, const std::vector<std::string> &args) {
     do_load_cmd("hal_lib", std::vector<std::string>());
     instance_count = 0;
     App(); // force rtapi_app to be created
-    if (args.size()) {
-        result = handle_command(args);
-        if (result != 0)
-            goto out;
-        if (force_exit || instance_count == 0)
-            goto out;
-    }
     //Process commands as long as master should not exit
     while(master_process_socket_command(fd));
-out:
     do_unload_cmd("hal_lib");
     pthread_cancel(queue_thread);
     pthread_join(queue_thread, nullptr);
@@ -946,7 +941,6 @@ int main(int argc, char **argv) {
         args.push_back(std::string(argv[i]));
     }
 
-become_master:
     int fd = socket(PF_UNIX, SOCK_STREAM, 0);
     if (fd == -1) {
         perror("socket");
@@ -969,24 +963,37 @@ int main(int argc, char **argv) {
         //If exit is called and master is not running, do not start master
         //and exit again
         if (args.size() == 1 && args[0] == "exit") {
+            rtapi_print_msg(RTAPI_MSG_ERR, "rtapi_app: exit received while not running\n");
             return 0;
         }
-        //If getrt is called and master is not running, do not start master
-        //execute and return
+        //If getrt is called and master is not running, do not start master.
+        //Instead, run the command directly
         //This is needed for the verify command in the realtime script
         //If master is started, this starts up the whole realtime chanin
         //and halrun -U is needed to exit again
         if (args.size() == 1 && args[0] == "getrt") {
             return do_getrt_cmd();
         }
-        int result = listen(fd, 10);
-        if (result != 0) {
-            perror("listen");
-            exit(1);
+        //Start a master on start command
+        if (args.size() == 1 && args[0] == "start") {
+            int result = listen(fd, 10);
+            if (result != 0) {
+                perror("listen");
+                exit(1);
+            }
+            pid_t pid = fork();
+            if (pid < 0){
+                perror("lisforkten");
+                exit(1);
+            }
+            if(pid == 0){
+                setsid(); // create a new session if we can...
+                result = master(fd);
+            }
+            return result;
         }
-        setsid(); // create a new session if we can...
-        result = master(fd, args);
-        return result;
+        fprintf(stderr, "error: No master found. Use realtime start to start one.\n");
+        exit(1);
     } else if (errno == EADDRINUSE) {
         struct timespec start, now;
         clock_gettime(CLOCK_MONOTONIC, &start);
@@ -1003,7 +1010,7 @@ int main(int argc, char **argv) {
         if (result < 0 && errno == ECONNREFUSED) {
             fprintf(stderr, "Waited 3 seconds for master.  giving up.\n");
             close(fd);
-            goto become_master;
+            exit(1);
         }
         if (result < 0) {
             fprintf(stderr, "connect %s: %s", addr.sun_path, strerror(errno));