- Copy signal mask out before THR_UNLOCK(), because THR_UNLOCK() may call
_thr_suspend_check() which messes sigmask saved in thread structure. - Don't suspend a thread has force_exit set. - In pthread_exit(), if there is a suspension flag set, wake up waiting- thread after setting PS_DEAD, this causes waiting-thread to break loop in suspend_common().
This commit is contained in:
parent
965a19847a
commit
9dbfb036ab
@ -247,6 +247,11 @@ create_stack(struct pthread_attr *pattr)
|
||||
static void
|
||||
thread_start(struct pthread *curthread)
|
||||
{
|
||||
sigset_t set;
|
||||
|
||||
if (curthread->attr.suspend == THR_CREATE_SUSPENDED)
|
||||
set = curthread->sigmask;
|
||||
|
||||
/*
|
||||
* This is used as a serialization point to allow parent
|
||||
* to report 'new thread' event to debugger or tweak new thread's
|
||||
@ -263,19 +268,20 @@ thread_start(struct pthread *curthread)
|
||||
|
||||
SIGEMPTYSET(set);
|
||||
SIGADDSET(set, SIGCANCEL);
|
||||
sigprocmask(SIG_UNBLOCK, &set, NULL);
|
||||
__sys_sigprocmask(SIG_UNBLOCK, &set, NULL);
|
||||
}
|
||||
|
||||
if (curthread->attr.suspend == THR_CREATE_SUSPENDED) {
|
||||
sigset_t set = curthread->sigmask;
|
||||
|
||||
#if 0
|
||||
/* Done in THR_UNLOCK() */
|
||||
_thr_ast(curthread);
|
||||
#endif
|
||||
|
||||
/*
|
||||
* Parent thread have stored signal mask for us,
|
||||
* we should restore it now.
|
||||
*/
|
||||
sigprocmask(SIG_SETMASK, &set, NULL);
|
||||
__sys_sigprocmask(SIG_SETMASK, &set, NULL);
|
||||
}
|
||||
|
||||
/* Run the current thread's start routine with argument: */
|
||||
|
@ -121,6 +121,10 @@ _pthread_exit(void *status)
|
||||
}
|
||||
THR_LOCK(curthread);
|
||||
curthread->state = PS_DEAD;
|
||||
if (curthread->flags & THR_FLAGS_NEED_SUSPEND) {
|
||||
curthread->cycle++;
|
||||
_thr_umtx_wake(&curthread->cycle, INT_MAX);
|
||||
}
|
||||
THR_UNLOCK(curthread);
|
||||
/*
|
||||
* Thread was created with initial refcount 1, we drop the
|
||||
|
@ -92,6 +92,9 @@ _thr_suspend_check(struct pthread *curthread)
|
||||
long cycle;
|
||||
int err;
|
||||
|
||||
if (curthread->force_exit)
|
||||
return;
|
||||
|
||||
err = errno;
|
||||
/*
|
||||
* Blocks SIGCANCEL which other threads must send.
|
||||
|
Loading…
x
Reference in New Issue
Block a user