Где ошибка при работе с потоками?

Question

[Кирилл Журавлев]

Имеется некая функция, в который идет много вычислений с матрицей. Решил разбить работу на два потока.

...
		thread even(bareissEven,a, k, p0);
		thread odd(bareissOdd,a, k, p0);
		even.detach();
		odd.detach();
...

Вызываемые функции

void bareissEven(matrix<cpp_int>& a, int k, cpp_int p0){
	for (int i = 0; i < a.size1(); i++)
	{
		if (i == k) continue;
		for (int j = 0; j < a.size2(); j++)
		{
			if (j == k) continue;
			if (j % 2 == 0) a(i, j) = ((a(k, k) * a(i, j)) - (a(i, k) * a(k, j))) / p0;
		}
	}
}

void bareissOdd(matrix<cpp_int>& a, int k, cpp_int p0) {
	for (int i = 0; i < a.size1(); i++)
	{
		if (i == k) continue;
		for (int j = 0; j < a.size2(); j++)
		{
			if (j == k) continue;
			if (j % 2 != 0) a(i, j) = ((a(k, k) * a(i, j)) - (a(i, k) * a(k, j))) / p0;
		}
	}
}

При компиляции получаю


Что я делаю не так?

Comments

[res2001]

Возможно перед тем как создать потоки надо объявить потоковые функции.

Проверять на четность можно гораздо эффективней:
for (int j = 1; j < a.size2(); j+=2)
В этом случае j всегда будет нечетным и дополнительные проверки будут не нужны.
Аналогично и для четных j.

[Кирилл Журавлев]

res2001, про четность спасибо. Потоковые функции объявлены прототипами ранее. Ушел от ошибки, убрав передачу по ссылке, но тогда считает неправильно.

[res2001]

Кирилл Журавлев, а, да, ссылки...используйте std::ref чтоб передать в потоковую функцию ссылку.

[Roman]

Кирилл Журавлев, зачем столько телодвижений?
одна функция, два потока.
передаете диапазоны [a, b] [b, c]

типа

matrix m(4, 4);
//...

void calc(matrix& a, int start, int stop, int k, int p0)
{
  for (int i = start; i < stop; i++)
  {
    if (i == k) continue;
    for (int j = 0; j < a[i].size(); j++)
    {
      if (j == k) continue;
      a(i, j) = ((a(k, k) * a(i, j)) - (a(i, k) * a(k, j))) / p0;
    }
  }
}

...

thread one(calc, ref(m), 0, 2, ... );
thread two(calc, ref(m), 2, 4, ... );

[Кирилл Журавлев]

Roman, первый раз работаю с потоками, отсюда столько телодвижений.

[Roman]

Кирилл Журавлев, не надо так остро воспринимать.
есть еще OpenMP там поинтереснее будет.

[Кирилл Журавлев]

Roman, воспринял не остро, так получилось. За наводку на OpenMP спасибо =)

[Евгений Шатунов]

Кирилл Журавлев, совет для более качественного дознания источника ошибки.
Окном Error List удобно пользоваться только для общего ознакомления с перечнем ошибок.
Сами ошибки лучше изучать в окне Output. Ошибка в Error List связана с позицией ошибки в Output, где и изложена более детальная информация о проблеме.
Текст из окна Output принесет вопросу гораздо больше пользы, чем мелкая картинка.

Answer 1

[qqrm]

Лучше пользоваться std::async, чем голыми тредами.