我正在尝试为每个正在进行的请求创建一个线程。这是我正在制作的一个简短示例。问题是我可以将第一个进入请求写入列表,但是当我尝试发送第二个请求时,它会在需要向我的函数“request_loop”发送信号时停止工作。
所以它在那之前完美无缺。有谁知道如何解决这个问题?该计划不希望进一步工作。
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <fcntl.h>
#include <errno.h>
#include <string.h>
#include "tcpsocket.h"
#include <sys/select.h>
#include <sys/time.h>
#include <sys/types.h>
#include <pthread.h>
#define NUM_HANDLER_THREADS 3 /* number of threads used to service requests */
pthread_mutex_t request_mutex = PTHREAD_MUTEX_INITIALIZER;
pthread_cond_t got_request = PTHREAD_COND_INITIALIZER;
unsigned char buffer[BUFSIZE];
unsigned char collect_buffers[2000];
struct timespec timeout;
struct timeval tv;
int num_requests = 0;
int num;
int rc = 0;
struct request {
int number;
struct request* next;
};
struct request* requests = NULL;
struct request* last_request = NULL;
这是我添加请求的功能。当我想将第二个帖子写入我的列表时,它就会停止工作......
void add_request(int request_num,pthread_mutex_t* p_mutex,pthread_cond_t* p_cond_var)
{
int rc;
struct request* a_request;
a_request = (struct request*)malloc(sizeof(struct request));
if (!a_request) { /* malloc failed?? */
fprintf(stderr, "add_request: out of memory\n");
exit(1);
}
a_request->number = request_num;
a_request->next = NULL;
rc = pthread_mutex_lock(p_mutex);
printf("NUMBER OF REQUESTS: %d\n",num_requests);
if (num_requests == 0) { /* special case - list is empty */
printf("list is empty\n");
requests = a_request;
last_request = a_request;
printf("REQUEST NUMBER:%d",a_request->number);
}else{
printf("created list\n");
last_request->next = a_request;
last_request = a_request;
}
num_requests++;
rc = pthread_mutex_unlock(p_mutex);
printf("send signal");
rc = pthread_cond_signal(p_cond_var);
printf("signal sended");
}
这是我想将数据写入列表的功能。之后我删除了请求:free(request);
struct request* get_request(pthread_mutex_t* p_mutex){
printf("NUMBER REQUESTS (get request): %d\n",num_requests);
int rc;
struct request* a_request;
rc = pthread_mutex_lock(p_mutex);
if (num_requests > 0) {
a_request = requests;
requests = a_request->next;
printf("REQUEST NUMBER:%d\n",a_request->number);
if (requests == NULL) { /* this was the last request on the list */
last_request = NULL;
printf("last request is NULL\n\n");
}
num_requests--; //remove REQUEST
}else { /* requests list is empty */
printf("list is empty\n");
a_request = NULL;
}
//remove request
free(a_request);
rc = pthread_mutex_unlock(p_mutex);
/* return the request to the caller. */
return a_request;
}
这是我处理请求的循环,直到没有请求为止
void* handle_requests_loop(void* data){
//rc = pthread_mutex_lock(&request_mutex);
printf("NUMBER REQUESTS (loop): %d\n",num_requests);
int rc;
struct request* a_request;
int thread_id = *((int*)data);
while(num_requests > 0){
printf("THREAD ID : %d\n",thread_id);
a_request = get_request(&request_mutex);
if(num_requests == 0){
printf("NO REQUESTS\n\n");
}else{
printf("THERE ARE REQUESTS\n\n");
}
}
while(num_requests == 0){
printf("wait");
rc = pthread_cond_wait(&got_request, &request_mutex);
}
}
我为此编写了一个生成2个线程的测试函数。第一个工作正常,另一个在需要向线程循环发送新信号时停止工作。
int main(int argc, char* argv[]){
printf("StartUp\n");
//---------------------------Create Threads--------------------------------
for (i=0; i<NUM_HANDLER_THREADS; i++) {
thr_id[i] = i;
pthread_create(&p_threads[i], NULL, handle_requests_loop,(void*)&thr_id[i]);
}
//--------------------------INTERRUPT--------------------------------------
int j;
for(j=1;j<3;j++){
printf("ADD REQUEST\n");
add_request(j, &request_mutex, &got_request);
delay.tv_sec = 5;
delay.tv_nsec = 10;
nanosleep(&delay,NULL);
}
return 0;
}
有谁知道如何解决这个问题?非常感谢!!
答案 0 :(得分:0)
我不知道你的具体问题是什么。我需要你更具体地说明当你只有一个接收器线程时会发生什么,并在调试器中运行并告诉我们每个线程遇到的代码行。例如,从你的文章中可以看出,它是主线程还是其中一个被“卡住”的子线程。当你只创建一个子线程(更不用说两个子线程)而不是三个子线程时,你是否观察到卡住情况还不清楚。
那说有一个错误,你可以清理一些事情,当你试图自己解决这些问题时,这会使你自己更容易。
错误是在handle_requests_loop()
的底部,您使用pthread_cond_wait()
呼叫&request_mutex
,即使您当前没有request_mutex
已锁定。这会导致未定义的行为(可能包括永久禁用调用pthread_cond_wait()
或 pthread_cond_signal()
的线程中的一个或两个。)
你可以清理的东西:
PTHREAD_MUTEX_INITIALIZER
初始化互斥锁。这会创建非递归互斥锁。如果同一个线程试图锁定互斥锁两次,它将自行死锁。您应该使用PTHREAD_MUTEX_RECURSIVE
属性进行初始化,或者非常小心不要双重锁定。例如:如果您在pthread_mutex_lock()
的第一行取消注释handle_requests_loop()
来电,那么当您在{{第5行调用pthread_mutex_lock()
时,您保证会自行陷入僵局 1}}。get_request()
中创建一些主题,但不要调用main()
。现在这可能很好,但是将来某个时候你可能不假思索地调用pthread_join()
引用main()堆栈上的一些数据, 会导致严重问题(未定义的行为。)pthread_create()
既没有handle_requests_loop()
语句,也没有致电return
。你需要一个或另一个(至少返回NULL)。pthread_exit()
vs requests
而不是只拥有一个变量。 (它看起来不对,但我无法弄清楚,所以它可能,纯粹的运气,可以。)last_request
底部附近释放a_request
,但随后将其返回get_request()
。从长远来看,这只能导致悲伤和伤害感情。