Changeset 625 for trunk/kernel/syscalls/sys_get_config.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_get_config.c (modified) (5 diffs)

trunk/kernel/syscalls/sys_get_config.c

@@ -2 +2 @@
  * sys_get_config.c - get hardware platform parameters.
  * 
- * Author    Alain Greiner (2016,2017,2018)
+ * Author    Alain Greiner (2016,2017,2018,2019)
  *  
  * Copyright (c) UPMC Sorbonne Universites
@@ -24 +24 @@
 #include <hal_kernel_types.h>
 #include <hal_uspace.h>
+#include <hal_vmm.h>
 #include <hal_special.h>
 #include <errno.h>
@@ -48 +49 @@
     process_t * process = this->process;
 
+#if (DEBUG_SYS_GET_CONFIG || CONFIG_INSTRUMENTATION_SYSCALLS)
+uint64_t     tm_start = hal_get_cycles();
+#endif
+
 #if DEBUG_SYS_GET_CONFIG
-uint64_t     tm_start;
-uint64_t     tm_end;
 tm_start = hal_get_cycles();
 if( DEBUG_SYS_GET_CONFIG < tm_start )
@@ -114 +117 @@
     hal_fence();
 
+#if (DEBUG_SYS_GET_CONFIG || CONFIG_INSTRUMENTATION_SYSCALLS)
+uint64_t     tm_end = hal_get_cycles();
+#endif
+
 #if DEBUG_SYS_GET_CONFIG
-tm_end = hal_get_cycles();
 if( DEBUG_SYS_GET_CONFIG < tm_end )
 printk("\n[DBG] %s : thread %x exit / process %x / cost %d / cycle %d\n",
@@ -121 +127 @@
 #endif
 
+#if CONFIG_INSTRUMENTATION_SYSCALLS
+hal_atomic_add( &syscalls_cumul_cost[SYS_GET_CONFIG] , tm_end - tm_start );
+hal_atomic_add( &syscalls_occurences[SYS_GET_CONFIG] , 1 );
+#endif
+
         return 0;