Skip to content
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 intermittent LogGeneratedClassesTest failures #18968

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

singh264
Copy link
Contributor

Fix intermittent LogGeneratedClassesTest failures by
removing use of lambda expression in static block of
InternalCRIUSupport, which avoids the unexpected
creation of lambda class files for InternalCRIUSupport
during the test.

Issue: #18556
Signed-off-by: Amarpreet Singh Amarpreet.A.Singh@ibm.com

singh264 added a commit to singh264/aqa-tests that referenced this pull request Feb 16, 2024
Depends-on: eclipse-openj9/openj9#18968
Issue: eclipse-openj9/openj9#18556
Signed-off-by: Amarpreet Singh <Amarpreet.A.Singh@ibm.com>
@singh264
Copy link
Contributor Author

@tajila requesting your review.

Could openj9 grinders be started on a x86-64 linux machine ub16x64j95 to reproduce the intermittent LogGeneratedClassesTest failures and verify if the code changes resolve the issue? I am unable to start openj9 grinders due to lack of permissions.

It seems like the test failures would be reproducible in openj9 grinders on ub16x64j95, where the test has previously failed and where I have also reproduced the issue locally. The test failures are not reproducible in the internal grinders on x86-64 linux machines when the test is run 1000 times for jdk11, jdk17 and jdk21.

@singh264 singh264 marked this pull request as ready for review February 16, 2024 17:49
@tajila
Copy link
Contributor

tajila commented Feb 16, 2024

Im not sure this is the right approach. The test used to pass before, so changing the test is not the solution. Unless the test was always broken?

@pshipton
Copy link
Member

@TobiAjila this isn't a test change.

@tajila
Copy link
Contributor

tajila commented Feb 16, 2024

My mistake, didnt look carefully

@tajila
Copy link
Contributor

tajila commented Feb 16, 2024

We only need unsafe for the registerRestoreEnvVariables so we can put that init code in a function and call when the restore hook is registered. That way we can keep the static init empty.

@singh264 singh264 force-pushed the openj9_issues_18556 branch 2 times, most recently from 443d49f to 5c1e9e5 Compare February 17, 2024 02:44
@singh264
Copy link
Contributor Author

@tajila re-requesting your review.

Fix intermittent LogGeneratedClassesTest failures by
removing use of lambda expression in static block of
InternalCRIUSupport, which avoids the unexpected
creation of lambda class files for InternalCRIUSupport
during the test.

Issue: eclipse-openj9#18556
Signed-off-by: Amarpreet Singh <Amarpreet.A.Singh@ibm.com>
@singh264
Copy link
Contributor Author

@tajila re-requesting your review.

@tajila
Copy link
Contributor

tajila commented Feb 20, 2024

jenkins test sanity plinux jdk17

@tajila
Copy link
Contributor

tajila commented Feb 20, 2024

jenkins test sanity alinux64 jdk21

@singh264
Copy link
Contributor Author

jenkins test sanity plinux jdk17
jenkins test sanity alinux64 jdk21

It seems like the ppc64le linux build and the aarch64 linux build failed due to omr changes and openj9 changes that were recently reverted.

10:22:33  /home/jenkins/workspace/Build_JDK17_ppc64le_linux_Personal/omr/compiler/optimizer/ValuePropagationCommon.cpp:2360:37: error: 'java_lang_StringCoding_implEncodeAsciiArray' is not a member of 'TR'; did you mean 'java_lang_StringCoding_implEncodeISOArray'?
10:22:33   2360 |    bool isAsciiEncoder = (rm == TR::java_lang_StringCoding_implEncodeAsciiArray) && !disableImplEncodeAsciiArray;
10:22:33        |                                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
10:22:33        |                                     java_lang_StringCoding_implEncodeISOArray

@tajila
Copy link
Contributor

tajila commented Feb 20, 2024

jenkins test sanity alinux64 jdk21

@tajila
Copy link
Contributor

tajila commented Feb 20, 2024

jenkins test sanity plinux jdk17

@singh264
Copy link
Contributor Author

jenkins test sanity alinux64 jdk21
jenkins test sanity plinux jdk17

The ppc64le linux build passed, but the aarch64 linux build failed due to a testDateScheduledBeforeCheckpointDone timeout.

