2

这是我正在玩的一个玩具示例，用于学习如何使用 TBB。Parallel::operator() 应该并行运行，但它有一个关键区域，一次只能由一个处理器访问，因此它打印的消息不会被打乱。我的问题是它无法编译并且编译器消息并没有真正帮助我。我究竟做错了什么？

此外，这是否被认为是在 parallel_for 中实现互斥锁的正确方法？

#include <iostream>
#include <vector>
#include <cmath>
#include <tbb/tbb.h>

typedef tbb::queuing_mutex Mutex;

struct Parallel
{
    Mutex mutex;
    std::vector<int> * values;

    Parallel(std::vector<int> * values_) : values(values_) {}

    void operator()( tbb::blocked_range< unsigned int > & range ) const {
        for(unsigned int i = range.begin(); i < range.end(); ++i) {
            {
                Mutex::scoped_lock lock(mutex);
                if ( (*values)[i] > 40)
                {
                    std::cout << "NO SCRAMBLING ALLOWED!\n";
                    std::cout.flush();
                }
                lock.release();
            }
        }
    }
};

int main() {
    const int someValue = 20000;

    std::vector<int> data(someValue);
    for(int i = 0; i < someValue; ++i) {
        data[i] = std::rand();
    }

    tbb::parallel_for( tbb::blocked_range<unsigned int>(0, data.size()),
                       Parallel(&data) );
}

贝娄是错误信息：

/path-to-src/main.cpp: In member function 'void Parallel::operator()(tbb::blocked_range<unsigned int>&) const':
/path-to-src/main.cpp:20:46: error: no matching function for call to 'tbb::queuing_mutex::scoped_lock::scoped_lock(const Mutex&)'
/path-to-src/main.cpp:20:46: note: candidates are:
In file included from /usr/include/tbb/tbb.h:65:0,
                 from /path-to-src/main.cpp:4:
/usr/include/tbb/queuing_mutex.h:80:9: note: tbb::queuing_mutex::scoped_lock::scoped_lock(tbb::queuing_mutex&)
/usr/include/tbb/queuing_mutex.h:80:9: note:   no known conversion for argument 1 from 'const Mutex {aka const tbb::queuing_mutex}' to 'tbb::queuing_mutex&'
/usr/include/tbb/queuing_mutex.h:77:9: note: tbb::queuing_mutex::scoped_lock::scoped_lock()
/usr/include/tbb/queuing_mutex.h:77:9: note:   candidate expects 0 arguments, 1 provided
/usr/include/tbb/queuing_mutex.h:66:11: note: tbb::queuing_mutex::scoped_lock::scoped_lock(const tbb::queuing_mutex::scoped_lock&)
/usr/include/tbb/queuing_mutex.h:66:11: note:   no known conversion for argument 1 from 'const Mutex {aka const tbb::queuing_mutex}' to 'const tbb::queuing_mutex::scoped_lock&'
4

2 回答 2

1

tbb::parallel_for 旨在不按编写的方式编译示例。它可以防止代码中的错误。tbb::parallel_for 的范围形式将仿函数按值复制到多个任务对象中。因此，每个任务都有一个单独的互斥锁副本，因此互斥锁不会提供预期的同步。

修复代码的方法是在 struct Parallel 之外声明互斥体，并通过指针传递它，类似于“值”的指针。

于 2013-09-18T14:50:47.867 回答
0

一切都是正确的，除了mutex必须要mutable在锁中使用它。此外，不需要释放锁。

typedef tbb::queuing_mutex Mutex;

struct Parallel
{
    mutable Mutex mutex;              // mutable so that we can use it in const member
    std::vector<int> * values;

    Parallel(std::vector<int> * values_) : values(values_) {}

    // note: this is a const member
    void operator()( tbb::blocked_range< unsigned int > & range ) const
    {
        for(unsigned int i = range.begin(); i < range.end(); ++i)
        {
            Mutex::scoped_lock lock(mutex);       // requires non-const argument
            if ( (*values)[i] > 40)
            {
                std::cout << "NO SCRAMBLING ALLOWED!\n";
                std::cout.flush();
            }
            // no need to release the lock: the destructor will do that for you
        }
    }
};

或者，您当然可以将成员函数设为非 const。

于 2013-09-18T15:12:00.450 回答