d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove the leaderp variable and just access p_leader directly. The

Browse Source 
p_leader field is not protected by the proc lock but is only set during
fork1() by the parent process and never changes.
This commit is contained in:
jhb 2002-10-15 00:03:40 +00:00
parent f8f30374dd
commit db8a406c24
1 changed files with 3 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
sys/kern/kern_descrip.c
View File

@ -258,7 +258,6 @@ kern_fcntl(struct thread *td, int fd, int cmd, intptr_t arg)
struct flock *flp;
int tmp, error = 0, flg = F_POSIX;
u_int newmin;
struct proc *leaderp;
mtx_lock(&Giant);
@ -379,9 +378,8 @@ kern_fcntl(struct thread *td, int fd, int cmd, intptr_t arg)
}
PROC_LOCK(p);
p->p_flag |= P_ADVLOCK;
leaderp = p->p_leader;
PROC_UNLOCK(p);
error = VOP_ADVLOCK(vp, (caddr_t)leaderp, F_SETLK,
error = VOP_ADVLOCK(vp, (caddr_t)p->p_leader, F_SETLK,
flp, flg);
break;
case F_WRLCK:
@ -389,18 +387,11 @@ kern_fcntl(struct thread *td, int fd, int cmd, intptr_t arg)
error = EBADF;
break;
}
PROC_LOCK(p);
p->p_flag |= P_ADVLOCK;
leaderp = p->p_leader;
PROC_UNLOCK(p);
error = VOP_ADVLOCK(vp, (caddr_t)leaderp, F_SETLK, flp,
error = VOP_ADVLOCK(vp, (caddr_t)p->p_leader, F_SETLK, flp,
flg);
break;
case F_UNLCK:
PROC_LOCK(p);
leaderp = p->p_leader;
PROC_UNLOCK(p);
error = VOP_ADVLOCK(vp, (caddr_t)leaderp, F_UNLCK, flp,
error = VOP_ADVLOCK(vp, (caddr_t)p->p_leader, F_UNLCK, flp,
F_POSIX);
break;
default: