diff options
author | ed <ed@FreeBSD.org> | 2008-09-16 14:57:23 +0000 |
---|---|---|
committer | ed <ed@FreeBSD.org> | 2008-09-16 14:57:23 +0000 |
commit | 0f8f4f624b9601d63a980ccf5c07f3e3109f98cd (patch) | |
tree | 3189f895c04a095e22df6f0c2d9b07814761a05f /sys/kern/kern_proc.c | |
parent | f145085b0dbbd1162ad9ad4972700926cb81df54 (diff) | |
download | FreeBSD-src-0f8f4f624b9601d63a980ccf5c07f3e3109f98cd.zip FreeBSD-src-0f8f4f624b9601d63a980ccf5c07f3e3109f98cd.tar.gz |
Fix minor TTY API inconsistency.
Unlike tty_rel_gone() and tty_rel_sess(), the tty_rel_pgrp() routine
does not unlock the TTY. I once had the idea to make the code call
tty_rel_pgrp() and tty_rel_sess(), picking up the TTY lock once. This
turned out a little harder than I expected, so this is how it works now.
It's a lot easier if we just let tty_rel_pgrp() unlock the TTY, because
the other routines do this anyway.
Diffstat (limited to 'sys/kern/kern_proc.c')
-rw-r--r-- | sys/kern/kern_proc.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/sys/kern/kern_proc.c b/sys/kern/kern_proc.c index 55f7e18..e45a489 100644 --- a/sys/kern/kern_proc.c +++ b/sys/kern/kern_proc.c @@ -506,7 +506,6 @@ pgdelete(pgrp) if (tp != NULL) { tty_lock(tp); tty_rel_pgrp(tp, pgrp); - tty_unlock(tp); } mtx_destroy(&pgrp->pg_mtx); |