我真的很难理解如何以允许线程同时运行的方式锁定和解锁互斥锁。现在我试图让每个线程做一些事情,然后等待所有线程准备好,然后重复它。这应该重复发生,所以我把它全部放在一个条件循环中。我的问题是,似乎我的广播从未被任何线程接收过,因为等待信号的每个线程都会永远等待,而发送信号的线程会继续。
我已经愚弄了我的代码,使其尽可能简单,同时仍然可编译和可运行:
#include <stdio.h>
#include <pthread.h>
#include <unistd.h>
#define num_threads 4
int repeat = 1;
int timesLooped = 0;
int waitingThreads = 0;
int threadNo = -1;
pthread_mutex_t mutex1;
pthread_cond_t condition;
//thread body
void* threadBody(void* arg){
//sensitive info: requires lock
if(pthread_mutex_lock(&mutex1)){ printf("error1\n"); }
threadNo++;
int threadId = threadNo;
//other info is set here as well in my code
if(pthread_mutex_unlock(&mutex1)){ printf("error2\n"); }
//main loop in the thread body
while(repeat == 1){
if(pthread_mutex_lock(&mutex1)){ printf("error3\n"); }
//wait until all threads are ready
while(waitingThreads < num_threads - 1){
printf(" %d! ", threadId);
waitingThreads++;
pthread_cond_wait(&condition, &mutex1);
printf(" %d ", threadId);
if(pthread_mutex_unlock(&mutex1)){ printf("error4\n"); }
}
//last thread will broadcast
if(waitingThreads == num_threads - 1){
printf("\n\nthread %d was last! broadcasting to let everyone proceed...", threadId);
timesLooped++;
waitingThreads = 0;
if(timesLooped == 3){
repeat = 0;
}
sleep(1);
pthread_cond_broadcast(&condition);
if(pthread_mutex_unlock(&mutex1)){ printf("error5\n"); }
}
}
printf("\n\nexiting thread %d\n", threadId);
pthread_exit((void*) arg);
}
int main(int argc, char** argv){
pthread_t threads[num_threads];
void* retval;
//create threads
for(long i = 0; i < num_threads; i++){
pthread_create(&threads[i], NULL, threadBody, (void*) i);
}
//join threads
for(long j = 0; j < num_threads; j++){
pthread_join(threads[j], &retval);
}
printf("\n\nDONE\n");
}
这给了我一个输出:
线程3是最后一个!广播让大家继续......
线程2是最后一个!广播让大家继续......
线程1是最后一个!广播让大家继续......
退出线程1(死锁,其他线程永不退出)
答案 0 :(得分:1)
您的计划中至少有两个错误。
您有数据竞争(请参阅此blog post)。
当线程0
到num_threads - 1
访问waitingThreads
中的if (waitingThreads == num_threads - 1) ...
时,他们会在锁定之外(即他们参加比赛)。
您不允许运行最后一个线程(这是您的主要问题)。
pthread_cond_wait
返回两个条件已发出信号和可以重新获取互斥锁。
但是你的最后一个帖子立即在释放它时重新获取互斥锁,因此没有其他线程可以继续。
我希望如果您在最后一个帖子中sleep
之后添加usleep
或pthread_mutex_unlock
,您的程序将按照您的预期开始工作。