Changeset 619 for trunk/kernel/kern/process.c

Timestamp:

Feb 12, 2019, 1:15:47 PM (5 years ago)

Author:

alain

Message:

1) Fix a bug in KSH : after the "load" command,

the [ksh] prompt is now printed after completion
of the loaded application.

2) Fix a bug in vmm_handle_cow() : the copy-on-write

use now a hal_remote_memcpy() to replicate the page content.

File:

• trunk/kernel/kern/process.c (modified) (7 diffs)

trunk/kernel/kern/process.c

@@ -528 +528 @@
     process_t        * process_ptr;       // local pointer on process copy
     reg_t              save_sr;           // for critical section
-    rpc_desc_t         rpc;               // shared RPC descriptor
     thread_t         * client;            // pointer on client thread
     xptr_t             client_xp;         // extended pointer on client thread
     process_t        * local;             // pointer on process copy in local cluster
     uint32_t           remote_nr;         // number of remote process copies
+    rpc_desc_t         rpc;               // shared RPC descriptor
+    uint32_t           responses;         // shared RPC responses counter
 
     client    = CURRENT_THREAD;
@@ -579 +580 @@
     thread_block( client_xp , THREAD_BLOCKED_RPC );
 
+    // initialize RPC responses counter
+    responses = 0;
+
     // initialize shared RPC descriptor
-    rpc.responses = 0;
+    // can be shared, because no out arguments
+    rpc.rsp       = &responses;
     rpc.blocking  = false;
     rpc.index     = RPC_PROCESS_SIGACTION;
@@ -608 +613 @@
             remote_nr++;
 
-            // atomically increment responses counter
-            hal_atomic_add( (void *)&rpc.responses , 1 );
+            // atomically increment RPC responses counter
+            hal_atomic_add( &responses , 1 );
 
 #if DEBUG_PROCESS_SIGACTION 
@@ -617 +622 @@
 #endif
             // call RPC in target cluster
-            rpc_process_sigaction_client( process_cxy , &rpc );
+            rpc_send( process_cxy , &rpc );
         }
     }  // end list of copies
@@ -685 +690 @@
 
 // check target process is an user process
-assert( (LPID_FROM_PID( process->pid ) != 0 ), "target process must be an user process" );
+assert( (LPID_FROM_PID( process->pid ) != 0 ),
+"process %x is not an user process\n", process->pid );
 
     // get target process owner cluster
@@ -773 +779 @@
 
 // check target process is an user process
-assert( (LPID_FROM_PID( process->pid ) != 0), "process %x not an user process", process->pid );
+assert( (LPID_FROM_PID( process->pid ) != 0),
+"process %x is not an user process\n", process->pid );
 
     // get lock protecting process th_tbl[]
@@ -831 +838 @@
 
 // check target process is an user process
-assert( ( process->pid != 0 ),
-"target process must be an user process" );
+assert( ( LPID_FROM_PID( process->pid ) != 0 ),
+"process %x is not an user process\n", process->pid );
 
     // get lock protecting process th_tbl[]