summaryrefslogtreecommitdiff
path: root/kernel/proc.c
diff options
context:
space:
mode:
authorFrans Kaashoek <[email protected]>2022-08-22 19:58:33 -0400
committerFrans Kaashoek <[email protected]>2022-08-22 19:58:33 -0400
commit2f0b4d698bc48f4efb4b9fdca20f415ec765bf41 (patch)
tree72650caac5471845c4f64d8185b88141ab2817ec /kernel/proc.c
parent2a391ebc8ba1fd0e6f0899277218d531fd5c7396 (diff)
downloadxv6-labs-2f0b4d698bc48f4efb4b9fdca20f415ec765bf41.tar.gz
xv6-labs-2f0b4d698bc48f4efb4b9fdca20f415ec765bf41.tar.bz2
xv6-labs-2f0b4d698bc48f4efb4b9fdca20f415ec765bf41.zip
Use pp instead of np to be more consistent
Diffstat (limited to 'kernel/proc.c')
-rw-r--r--kernel/proc.c24
1 files changed, 12 insertions, 12 deletions
diff --git a/kernel/proc.c b/kernel/proc.c
index 3bb2618..1ed3ee5 100644
--- a/kernel/proc.c
+++ b/kernel/proc.c
@@ -390,7 +390,7 @@ exit(int status)
int
wait(uint64 addr)
{
- struct proc *np;
+ struct proc *pp;
int havekids, pid;
struct proc *p = myproc();
@@ -399,27 +399,27 @@ wait(uint64 addr)
for(;;){
// Scan through table looking for exited children.
havekids = 0;
- for(np = proc; np < &proc[NPROC]; np++){
- if(np->parent == p){
+ for(pp = proc; pp < &proc[NPROC]; pp++){
+ if(pp->parent == p){
// make sure the child isn't still in exit() or swtch().
- acquire(&np->lock);
+ acquire(&pp->lock);
havekids = 1;
- if(np->state == ZOMBIE){
+ if(pp->state == ZOMBIE){
// Found one.
- pid = np->pid;
- if(addr != 0 && copyout(p->pagetable, addr, (char *)&np->xstate,
- sizeof(np->xstate)) < 0) {
- release(&np->lock);
+ pid = pp->pid;
+ if(addr != 0 && copyout(p->pagetable, addr, (char *)&pp->xstate,
+ sizeof(pp->xstate)) < 0) {
+ release(&pp->lock);
release(&wait_lock);
return -1;
}
- freeproc(np);
- release(&np->lock);
+ freeproc(pp);
+ release(&pp->lock);
release(&wait_lock);
return pid;
}
- release(&np->lock);
+ release(&pp->lock);
}
}