Какой уровень детализации блокировки хорош в параллельных структурах данных?

Я довольно новичок в многопоточности, у меня есть однопоточное приложение для анализа данных, которое имеет большой потенциал для распараллеливания, и, хотя наборы данных большие, это не близко к насыщению чтения / записи на жестком диске, поэтому я думаю, Я должен воспользоваться поддержкой потоков, которая сейчас есть в стандарте, и попытаться ускорить работу зверя.

После некоторого исследования я решил, что потребитель-производитель является хорошим подходом для чтения данных с диска и его обработки, и я начал писать пул объектов, который станет частью кольцевого буфера, где производители будут помещать данные, а потребители — данные. Когда я писал класс, мне казалось, что я слишком хорошо разбираюсь в том, как я управляю блокировкой и освобождением членов данных. Такое ощущение, что половина кода блокируется и разблокируется, и как будто существует безумное количество объектов синхронизации, плавающих вокруг.

Итак, я прихожу к вам с объявлением класса и примером функции, и этот вопрос: это слишком мелко? Недостаточно мелкозернистый? Плохо продумано?

struct PoolArray
{
public:
Obj* arr;
uint32 used;
uint32 refs;
std::mutex locker;
};

class SegmentedPool
{
public: /*Construction and destruction cut out*/
void alloc(uint32 cellsNeeded, PoolPtr& ptr);
void dealloc(PoolPtr& ptr);
void clearAll();
private:
void expand();

//stores all the segments of the pool
std::vector< PoolArray<Obj> > pools;
ReadWriteLock poolLock;

//stores pools that are empty
std::queue< int > freePools;
std::mutex freeLock;

int currentPool;
ReadWriteLock currentLock;
};

void SegmentedPool::dealloc(PoolPtr& ptr)
{
//find and access the segment
poolLock.lockForRead();
PoolArray* temp = &(pools[ptr.getSeg()]);
poolLock.unlockForRead();
//reduce the count of references in the segment
temp->locker.lock();
--(temp->refs);
//if the number of references is now zero then set the segment back to unused
//and push it onto the queue of empty segments so that it can be reused
if(temp->refs==0)
{
temp->used=0;
freeLock.lock();
freePools.push(ptr.getSeg());
freeLock.unlock();
}
temp->locker.unlock();
ptr.set(NULL,-1);
}

Несколько объяснений:
First PoolPtr — это глупый маленький указатель, похожий на объект, который хранит указатель и номер сегмента в пуле, из которого поступил указатель.

Во-вторых, все это «шаблонизировано», но я взял эти строки, чтобы попытаться уменьшить длину блока кода

Третий ReadWriteLock — это то, что я собрал, используя мьютекс и пару условных переменных.