Changeset 630 for soft/giet_vm/giet_drivers/hba_driver.c

Timestamp:

Jul 18, 2015, 3:04:15 PM (9 years ago)

Author:

alain

Message:

Update the peripheral drivers using descheduling,
to comply with the modified NORUN bit-vector in task context.

File:

• soft/giet_vm/giet_drivers/hba_driver.c (modified) (5 diffs)

soft/giet_vm/giet_drivers/hba_driver.c

@@ -23 +23 @@
 #include <vmem.h>
 
-///////////////////////////////////////////////////////////////////////////////////
+//////////////////////////////////////////////////////////////////////////////////
+//      Extern variables
+//////////////////////////////////////////////////////////////////////////////////
+
+// allocated in the boot.c or kernel_init.c files
+extern static_scheduler_t* _schedulers[X_SIZE][Y_SIZE][NB_PROCS_MAX]; 
+
+//////////////////////////////////////////////////////////////////////////////////
 //               Global variables
-///////////////////////////////////////////////////////////////////////////////////
-
 //////////////////////////////////////////////////////////////////////////////////
 // The global variable hba_boot_mode defines the way the HBA component is used
 // and must be defined in both kernel_init.c and boot.c files.
-// - during the boot phase, only one processor has access to the HBA in synchronous
-//   mode, there is no need for the allocator to use a lock
+// - during the boot phase, only one processor access the HBA in synchronous
+//   mode. There is no need for the allocator to use a lock.
 // - after the boot phase, the HBA device can be used by several processors. The
 //   allocator is protected by a sqt_lock.
@@ -278 +283 @@
     // in descheduling mode, we deschedule the task
     // and use an interrupt to reschedule the task.
-    // We need a critical section, because we must reset the RUN bit
+    // We need a critical section, because we must set the NORUN bit
         // before to launch the transfer, and we don't want to be 
     // descheduled between these two operations. 
@@ -303 +308 @@
         _it_disable( &save_sr ); 
 
-        // reset runnable 
-        _set_task_slot( x, y, p, ltid, CTX_RUN_ID, 0 );  
-
+        // Set NORUN_MASK_IOC bit 
+        static_scheduler_t* psched  = (static_scheduler_t*)_schedulers[x][y][p];
+        unsigned int*       ptr     = &psched->context[ltid][CTX_NORUN_ID];
+        _atomic_or( ptr , NORUN_MASK_IOC );
+      
         // start HBA transfer
         _hba_set_register( HBA_PXCI, (1<<cmd_id) );
@@ -432 +439 @@
 
             // identify waiting task 
-            unsigned int remote_procid  = _hba_gtid[cmd_id]>>16;
-            unsigned int ltid           = _hba_gtid[cmd_id] & 0xFFFF;
-            unsigned int remote_cluster = remote_procid >> P_WIDTH;
-            unsigned int remote_x       = remote_cluster >> Y_WIDTH;
-            unsigned int remote_y       = remote_cluster & ((1<<Y_WIDTH)-1);
-            unsigned int remote_p       = remote_procid & ((1<<P_WIDTH)-1);
+            unsigned int procid  = _hba_gtid[cmd_id]>>16;
+            unsigned int ltid    = _hba_gtid[cmd_id] & 0xFFFF;
+            unsigned int cluster = procid >> P_WIDTH;
+            unsigned int x       = cluster >> Y_WIDTH;
+            unsigned int y       = cluster & ((1<<Y_WIDTH)-1);
+            unsigned int p       = procid & ((1<<P_WIDTH)-1);
  
-            // re-activates waiting task
-            _set_task_slot( remote_x,
-                            remote_y,
-                            remote_p,
-                            ltid,
-                            CTX_RUN_ID,
-                            1 );
+            // Reset NORUN_MASK_IOC bit 
+            static_scheduler_t* psched  = (static_scheduler_t*)_schedulers[x][y][p];
+            unsigned int*       ptr     = &psched->context[ltid][CTX_NORUN_ID];
+            _atomic_and( ptr , ~NORUN_MASK_IOC );
 
             // send a WAKUP WTI to processor running the waiting task 
-            _xcu_send_wti( remote_cluster , 
-                           remote_p , 
+            _xcu_send_wti( cluster , 
+                           p , 
                            0 );          // don't force context switch
 
@@ -457 +461 @@
         "  resume task %d running on P[%d,%d,%d]\n",
         cmd_id , _get_proctime() ,
-        ltid , remote_x , remote_y , remote_p );
+        ltid , x , y , p );
 #endif 
         }