Changeset 391 for soft/giet_vm/giet_kernel/kernel_init.c

Timestamp:

Aug 11, 2014, 9:19:06 PM (10 years ago)

Author:

alain

Message:

Idle task stack implementation. The idle stack was previously implemented in the
seg_kernel data, and this was a scalabity bottleneck. i
A 4 Kbytes stack for idle stack is now implemented in the processor scheduler segment.

File:

• soft/giet_vm/giet_kernel/kernel_init.c (modified) (6 diffs)

soft/giet_vm/giet_kernel/kernel_init.c

@@ -14 +14 @@
 // and where there is one physical memory bank per cluster.
 //
-// The kernel_init() function is executed in parallel by all procesors, 
-// and completes the system initialisation that has been started by processor[0,0,0] 
+// The kernel_init() function is executed sequencially by all procesors. 
+// It completes the system initialisation that has been started by processor[0,0,0] 
 // in the boot_init() function. It makes the following assuptions, regarding the work 
 // bone by the boot code:
@@ -100 +100 @@
 
 #if !defined(IDLE_TASK_INDEX) 
-# error: You must define IDLE_TASK_INDEX in the giet_config.h file
+# error: You must define IDLE_TASK_INDEX in the ctx_handler.h file
+#endif
+
+#if !defined(IDLE_TASK_STACK) 
+# error: You must define IDLE_TASK_STACK in the ctx_handler.h file
 #endif
 
@@ -129 +133 @@
 
 ////////////////////////////////////////////////////////////////////////////////////
-// stacks for the "idle" tasks (64 bytes for each processor)
-////////////////////////////////////////////////////////////////////////////////////
-
-__attribute__((section (".kdata"))) 
-volatile unsigned int _idle_stack[X_SIZE * Y_SIZE * NB_PROCS_MAX * 16 ]; 
-
-////////////////////////////////////////////////////////////////////////////////////
-// Synchonisation Barrier before jumping to user code
+// Synchonisation barrier before jumping to user code
 ////////////////////////////////////////////////////////////////////////////////////
 
@@ -155 +152 @@
     while( pid != _init_barrier ) asm volatile ( "nop" );
 
-    // Step 1 : each processor get its scheduler virtual address
+    // Step 1 : each processor get its scheduler virtual address from CP0 register
     //          and contribute to initialise the _schedulers[] array
 
@@ -290 +287 @@
 
     // step 6 : each processor updates the idle_task context:
-    //          (only CTX_SP, CTX_RA, CTX_EPC).
-    //          The stack size is 512 bytes, reserved in seg_kdata.
+    //          (CTX_SP, CTX_RA, CTX_EPC).
+    //          The 4 Kbytes idle stack is implemented in the scheduler.
     //          The PTPR register, the CTX_PTPR and CTX_PTAB slots 
     //          have been initialised in boot code.
 
-    unsigned int p = ((x * Y_SIZE) + y) * NB_PROCS_MAX + lpid; 
-
-    unsigned int stack = (unsigned int)_idle_stack + ((p + 1) * 64);
-
-    _set_task_slot( global_pid, IDLE_TASK_INDEX, CTX_SP_ID,  stack);
+    unsigned int pstack = ((unsigned int)psched) + 0x2000;
+
+    _set_task_slot( global_pid, IDLE_TASK_INDEX, CTX_SP_ID,  pstack);
     _set_task_slot( global_pid, IDLE_TASK_INDEX, CTX_RA_ID,  (unsigned int) &_ctx_eret);
     _set_task_slot( global_pid, IDLE_TASK_INDEX, CTX_EPC_ID, (unsigned int) &_idle_task);
 
 #if GIET_DEBUG_INIT
-_printf("\n[GIET DEBUG INIT] Processor[%d,%d,%d] initializes IDLE task\n",
-        x, y, lpid );
+_printf("\n[GIET DEBUG INIT] Processor[%d,%d,%d] initializes IDLE task\n"
+        " - stack_base = %x\n"
+        " - stack_size = 0x1000\n",
+        x, y, lpid, pstack - 0x1000 );
 #endif
 
@@ -341 +338 @@
     // busy waiting until all processors synchronized
     while ( _init_barrier != NB_TOTAL_PROCS );
+
+#if GIET_DEBUG_INIT
+_printf("\n[GIET DEBUG INIT] Processor[%d,%d,%d] initializes registers at cycle %d\n"
+        " - sp   = %x\n"
+        " - sr   = %x\n"
+        " - ptpr = %x\n"
+        " - epc  = %x\n",
+        x, y, lpid, _get_proctime(),
+        sp_value, sr_value, ptpr_value, epc_value );
+#endif
 
     // set registers and jump to user code