std::thread 处的分段错误

Segmentation fault at std::thread

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 is inverse subpixel

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或更多的时候会出现这种情况。它不会从第一次崩溃。即使我尝试只调用一个线程,结果也是一样的。

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];
}

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