-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
segfaults and other stability issues #24
Comments
|
|
We've been using non-parallel version of this script for a while now. Instead, we just run this script several times on different ranges. See whateverable issue #24 and RT #129781
|
Otherwise we choke ourselves trying to process the output. See RT #130370 Partially addresses #24
As of today, there are no segfaults. I'd still have to write tests for some cases mentioned here, but generally it is not an issue anymore. |
For a long time this script was suffering due to various instabilities in rakudo (see issue #24). As of today, some of the bugs were fixed, but at the same time this script itself was wiggled into some stable state. I am afraid to touch it. Therefore, committing what we have without any clean up. In fact, there are some issues I can see right now, but the script is proven to work in practice… 🙈
Getting stuff like this:
Didn't look into it deeply at all, but leaving a note here anyway. Can be reproduced by running |
I’m getting those a lot too (happening debugging HTTP::Server::Async issues)
On July 27, 2017 at 11:18:45 AM, Aleks-Daniel Jakimenko-Aleksejev (notifications@github.com<mailto:notifications@github.com>) wrote:
OK, bots are not stable anymore. I think it's due to rakudo/rakudo@9658dd9<rakudo/rakudo@9658dd98c9>.
Getting stuff like this:
MoarVM panic: Internal error: invalid thread ID 284 in GC work pass
Didn't look into it deeply at all, but leaving a note here anyway.
Can be reproduced by running t/bisectable.p6 on the server (sometimes you may get lucky and the whole file will pass, but usually it crashes half way through).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<#24 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AB75kKe5t-nLzDuQEw_fqm66YFsjJYXQks5sSNSBgaJpZM4JovXH>.
|
Seems to be alright now after fixes by @jnthn++. |
|
The leakage was reported in RT #131879, and right now it is fixed in a way that it does not leak as much anymore. The memory usage increases if you keep throwing non-existent commits into the bots, but given 16GB of RAM on the server this is hardly a problem. Right now, the bots are stable. |
|
Right now Quotable does not work anyway (see issue #24), but it should start working as soon as the issue is resolved.
RT #131961 is resolved, waiting for the next bug to appear now. |
|
|
Well, I guess it's not. Things are still broken though. |
|
OK, RT #132191 turned out to be an issue in IRC-Client (it was relying on a rakudo bug). Now there are at least two other problems. Bisectable fails with this output:
There is no reason why test 61 would fail. Actually, it passes if you put it higher in that file. I don't know what's going on there, but most likely it's an issue in rakudo. The second problem is that it runs some other test after the first test failed. Why? It should not be like that. |
Ah OK, the ‘_’ test is an issue in whateverable. Nevermind that. Why does it fail in the first place is beyond me however. |
The code involves a lot of calls to Text::Diff::Sift4 module, but nothing special really. This issue didn't exist a few releases ago, and I really am not sure when this happened exactly. The same test works fine in |
Alright, some progress on that! First of all, it doesn't hang, it segfaults. The reason I was thinking that it hangs is because the test suite does not really detect if the bot process dies unexpectedly, so there was no easy way to notice. Now I have some code that will help notice the issue in the future, will commit that soon. Now, the segfault happens in the react block here: So, that's easy now, right? Just run it under valgrind and you'll immediately see the issue… Ha-ha. Nope. You run it under valgrind, and the issue goes away. 💩 I'm suspecting that we may be seeing something like rakudo/rakudo#1202 here, but it's hard to tell. |
Same issue under gdb: https://gist.github.com/MasterDuke17/0312dd2af1e3b2b498d91cfacc45343c |
|
|
On this line: I cannot reproduce so we will just let it be… |
|
See issue #24 and rakudo/rakudo#1259
|
Again, there's nothing special with this test. And if you look closely, previous tests have been commented out because they were causing another segv previously. Here's the ticket: rakudo/rakudo#1259 |
Bots no longer leak memory like crazy, so that issue is resolved. Bisectable still can't get through its tests though. |
OK issue #296 can be workarounded like this: -my $host-arch = $*KERNEL.hardware;
+my $host-arch = ‘x86_64’;
$host-arch = ‘amd64’|‘x86_64’ if $host-arch eq ‘amd64’|‘x86_64’;
-$host-arch = $*KERNEL.name ~ ‘-’ ~ $host-arch;
+$host-arch = ‘linux’ ~ ‘-’ ~ $host-arch; Heh. Not committing this to the repo because I'm hoping it'll get resolved relatively quickly. |
Could you try this diff and see if that makes a difference?
$ git diff
diff --git a/src/core/Kernel.pm6 b/src/core/Kernel.pm6
index 1cde4c4..7ce4cf8 100644
--- a/src/core/Kernel.pm6
+++ b/src/core/Kernel.pm6
@@ -180,8 +180,8 @@ class Kernel does Systemic {
}
}
…-Rakudo::Internals.REGISTER-DYNAMIC: '$*KERNEL', {
+#Rakudo::Internals.REGISTER-DYNAMIC: '$*KERNEL', {
PROCESS::<$KERNEL> := Kernel.new;
-}
+#}
If it does, then it’s something in the auto-vivification of dynamic variables that’s to blame, and not something specific to $*KERNEL.
On 7 Mar 2018, at 22:34, Aleks-Daniel Jakimenko-Aleksejev ***@***.***> wrote:
OK issue #296 can be workarounded like this:
-my $host-arch = $*KERNEL.hardware;
+my $host-arch = ‘x86_64’;
$host-arch = ‘amd64’|‘x86_64’ if $host-arch eq ‘amd64’|‘x86_64’;
-$host-arch = $*KERNEL.name ~ ‘-’ ~ $host-arch;
+$host-arch = ‘linux’ ~ ‘-’ ~ $host-arch;
Heh. Not committing this to the repo because I'm hoping it'll get resolved relatively quickly.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
@lizmat tried that, no difference. Further discussion here: rakudo/rakudo#1595 |
|
|
In 2023, I hit a few issues that were part of #388:
|
See this: http://irclog.perlgeek.de/perl6/2016-08-19#i_13055233
Pretty sure that it is not our fault, but we have to rakudobug it.
The text was updated successfully, but these errors were encountered: