Changeset 626 for trunk/kernel/syscalls/sys_thread_create.c

Timestamp:

Apr 29, 2019, 7:25:09 PM (5 years ago)

Author:

alain

Message:

This version has been tested on the sort multithreaded application
for TSAR_IOB architectures ranging from 1 to 8 clusters.
It fixes three bigs bugs:
1) the dev_ioc device API has been modified: the dev_ioc_sync_read()
and dev_ioc_sync_write() function use now extended pointers on the
kernel buffer to access a mapper stored in any cluster.
2) the hal_uspace API has been modified: the hal_copy_to_uspace()
and hal_copy_from_uspace() functions use now a (cxy,ptr) couple
to identify the target buffer (equivalent to an extended pointer.
3) an implementation bug has been fixed in the assembly code contained
in the hal_copy_to_uspace() and hal_copy_from_uspace() functions.

File:

• trunk/kernel/syscalls/sys_thread_create.c (modified) (2 diffs)

trunk/kernel/syscalls/sys_thread_create.c

@@ -106 +106 @@
             }
         
-            hal_copy_from_uspace( &kern_attr , user_attr , sizeof(pthread_attr_t) );
+            hal_copy_from_uspace( local_cxy, 
+                              &kern_attr,
+                              user_attr,
+                              sizeof(pthread_attr_t) );
     }
 
@@ -212 +215 @@
         // returns trdid to user space
         trdid = hal_remote_l32( XPTR( child_cxy , &child_ptr->trdid ) );
-        hal_copy_to_uspace( trdid_ptr , &trdid , sizeof(pthread_t) );
+        hal_copy_to_uspace( local_cxy,
+                        &trdid,
+                        trdid_ptr,
+                        sizeof(pthread_t) );
 
     // activate new thread