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

In kse_release(), check if current thread is bound

Browse Source 
and current kse mailbox was already initialized, also
prevent last thread from exiting unless we figure out
how to safely support null thread proc.
This commit is contained in:
David Xu 2002-11-14 06:06:45 +00:00
parent e3c29144fe
commit ca161eb6e9
2 changed files with 32 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
sys/kern/kern_kse.c
View File

@ -341,12 +341,24 @@ kse_release(struct thread *td, struct kse_release_args *uap)
struct proc *p;
p = td->td_proc;
/* KSE-enabled processes only, please. */
if (p->p_flag & P_KSES) {
/* KSE-enabled processes only */
if (!(p->p_flag & P_KSES))
return (EINVAL);
/*
* Must be a bound thread. And kse must have a mailbox ready,
* if not, the kse would can not generate an upcall.
*/
if (!(td->td_flags & TDF_UNBOUND) && (td->td_kse->ke_mailbox != NULL)) {
PROC_LOCK(p);
mtx_lock_spin(&sched_lock);
thread_exit();
/* NOTREACHED */
/* prevent last thread from exiting */
if (p->p_numthreads > 1) {
thread_exit();
/* NOTREACHED */
} else {
mtx_unlock_spin(&sched_lock);
PROC_UNLOCK(p);
}
}
return (EINVAL);
}

20
sys/kern/kern_thread.c
View File

@ -341,12 +341,24 @@ kse_release(struct thread *td, struct kse_release_args *uap)
struct proc *p;
p = td->td_proc;
/* KSE-enabled processes only, please. */
if (p->p_flag & P_KSES) {
/* KSE-enabled processes only */
if (!(p->p_flag & P_KSES))
return (EINVAL);
/*
* Must be a bound thread. And kse must have a mailbox ready,
* if not, the kse would can not generate an upcall.
*/
if (!(td->td_flags & TDF_UNBOUND) && (td->td_kse->ke_mailbox != NULL)) {
PROC_LOCK(p);
mtx_lock_spin(&sched_lock);
thread_exit();
/* NOTREACHED */
/* prevent last thread from exiting */
if (p->p_numthreads > 1) {
thread_exit();
/* NOTREACHED */
} else {
mtx_unlock_spin(&sched_lock);
PROC_UNLOCK(p);
}
}
return (EINVAL);
}