-
Notifications
You must be signed in to change notification settings - Fork 737
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
cmdLineTester_criu_nonPortableRestore MethodTypeDeadlockTest crash #18800
Comments
@TobiAjila pls help direct this. |
https://openj9-jenkins.osuosl.org/job/Test_openjdk17_j9_sanity.functional_aarch64_linux_OMR_testList_1/433
|
This may be a regression. @ThanHenderson can you please take a look at this |
I reproduced a segmentation fault for For I'll investigate to see if this is indeed a recent regression or just such an intermittent problem that it hadn't shown up for a while. |
This is an issue with the tracing mechanism, and doesn't show up when using |
This is what a normal
But in the failing cases the openj9/runtime/rastrace/trclog.c Line 2140 in 5d03ee9
produces garbage data, similar to what is seen below, which causes problems at various points downstream in the tracing pipeline
|
After a long period of testing and a pseudo-binary search of commits not included in 0.43 (which doesn't crash), I've identified #18745 as the commit that introduces this regression. |
I will have a look it. |
This is actually not the case. After that commit, they just seem to happen more frequently. Running the commit before for ~50,000 iterations I observed the failure 4 times. |
The Line 2878 in 43f3707
openj9/runtime/vm/CRIUHelpers.cpp Line 903 in 43f3707
I was able to reproduce the crash in a fyre x64 Linux image ~10/1000 runs, removing the @ThanHenderson could you try a build in your test environment? |
This patch fixes eclipse-openj9#18800. When the CRIUSupport APIs were moved into java.base, the line that previously registered the j9criu module for tracing was not removed. Therefore, the j9vm module is being double registered, causing downstream effects in the tracing mechanism. This patch removes the registration during the CRIU API setup phase. Issues: eclipse-openj9#18800 Signed-off-by: Nathan Henderson <[email protected]>
https://openj9-jenkins.osuosl.org/job/Test_openjdk11_j9_sanity.functional_aarch64_linux_Nightly_testList_0/588/
cmdLineTester_criu_nonPortableRestore_8
-Xgcpolicy:gencon -Xgcthreads1
https://openj9-artifactory.osuosl.org/artifactory/ci-openj9/Test/Test_openjdk11_j9_sanity.functional_aarch64_linux_Nightly_testList_0/588/functional_test_output.tar.gz
The text was updated successfully, but these errors were encountered: