-
Notifications
You must be signed in to change notification settings - Fork 721
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
crash vmState=0x0005ffff building xmac jdk22 #18321
Comments
@hzongaro fyi |
If it doesn't work on restart then it blocks accepting new jdk22 levels. The restart also failed |
Failed in the next build as well so this seems a blocker. https://openj9-jenkins.osuosl.org/job/Build_JDKnext_x86-64_mac_OpenJDK/536/ |
I don't see a port library function in the native stack. A C++ exception is raised from
|
Sorry about that - I'm outside of my area of understanding here. For some reason I thought the port library was somehow involved in dealing with exceptions. |
Yeah the jit is expected to recover from any exception that's thrown using Has the version of libunwind changed on mac? |
I had been wondering whether the problem could have somehow been related commit 457d8a3a5, but I can reproduce the crash without that change. I can also reproduce the successful build with the SHAs from https://openj9-jenkins.osuosl.org/job/Pipeline_Build_Test_JDKnext_x86-64_mac/605/, so I'm trying to narrow down what change could have triggered this, in case that provides some clue. |
I've been trying to figure out if https://bugs.llvm.org/show_bug.cgi?id=20800 (llvm/llvm-project@c578567) applies to us (found this issue from this mailing list thread). The xcode version is 12.4 which based on this means |
Thanks, Irwin @dsouzai. I attempted two internal x86-64_mac builds building the compiler: Assuming that I'm setting the option in the right place (our builds are a real black box to me) using Peter @pshipton, I'm not sure how to proceed in verifying whether this really is an instance of the bug that Irwin identified, and whether we should go ahead with this as a work-around. |
@keithc-ca any opinions? @hzongaro if you create a PR with hzongaro@0210b618899 it can be reviewed, PR tested, and merged. |
Do you want to do any perf testing in advance, or release it and wait to see if regular perf testing picks up on the change? I expect it could be restricted to jdk22+ since we don't (yet) see the problem in earlier versions. |
I think that's a good suggestion. |
After thinking about it some more, I no longer think it should be version-specific. If we need |
That's quite true. How would you feel if I made the version-specific change, just to get the JDKNext acceptance build working, while we evaluate the performance impact? |
Sure, that sounds like a reasonable plan. |
https://openj9-jenkins.osuosl.org/job/Build_JDKnext_x86-64_mac_OpenJDK/535
https://openj9-artifactory.osuosl.org/artifactory/ci-openj9/Build_JDKnext_x86-64_mac_OpenJDK/535/Build_JDKnext_x86-64_mac_OpenJDK-535-20231020-140300-diagnostics.tar.gz
The text was updated successfully, but these errors were encountered: