Skip to content

zLinux ASSERTION FAILED VMAccess.cpp:355: (0 == ((vmThread)->publicFlags & J9_PUBLIC_FLAGS_VM_ACCESS)) #17334

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

Closed
pshipton opened this issue May 4, 2023 · 3 comments

Comments

@pshipton
Copy link
Member

pshipton commented May 4, 2023

https://openj9-jenkins.osuosl.org/job/Test_openjdk8_j9_sanity.openjdk_s390x_linux_Nightly/535/
java/lang/ThreadGroup/Suspend.java

20:02:29  00:01:29.274 0x17d7400    j9vm.228    *   ** ASSERTION FAILED ** at /home/jenkins/workspace/Build_JDK8_s390x_linux_Nightly/openj9/runtime/vm/VMAccess.cpp:355: (0 == ((vmThread)->publicFlags & J9_PUBLIC_FLAGS_VM_ACCESS))

This was also seen in the last OMR acceptance build, see https://openj9.slack.com/archives/CDS7QE9HB/p1683147702449159

Changes from last nightly build:
275bb5a...0fe9c4b
eclipse-openj9/openj9-omr@7ddb743...5dcc2f2

Since it was seen in OMR acceptance we can rule out some OpenJ9 changes. Also the OMR changes were already tested in the previous OMR acceptance without any failures.
275bb5a...aa5727e

I think #16943 must be the cause. It changes some non Java 19 specific code in Java_java_lang_Thread_resumeImpl

@dipak-bagadiya @babsingh Unless there is a quick fix we should revert this.

@babsingh
Copy link
Contributor

babsingh commented May 4, 2023

I am in favour of reverting it. We did see a similar error in JDK19 but not in JDK20: #16943 (comment). @dipak-bagadiya was unable to reproduce the error with JDK20 locally. We will re-merge the code changes once this failure is resolved. Created #17335 to revert the code changes from #16943.

#16689 has been re-opened. adoptium/aqa-tests#4560 has been opened to again exclude WaitNotifySuspendedVThreadTest.

@pshipton
Copy link
Member Author

pshipton commented May 4, 2023

Since the change is reverted, this can be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants