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

Timestamp:

Oct 7, 2015, 12:05:03 PM (9 years ago)

Author:

alain

Message:

Cosmetic.

File:

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

soft/giet_vm/giet_drivers/hba_driver.c

@@ -28 +28 @@
 
 // allocated in the boot.c or kernel_init.c files
-extern static_scheduler_t* _schedulers[X_SIZE][Y_SIZE][NB_PROCS_MAX]; 
+extern static_scheduler_t*    _schedulers[X_SIZE][Y_SIZE][NB_PROCS_MAX]; 
+extern unsigned int           _hba_boot_mode;  
 
 //////////////////////////////////////////////////////////////////////////////////
@@ -36 +37 @@
 // and must be defined in both kernel_init.c and boot.c files.
 // - 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.
+//   mode. There is no need to use a lock.
+// - during the execution phase, the HBA device can be used by several processors. 
+//   The slot allocator is protected by a sqt_lock.
 //////////////////////////////////////////////////////////////////////////////////
         
-extern unsigned int _hba_boot_mode;
-
-__attribute__((section(".kdata")))
-sqt_lock_t          _hba_allocator_lock  __attribute__((aligned(64)));
+__attribute__((section(".kdata")))
+sqt_lock_t          _hba_lock  __attribute__((aligned(64)));
 
 // state of each slot (allocated to a thread or not)
@@ -98 +97 @@
 ///////////////////////////////////////////////////////////////////////////////
 // This blocking fonction allocates a free command index to the thread. 
-// The hba_allocator_lock is not used in boot mode.
+// The hba_lock is not used in boot mode.
 // It returns the allocated command index (between 0 and 31)
 ///////////////////////////////////////////////////////////////////////////////
@@ -109 +108 @@
     while ( found == 0)
     {
-        if ( !_hba_boot_mode )
-            _sqt_lock_acquire(&_hba_allocator_lock);
+        if ( !_hba_boot_mode ) _sqt_lock_acquire(&_hba_lock);
 
         for ( c = 0; c < 32 ; c++ )
@@ -123 +121 @@
         }
 
-        if ( !_hba_boot_mode )
-            _sqt_lock_release(&_hba_allocator_lock);
+        if ( !_hba_boot_mode ) _sqt_lock_release(&_hba_lock);
     }
 
@@ -379 +376 @@
     // initialise allocator lock if not in boot mode
     if ( !_hba_boot_mode )
-        _sqt_lock_init(&_hba_allocator_lock);
+        _sqt_lock_init(&_hba_lock);
 
     // initialise Command Descriptors in Command List, allocated command table
@@ -390 +387 @@
         _hba_cmd_list[c].ctba  = (unsigned int)(paddr);
         _hba_cmd_list[c].ctbau = (unsigned int)(paddr>>32);
-        _hba_allocated_cmd[c] = 0;
-        _hba_active_cmd[c] = 0;
+        _hba_allocated_cmd[c]  = 0;
+        _hba_active_cmd[c]     = 0;
     }