将包含另一个结构的结构传递给pthread_create

时间:2011-03-23 14:55:37

标签: c pointers pthreads structure void-pointers

所以我试图传递一个包含多个变量的结构,包括另一个处理位图信息的结构。但是,我的代码在某处失败了,因为它在解除引用结构中包含的信息的指针方面吐出了“解除指向不完整类型的指针”的错误。我知道这里有很多问题要处理这个问题,但我已经尝试实现那里所说的并且失败了。

这是main()的相关代码,包括有关初始化的编辑:

pthread_t threads[thread_num];
pthread_attr_t attr;
int rc;
void *status;
pthread_attr_init(&attr);
pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_JOINABLE);

// Create a bitmap of the appropriate size.
struct bitmap *bitm = bitmap_create(image_width,image_height);

struct thread_args *arguments = (struct thread_args*) malloc(sizeof(struct thread_args));
arguments->bm = bitm;
arguments->xmin = xcenter-scale;
arguments->xmax = xcenter+scale;
arguments->ymin = ycenter-scale;
arguments->ymax = ycenter+scale;
arguments->max = max;
// Compute the Mandelbrot image
for(int i=0;i<thread_num;i++){
    arguments->thread_id = i;
if(pthread_create(&threads[i], NULL, compute_image, (void *)arguments)<0){
   printf("ERROR; return code from pthread_create() is %d\n", rc);
}
 }

 pthread_attr_destroy(&attr);
 for(int t=0; t<thread_num; t++) {
    rc = pthread_join(threads[t], &status);
    if (rc) {
        printf("ERROR; return code from pthread_join() is %d\n", rc);
     exit(-1);
     }
 }

这是函数的相关代码,作为pthread_create的参数传递:

void* compute_image(void *threadargs ){
   int i,j;
   struct thread_data *my_data = (struct thread_args*) malloc(sizeof(struct thread_args));

   my_data = (struct thread_data *) threadargs;
   int width = bitmap_width(my_data->bm);
   int height = bitmap_height(my_data->bm);
   int threads = my_data->threads;
int thread_id = my_data->thread_id;
double xmin = my_data->xmin;
double xmax = my_data->xmax;
double ymin = my_data->ymin;
double ymax = my_data->ymax;
int max = my_data->max;


// For every pixel in the image...

for(j=height/threads*thread_id;j<height/threads*(thread_id+1);j++) {

    for(i=0;i<width;i++) {

        // Determine the point in x,y space for that pixel.
        double x = xmin + i*(xmax-xmin)/width;
        double y = ymin + j*(ymax-ymin)/height;

        // Compute the iterations at that point.
        int iters = iterations_at_point(x,y,max);

        // Set the pixel in the bitmap.
        bitmap_set(my_data->bm,i,j,iters);
    }
}

}

这是结构:

struct thread_args{
    int thread_id;
    int threads;
    struct bitmap *bm;
    double xmin;
    double xmax;
    double ymin;
    double ymax;
    int max;
};

2 个答案:

答案 0 :(得分:2)

我发现您的代码有两个问题:

  1. 您定义struct thread_args,但在compute_image()中,您使用struct thread_data。这些不一样,而且(我猜)你的意思是thread_args,而不是thread_data。这很可能解释了你得到的编译错误。您可以考虑将此struct image_data命名为;这可能不太可能引起混淆。
  2. 您使用my_data d内存初始化malloc(),然后立即从threadargs的强制转换中重新分配。这是一个内存泄漏,没有充分的理由。只需删除malloc初始化。

答案 1 :(得分:0)

这是您放置struct thread_args定义的问题。当编译器只有原型可用时,它对结构成员一无所知。您需要将结构定义保留在包含的头文件中,或者在包含定义的src文件中创建访问器函数。