xfnw/plan9fox
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

kernel: improve diagnostics by reversing the roles of Proc.parentpid and Proc.parent

Browse source 
for better system diagnostics, we *ALWAYS* want to record the parent
pid of a user process, regardless of if the child will post a wait
record on exit or not.

for that, we reverse the roles of Proc.parent and Proc.parentpid so
Proc.parentpid will always be set on rfork() and the Proc.parent
pointer will point to the parent's Proc structure or is set to nil
when no wait record should be posted on exit (RFNOWAIT flag).

this means that we can get the pid of the original parent process
from /proc, regardless of the the child having rforked with the
RFNOWAIT flag. this improves the output of pstree(1) somewhat if
the parent is still alive. note that theres no guarantee that the
parent pid is still valid.

the conditions are unchanged:

a user process that will post wait record has:

up->kp == 0 && up->parent != nil && up->parent->pid == up->parentpid

the boot process is:

up->kp == 0 && up->parent == nil && up->parentpid == 0

and kproc's have:

up->kp != 0 && up->parent == nil && up->parentpid == 0
This commit is contained in:
cinap_lenrek 2019-12-01 17:57:14 +01:00
parent 21a599743e
commit 08d6b0f043
2 changed files with 12 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
sys/src/9/port/proc.c
View file

@ -1130,7 +1130,14 @@ pexit(char *exitstr, int freemem)
* if not a kernel process and have a parent,
* do some housekeeping.
*/
if(up->kp == 0 && up->parentpid != 0) {
if(up->kp)
goto Nowait;
p = up->parent;
if(p != nil){
if(p->pid != up->parentpid || p->state == Broken)
goto Nowait;
wq = smalloc(sizeof(Waitq));
wq->w.pid = up->pid;
utime = up->time[TUser] + up->time[TCUser];
@ -1143,7 +1150,6 @@ pexit(char *exitstr, int freemem)
else
wq->w.msg[0] = '\0';
p = up->parent;
lock(&p->exl);
/*
* Check that parent is still alive.
@ -1171,12 +1177,13 @@ pexit(char *exitstr, int freemem)
if(wq != nil)
free(wq);
}
else if(up->kp == 0 && up->parent == nil){
else if(up->parentpid == 0){
if(exitstr == nil)
exitstr = "unknown";
panic("boot process died: %s", exitstr);
}
Nowait:
if(!freemem)
addbroken(up);

4
sys/src/9/port/sysproc.c
View file

@ -181,9 +181,9 @@ sysrfork(va_list list)
*/
forkchild(p, up->dbgreg);
p->parent = up;
p->parentpid = up->pid;
if((flag&RFNOWAIT) == 0){
p->parentpid = up->pid;
p->parent = up;
lock(&up->exl);
up->nchild++;
unlock(&up->exl);