別スレッドにいなくても、SyncLockとMemoryBarrierを呼び出しても安全ですか？

Question

フォローアップthis post。私の目標は、一度に1つだけのCalculateを持つことですので、私はSyncLockを追加しました：

Public Sub Calculate(Optional inBack As Boolean = True) 
    If Not inBack Then 
     InternalCalculate(-1, False) 
    Else 
     If CalcThread IsNot Nothing Then 
      CalcThread.Abort() ' yes, I will replace this 
      CalcThread = Nothing ' XXX 
     End If 
     If CalcThread Is Nothing Then 
      CalcThread = New Threading.Thread(AddressOf InternalCalculate) 
      CalcThread.IsBackground = True 
     End If 
     CalcThread.Start() 
    End If 
End Sub 

Private Sub InternalCalculate(Optional Line As Integer = -1, Optional isBack As Boolean = True) 
    Dim Lock As New Object 
    SyncLock Lock 
     Threading.Thread.MemoryBarrier() ' do this BEFORE a write, right? 
     isRunning = true 
     'do the expensive stuff 
    End SyncLock 
End Sub 

注意isBack。これが偽の場合、コードはmainで実行されるだけです。これは、単一のLineを再計算するときによく使用されます。だから私の質問は、これらの2つのラインの安全性についてです：

SyncLock Lock 
Threading.Thread.MemoryBarrier() 

それは私がメインで実行されているコードでこれらを呼び出す場合、何が起こるかthe documentationに私には明らかではありません。私はコードを追加しましたが、それは正常に実行されるようですが、私は自分がAbortのような別の問題に開かれていないことを確認したいと思います。これらはスレッド化と非スレッド化の両方で使用できますか？

Answer 1

はい、シングルスレッドコードで使用するのは安全です。

InternalCalculateには、同時にコードを実行できる問題があります。 InternalCalculateを呼び出すたびに、新しいLockオブジェクトが作成されます。 Synclockは、同じオブジェクトを使用する場合にのみスレッドをブロックするので、Lockを読み取り専用メンバー変数にします。

複数のスレッドを実行すると、2番目、3番目、4番目などは、ロックを取得する最初のスレッドが同期ロックブロックを終了するまで待機します。その後、他の人は行くでしょう。したがって、これが一度だけ実行されるべきコードであれば、ダブルチェックロックパターンをチェックアウトする必要があります。

揮発性データの読み書きにMemoryBarriersを追加する代わりに、System.Threading.Thread.VolatileReadとSystem.Threading.Thread.VolatileWriteを使用することをお勧めします。次に、読み書きを行う順序を覚えておく必要はありません。