вот мой случай
за исключением того, что там 2 потока работают с разными кусками вектора (0-10 и 10-20), поэтому проблемы нет
а у меня оба работают с 0-20 и проблема есть
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
Проблема в том, что mutex берется первым потоком, а второй поток не может работать, пока он не будет освобожден.
Таким образом, по сути, вы получаете многопоточное приложение, в котором все потоки выполняют свою работу последовательно.
Вы можете переместить свою охрану в for-loop
for(int i=start; i<end; i++) {
std::lock_guard<std::mutex> guard(mtx);
vec[i] = i;
}
Это даст потокам возможность работать вместе над пройденным вектором.
BUT:
Вы должны принять во внимание, что несколько потоков не всегда дают производительность boost jut, потому что параллелизм != параллелизм .
Я бы ожидал, что это приложение таким образом будет на самом деле медленнее, чем однопоточная реализация, потому что следующее:
Потоки блокируют друг друга с помощью mutex , поэтому одновременно работает только один поток
Вы тратите время на переключение контекста между потоками
ИДЕЯ РЕШЕНИЯ:
Если вы хотите запустить его действительно параллельно, вам нужно заставить потоки работать с независимыми данными, а затем объединить результат.
у меня потоки нужны для чтения, а на запись операций в разы меньше
поэтому можно было бы либо лочить вектор просто (его ведь читать можно будет?), но это затраты, как выше описано
либо запись делать одним потоком (если можно одновременно в многопоточном приложении такое сделать?
По идее можно, ведь я могу внутри ф-и, которая вызывается на событии, сделать разделение на чтение/запись и параллелить только чтение...)
(ушёл читать хабр)
