使用线程的Hello World损坏的内存

Question

我正在通过一些简单的 pthread 示例 llnl.computing.gov pthreads 教程。网站上的程序打印出 threadid 的地址，但我想将 id 的地址传递给 PrintHello，然后使用取消引用地址来获取 id。我认为在那里睡觉时，每个线程都应该打印 8（线程数）。代码是

#include <pthread.h>
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#define NUM_THREADS  8
void *PrintHello(void *threadid)
{
   long *taskid = (long *)threadid;
   sleep(1);
   printf("Hello from thread %ldn", *taskid);
   pthread_exit(NULL);
} 
int main(int argc, char *argv[])
{
  pthread_t threads[NUM_THREADS];
  int rc;
  long t;
  for(t=0;t<NUM_THREADS;t++) {
    printf("Creating thread %ldn", t);
    rc = pthread_create(&threads[t], NULL, PrintHello, (void *) &t);
    if (rc) {
      printf("ERROR; return code from pthread_create() is %dn", rc);
      exit(-1);
    }
 }
 pthread_exit(NULL);
}

当我在Cygwin中编译并运行它时，它会出现堆栈损坏错误。如果我将 PrintHello 重写为：

void *PrintHello(void *threadid)
{
  long taskid = (long) threadid;
  sleep(1);
  printf("Hello from thread %ldn", taskid);
  pthread_exit(NULL);
 }

它

没有错误，它只是打印地址，我想取消引用地址并从main获取t的值。

有没有人对如何实现这一目标有一些指示？我知道我可以t传递给pthread_create而不是&t但我想这样做是为了学习目的。

Answer 1

当您从主线程调用pthread_exit(NULL)时，它会终止该线程。 此时，main函数中的任何局部变量（包括t）都将被销毁，不能再使用。

如果主线程在所有工作线程都完成使用 t 之前退出（通过您通过 pthread_create 传递给它们的指针），则程序将表现出未定义的行为。

该程序包含一个争用条件，因为从工作线程访问变量t和从主线程销毁变量t是不同步的。 解决此问题的一种方法是在退出之前让主线程与每个工作线程连接（通过 pthread_join ）。

Answer 2

1）你传递t的地址，所以每个线程都会得到一个指向同一变量的指针，这不是线程ID，而是一个值不断变化的long。 您正在main中修改变量，并在每个其他线程中读取它，这是一场数据竞赛。

2）可能发生的情况是，当新线程执行时，main中的循环已经完成并且变量已经超出范围。当main退出时，其局部变量将从堆栈中弹出，因此当其他线程访问t时，它不再在堆栈上。你的程序中没有任何同步来防止这种情况，所以你在那里有另一个数据竞争。您需要在main等待线程完成，您可以通过调用pthread_join等待每个线程来完成，但这仍然不会改变其他线程在另一个线程写入变量时尝试读取变量的事实。

3）没有必要在那里调用pthread_exit，从函数或main返回会自动退出线程（就像调用exit(0)导致main()完成一样）

Answer 3

一些指示？好吧，您的线程函数中有很多它们...

问题：您无法安全地传递局部变量的地址 - 当主变量退出时，它会超出范围。您需要将指针声明为静态全局变量，或者 malloc（） sizeof（long） 内存并使用它。