Czy to (volatile bool) zawsze jest bezpieczne?

Question

Zastanawiam się, czy jest to całkowicie bezpieczne dla wątków i czy zmienne słowo kluczowe powinno być na miejscu.

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; 
    } 
} 

Myślę, że tak, ale chcę dwukrotnie sprawdzić, ponieważ nie jestem w 100% pewny, że chcę tylko mieć pewność.

Myślę, że zmienne słowo kluczowe jest wymagane, ponieważ wtedy nigdy nie byłoby możliwe, aby wartość była przechowywana w rejestrach lub podobne optymalizacje.

Answer 1

Co odpowiedział SLaks jest poprawny, oczywiście, ale aby odpowiedzieć na Twoje pytanie: tak w obu przypadkach: jest to bezpieczne, to powinny być zadeklarowane zmienne.

Answer 2

Powinieneś wymienić całą konstrukcję na ManualResetEvent, która jest zarówno wątkowa, jak i szybsza.

private static readonly ManualResetEvent ev = new ManualResetEvent(); 

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

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