Skip to content

Clean monitor enter records during VM shutdown#24080

Merged
tajila merged 1 commit into
eclipse-openj9:masterfrom
thallium:mon-enter-clean-up
Jul 16, 2026
Merged

Clean monitor enter records during VM shutdown#24080
tajila merged 1 commit into
eclipse-openj9:masterfrom
thallium:mon-enter-clean-up

Conversation

@thallium

@thallium thallium commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@thallium

thallium commented Jun 4, 2026

Copy link
Copy Markdown
Contributor Author

@tajila can you launch tests again?

@tajila

tajila commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

jenkins test sanity alinux64 jdk21,jdk25

@thallium thallium force-pushed the mon-enter-clean-up branch from 61263ee to 36ac150 Compare June 8, 2026 18:54
@thallium

thallium commented Jun 8, 2026

Copy link
Copy Markdown
Contributor Author

@tajila I moved the clean-up code to protectedDestroyJavaVM(), can you launch the test again if the change looks good to you?

@tajila

tajila commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

jenkins test sanity alinux64 jdk21,jdk25

@thallium

thallium commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

My personal build https://hyc-runtimes-jenkins.swg-devops.com/view/OpenJ9%20-%20Personal/job/Pipeline-Build-Test-Personal/33821/ passed sanity.functional and sanity.openjdk, so I'm expecting the running https://openj9-jenkins.osuosl.org/job/Pipeline_Build_Test_JDK25_aarch64_linux/430/ to pass as well.

Just fixed compile issue on JDK21.

@thallium thallium force-pushed the mon-enter-clean-up branch from 36ac150 to 2e0546a Compare June 9, 2026 13:43
@tajila

tajila commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

jenkins test sanity alinux64 jdk21,jdk25

@thallium

thallium commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

testJITServer failure in https://openj9-jenkins.osuosl.org/job/Pipeline_Build_Test_JDK25_aarch64_linux/430/ seems to be known issue.

@thallium

thallium commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

Also failed on cmdLineTester_jfr_0 Testing: runWorkload 2 - approx 2mins because of timeout:

***[TEST INFO 2026/06/09 17:08:22] ProcessKiller detected a timeout after 3000000 milliseconds!***

Might be the same as #23120.

@tajila

tajila commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

jenkins test sanity alinux64 jdk21,jdk25

1 similar comment
@tajila

tajila commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

jenkins test sanity alinux64 jdk21,jdk25

@tajila

tajila commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

failures in cmdLineTester_jfr_0

@thallium

Copy link
Copy Markdown
Contributor Author

I think it was just a machine being slow or something? I'm rerunning with grinder https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/60906/ and it's passing so far but spent ~10mins on it. I think the extra virtual thread load makes it a lot heavier. When I was running the test with nightly build on my local VM, I even got native OOM for some reason.

@tajila

tajila commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

jenkins test sanity alinux64 jdk21,jdk25

@tajila tajila merged commit 9ad47ec into eclipse-openj9:master Jul 16, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants