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

Timestamp:

Jan 1, 2015, 8:23:48 PM (9 years ago)

Author:

alain

Message:

1) The NIC, IOC, DMA and HBA drivers have been adapted to support the new _v2p_translate() function prototype (returns void).
2) The _mmc_inval() and _mmc_sync() functions does not use anymore the hard lock in the MMC, but use a soft spin_lock.
3) The NIC driver does not use anymore the GIET_NIC_BUFSIZE, GIET_NIC_NBUFS, and GIET_NIC_TIMEOUT parameters (removed from giet_config.h file).
4) The NIC driver registers map has been modified to support 64 bytes buffer descriptors for chained buffers.

File:

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

soft/giet_vm/giet_drivers/hba_driver.c

@@ -173 +173 @@
 
         // get ppn and flags 
-        ko = _v2p_translate( (page_table_t*)user_pt_vbase,
-                              vpn,
-                              &ppn,
-                              &flags );
+        _v2p_translate( (page_table_t*)user_pt_vbase,
+                        vpn,
+                        &ppn,
+                        &flags );
 
         // check access rights
-        if ( ko )
-        {
-            _puts("[GIET ERROR] in _hba_set_cmd() : user buffer unmapped\n");
-            return -1;
-        }
         if ((flags & PTE_U) == 0)
         {
@@ -261 +256 @@
     unsigned int ppn;
     unsigned int flags;
-    unsigned int fail;
     unsigned int vbase;
     unsigned int c;               // c == command index
@@ -281 +275 @@
     // Command list physical addresse
     vbase = (unsigned int)(&hba_cmd_list[channel]);
-    fail = _v2p_translate( (page_table_t*)pt,
-                           vbase>>12,
-                           &ppn,
-                           &flags );
-    if ( fail )
-    {
-        _puts("[GIET ERROR] in _hba_init() : command list unmapped\n");
-        return -1;
-    }
+    _v2p_translate( (page_table_t*)pt,
+                     vbase>>12,
+                     &ppn,
+                     &flags );
     hba_cmd_list_paddr[channel] = ((paddr_t)ppn) | (vbase & 0xFFF);
 
@@ -296 +285 @@
     {
         vbase = (unsigned int)(&hba_cmd_table[channel][c]);
-        fail = _v2p_translate( (page_table_t*)pt,
-                               vbase>>12,
-                               &ppn,
-                               &flags );
-        if ( fail )
-        {
-            _puts("[GIET ERROR] in _hba_init() : command table unmapped\n");
-            return -1;
-        }
+        _v2p_translate( (page_table_t*)pt,
+                         vbase>>12,
+                         &ppn,
+                         &flags );
         hba_cmd_table_paddr[channel][c] = ((paddr_t)ppn) | (vbase & 0xFFF);
     }