std :: thread上的分段错误

时间:2015-11-14 21:27:06

标签: c++ segmentation-fault stdthread

typedef struct _ppm_struct* ppm_struct;
typedef unsigned char ppm_subpixel;

typedef ppm_subpixel (*ppm_pixel_func)(ppm_subpixel);
struct ppm_pixel {
   ppm_subpixel red;
   ppm_subpixel green;
   ppm_subpixel blue;
};

struct _ppm_struct {
   unsigned int width;
   unsigned int height;
   unsigned int max_color;
   ppm_pixel *firstpixel;
};

所以我有这个函数调用:

void pppm_color_pixel(ppm_struct p, ppm_struct out, ppm_pixel_func func, int threads) {
//ppm_pixel_func func2 = &inverse_pixel;
unsigned int w = p->width;
unsigned int h = p->height;
ppm_pixel *iterator = p->firstpixel;
std::thread th[threads];

long long total_pixels = w*h;
unsigned int count_for_thread = total_pixels/threads;

for(int j = 0; j<threads; ++j) {

   th[j] = std::thread(pppm_color_chunk_pixel,func, *iterator,count_for_thread);
   iterator = &iterator[count_for_thread];
}

for(int j = 0; j<threads; ++j) {
    th[j].join();
}

接下来的两个函数:funct是逆子像素

ppm_subpixel inverse_subpixel(ppm_subpixel subpixel) {
    subpixel = 255- subpixel;
    return subpixel;
}

void pppm_color_chunk_pixel(ppm_pixel_func func, ppm_pixel start_pixel, unsigned int count) {
    ppm_pixel *iterator = &start_pixel;
    for(unsigned int i = 0; i< count; ++i) {
        iterator[i].red = func(iterator[i].red);
        iterator[i].green = func(iterator[i].green);
        iterator[i].blue = func(iterator[i].blue);
    }
}

主要:

pppm_color_pixel(p,out, &inverse_subpixel,pppm_get_max_cores());

问题在于,当我运行它时,它会向我显示分段错误。这发生在这一行:

iterator[i].green = func(iterator[i].green);

当发生这种情况时,我真正不理解的是i = 2或更多。它不会从第一次崩溃。即使我尝试只调用一个线程,结果也是一样的。

1 个答案:

答案 0 :(得分:1)

pppm_color_chunk_pixel接受ppm_pixel的副本。由于它是按值复制的,因此将*iterator传递给它只会导致复制一个元素。在&start_pixel之后迭代任何内容都会失败。 由于std::thread不通过引用传递对象,因此您还需要一个引用包装器。

void pppm_color_chunk_pixel(ppm_pixel_func func, ppm_pixel& start_pixel, unsigned int count) {
ppm_pixel* iterator = &start_pixel;
for (unsigned int i = 0; i < count; ++i) {
    iterator[i].red   = func(iterator[i].red);
    iterator[i].green = func(iterator[i].green);
    iterator[i].blue  = func(iterator[i].blue);
}

致电:

for (int j = 0; j < threads; ++j) {
   th[j] = std::thread(pppm_color_chunk_pixel, func, std::ref(*iterator), count_for_thread);
   iterator = &iterator[count_for_thread];
}

我没有测试过,但我猜这是你的问题。