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

Timestamp:

Apr 10, 2019, 10:09:39 AM (5 years ago)

Author:

alain

Message:

Fix a bug in the vmm_remove_vseg() function: the physical pages
associated to an user DATA vseg were released to the kernel when
the target process descriptor was in the reference cluster.
This physical pages release should be done only when the page
forks counter value is zero.
All other modifications are cosmetic.

File:

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

trunk/kernel/syscalls/sys_barrier.c

@@ -2 +2 @@
  * sys_barrier.c - Access a POSIX barrier.
  * 
- * authors       Alain Greiner (2016,2017,2018)
+ * authors       Alain Greiner (2016,2017,2018,2019)
  *
  * Copyright (c) UPMC Sorbonne Universites
@@ -25 +25 @@
 #include <hal_special.h>
 #include <hal_uspace.h>
+#include <hal_vmm.h>
 #include <errno.h>
 #include <thread.h>
@@ -56 +57 @@
     process_t * process = this->process;
 
+#if (DEBUG_SYS_BARRIER || CONFIG_INSTRUMENTATION_SYSCALLS)
+uint64_t     tm_start = hal_get_cycles();
+#endif
+
 #if DEBUG_SYS_BARRIER
-uint64_t   tm_start;
-uint64_t   tm_end;
-tm_start = hal_get_cycles();
 if( DEBUG_SYS_BARRIER < tm_start )
 printk("\n[%s] thread[%x,%x] enters for %s / count %d / cycle %d\n",
@@ -184 +186 @@
         }  // end switch
 
+    hal_fence();
+
+#if (DEBUG_SYS_BARRIER || CONFIG_INSTRUMENTATION_SYSCALLS)
+uint64_t     tm_end = hal_get_cycles();
+#endif
+
 #if DEBUG_SYS_BARRIER
-tm_end = hal_get_cycles();
 if( DEBUG_SYS_BARRIER < tm_end )
-printk("\n[%s] thread[%x,%x] exit for %s / cost %d / cycle %d\n",
-__FUNCTION__, process->pid, this->trdid, sys_barrier_op_str(operation), 
-(uint32_t)(tm_end - tm_start), (uint32_t)tm_end );
+printk("\n[%s] thread[%x,%x] exit for %s / cycle %d\n",
+__FUNCTION__, process->pid, this->trdid, sys_barrier_op_str(operation), (uint32_t)tm_end );
+#endif
+
+#if CONFIG_INSTRUMENTATION_SYSCALLS
+hal_atomic_add( &syscalls_cumul_cost[SYS_BARRIER] , tm_end - tm_start );
+hal_atomic_add( &syscalls_occurences[SYS_BARRIER] , 1 );
 #endif