github.com/moarvm/moarvm | IRC logs at colabti.org/irclogger/irclogger_logs/moarvm
Set by AlexDaniel on 12 June 2018.
00:04 anatofuz joined
japhb timotimo: Yeah, I think so. 00:05
Geth MoarVM: 72befa1647 | (Timo Paulssen)++ | src/jit/core_templates.expr
The exprjit of sp_bind_o caused wrongness

even though the sp_bind_o looked correct to many eyes. we expect the problem is something that used to not go through the exprjit because sp_bind_o caused the exprjit to bail out earlier is now active
00:10
timotimo damnit, the other stuff got merged into moar master directly after the bump 00:19
so i'll have to also revert that and re-revert it back
Geth MoarVM: b2dc3f3ba2 | (Timo Paulssen)++ | 19 files
Revert "Merge pull request #1139 from MasterDuke17/add_context_to_error_messages"

This reverts commit bfe499dd6b891bc5101543337a7082ad356d0b16, reversing changes made to 2ac137c3d879e7e77c37522fcd7661fd9e4f6203.
00:23
00:28 travis-ci joined
travis-ci MoarVM build passed. Timo Paulssen 'The exprjit of sp_bind_o caused wrongness 00:28
travis-ci.org/MoarVM/MoarVM/builds/585351219 github.com/MoarVM/MoarVM/compare/b...befa1647f7
00:28 travis-ci left 00:38 lucasb left 00:42 travis-ci joined
travis-ci MoarVM build passed. Timo Paulssen 'Revert "Merge pull request #1139 from MasterDuke17/add_context_to_error_messages" 00:42
travis-ci.org/MoarVM/MoarVM/builds/585353748 github.com/MoarVM/MoarVM/compare/7...dc3f3ba2f0
00:42 travis-ci left
timotimo really gotta look into this "coz compiler" thing 00:43
especially seeing if we can steal the way it works and put it into moarvm 00:44
00:53 anatofuz left 01:02 anatofuz joined 02:26 anatofuz left, anatofuz joined 02:29 anatofuz left, anatofuz joined 02:33 anatofuz left 02:34 anatofuz joined 02:38 anatofuz left 02:39 anatofuz joined 02:49 anatofuz left 02:50 anatofuz joined 03:48 sena_kun joined 04:10 anatofuz left 04:11 anatofuz joined 04:22 sena_kun left 05:04 robertle_ left 06:16 anatofuz left 06:22 domidumont joined 06:24 anatofuz joined, anatofuz left 06:25 brrt joined 06:27 sena_kun joined 06:37 anatofuz joined
nwc10 good *, #moarvm 06:44
brrt good * nwc10 06:54
07:04 anatofuz left 07:12 anatofuz joined 07:22 sena_kun left 07:38 zakharyas joined 07:41 brrt left 08:21 anatofuz left 08:22 anatofuz joined 08:46 sena_kun joined, robertle left 08:48 robertle joined 09:14 sena_kun left 09:36 anatofuz left 09:37 anatofuz joined 09:58 anatofuz left 10:02 anatofuz joined 10:03 robertle left 10:06 anatofuz left 10:10 robertle joined
timotimo uh oh, looks like more than just sp_bind_o is wrong 10:48
i wonder if i should revert all of them for now and put them into a branch 10:49
nwc10 suggests
1) step away from the keyboard
2) make coffee (or something)
3) ponder this
and after a delay, see if you have the same answer in your head
timotimo i've just been away from the keyboard for a whole bunch of hours :P
nwc10 aha. pre-delayed 10:50
as to the actual question, I don't know
timotimo i regularly stock up on away-from-keyboard time
nwc10 I'm not sure what I'm going to do for away-from-keyboard time once I've finished flattening the garden
11:12 sena_kun joined 11:18 anatofuz joined
timotimo terminate called after throwing an instance of 'dwarf::format_error' 11:20
what(): DW_FORM_sec_offset not expected for attribute (DW_AT)0x2119
:|
11:24 anatofuz left 11:25 anatofuz joined
nine timotimo: moving them off into a branch would indeed be nice for others who work on the VM. Especially since they cause mysterious and hard to reproduce issues. 11:53
Geth MoarVM: 986e05beab | (Timo Paulssen)++ | 2 files
Revert "exprjit sp_p6oget_bi"

