Question

我正在运行以下程序来实现计时器。当线程在从先前运行的线程接收到条件变量的信号后唤醒时，它创建一个计时器并在计时器到期时向下一个线程发送信号。我希望它运行一段时间，但计时器在一些运行后停止滴答。

//Import 
#define _POSIX_C_SOURCE 199309
#include <sched.h>
#include <unistd.h>
#include <sys/wait.h>
#include <stdio.h>
#include <stdlib.h>
#include <pthread.h>
#include <unistd.h>
#include <time.h>
#include <sys/time.h>
#include <signal.h>
#include <errno.h>
#define NUM_THREADS 10

#define CLOCKID CLOCK_REALTIME
#define SIG SIGUSR1
timer_t timerid;

pthread_cond_t   condA[NUM_THREADS+1]  = PTHREAD_COND_INITIALIZER;
pthread_mutex_t  mutex  = PTHREAD_MUTEX_INITIALIZER;

pthread_t tid[NUM_THREADS];
int state = 0;

static void handler(int sig, siginfo_t *si, void *uc)
{
    if(si->si_value.sival_ptr != &timerid){
        printf("Stray signal\n");
    } else {
        //printf("Caught signal %d from timer\n", sig);
    }

        pthread_cond_signal(&condA[state]);
}

void *threadA(void *data_)
{
    int i = 0, s;
    long int loopNum, j;

    int turn = (intptr_t)data_;

    struct timeval tval_result;

    // Timer's part starts

    struct sigevent sev;
    struct itimerspec its;
    long long freq_nanosecs;
    sigset_t mask;
    struct sigaction sa;
    // TImer'spart ends

    while(1)
    {

        /* Wait for state A */
        pthread_mutex_lock(&mutex);

        for (;state != turn;)
        {
            s = pthread_cond_wait(&condA[turn], &mutex);
            if (s != 0)
                perror("pthread_cond_wait");
           // printf("main(): state = %d\n", state);
        }
        pthread_mutex_unlock(&mutex);

        //do stuff
        for(j=0;j<10000;j++)
            {//some dummy time consuming works}

        sa.sa_flags = SA_SIGINFO;
        sa.sa_sigaction = handler;
        sigemptyset(&sa.sa_mask);
        sigaction(SIG, &sa, NULL);

        sev.sigev_notify = SIGEV_SIGNAL;
        sev.sigev_signo = SIG;
        sev.sigev_value.sival_ptr = &timerid;
        timer_create(CLOCKID, &sev, &timerid);
        /* Start the timer */

        its.it_value.tv_sec = 0;
        its.it_value.tv_nsec = 2000;
        its.it_interval.tv_sec = 0;
        its.it_interval.tv_nsec = 0;

        timer_settime(timerid, 0, &its, NULL);
        pthread_mutex_lock(&mutex);
        state = (state +1)%NUM_THREADS;
        //pthread_cond_signal(&condA[state]);
        pthread_mutex_unlock(&mutex);
        // Timer's code ends
    }

}

int main(int argc, char *argv[])
{
    int data = 0;
    int err;

    while(data < NUM_THREADS)
    {
        //create our threads
        err = pthread_create(&tid[data], NULL, threadA, (void *)(intptr_t)data);
        if(err != 0)
            printf("\ncan't create thread :[%s]", strerror(err));
        else
           // printf("\n Thread created successfully\n");

        data++;
    }

    pthread_exit(NULL);
}

虽然没有执行printf语句，为什么一段时间后它会冻结？如果不。计时器是有限的，我应该用什么其他策略来解决这个问题？

Answer 1

POSIX说：

在异步调用的信号处理程序中使用pthread_cond_signal（）函数是不安全的。

最有可能最终破坏pthread_cond_wait / pthread_cond_signal的状态，任何事情都可能发生。

不要混合线程和信号处理程序，它只会导致疯狂。在信号处理程序中你很少允许做的事情，与线程相关的事情就更少了，很难确保正确的线程最终处理正确的信号等等。

如果您正在进行线程，无论如何都要在一个线程中实现一个计时器，计算下一次事件需要睡多少时间（不要将它硬编码到您的计时器期间，因为那会使你的计时器漂移），睡得那么多并打电话给pthread_cond_signal。

此外，裸体pthread_cond_signal电话的形式很糟糕，而且通常是个错误。您可能会感到不幸并且在其他线程执行pthread_cond_wait之前调用它并且您的信号将丢失。通常要做的是设置一个变量（由互斥锁保护，这就是pthread_cond_signal想要互斥锁的原因）然后发出变量设置的信号。

如果您认为这工作太多，在这种情况下条件变量可能不是正确的机制，您应该使用信号量。顺便说一句sem_post根据POSIX从信号处理程序调用是合法的，但我仍然认为将线程与信号混合是一个坏主意。

定时器会在一段时间后冻结

1 个答案: