diff options
author | Takashi Kokubun <takashikkbn@gmail.com> | 2020-12-20 21:54:27 -0800 |
---|---|---|
committer | Takashi Kokubun <takashikkbn@gmail.com> | 2020-12-20 22:03:20 -0800 |
commit | a574df14e45b8b5a1de7bfe949e08b61ae51b0bd (patch) | |
tree | b3378de36e14145f78a5246a83eb3d702f31ff1c /vm.c | |
parent | 74a78778362a7ad9aa246f2cb1a30174a6007365 (diff) |
Stop marking unit_queue
The original motivation of this marking was https://github.com/k0kubun/yarv-mjit/issues/20.
As wanabe said, there are multiple options to mitigate the issue, and
Eric Wong introduced another fix at 143776f6fe by checking unit->iseq
inside the lock.
Therefore this particular condition has been covered in two ways, and
the script given by wanabe no longer crashes without mjit_mark().
Diffstat (limited to 'vm.c')
-rw-r--r-- | vm.c | 2 |
1 files changed, 0 insertions, 2 deletions
@@ -2594,8 +2594,6 @@ rb_vm_mark(void *ptr) rb_gc_mark_values(RUBY_NSIG, vm->trap_list.cmd); rb_id_table_foreach_values(vm->negative_cme_table, vm_mark_negative_cme, NULL); - - mjit_mark(); } RUBY_MARK_LEAVE("vm"); |