È questo (booster volatile) sempre thread safe?

Question

Mi chiedo se questo è completamente sicuro per i thread e se la parola chiave volatile debba essere presente o meno.

using System.Threading; 

class Program 
{ 
    private static volatile bool _restart = true; 

    private static void Main() 
    { 
     while (_restart) 
     { 
      // Do stuff here every time for as long as _restart is true 
      Thread.Sleep(1); 
     } 
    } 

    private static void SomeOtherThread() 
    { 
     Thread.Sleep(1000); 
     _restart = false; 
    } 
} 

penso che sia, ma voglio ricontrollare, dal momento che io non sono sicuro al 100% voglio solo essere sicuro.

Penso che la parola chiave volatile sia richiesta perché non sarebbe mai possibile avere il valore memorizzato nella cache nei registri o in uguali ottimizzazioni.

Answer 1

Cosa risposto SLaks è corretto, naturalmente, ma per rispondere alla tua domanda: sì, su entrambi i fronti: è sicuro, deve essere dichiarata volatile.

Answer 2

È necessario sostituire l'intero costrutto con uno ManualResetEvent, che è sia thread-safe che più veloce.

private static readonly ManualResetEvent ev = new ManualResetEvent(); 

private static void Main() 
{ 
    ev.WaitOne() 
} 

private static void SomeOtherThread() 
{ 
    Thread.Sleep(1000); 
    ev.Set(); 
}