Testing: Create CRIU checkpoint image and restore three times - testDateScheduledBeforeCheckpointDone
Test start time: 2024/02/20 20:11:07 Coordinated Universal Time
Running command: bash /home/jenkins/workspace/Test_openjdk21_j9_sanity.functional_aarch64_linux_Personal_testList_0/aqa-tests/TKG/../../jvmtest/functional/cmdLineTests/criu/criuScript.sh /home/jenkins/workspace/Test_openjdk21_j9_sanity.functional_aarch64_linux_Personal_testList_0/aqa-tests/TKG/../../jvmtest/functional/cmdLineTests/criu /home/jenkins/workspace/Test_openjdk21_j9_sanity.functional_aarch64_linux_Personal_testList_0/jdkbinary/j2sdk-image/bin/java " -Xgcpolicy:optthruput " "org.openj9.criu.TimeChangeTest testDateScheduledBeforeCheckpointDone" 3 3 false false
Time spent starting: 11 milliseconds
***[TEST INFO 2024/02/20 20:16:07] ProcessKiller detected a timeout after 300000 milliseconds!***
***[TEST INFO 2024/02/20 20:16:07] executing /usr/bin/gdb -batch -x /tmp/debugger687231450370899115.txt bash 215349***
GDB OUT No shared libraries loaded at this time.
INFO: Running '/usr/bin/gdb' failed with rc = 1
GDB ERR Could not attach to process.  If your uid matches the uid of the target
GDB ERR process, check the setting of /proc/sys/kernel/yama/ptrace_scope, or try
GDB ERR again as the root user.  For more details, see /etc/sysctl.d/10-ptrace.conf
GDB ERR ptrace: Operation not permitted.
GDB ERR /home/jenkins/workspace/Test_openjdk21_j9_sanity.functional_aarch64_linux_Personal_testList_0/aqa-tests/TKG/output_1708455580932/cmdLineTester_criu_nonPortableRestore_5/215349: No such file or directory.
GDB ERR /tmp/debugger687231450370899115.txt:2: Error in sourced command file:
GDB ERR The program has no registers now.

INFO: Sleep for 60000 millis before next capture.
***[TEST INFO 2024/02/20 20:17:07] executing /usr/bin/gdb -batch -x /tmp/debugger687231450370899115.txt bash 215349***
GDB OUT No shared libraries loaded at this time.
INFO: Running '/usr/bin/gdb' failed with rc = 1
GDB ERR Could not attach to process.  If your uid matches the uid of the target
GDB ERR process, check the setting of /proc/sys/kernel/yama/ptrace_scope, or try
GDB ERR again as the root user.  For more details, see /etc/sysctl.d/10-ptrace.conf
GDB ERR ptrace: Operation not permitted.
GDB ERR /home/jenkins/workspace/Test_openjdk21_j9_sanity.functional_aarch64_linux_Personal_testList_0/aqa-tests/TKG/output_1708455580932/cmdLineTester_criu_nonPortableRestore_5/215349: No such file or directory.
GDB ERR /tmp/debugger687231450370899115.txt:2: Error in sourced command file:
GDB ERR The program has no registers now.

***[TEST INFO 2024/02/20 20:17:08] executing kill -ABRT 215349***
Time spent executing: 360797 milliseconds
Test result: FAILED
Output from test:
 [OUT] start running script
 [OUT] export GLIBC_TUNABLES=glibc.cpu.hwcaps=-XSAVEC,-XSAVE,-AVX2,-ERMS,-AVX,-AVX_Fast_Unaligned_Load
 [OUT] export LD_BIND_NOT=on
 [OUT] /home/jenkins/workspace/Test_openjdk21_j9_sanity.functional_aarch64_linux_Personal_testList_0/jdkbinary/j2sdk-image/bin/java -XX:+EnableCRIUSupport  -Xgcpolicy:optthruput  -cp /home/jenkins/workspace/Test_openjdk21_j9_sanity.functional_aarch64_linux_Personal_testList_0/aqa-tests/TKG/../../jvmtest/functional/cmdLineTests/criu/criu.jar org.openj9.criu.TimeChangeTest testDateScheduledBeforeCheckpointDone 3 3
 [ERR] /home/jenkins/workspace/Test_openjdk21_j9_sanity.functional_aarch64_linux_Personal_testList_0/aqa-tests/TKG/../../jvmtest/functional/cmdLineTests/criu/criuScript.sh: line 41: 215350 Killed                  $2 -XX:+EnableCRIUSupport $3 -cp "$1/criu.jar" $4 $5 $6 > testOutput 2>&1

***[TEST INFO 2024/02/20 20:17:08] kill -ABRT signal sent***
***[TEST INFO 2024/02/20 20:17:08] ABRT completed***

@tajila tajila merged commit f3b5cbd into eclipse-openj9:master Feb 21, 2024
5 of 7 checks passed
JasonFengJ9 pushed a commit to adoptium/aqa-tests that referenced this pull request Feb 21, 2024
Depends-on: eclipse-openj9/openj9#18968
Issue: eclipse-openj9/openj9#18556

Signed-off-by: Amarpreet Singh <Amarpreet.A.Singh@ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants