Question

我正在阅读＆＃34; C ++并发行动＆＃34;作者：Anthony Williams，我遇到了这段代码，一个简单的线程池实现。

class thread_pool
{
    std::atomic_bool done;
    thread_safe_queue<std::function<void()> > work_queue;
    std::vector<std::thread> threads;
    join_threads joiner;

    void worker_thread()
    {
        while(!done) 
        {
            std::function<void()> task;
            if(work_queue.try_pop(task))            
            {
                task(); 
            }
            else
            {
                std::this_thread::yield();
            }
       }
    }
    public:
    thread_pool():
        done(false),joiner(threads)
    {
        unsigned const thread_count=std::thread::hardware_concurrency();
        try 
        {

           for(unsigned i=0;i<thread_count;++i)
           {
              threads.push_back(
                 std::thread(&thread_pool::worker_thread,this));
           } 
        }
        catch(...)
        {
           done=true;
           throw;
        }
     }

    ~thread_pool()
    {
        done=true;
    }

    template<typename FunctionType>
    void submit(FunctionType f)
    {
        work_queue.push(std::function<void()>(f));
    }
};

P.S。 join_threads是一个简单的类，它在破坏和连接时加入线程 thread_safe_queue是一个......线程安全的队列！

我的问题是关于布尔标志 std :: atomic_bool已完成。我已经读过，使用默认赋值运算符与使用顺序一致的内存排序相同。

做=真。 - ＆GT; done.store（true，std :: memory_order_seq_cst）

在这种情况下真的有必要吗？使用发布/获取订购甚至是轻松的是不够的？工作线程只是循环遍布bool值，显然没有任何其他内存访问权限可以同步。

我过度优化还是错过了什么？

Answer 1

我不认为你被误解了。顺序一致的访问比最低要求更受限制。

在这种情况下，使用std::atomic::operator=具有简单的优点（即更清晰的代码），并且不太可能引入任何性能问题 - 特别是在大多数平台上，原子布尔值非常接近地映射到处理器操作

std :: atomic_bool标志的内存顺序

1 个答案: