From e4cd5838fc6a816146fecf94f1c33441ff61ec89 Mon Sep 17 00:00:00 2001 From: antirez Date: Tue, 27 Mar 2012 13:48:53 +0200 Subject: [PATCH] Mask SIGALRM everything but in the main thread. This is required to ensure that the signal will be delivered to the main thread when the watchdog timer expires. --- src/bio.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/src/bio.c b/src/bio.c index eaac8e40..aa2cdf9f 100644 --- a/src/bio.c +++ b/src/bio.c @@ -108,9 +108,18 @@ void bioCreateBackgroundJob(int type, void *arg1, void *arg2, void *arg3) { void *bioProcessBackgroundJobs(void *arg) { struct bio_job *job; unsigned long type = (unsigned long) arg; + sigset_t sigset; pthread_detach(pthread_self()); pthread_mutex_lock(&bio_mutex[type]); + /* Block SIGALRM so we are sure that only the main thread will + * receive the watchdog signal. */ + sigemptyset(&sigset); + sigaddset(&sigset, SIGALRM); + if (pthread_sigmask(SIG_BLOCK, &sigset, NULL)) + redisLog(REDIS_WARNING, + "Warning: can't mask SIGALRM in bio.c thread: %s", strerror(errno)); + while(1) { listNode *ln; -- 2.45.2