Changeset 637 for trunk/kernel/syscalls/sys_barrier.c

Timestamp:

Jul 18, 2019, 2:06:55 PM (5 years ago)

Author:

alain

Message:

Introduce the non-standard pthread_parallel_create() system call
and re-write the <fft> and <sort> applications to improve the
intrinsic paralelism in applications.

File:

• trunk/kernel/syscalls/sys_barrier.c (modified) (9 diffs)

trunk/kernel/syscalls/sys_barrier.c

@@ -33 +33 @@
 #include <remote_barrier.h>
 
-#if DEBUG_SYS_BARRIER
 //////////////////////////////////////////////////////
 static char * sys_barrier_op_str( uint32_t operation )
@@ -42 +41 @@
         else                                    return "undefined";
 }
-#endif
 
 //////////////////////////////////
@@ -74 +72 @@
 
 #if DEBUG_SYSCALLS_ERROR
-printk("\n[ERROR] in %s : unmapped barrier %x / thread %x / process %x\n",
-__FUNCTION__ , vaddr , this->trdid , process->pid );
+printk("\n[ERROR] in %s for %s : unmapped barrier %x / thread[%x,%x]\n",
+__FUNCTION__, sys_barrier_op_str(operation), vaddr, process->pid, this->trdid );
 #endif
         this->errno = error;
@@ -94 +92 @@
 
 #if DEBUG_SYSCALLS_ERROR
-printk("\n[ERROR] in %s : unmapped barrier attributes %x / thread %x / process %x\n",
-__FUNCTION__ , attr , this->trdid , process->pid );
+printk("\n[ERROR] in %s for INIT : unmapped barrier attributes %x / thread[%x,%x]\n",
+__FUNCTION__ , attr , process->pid , this->trdid );
 #endif
                     this->errno = EINVAL;
@@ -102 +100 @@
   
                 // copy barrier attributes into kernel space
-                hal_copy_from_uspace( local_cxy,
-                                      &k_attr,
-                                      (void*)attr,
+                hal_copy_from_uspace( XPTR( local_cxy , &k_attr ),
+                                      (void *)attr,
                                       sizeof(pthread_barrierattr_t) );
 
@@ -111 +108 @@
 
 #if DEBUG_SYSCALLS_ERROR
-printk("\n[ERROR] in %s : wrong arguments / count %d / x_size %d / y_size %d / nthreads %x\n",
+printk("\n[ERROR] in %s for INIT : count (%d) != x_size (%d) * y_size (%d) * nthreads (%x)\n",
 __FUNCTION__, count, k_attr.x_size, k_attr.y_size, k_attr.nthreads );
 #endif
@@ -131 +128 @@
 
 #if DEBUG_SYSCALLS_ERROR
-printk("\n[ERROR] in %s : cannot create barrier %x / thread %x / process %x\n",
-__FUNCTION__ , vaddr , this->trdid , process->pid );
+printk("\n[ERROR] in %s for INIT : cannot create barrier %x / thread[%x,%x]\n",
+__FUNCTION__ , vaddr , process->pid , this->trdid );
 #endif
                 this->errno = ENOMEM;
@@ -148 +145 @@
 
 #if DEBUG_SYSCALLS_ERROR
-printk("\n[ERROR] in %s : barrier %x not registered / thread %x / process %x\n",
-__FUNCTION__ , (intptr_t)vaddr , this->trdid , process->pid );
+printk("\n[ERROR] in %s for WAIT : barrier %x not registered / thread[%x,%x]\n",
+__FUNCTION__ , (intptr_t)vaddr , process->pid, this->trdid );
 #endif
                 this->errno = EINVAL;
@@ -169 +166 @@
 
 #if DEBUG_SYSCALLS_ERROR
-printk("\n[ERROR] in %s : barrier %x not registered / thread %x / process %x\n",
-__FUNCTION__ , (intptr_t)vaddr , this->trdid , process->pid );
+printk("\n[ERROR] in %s for DESTROY : barrier %x not registered / thread[%x,%x]\n",
+__FUNCTION__ , (intptr_t)vaddr , process->pid, this->trdid );
 #endif
                 this->errno = EINVAL;