diff options
| author | Lukas Larsson <[email protected]> | 2018-12-13 16:34:38 +0100 |
|---|---|---|
| committer | Lukas Larsson <[email protected]> | 2018-12-13 17:08:55 +0100 |
| commit | 27ff974bfa155ad31dcdc338d7fecf36b08a0516 (patch) | |
| tree | 13c02aedb0e3f51a5663e820d356206d9efef465 /erts/emulator/beam/erl_proc_sig_queue.h | |
| parent | be749b4f83f3b3a2fa3bd8e5a3e5e28467b137b2 (diff) | |
| download | otp-27ff974bfa155ad31dcdc338d7fecf36b08a0516.tar.gz otp-27ff974bfa155ad31dcdc338d7fecf36b08a0516.tar.bz2 otp-27ff974bfa155ad31dcdc338d7fecf36b08a0516.zip | |
erts: Fix seq_trace:reset_trace dirty gc bug
When seq_trace:reset_trace could be called while a
process was doing a dirty GC. This triggered a race
where all signals was moved to the internal signal
queue during the GC which in turn caused the a heap
overrun problem.
This fix makes it so that the main and msgq lock are
taken before the clear. This will make sure that we
are allowed to do the clear.
Diffstat (limited to 'erts/emulator/beam/erl_proc_sig_queue.h')
| -rw-r--r-- | erts/emulator/beam/erl_proc_sig_queue.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/erts/emulator/beam/erl_proc_sig_queue.h b/erts/emulator/beam/erl_proc_sig_queue.h index 3fc2d06b2d..6b065a7add 100644 --- a/erts/emulator/beam/erl_proc_sig_queue.h +++ b/erts/emulator/beam/erl_proc_sig_queue.h @@ -989,8 +989,7 @@ erts_proc_sig_fetch(Process *proc) Sint res = 0; ErtsSignal *sig; - ERTS_LC_ASSERT(erts_thr_progress_is_blocking() - || ERTS_PROC_IS_EXITING(proc) + ERTS_LC_ASSERT(ERTS_PROC_IS_EXITING(proc) || ((erts_proc_lc_my_proc_locks(proc) & (ERTS_PROC_LOCK_MAIN | ERTS_PROC_LOCK_MSGQ)) |