From ccefd921e76ba4ca7f3d6b54691315c0718b039c Mon Sep 17 00:00:00 2001
From: Richard Braun <rbraun@sceen.net>
Date: Wed, 31 May 2017 21:11:29 +0200
Subject: kern/shell: separate initialization into two steps

The shell must be able to register commands early. As a result, it's now
initialized in two steps, one of which starts the shell thread.
---
 kern/kernel.c |  2 +-
 kern/shell.c  | 10 ++++++++--
 kern/shell.h  |  5 +++++
 3 files changed, 14 insertions(+), 3 deletions(-)

(limited to 'kern')

diff --git a/kern/kernel.c b/kern/kernel.c
index 05cf8392..1d61b0ca 100644
--- a/kern/kernel.c
+++ b/kern/kernel.c
@@ -50,8 +50,8 @@ kernel_main(void)
     work_setup();
     llsync_setup();
     sref_setup();
-    shell_setup();
     vm_page_info();
+    shell_start();
 
 #ifdef X15_RUN_TEST_MODULE
     test_setup();
diff --git a/kern/shell.c b/kern/shell.c
index c4675bf4..ba756a5f 100644
--- a/kern/shell.c
+++ b/kern/shell.c
@@ -1158,8 +1158,6 @@ shell_run(void *arg)
 void __init
 shell_setup(void)
 {
-    struct thread_attr attr;
-    struct thread *thread;
     unsigned long i;
     int error;
 
@@ -1169,6 +1167,14 @@ shell_setup(void)
         error = shell_cmd_register(&shell_default_cmds[i]);
         error_check(error, "shell_cmd_register");
     }
+}
+
+void __init
+shell_start(void)
+{
+    struct thread_attr attr;
+    struct thread *thread;
+    int error;
 
     thread_attr_init(&attr, THREAD_KERNEL_PREFIX "shell");
     thread_attr_set_detached(&attr);
diff --git a/kern/shell.h b/kern/shell.h
index 5ff4920d..a33abbef 100644
--- a/kern/shell.h
+++ b/kern/shell.h
@@ -55,6 +55,11 @@ void shell_cmd_init(struct shell_cmd *cmd, const char *name,
  */
 void shell_setup(void);
 
+/*
+ * Start the shell thread.
+ */
+void shell_start(void);
+
 /*
  * Register a shell command.
  *
-- 
cgit v1.2.3