Use a linker-table based system to automatically mark and start up

permanent processes, rather than requiring each one to have its own
initialisation function.

diff --git a/src/core/main.c b/src/core/main.c
index 64e098ca32743325290fcd7c63f098602f41c203..5b01df9cc6db6b8f471bdf018830746274871360 100644 (file)
--- a/src/core/main.c
+++ b/src/core/main.c
@@ -16,6 +16,7 @@ Literature dealing with the network protocols:
 
 #include <gpxe/heap.h>
 #include <gpxe/init.h>
+#include <gpxe/process.h>
 #include <gpxe/device.h>
 #include <gpxe/shell.h>
 #include <gpxe/shell_banner.h>
@@ -29,8 +30,10 @@ Literature dealing with the network protocols:
  * Call this function only once, before doing (almost) anything else.
  */
 static void startup ( void ) {
-       hide_etherboot();
        init_heap();
+       init_processes();
+
+       hide_etherboot();
        call_init_fns();
        probe_devices();
 }

diff --git a/src/core/process.c b/src/core/process.c
index 0583a398cc139cc6d35529c4d58e689b5f00fbb9..bd35e614f9f115e6626b0199b15e7f010d5fa217 100644 (file)
--- a/src/core/process.c
+++ b/src/core/process.c
@@ -30,6 +30,12 @@
 /** Process run queue */
 static LIST_HEAD ( run_queue );
 
+/** Registered permanent processes */
+static struct process processes[0]
+       __table_start ( struct process, processes );
+static struct process processes_end[0]
+       __table_end ( struct process, processes );
+
 /**
  * Add process to process list
  *
@@ -72,3 +78,15 @@ void step ( void ) {
                break;
        }
 }
+
+/**
+ * Initialise processes
+ *
+ */
+void init_processes ( void ) {
+       struct process *process;
+
+       for ( process = processes ; process < processes_end ; process++ ) {
+               process_add ( process );
+       }
+}

diff --git a/src/include/gpxe/init.h b/src/include/gpxe/init.h
index bd4cd97d4f5e1d5c98fdcce9e7dff1742b713bf3..b13674ec70c87a954714d49c2efe2e0c932bc293 100644 (file)
--- a/src/include/gpxe/init.h
+++ b/src/include/gpxe/init.h
@@ -42,7 +42,6 @@ struct init_fn {
 #define        INIT_LOADBUF    08
 #define        INIT_PCMCIA     09
 #define        INIT_RPC        11
-#define INIT_PROCESS   12
 
 /* Macro for creating an initialisation function table entry */
 #define INIT_FN( init_order, init_func, reset_func, exit_func )        \

diff --git a/src/include/gpxe/process.h b/src/include/gpxe/process.h
index c0837fa4f94cd82631023b68b6836d906951898e..595787c0f50cb7b6e2d1eadb21735287f734ae06 100644 (file)
--- a/src/include/gpxe/process.h
+++ b/src/include/gpxe/process.h
@@ -9,6 +9,7 @@
 
 #include <gpxe/list.h>
 #include <gpxe/refcnt.h>
+#include <gpxe/tables.h>
 
 /** A process */
 struct process {
@@ -33,6 +34,7 @@ struct process {
 extern void process_add ( struct process *process );
 extern void process_del ( struct process *process );
 extern void step ( void );
+extern void init_processes ( void );
 
 /**
  * Initialise process without adding to process list
@@ -62,4 +64,13 @@ process_init ( struct process *process,
        process_add ( process );
 }
 
+/**
+ * Declare a permanent process
+ *
+ * Permanent processes will be automatically added to the process list
+ * at initialisation time.
+ */
+#define __permanent_process \
+       __table ( struct process, processes, 01 )
+
 #endif /* _GPXE_PROCESS_H */

diff --git a/src/net/netdevice.c b/src/net/netdevice.c
index 971830d9707b684f6038039ad164edd1fc5edbf0..fb4612bc9ae552e311d4843ab05f7695e4f05e3b 100644 (file)
--- a/src/net/netdevice.c
+++ b/src/net/netdevice.c
@@ -448,13 +448,6 @@ static void net_step ( struct process *process __unused ) {
 }
 
 /** Networking stack process */
-static struct process net_process = {
+struct process net_process __permanent_process = {
        .step = net_step,
 };
-
-/** Initialise the networking stack process */
-static void init_net ( void ) {
-       process_add ( &net_process );
-}
-
-INIT_FN ( INIT_PROCESS, init_net, NULL, NULL );

diff --git a/src/net/retry.c b/src/net/retry.c
index f8c34b8131b6baab247da13900345bd3036498ff..6734968f51c1c6314896b309541c333f52bf432b 100644 (file)
--- a/src/net/retry.c
+++ b/src/net/retry.c
@@ -167,13 +167,6 @@ static void retry_step ( struct process *process __unused ) {
 }
 
 /** Retry timer process */
-static struct process retry_process = {
+struct process retry_process __permanent_process = {
        .step = retry_step,
 };
-
-/** Initialise the retry timer module */
-static void init_retry ( void ) {
-       process_add ( &retry_process );
-}
-
-INIT_FN ( INIT_PROCESS, init_retry, NULL, NULL );