eclipse-openj9 / openj9

Eclipse OpenJ9: A Java Virtual Machine for OpenJDK that's optimized for small footprint, fast start-up, and high throughput. Builds on Eclipse OMR (https://github.com/eclipse/omr) and combines with the Extensions for OpenJDK for OpenJ9 repo.
Other
3.28k stars 722 forks source link

jcstress_BoundedSingleTest_0_FAILED NullPointerException: Cannot read field "sync" because "this" is null #16506

Open JasonFengJ9 opened 1 year ago

JasonFengJ9 commented 1 year ago

Failure link

From an internal build(paix823):

openjdk version "19.0.1" 2022-10-18
IBM Semeru Runtime Open Edition 19.0.1+10 (build 19.0.1+10)
Eclipse OpenJ9 VM 19.0.1+10 (build master-a2475ea96, JRE 19 AIX ppc64-64-Bit Compressed References 20221018_15 (JIT enabled, AOT enabled)
OpenJ9   - a2475ea96
OMR      - 7d8ea8b6b
JCL      - 5168276e58 based on jdk-19.0.1+10)

Rerun in Grinder - Change TARGET to run only the failed test targets.

Optional info

Failure output (captured from console output)

[2023-01-01T02:13:10.943Z] variation: NoOptions
[2023-01-01T02:13:10.943Z] JVM_OPTIONS:  

[2023-01-01T02:16:44.483Z]     Unrecoverable error while running
[2023-01-01T02:16:44.483Z]     java.lang.NullPointerException: Cannot read field "sync" because "this" is null
[2023-01-01T02:16:44.483Z]      at java.base/java.util.concurrent.CountDownLatch.countDown(CountDownLatch.java:290)
[2023-01-01T02:16:44.483Z]      at org.openjdk.jcstress.tests.countdownlatch.racy.BoundedSingleTest.actor1(BoundedSingleTest.java:60)
[2023-01-01T02:16:44.483Z]      at org.openjdk.jcstress.tests.countdownlatch.racy.BoundedSingleTest_jcstress.run_actor1(BoundedSingleTest_jcstress.java:256)
[2023-01-01T02:16:44.483Z]      at org.openjdk.jcstress.tests.countdownlatch.racy.BoundedSingleTest_jcstress.task_actor1(BoundedSingleTest_jcstress.java:239)
[2023-01-01T02:16:44.483Z]      at org.openjdk.jcstress.tests.countdownlatch.racy.BoundedSingleTest_jcstress.access$100(BoundedSingleTest_jcstress.java:19)
[2023-01-01T02:16:44.483Z]      at org.openjdk.jcstress.tests.countdownlatch.racy.BoundedSingleTest_jcstress$6.internalRun(BoundedSingleTest_jcstress.java:156)
[2023-01-01T02:16:44.483Z]      at org.openjdk.jcstress.infra.runners.CounterThread.run(CounterThread.java:38)

[2023-01-01T02:16:45.162Z] jcstress_BoundedSingleTest_0_FAILED

This is a new test added on Dec. 2022, and didn't pass before.

Similar failures: jcstress_UnboundedSingleTest_0_FAILED

Also occurred at JDK17

pshipton commented 1 year ago

@JasonFengJ9 all the failures seem to be on AIX. Is the test running and passing on other platforms?

JasonFengJ9 commented 1 year ago

all the failures seem to be on AIX. Is the test running and passing on other platforms?

Added AIX label, the test in question passed in other platforms.

pshipton commented 1 year ago

@llxia fyi @JasonFengJ9 does this pass on OpenJDK?

JasonFengJ9 commented 1 year ago

does this pass on OpenJDK?

13:07:04  openjdk version "19.0.1-beta" 2022-10-18
13:07:04  OpenJDK Runtime Environment Temurin-19+36-202210180349 (build 19.0.1-beta+36-202210180349)
13:07:04  OpenJDK 64-Bit Server VM Temurin-19+36-202210180349 (build 19.0.1-beta+36-202210180349, mixed mode)

5x grinder w/ RI all passed

Note: this is a temurin build on Oct. 2022, there are no newer JDK19 AIX nightly builds available.

10x grinder w/ latest JDK17 temurin nightly - all passed

14:05:23  openjdk version "17.0.6-beta" 2023-01-17
14:05:23  OpenJDK Runtime Environment Temurin-17.0.6+9-202301032339 (build 17.0.6-beta+9-202301032339)
pshipton commented 1 year ago

I was just about to ask for 17 but I see it's been added to the previous comment.

pshipton commented 1 year ago

I see 17 Temurin passed as well.

pshipton commented 1 year ago

I don't know what component is at fault. @JasonFengJ9 can you try one more on OpenJ9 with -Xint pls.

JasonFengJ9 commented 1 year ago

JDK17 -Xint 10x grinder - all passed

0xdaryl commented 1 year ago

Moving out to 0.40.

@zl-wang : since this issue only appeared on AIX could you assign for investigation please?

zl-wang commented 1 year ago

@midronij please take a look ...

0xdaryl commented 1 year ago

Moving to 0.41.

hzongaro commented 1 year ago

@midronij, when you have the opportunity, may I ask you to provide an update on the investigation of this problem? I'll move this to the 0.43 release in the meanwhile.

midronij commented 1 year ago

@hzongaro very sorry for the delay. Unfortunately, as I've been occupied with other work, I haven't had much chance to investigate this issue in earnest. If it's alright, I will pick this item back up again as soon as I'm able.

zl-wang commented 1 year ago

@bhavanisn please take this over (at lower priority)