diff options
| author | Patrik Nyblom <[email protected]> | 2011-03-02 14:23:22 +0100 |
|---|---|---|
| committer | Patrik Nyblom <[email protected]> | 2011-03-02 14:23:22 +0100 |
| commit | c971c489dc8318ca4e92e9ba40d739dc4ee50982 (patch) | |
| tree | fed367a9d7eefc3201b1d23cdbb21744fa49e4ca /erts/emulator/sys | |
| parent | e12038dc74b081e6b5dfd2b3f266dc01a09ed523 (diff) | |
| download | otp-c971c489dc8318ca4e92e9ba40d739dc4ee50982.tar.gz otp-c971c489dc8318ca4e92e9ba40d739dc4ee50982.tar.bz2 otp-c971c489dc8318ca4e92e9ba40d739dc4ee50982.zip | |
Remove race in main thread stealing due to reading and writing to pipe from same thread
Diffstat (limited to 'erts/emulator/sys')
| -rw-r--r-- | erts/emulator/sys/unix/sys.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/erts/emulator/sys/unix/sys.c b/erts/emulator/sys/unix/sys.c index 5f38d73359..bafbbb0f6c 100644 --- a/erts/emulator/sys/unix/sys.c +++ b/erts/emulator/sys/unix/sys.c @@ -2997,10 +2997,12 @@ init_smp_sig_notify(void) #ifdef __DARWIN__ int erts_darwin_main_thread_pipe[2]; +int erts_darwin_main_thread_result_pipe[2]; -static void initialize_darwin_main_thread_pipe(void) +static void initialize_darwin_main_thread_pipes(void) { - if (pipe(erts_darwin_main_thread_pipe) < 0) { + if (pipe(erts_darwin_main_thread_pipe) < 0 || + pipe(erts_darwin_main_thread_result_pipe) < 0) { erl_exit(1,"Fatal error initializing Darwin main thread stealing"); } } @@ -3011,7 +3013,7 @@ erts_sys_main_thread(void) { erts_thread_disable_fpe(); #ifdef __DARWIN__ - initialize_darwin_main_thread_pipe(); + initialize_darwin_main_thread_pipes(); #endif /* Become signal receiver thread... */ #ifdef ERTS_ENABLE_LOCK_CHECK @@ -3039,7 +3041,7 @@ erts_sys_main_thread(void) read(erts_darwin_main_thread_pipe[0],&func,sizeof(void* (*)(void*))); read(erts_darwin_main_thread_pipe[0],&arg, sizeof(void*)); resp = (*func)(arg); - write(erts_darwin_main_thread_pipe[1],&resp,sizeof(void *)); + write(erts_darwin_main_thread_result_pipe[1],&resp,sizeof(void *)); } #else #ifdef DEBUG |