-
Notifications
You must be signed in to change notification settings - Fork 118
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
Fix test suite on platforms without compaction support #355
Conversation
7d7bdef
to
2501bb6
Compare
Hi, I noted in #354 that this seemed the obvious fix to me and so I had already tried it, but this commit causes the test suite to hang. Output is just:
and then it spins chewing one CPU core. |
If you remove the If so I'm happy to take a PR that detect your environment and just skips this part. |
Hmmm, no it still hangs, so this must not be the cause. I provide free shell access to all of the machines I use to reproduce these bugs on, would this be useful to look into it? |
I might if I have time to kill. In the meantime, what happens if you send a SEGV signal? That should trigger the ruby crash report and give you a C-backtrace of where ruby is hanging. |
|
Thanks, ok so this is very weird. It's literally stuck loading the code, it hasn't even started the test suite yet. I really have little idea what it could be. |
Recompiled everything with -O0, debug assertions, etc. Slightly better backtrace here - the exact line varies slightly depending on when I stop it, but everything up to the top 2 frames or so is consistent.
|
Well...interesting discovery, everything works on Ruby 3.2! |
Interesting backtrace. I wonder if somehow it's a Ruby 3.1 GC bug (e.g. @peterzhu2118 does that ring any bell? But good news on 3.2 working! |
See: msgpack/msgpack-ruby#355 Bug: https://bugs.gentoo.org/917362 Signed-off-by: Matoro Mahri <matoro_gentoo@matoro.tk>
See: msgpack/msgpack-ruby#355 Bug: https://bugs.gentoo.org/917362 Signed-off-by: Matoro Mahri <matoro_gentoo@matoro.tk>
See: msgpack/msgpack-ruby#355 Bug: https://bugs.gentoo.org/917362 Signed-off-by: Matoro Mahri <matoro_gentoo@matoro.tk>
See: msgpack/msgpack-ruby#355 Bug: https://bugs.gentoo.org/917362 Signed-off-by: Matoro Mahri <matoro_gentoo@matoro.tk> Signed-off-by: Sam James <sam@gentoo.org>
Fix: #354