Changeset 637 for trunk/kernel/syscalls/sys_mkfifo.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_mkfifo.c (modified) (5 diffs)

trunk/kernel/syscalls/sys_mkfifo.c

@@ -2 +2 @@
  * sys_mkfifo.c - creates a named FIFO file.
  *
- * Author    Alain Greiner (2016,2017)
+ * Author    Alain Greiner (2016,2017,2018,2019)
  *
  * Copyright (c) UPMC Sorbonne Universites
@@ -33 +33 @@
                  uint32_t  mode __attribute__((unused)) )
 {
-    error_t        error;
     char           kbuf[CONFIG_VFS_MAX_PATH_LENGTH];
 
@@ -39 +38 @@
     process_t * process = this->process;
 
+#if (DEBUG_SYS_MKFIFO || CONFIG_INSTRUMENTATION_SYSCALLS)
+uint64_t     tm_start = hal_get_cycles();
+#endif
+
+#if DEBUG_SYS_MKFIFO
+if( DEBUG_SYS_MKFIFO < tm_end )
+printk("\n[%s] thread[%x,%x] enter for <%s> / cycle %d\n",
+__FUNCTION__, process->pid, this->trdid, pathname, (uint32_t)tm_end );
+#endif
+ 
     // check fd_array not full
     if( process_fd_array_full() )
     {
-        printk("\n[ERROR] in %s : file descriptor array full for process %x\n",
-               __FUNCTION__ , process->pid );
+
+#if DEBUG_SYSCALLS_ERROR
+printk("\n[ERROR] in %s : file descriptor array full for process %x\n",
+__FUNCTION__ , process->pid );
+#endif
         this->errno = ENFILE;
         return -1;
@@ -51 +63 @@
     if( hal_strlen_from_uspace( pathname ) >= CONFIG_VFS_MAX_PATH_LENGTH )
     {
-        printk("\n[ERROR] in %s : pathname too long\n", __FUNCTION__ );
+
+#if DEBUG_SYSCALLS_ERROR
+printk("\n[ERROR] in %s : pathname too long\n", __FUNCTION__ );
+#endif
         this->errno = ENFILE;
         return -1;
@@ -57 +72 @@
 
     // copy pathname in kernel space
-    hal_strcpy_from_uspace( kbuf , pathname , CONFIG_VFS_MAX_PATH_LENGTH );
+    hal_strcpy_from_uspace( XPTR( local_cxy , kbuf ),
+                            pathname,
+                            CONFIG_VFS_MAX_PATH_LENGTH );
 
     printk("\n[ERROR] in %s : not implemented yet\n", __FUNCTION__ );
     return -1;
 
-    if( error )
-    {
-        printk("\n[ERROR] in %s : cannot create named FIFO %s\n",
-               __FUNCTION__ , kbuf );
-        this->errno = error;
-        return -1;
-    }
+#if (DEBUG_SYS_MKFIFO || CONFIG_INSTRUMENTATION_SYSCALLS)
+uint64_t     tm_end = hal_get_cycles();
+#endif
 
-    return 0;
+#if DEBUG_SYS_MKFIFO
+if( DEBUG_SYS_MKFIFO < tm_end )
+printk("\n[%s] thread[%x,%x] exit for <%s> / cycle %d\n",
+__FUNCTION__, process->pid, this->trdid, pathname, (uint32_t)tm_end );
+#endif
+ 
+#if CONFIG_INSTRUMENTATION_SYSCALLS
+hal_atomic_add( &syscalls_cumul_cost[SYS_MKFIFO] , tm_end - tm_start );
+hal_atomic_add( &syscalls_occurences[SYS_MKFIFO] , 1 );
+#endif
 
 } // end sys_mkfifo()