mirror of
https://github.com/fluencelabs/musl
synced 2025-06-28 14:11:56 +00:00
major improvements to cancellation handling
- there is no longer any risk of spoofing cancellation requests, since the cancel flag is set in pthread_cancel rather than in the signal handler. - cancellation signal is no longer unblocked when running the cancellation handlers. instead, pthread_create will cause any new threads created from a cancellation handler to unblock their own cancellation signal. - various tweaks in preparation for POSIX timer support.
This commit is contained in:
@ -18,6 +18,7 @@ struct __libc {
|
|||||||
FILE *ofl_head;
|
FILE *ofl_head;
|
||||||
int ofl_lock;
|
int ofl_lock;
|
||||||
void (*lockfile)(FILE *);
|
void (*lockfile)(FILE *);
|
||||||
|
void (*sigtimer)();
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
||||||
|
@ -41,10 +41,14 @@ struct pthread {
|
|||||||
long off;
|
long off;
|
||||||
void *pending;
|
void *pending;
|
||||||
} robust_list;
|
} robust_list;
|
||||||
|
int unblock_cancel;
|
||||||
};
|
};
|
||||||
|
|
||||||
struct __timer {
|
struct __timer {
|
||||||
struct sigevent sev;
|
int timerid;
|
||||||
|
union sigval val;
|
||||||
|
void (*notify)(union sigval);
|
||||||
|
pthread_t thread;
|
||||||
};
|
};
|
||||||
|
|
||||||
#define __SU (sizeof(size_t)/sizeof(int))
|
#define __SU (sizeof(size_t)/sizeof(int))
|
||||||
@ -73,7 +77,6 @@ struct __timer {
|
|||||||
|
|
||||||
#define SIGCANCEL 32
|
#define SIGCANCEL 32
|
||||||
#define SIGSYSCALL 33
|
#define SIGSYSCALL 33
|
||||||
#define SIGTIMER 32 /* ?? */
|
|
||||||
|
|
||||||
int __set_thread_area(void *);
|
int __set_thread_area(void *);
|
||||||
int __libc_sigaction(int, const struct sigaction *, struct sigaction *);
|
int __libc_sigaction(int, const struct sigaction *, struct sigaction *);
|
||||||
|
@ -2,5 +2,6 @@
|
|||||||
|
|
||||||
int pthread_cancel(pthread_t t)
|
int pthread_cancel(pthread_t t)
|
||||||
{
|
{
|
||||||
|
t->cancel = 1;
|
||||||
return pthread_kill(t, SIGCANCEL);
|
return pthread_kill(t, SIGCANCEL);
|
||||||
}
|
}
|
||||||
|
@ -47,20 +47,19 @@ void __pthread_unwind_next(struct __ptcb *cb)
|
|||||||
static void docancel(struct pthread *self)
|
static void docancel(struct pthread *self)
|
||||||
{
|
{
|
||||||
struct __ptcb cb = { .__next = self->cancelbuf };
|
struct __ptcb cb = { .__next = self->cancelbuf };
|
||||||
sigset_t set;
|
|
||||||
self->canceldisable = 1;
|
self->canceldisable = 1;
|
||||||
self->cancelasync = 0;
|
self->cancelasync = 0;
|
||||||
sigemptyset(&set);
|
|
||||||
sigaddset(&set, SIGCANCEL);
|
|
||||||
__libc_sigprocmask(SIG_UNBLOCK, &set, 0);
|
|
||||||
__pthread_unwind_next(&cb);
|
__pthread_unwind_next(&cb);
|
||||||
}
|
}
|
||||||
|
|
||||||
static void cancel_handler(int sig, siginfo_t *si, void *ctx)
|
static void cancel_handler(int sig, siginfo_t *si, void *ctx)
|
||||||
{
|
{
|
||||||
struct pthread *self = __pthread_self();
|
struct pthread *self = __pthread_self();
|
||||||
if (si->si_code > 0 || si->si_pid != self->pid) return;
|
if (!self->cancel) {
|
||||||
self->cancel = 1;
|
if (si->si_code == SI_TIMER && libc.sigtimer)
|
||||||
|
libc.sigtimer(sig, si, ctx);
|
||||||
|
return;
|
||||||
|
}
|
||||||
if (self->canceldisable) return;
|
if (self->canceldisable) return;
|
||||||
if (self->cancelasync || (self->cancelpoint==1 && PC_AT_SYS(ctx)))
|
if (self->cancelasync || (self->cancelpoint==1 && PC_AT_SYS(ctx)))
|
||||||
docancel(self);
|
docancel(self);
|
||||||
@ -176,6 +175,12 @@ static void init_threads()
|
|||||||
static int start(void *p)
|
static int start(void *p)
|
||||||
{
|
{
|
||||||
struct pthread *self = p;
|
struct pthread *self = p;
|
||||||
|
if (self->unblock_cancel) {
|
||||||
|
sigset_t set;
|
||||||
|
sigemptyset(&set);
|
||||||
|
sigaddset(&set, SIGCANCEL);
|
||||||
|
__libc_sigprocmask(SIG_UNBLOCK, &set, 0);
|
||||||
|
}
|
||||||
pthread_exit(self->start(self->start_arg));
|
pthread_exit(self->start(self->start_arg));
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
@ -220,6 +225,7 @@ int pthread_create(pthread_t *res, const pthread_attr_t *attr, void *(*entry)(vo
|
|||||||
new->tsd = (void *)tsd;
|
new->tsd = (void *)tsd;
|
||||||
new->detached = attr->_a_detach;
|
new->detached = attr->_a_detach;
|
||||||
new->attr = *attr;
|
new->attr = *attr;
|
||||||
|
new->unblock_cancel = self->cancel;
|
||||||
memcpy(new->tlsdesc, self->tlsdesc, sizeof new->tlsdesc);
|
memcpy(new->tlsdesc, self->tlsdesc, sizeof new->tlsdesc);
|
||||||
new->tlsdesc[1] = (uintptr_t)new;
|
new->tlsdesc[1] = (uintptr_t)new;
|
||||||
stack = (void *)((uintptr_t)new-1 & ~(uintptr_t)15);
|
stack = (void *)((uintptr_t)new-1 & ~(uintptr_t)15);
|
||||||
|
Reference in New Issue
Block a user