Welcome to the main channel on the development of MoarVM, a virtual machine for NQP and Rakudo (moarvm.org). This channel is being logged for historical purposes.
Set by lizmat on 24 May 2021.
MasterDuke nine: do you not get an error locally with github.com/MoarVM/MoarVM/pull/1511 ? 09:07
i've gotten some random segfaults when building nqp and rakudo with it 09:18
and running rakudo tests, but i haven't been able to catch any in gdb yet 09:28
nine MasterDuke: huh...no, that PR is actually supposed to fix such things :D 09:52
No coredumps either? 09:53
MasterDuke hm, i can check
i always forget i have to use coredumpctl 09:54
gist.github.com/MasterDuke17/e028a...657eca4b8e 09:55
updated with a little more info 09:57
and some more 09:58
nine But, but, this is before my newly added code? 10:08
Btw I didn't really test it after changing it to MVM_free_null. 10:09
Can you change it back and test again?
MasterDuke k 10:29
oh, but CI was showing all fails on your first version too 10:35
nine: same segfault 10:47
backtrace looks the same 10:48
MasterDuke and no surprise since CI showed it too, but same results with clang 10:59
nine Oh, now I see it, too 11:09
lizmat And another Rakudo Weekly News hits the Net: rakudoweekly.blog/2021/06/29/2021-26-r3/ 11:11
nine MasterDuke: cought it in rr 11:44
Well, well, in MVMStaticFrame *sf = (MVMStaticFrame *)MVM_repr_at_pos_o(tc, check_frames, i); we actually get a VMNull instead of a static frame. 11:48
nine Ah, it's simply a good old GC issue 12:45
MasterDuke: pushed a fixed version :) 12:50
MasterDuke nice, and CI seems happy with it so far 13:03
nine slowly exhales 13:04
MasterDuke nine: that missing MVMROOT was a pre-existing problem wasn't it? i.e., the first version of your PR just made it more likely to trigger somehow 13:56
nine MasterDuke: no, because before my change there was no point at which we could have entered the GC. There is now with MVM_gc_mark_thread_blocked(tc); 13:58
MasterDuke ah