Changeset 223

Timestamp:

Nov 13, 2012, 6:20:39 PM (12 years ago)

Author:

meunier

Message:

Correction d'un bug dans drivers.c (fonctions _ioc_completed et _dma_completed) :
on relâche le lock après avoir remis à 0 les variables de synchro _ioc_done et _dma_done (+ pour le dma, lecture du status avant de relâcher le lock)
(+ ajout d'un sync dans les 2 cas avant relâchement du lock pour tsar)

File:

• soft/giet_vm/sys/drivers.c (modified) (3 diffs)

soft/giet_vm/sys/drivers.c

@@ -645 +645 @@
 
     // reset synchronization variables
-    _ioc_lock =0;
-    _ioc_done =0;
+    _ioc_done = 0;
+    asm volatile ("sync");
+    _ioc_lock = 0;
 
     return ret;
@@ -964 +965 @@
     unsigned int    task_id = _get_current_task_id();
     unsigned int    dma_id  = _get_context_slot( task_id, CTX_DMA_ID );
+    unsigned int    dma_ret;
 
     // busy waiting with a pseudo random delay between bus access
@@ -1001 +1003 @@
 
     // reset synchronization variables
+    _dma_done[dma_id] = 0;
+    dma_ret = dma_status[dma_id];
+    asm volatile("sync\n")
     _dma_lock[dma_id] = 0;
-    _dma_done[dma_id] = 0;
-
-    return _dma_status[dma_id];
+
+    return dma_ret;
 
 #else //NB_DMAS_MAX == 0