-
-
Notifications
You must be signed in to change notification settings - Fork 13
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
SIGSEGV during C2 CompileTask #1191
Comments
@victornoel which version of Jackson are you usin and do you experience these crashes with Jacoco diasble? |
Hi @karianna, sorry for the delay. So what I can already tell is that with 21.0.4+7 we don't get the problem (the CI has been using it for 4-5 days). We use Jackson 2.17.2 (and seen it happen with 2.17.1 too. We do use jacoco (0.8.11) indeed, and I haven't had the opportunity to test without it, but I will try to set that up as soon as I can. |
Thanks @victornoel - It could well be a genuine 21.0.5 bug, but it could also be a genuine 21.0.5 C2 change that jacoco needs to adjust for (this is a not uncommon problem with trivial versions of Java and tools like it). If we can eliminate jacoco then it's easire for me to get this properly looked at upstream :-) |
@karianna yep, it makes sense, I will try it and come back to you. I'm not blocked by this btw |
I have setup a jacoco-less CI pipeline that runs regularly using maven:3-eclipse-temurin-21, we will see how it behaves after a few days :) |
Hello! I have the same issue in my company's pipeline after making a refactor on the code. Our GitHub action workers with It happens in 90% of the executions of the pipeline for the refactored code. Unfortunately, I can't provide it due to privacy policies, but if I am able to reproduce it, I'll let you know. Cheers! Hereby, I attach the
Update: when I ran the same pipeline with temurin |
@vzedano hi, since you are so "lucky" that it fails often, would you mind checking if it fails with jacoco disabled (if you use it)? Is it also using Jackson and falling on the same place as in my report? Thanks |
Hi @karianna, after a week of running latest jdk 21 with jacoco disabled, I did not see the issue appear. What would be the next step then? Thanks! |
@victornoel - I'd report this to the Jacoco team in that case. https://groups.google.com/g/jacoco |
@karianna will do, thx for all the help! I will report back here before closing this ticket. |
I created jacoco/jacoco#1801 and also posted a message in the ML. |
Hey @karianna, so I'm here acting just as a messenger (:P), it seems that jacoco people are happy to help us finding the problem, but they basically say (see jacoco/jacoco#1801 (comment)) that they don't see how they could fix this in jacoco to avoid such a bug because jacoco is about bytecode enhancement. They did say that many time in the past helped track down JDK bugs and so they can help. Luckily it seems that @Godin may have also encountered that issue at its job but it's hard to reproduce. Ideally if we can get output from a fastdebug build of the JDK, that would greatly help. Do you know if we have such an image at adoptium? Cheers |
++ |
Please provide a brief summary of the bug
Hi, it seems that since we started using eclipse-temurin:21-jdk-noble (21.0.5+11) in our CI (via maven:3-eclipse-temurin-21), we get random JVM crashes during execution of the C2 compiler. It's always with the same trace and always during the compilation of
com.fasterxml.jackson.databind.introspect.POJOPropertiesCollector::_addFields
.I didn't find anything resembling this error in the OpenJDK JIRA nor anywhere on the internet :/
Did you test with the latest update version?
Please provide steps to reproduce where possible
Not sure how, it's quite random
Expected Results
No crash :P
Actual Results
Random crashes :)
What Java Version are you using?
OpenJDK 64-Bit Server VM Temurin-21.0.5+11 (21.0.5+11-LTS, mixed mode, sharing, tiered, compressed oops, compressed class ptrs, g1 gc, linux-amd64)
What is your operating system and platform?
Ubuntu 24.04.1 LTS on x86
How did you install Java?
Docker image
maven:3-eclipse-temurin-21
Did it work before?
Yes, I think in the previous version (21.0.4+7) but it's hard to exactly know.
Did you test with other Java versions?
Because it's using a docker image, I don't have a way to use an older version of Java unfortunately, but I will try to find a way to do that next.
Relevant log output
The text was updated successfully, but these errors were encountered: