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

Timestamp:

Nov 19, 2020, 11:45:52 PM (3 years ago)

Author:

alain

Message:

1) Introduce up to 4 command lines arguments in the KSH "load" command.
These arguments are transfered to the user process through the
argc/argv mechanism, using the user space "args" vseg.

2) Introduce the named and anonymous "pipes", for inter-process communication
through the pipe() and mkfifo() syscalls.

3) Introduce the "chat" application to validate the two above mechanisms.

4) Improve printk() and assert() fonctions in printk.c.

File:

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

trunk/kernel/syscalls/sys_barrier.c

@@ -33 +33 @@
 #include <remote_barrier.h>
 
-//////////////////////////////////////////////////////
-static char * sys_barrier_op_str( uint32_t operation )
-{
-        if     ( operation == BARRIER_INIT    ) return "INIT";
-        else if( operation == BARRIER_DESTROY ) return "DESTROY";
-        else if( operation == BARRIER_WAIT    ) return "WAIT";
-        else                                    return "undefined";
-}
-
 //////////////////////////////////
 int sys_barrier( intptr_t   vaddr,
@@ -55 +46 @@
     process_t * process = this->process;
 
-#if (DEBUG_SYS_BARRIER || CONFIG_INSTRUMENTATION_SYSCALLS)
+#if DEBUG_SYS_BARRIER || DEBUG_SYSCALLS_ERROR || CONFIG_INSTRUMENTATION_SYSCALLS
 uint64_t     tm_start = hal_get_cycles();
 #endif
@@ -72 +63 @@
 
 #if DEBUG_SYSCALLS_ERROR
+if( DEBUG_SYSCALLS_ERROR < (uint32_t)tm_start ) 
 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 );
@@ -92 +84 @@
 
 #if DEBUG_SYSCALLS_ERROR
+if( DEBUG_SYSCALLS_ERROR < (uint32_t)tm_start ) 
 printk("\n[ERROR] in %s for INIT : unmapped barrier attributes %x / thread[%x,%x]\n",
 __FUNCTION__ , attr , process->pid , this->trdid );
@@ -108 +101 @@
 
 #if DEBUG_SYSCALLS_ERROR
+if( DEBUG_SYSCALLS_ERROR < (uint32_t)tm_start ) 
 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 );
@@ -128 +122 @@
 
 #if DEBUG_SYSCALLS_ERROR
+if( DEBUG_SYSCALLS_ERROR < (uint32_t)tm_start ) 
 printk("\n[ERROR] in %s for INIT : cannot create barrier %x / thread[%x,%x]\n",
 __FUNCTION__ , vaddr , process->pid , this->trdid );
@@ -145 +140 @@
 
 #if DEBUG_SYSCALLS_ERROR
+if( DEBUG_SYSCALLS_ERROR < (uint32_t)tm_start ) 
 printk("\n[ERROR] in %s for WAIT : barrier %x not registered / thread[%x,%x]\n",
 __FUNCTION__ , (intptr_t)vaddr , process->pid, this->trdid );
@@ -166 +162 @@
 
 #if DEBUG_SYSCALLS_ERROR
+if( DEBUG_SYSCALLS_ERROR < (uint32_t)tm_start ) 
 printk("\n[ERROR] in %s for DESTROY : barrier %x not registered / thread[%x,%x]\n",
 __FUNCTION__ , (intptr_t)vaddr , process->pid, this->trdid );
@@ -181 +178 @@
         default:
         {
-            assert ( false, "illegal operation type <%x>", operation );
+            assert( __FUNCTION__, false, "illegal operation type <%x>", operation );
         }
         }  // end switch