El siguiente fragmento de código de muestra "de ámbito local objeto estático inicialización" no es seguro para subprocesos:
#include <windows.h>
#include <stdio.h>
#include <process.h>
struct X {
~X() { puts("~X()"); }
int i_ ;
void print(void) {
printf("thread id=%u, i = %d\n", GetCurrentThreadId(), i_);
}
X(int i) {
puts("begin to sleep 10 seconds");
Sleep(1000 * 10);
i_ = i;
printf("X(int) i = %d\n", i_);
puts("end");
}
};
X & getX()
{
static X static_x(1000);
return static_x;
}
void thread_proc(void *)
{
X & x = getX();
x.print();
}
int main(int argc, char *argv[])
{
HANDLE all_threads[2] = {};
all_threads[0] = HANDLE(_beginthread(thread_proc, 0, 0));
printf("First thread Id: %u\n", GetThreadId(all_threads[0]));
Sleep(1000);
all_threads[1] = HANDLE(_beginthread(thread_proc, 0, 0));
printf("Second thread Id: %u\n", GetThreadId(all_threads[1]));
WaitForMultipleObjects(_countof(all_threads), all_threads, TRUE, 1000 * 20);
puts("main exit");
return 0;
}
La salida será (por supuesto ID del tema será diferente en su máquina):
First thread Id: 20104
begin to sleep 10 seconds
Second thread Id: 20248
thread id=20248, i = 0
X(int) i = 4247392
end
thread id=20104, i = 1000
main exit
~X()
Antes de las primeras vueltas de rosca que significa ctor del Singleton se llama y volvió, el segundo hilo de obtener el objeto no inicializado y lo llaman es el método miembro (porque el objeto estático es en el segmento BSS, que será initilized a cero después de loader cargar el ejecutable) y obtener el valor incorrecto: 0.
Encendido de montaje lista por/fasc /Fastatic.asm obtendrá el código de montaje para la función getX():
01: [email protected]@[email protected]@XZ PROC ; getX
02:
03: ; 20 : {
04:
05: 00000 55 push ebp
06: 00001 8b ec mov ebp, esp
07:
08: ; 21 : static X static_x(1000);
09:
10: 00003 a1 00 00 00 00 mov eax, DWORD PTR [email protected][email protected]@[email protected]@[email protected]
11: 00008 83 e0 01 and eax, 1
12: 0000b 75 2b jne SHORT [email protected]
13: 0000d 8b 0d 00 00 00
14: 00 mov ecx, DWORD PTR [email protected][email protected]@[email protected]@[email protected]
15: 00013 83 c9 01 or ecx, 1
16: 00016 89 0d 00 00 00
17: 00 mov DWORD PTR [email protected][email protected]@[email protected]@[email protected], ecx
18: 0001c 68 e8 03 00 00 push 1000 ; 000003e8H
19: 00021 b9 00 00 00 00 mov ecx, OFFSET [email protected][email protected]@[email protected]@[email protected]@A
20: 00026 e8 00 00 00 00 call [email protected]@[email protected]@Z ; X::X
21: 0002b 68 00 00 00 00 push OFFSET [email protected][email protected]@[email protected]@[email protected] ; `getX'::`2'::`dynamic atexit destructor for 'static_x''
22: 00030 e8 00 00 00 00 call _atexit
23: 00035 83 c4 04 add esp, 4
24: [email protected]:
25:
26: ; 22 : return static_x;
27:
28: 00038 b8 00 00 00 00 mov eax, OFFSET [email protected][email protected]@[email protected]@[email protected]@A
29:
30: ; 23 : }
En la línea 10 el símbolo críptico [? $ S1 @? 1 ?? getX @@ YAAAX @@ XZ @ 4IA] es el indicador global (también en BSS) que indica si el singleton está codificado o no, será marcado como verdadero por la línea 14-17, justo antes de llamar al ctor, ese es el problema, esto también explica por qué el segundo subproceso recibió inmediatamente el objeto singleton no inicializado y felizmente lo llamó función de miembro. No hay ningún código relacionado con seguridad de subprocesos insertado por el compilador.
Ver http: // stackoverflow.com/questions/164496/how-can-i-create-a-thread-safe-singleton-pattern-in-windows – MerickOWA
@MerickOWA: Esto data del '08 y no proporciona una respuesta clara y aceptada globalmente. Además, ni siquiera cubre VC2010 (debido a la edad del tema). –
@ IC3M4N VS2010 se implementó antes de que se lanzara C++ 11, si no implementa la construcción segura de subprocesos de variables locales estáticas, entonces se le deja usar técnicas que han existido por muchos años. No veo nada que no se aplique a VS2010 – MerickOWA