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

Timestamp:

Nov 10, 2018, 2:33:26 PM (6 years ago)

Author:

alain

Message:

Fix various bugs in sys_stat() and sys_mmap() functions.
Improve debug in other functions.

File:

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

trunk/kernel/syscalls/sys_thread_create.c

@@ -62 +62 @@
         process    = parent->process;
 
+#if (DEBUG_SYS_THREAD_CREATE || CONFIG_INSTRUMENTATION_SYSCALLS)
+uint64_t     tm_start = hal_get_cycles();
+#endif
+
 #if DEBUG_SYS_THREAD_CREATE
-uint64_t tm_start;
-uint64_t tm_end;
 tm_start = hal_get_cycles();
 if( DEBUG_SYS_THREAD_CREATE < tm_start )
-printk("\n[DBG] %s : thread %x in process %x enter / cycle %d\n",
-__FUNCTION__, parent->trdid, process->pid, (uint32_t)tm_start );
+printk("\n[%s] thread[%x,%x] enter / cycle %d\n",
+__FUNCTION__, process_pid, parent->trdid, (uint32_t)tm_start );
 #endif
 
@@ -78 +80 @@
 
 #if DEBUG_SYSCALLS_ERROR
-printk("\n[ERROR] in %s : thread %x in process %x / trdid buffer %x unmapped %x\n",
-__FUNCTION__, parent->trdid, process->pid, (intptr_t)trdid_ptr );
+printk("\n[ERROR] in %s : thread[%x,%x] / trdid buffer %x unmapped %x\n",
+__FUNCTION__, process->pid, parent->trdid, (intptr_t)trdid_ptr );
 vmm_display( process , false );
 #endif
@@ -95 +97 @@
 
 #if DEBUG_SYSCALLS_ERROR
-printk("\n[ERROR] in %s : thread %x in process %x / user_attr buffer unmapped %x\n",
-__FUNCTION__, parent->trdid, process->pid, (intptr_t)user_attr );
+printk("\n[ERROR] in %s : thread[%x,%x] / user_attr buffer unmapped %x\n",
+__FUNCTION__, process->pid, parent->trdid, (intptr_t)user_attr );
 vmm_display( process , false );
 #endif
@@ -113 +115 @@
 
 #if DEBUG_SYSCALLS_ERROR
-printk("\n[ERROR] in %s : thread %x in process %x / start_func unmapped %x\n",
-__FUNCTION__, parent->trdid, process->pid, (intptr_t)start_func );
+printk("\n[ERROR] in %s : thread[%x,%x] / start_func unmapped %x\n",
+__FUNCTION__, process->pid, parent->trdid, (intptr_t)start_func );
 vmm_display( process , false );
 #endif
@@ -130 +132 @@
 
 #if DEBUG_SYSCALLS_ERROR
-printk("\n[ERROR] in %s : thread %x in process %x / start_args buffer unmapped %x\n",
-__FUNCTION__, parent->trdid, process->pid, (intptr_t)start_args );
+printk("\n[ERROR] in %s : thread[%x,%x] / start_args buffer unmapped %x\n",
+__FUNCTION__, process->pid, parent->trdid, (intptr_t)start_args );
 vmm_display( process , false );
 #endif
@@ -149 +151 @@
 
 #if DEBUG_SYSCALLS_ERROR
-printk("\n[ERROR] in %s : thread %x in process %x / illegal target cluster %x\n",
-__FUNCTION__, parent->trdid, process->pid, kern_attr.cxy );
+printk("\n[ERROR] in %s : thread[%x,%x] / illegal target cluster %x\n",
+__FUNCTION__, process->pid, parent->trdid, kern_attr.cxy );
 #endif
                             parent->errno = EINVAL;
@@ -200 +202 @@
 
 #if DEBUG_SYSCALLS_ERROR
-printk("\n[ERROR] in %s : thread %x in process %x cannot create new thread\n",
-__FUNCTION__ , parent->trdid, process->pid );
+printk("\n[ERROR] in %s : thread[%x,%x] cannot create new thread\n",
+__FUNCTION__ , process->pid, parent->trdid );
 #endif
                 parent->errno = ENOMEM;
@@ -216 +218 @@
     hal_fence();
 
+#if (DEBUG_SYS_THREAD_CREATE || CONFIG_INSTRUMENTATION_SYSCALLS)
+uint64_t     tm_end = hal_get_cycles();
+#endif
+
 #if DEBUG_SYS_THREAD_CREATE
-tm_end = hal_get_cycles();
 if( DEBUG_SYS_THREAD_CREATE < tm_end )
-printk("\n[DBG] %s : thread %x in process %x created thread %x / cycle %d\n",
-__FUNCTION__, parent->trdid, process->pid, child_ptr->trdid, (uint32_t)tm_end );
+printk("\n[%s] thread[%x,%x] created thread %x / cycle %d\n",
+__FUNCTION__, process->pid, parent->trdid, child_ptr->trdid, (uint32_t)tm_end );
+#endif
+
+#if CONFIG_INSTRUMENTATION_SYSCALLS
+hal_atomic_add( &syscalls_cumul_cost[SYS_THREAD_CREATE] , tm_end - tm_start );
+hal_atomic_add( &syscalls_occurences[SYS_THREAD_CREATE] , 1 );
 #endif