マルチスレッドでのunique_ptr

Question

unique_ptrをマルチスレッドで使用することについていくつかの心配があります。mutexはありません。私は以下の簡単なコードを書きました。見てください。 unique_ptr != nullptrをチェックすると、スレッドセーフなのでしょうか？

class BigClassCreatedOnce 
{ 
public: 
    std::atomic<bool> var; 

    // A lot of other stuff 
}; 

BigClassCreatedOnceクラスのインスタンスは一度だけ作成されますが、私は、スレッド間でそれを使用することが安全であるかわかりません。

class MainClass 
{ 
public: 
    // m_bigClass used all around the class from the Main Thread 

    MainClass() 
     : m_bigClass() 
     , m_thread() 
    { 
     m_thread = std::thread([this]() { 
      while (1) 
      { 
       methodToBeCalledFromThread(); 
       std::this_thread::sleep_for(std::chrono::milliseconds(1)); 
      } 
     }); 

     // other stuff here 

     m_bigClass.reset(new BigClassCreatedOnce()); // created only once 
    } 

    void methodToBeCalledFromThread() 
    { 
     if (!m_bigClass) // As I understand this is not safe 
     { 
      return; 
     } 

     if (m_bigClass->var.load()) // As I understand this is safe 
     { 
      // does something 
     } 
    } 

    std::unique_ptr<BigClassCreatedOnce> m_bigClass; 
    std::thread m_thread; 
}; 

サンプルを簡略化するために無限ループに入れています。私はunique_ptr != nullptrをチェックすると

int main() 
{ 
    MainClass ms; 
    while (1) 
    { 
     std::this_thread::sleep_for(std::chrono::milliseconds(1)); 
    } 
} 

Answer 1

、それは

いいえ、それはスレッドセーフではありませんスレッドセーフです。複数のスレッドがあり、そのうちの少なくとも1つが共有データに書き込む場合、同期が必要です。あなたがデータレースを持っていない場合、それは未定義の動作です。

m_bigClass.reset(new BigClassCreatedOnce()); // created only once 

と

if (!m_bigClass) 

の両方が同時に起こるので、データ競合であることができます。

私も

if (m_bigClass->var.load()) 

はスレッドセーフではないにもされていることを指摘したいと思います。 var.load()ですが、m_bigClassのアクセスはありませんので、そこにデータ競争もあります。