Changeset 670 for trunk/kernel/syscalls/sys_chmod.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_chmod.c (modified) (5 diffs)

trunk/kernel/syscalls/sys_chmod.c

@@ -2 +2 @@
  * sys_chmod.c - Change file access rights.
  *
- * Author    Alain Greiner  (2016,2017,2018,2019)
+ * Author       Alain Greiner  (2016,2017,2018,2019,2020)
  *
  * Copyright (c) 2015 UPMC Sorbonne Universites
@@ -29 +29 @@
 #include <thread.h>
 #include <process.h>
-
 #include <syscalls.h>
 
@@ -36 +35 @@
                uint32_t   rights __attribute__((unused)) )
 {
-    error_t     error;
     char        kbuf[CONFIG_VFS_MAX_PATH_LENGTH];
 
     thread_t  * this    = CURRENT_THREAD;
-    process_t * process = this->process;
+
+#if DEBUG_SYS_CHMOD || DEBUG_SYSCALLS_ERROR || CONFIG_INSTRUMENTATION_SYSCALLS
+process_t * process  = this->process;
+uint64_t    tm_start = hal_get_cycles();
+#endif
 
     // check pathname length
@@ -47 +49 @@
 
 #if DEBUG_SYSCALLS_ERROR
-printk("\n[ERROR] in %s : pathname too long / thread %x in process %x\n",
+if( DEBUG_SYSCALLS_ERROR < (uint32_t)tm_start )
+printk("\n[ERROR] in %s : thread[%x,%x] / pathname too long\n",
 __FUNCTION__, this->trdid, process->pid );
 #endif
@@ -59 +62 @@
                             CONFIG_VFS_MAX_PATH_LENGTH );
 
+
+
+
     printk("\n[ERROR] in %s : not implemented yet\n", __FUNCTION__ );
     return -1;
 
-    // get cluster and local pointer on reference process
-    // xptr_t      ref_xp  = process->ref_xp;
-    // process_t * ref_ptr = (process_t *)GET_PTR( ref_xp );
-    // cxy_t       ref_cxy = GET_CXY( ref_xp );
 
-    // call the relevant VFS function
-    // error = vfs_chmod( cwd_xp,
-    //                    kbuf,
-    //                    rights );
 
-    if( error )
-    {
-        printk("\n[ERROR] in %s : cannot remove directory %s\n",
-        __FUNCTION__ , kbuf );
-        this->errno = error;
-        return -1;
-    }
+
+    hal_fence();
+
+#if (DEBUG_SYS_CHMOD || CONFIG_INSTRUMENTATION_SYSCALLS)
+uint64_t     tm_end = hal_get_cycles();
+#endif
+
+#if DEBUG_SYS_CHMOD
+if( DEBUG_SYS_CHMOD < tm_end )
+printk("\n[%s] thread[%x,%x] exit for / cycle %d\n",
+__FUNCTION__, process->pid, this->trdid, (uint32_t)tm_end );
+#endif
+ 
+#if CONFIG_INSTRUMENTATION_SYSCALLS
+hal_atomic_add( &syscalls_cumul_cost[SYS_CHMOD] , tm_end - tm_start );
+hal_atomic_add( &syscalls_occurences[SYS_CHMOD] , 1 );
+#endif
 
     return 0;