Changeset 440 for trunk/hal/tsar_mips32/drivers/soclib_bdv.c

Timestamp:

May 3, 2018, 5:51:22 PM (6 years ago)

Author:

alain

Message:

1/ Fix a bug in the Multithreaded "sort" applicationr:
The pthread_create() arguments must be declared as global variables.
2/ The exit syscall can be called by any thread of a process..

File:

• trunk/hal/tsar_mips32/drivers/soclib_bdv.c (modified) (4 diffs)

trunk/hal/tsar_mips32/drivers/soclib_bdv.c

@@ -66 +66 @@
     // get client thread cluster and local pointer
     cxy_t      th_cxy = GET_CXY( th_xp );
-    thread_t * th_ptr = (thread_t *)GET_PTR( th_xp );
+    thread_t * th_ptr = GET_PTR( th_xp );
 
     // get command arguments and extended pointer on IOC device
@@ -91 +91 @@
     // get IOC device cluster and local pointer
     cxy_t      ioc_cxy = GET_CXY( ioc_xp );
-    chdev_t  * ioc_ptr = (chdev_t *)GET_PTR( ioc_xp );
-
-    // get extended pointer on SOCLIB-BDV peripheral
-    xptr_t     bdv_xp = hal_remote_lw( XPTR( ioc_cxy , &ioc_ptr->base ) );
-
-    // get SOCLIB_BDV device cluster and local pointer
-    cxy_t      bdv_cxy = GET_CXY( bdv_xp );
-    uint32_t * bdv_ptr = (uint32_t *)GET_PTR( bdv_xp );
+    chdev_t  * ioc_ptr = GET_PTR( ioc_xp );
+
+    // get cluster and pointers for SOCLIB-BDV peripheral segment base
+    xptr_t     seg_xp  = (xptr_t)hal_remote_lwd( XPTR( ioc_cxy , &ioc_ptr->base ) );
+    cxy_t      seg_cxy = GET_CXY( seg_xp );
+    uint32_t * seg_ptr = GET_PTR( seg_xp );
 
     // split buffer address in two 32 bits words
@@ -110 +108 @@
 
     // set SOCLIB_BDV registers to start one I/O operation
-    hal_remote_sw( XPTR( bdv_cxy , bdv_ptr + BDV_IRQ_ENABLE_REG ) , 1       );
-    hal_remote_sw( XPTR( bdv_cxy , bdv_ptr + BDV_BUFFER_REG     ) , buf_lsb ); 
-    hal_remote_sw( XPTR( bdv_cxy , bdv_ptr + BDV_BUFFER_EXT_REG ) , buf_msb ); 
-    hal_remote_sw( XPTR( bdv_cxy , bdv_ptr + BDV_LBA_REG        ) , lba     );
-    hal_remote_sw( XPTR( bdv_cxy , bdv_ptr + BDV_COUNT_REG      ) , count   );
-    hal_remote_sw( XPTR( bdv_cxy , bdv_ptr + BDV_OP_REG         ) , op      );
+    hal_remote_sw( XPTR( seg_cxy , seg_ptr + BDV_IRQ_ENABLE_REG ) , 1       );
+    hal_remote_sw( XPTR( seg_cxy , seg_ptr + BDV_BUFFER_REG     ) , buf_lsb ); 
+    hal_remote_sw( XPTR( seg_cxy , seg_ptr + BDV_BUFFER_EXT_REG ) , buf_msb ); 
+    hal_remote_sw( XPTR( seg_cxy , seg_ptr + BDV_LBA_REG        ) , lba     );
+    hal_remote_sw( XPTR( seg_cxy , seg_ptr + BDV_COUNT_REG      ) , count   );
+    hal_remote_sw( XPTR( seg_cxy , seg_ptr + BDV_OP_REG         ) , op      );
 
     // waiting policy  depends on the command type
@@ -126 +124 @@
         while (1)
         {
-            status = hal_remote_lw( XPTR( bdv_cxy , bdv_ptr + BDV_STATUS_REG ) );
+            status = hal_remote_lw( XPTR( seg_cxy , seg_ptr + BDV_STATUS_REG ) );
 
             if( status == BDV_READ_SUCCESS ) // successfully completed