This reverts commit 0674a1e6940e8c7945dd6e5620ad3edc28414f43.
11:56
MoarVM: a0ff25c746 | (Timo Paulssen)++ | src/jit/core_templates.expr
Revert "exprjit: templates for sp_get_i* and sp_get_o"

This reverts commit acfb5949967065694a058a31c159f99bfb730469.
nine Oh, I think I misdiagnosed the spesh plugin race condition issue. I thought that resolve_using_guards was using a plugin_state that was already freed by a concurrent add_resolution_to_guard_set. But this already uses MVM_fixed_size_free_at_safepoint. 12:39
I bet what actually happens is that this old plugin_state is no longer anchored somewhere where the GC would find it, so the guards it contains can become outdated. 12:41
Usually it would hang off the MVMStaticFrameSpesh but add_resolution_to_guard_set replaces it with the new version. 12:42
The bad news is that this is not just a case of adding an MVMROOT and be done with it. The good news is that this is still in the realm of problems that I know how to solve :) 12:44
Guest15407 timotimo: your templates are probably good, the problem lies elsewhere 12:49
timotimo yeah, but making them available causes trouble
Guest15407 true true :)
nine: what do you make of os_handle here? github.com/MoarVM/MoarVM/blob/mast...ops.c#L496 12:50
12:57 Kaypie is now known as Kaiepi
nine Guest15407: looks suspicious. Unless someone else keeps req->data up to date 13:19
13:32 evalable6 left 13:40 camelia joined 13:42 lucasb joined 13:44 evalable6 joined 13:45 camelia left 13:47 camelia joined
Guest15407 nine: I have seen the mutex (line 500) suddenly become 0x0, see gist.github.com/dogbert17/bd5a41e0...d8a1536915 14:05
14:15 anatofuz left 14:17 anatofuz joined 14:44 anatofuz left 14:45 anatofuz joined 14:48 zakharyas left 14:49 anatofuz left 15:11 zakharyas joined, Guest15407 left 15:25 domidumont left
lizmat and yet another Perl 6 Weekly hits the Net: p6weekly.wordpress.com/2019/09/16/...essinging/ 15:41
15:47 robertle left 15:51 anatofuz joined 16:02 anatofuz left 16:09 zakharyas left
MasterDuke timotimo: why did my PR merge have to be reverted? couldn't just the template commits have been reverted and then a new bump? 16:46
timotimo then the bump would have had the merge in it, too 16:47
MasterDuke also to everybody, my bad on asking for the bump in the first place
timotimo but the merge was specifically delayed after the bump
MasterDuke oh, ha. right
heh, forgot about that. timotimo++
17:15 domidumont joined 17:18 anatofuz joined 17:22 anatofuz left 17:44 sena_kun left 17:45 Kaiepi left 17:46 Kaiepi joined 17:51 robertle joined
nine It occured to me that I totally wasted those GC fixes I did. I could have kept a pile of them, just pushing one per week and ensure my mention in the weekly for almost a year in advance! Then I could have become known in the community as the possibly corrupt memory guy...or....wait...nah, better not :D 18:04
MasterDuke so many different options for hyphens to change meaning there: possibly-corrupt memory guy, possibly corrupt-memory guy, possibly corrupt memory-guy, possibly corrupt-memory-guy, possibly-corrupt-memory guy, possibly-corrupt memory-guy, possibly-corrupt-memory-guy 18:08
18:50 domidumont left 19:04 Kaiepi left 19:05 Kaiepi joined 19:07 Kaiepi left, Kaiepi joined 19:20 anatofuz joined
nine I wonder...how terrible would this patch be? gist.github.com/niner/19c93f7b96ae...7986d10e76 19:20
19:23 sena_kun joined 19:51 AlexDaniel left, AlexDaniel joined, AlexDaniel left, AlexDaniel joined 19:53 anatofuz left 19:58 Kaiepi left 19:59 brrt joined 20:01 Kaiepi joined, Kaiepi left, Kaiepi joined 20:21 robertle left 20:40 sena_kun left
brrt \o 20:42
20:54 Ven`` joined
timotimo nine: did you mean to root temp_root_ptr? 21:16
21:20 tellable6 left 21:22 Ven`` left 21:38 anatofuz joined 21:41 Kaiepi left 21:50 Kaiepi joined 22:19 anatofuz left 22:48 anatofuz joined 22:53 anatofuz left 22:54 anatofuz joined 22:58 anatofuz left 23:03 anatofuz joined 23:06 anatofuz left 23:10 anatofuz joined 23:40 brrt left 23:56 anatofuz left