c - 试图理解 POSIX 线程

Question

我试图掌握 POSIX 线程的使用，并创建了一个简单的程序，该程序只需将全局变量递增 10。有时它会一直运行良好，其他情况下会在随机点出现故障。虽然这种类型的问题似乎是线程的常见问题，但我无法理解为什么在这样一个简单的示例中会发生这种情况。我目前的想法是，由于未加入线程，父级可能在此之前结束，但如果我尝试加入，我会在第一个线程上出现段错误...... Join 语句被保留但被注释掉了。 Join 是否以正确的语法使用？没有执行连接会导致段错误吗？

#include <stdio.h>
#include <stdlib.h>
#include <pthread.h>
#include <errno.h>
#include <string.h>

#define NUMPHILO 5

thread_t threads[NUMPHILO];  //array of threads 
pthread_mutex_t mutex;        // initialize mutex

int x = 5;          //for test
int y = 10;

int philofunct(){
    pthread_mutex_lock (&mutex);
    x = x+y;
    pthread_mutex_unlock (&mutex);
    printf("Philo fucntion x = %d\n",x);
    return x;
}


int main(){

    pthread_mutex_init(&mutex, NULL);

    for(int i = 0; i < NUMPHILO; i++){  
        printf("creating thread[%i]\n",i);
        if( pthread_create(&threads[i], NULL, (void *)philofunct(),(void *) &i) != 0)
            fprintf(stderr, "%s", strerror(errno));
//      pthread_join(threads[i],NULL);  
//      printf("Threads Joined\n");
    }
    pthread_mutex_destroy(&mutex);
    printf("Threads created\n");
}

输出:

creating thread[0]
Philo fucntion x = 15
creating thread[1]
Philo fucntion x = 25
creating thread[2]
Philo fucntion x = 35
creating thread[3]
Philo fucntion x = 45
creating thread[4]
Philo fucntion x = 55
Threads created

下次运行:

creating thread[0]
Philo fucntion x = 15
creating thread[1]
Philo fucntion x = 25
creating thread[2]
Philo fucntion x = 35
Segmentation fault (core dumped)

一如既往，非常感谢任何帮助。

Answer 1

你遇到的主要问题是:

    if( pthread_create(&threads[i], NULL, (void *)philofunct(),(void *) &i) != 0)

仔细看，您是在调用 philofunct() 而不是将函数指针传递给它。更糟糕的是，您正在传递 philofunct() 的返回值，它是一些整数，例如 15，作为指向 pthread_create() 的函数指针.自然地，当线程尝试从诸如 15 的地址执行代码时，它会 segv。

一旦你像这样正确定义了 philofunct():

void *philofunct(void *arg)

然后您可以调用 pthread_create() 而无需强制转换:

    if( pthread_create(&threads[i], NULL, philofunct,(void *) &i) != 0)

另外，如果你想加入你的线程，你应该在它们全部运行后加入你的线程。如果您在创建循环中加入线程，您将在创建下一个线程之前等待每个线程完成。所以你会这样写:

for (i=0;i<NUMPHILO; i++) {
    pthread_create(&threads[i], ...);
    // Other stuff...
}
for (i=0;i<NUMPHILO; i++)
    pthread_join(threads[i], NULL);