RFR: 8291555: Implement alternative fast-locking scheme [v25]

Roman Kennke rkennke at openjdk.org
Tue Mar 14 18:22:30 UTC 2023


On Tue, 14 Mar 2023 18:16:36 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

> I kicked off a round of Mach5 Tier1 testing last night. I got 133 SA test failures that are probably fixed by v24. runtime/logging/MonitorInflationTest.java also failed on all 5 configs tested in Tier1:
> 
> ```
> java.lang.RuntimeException: 'inflate(has_locker):' missing from stdout/stderr
> 	at jdk.test.lib.process.OutputAnalyzer.shouldContain(OutputAnalyzer.java:221)
> 	at MonitorInflationTest.analyzeOutputOn(MonitorInflationTest.java:41)
> 	at MonitorInflationTest.main(MonitorInflationTest.java:56)
> 	at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103)
> 	at java.base/java.lang.reflect.Method.invoke(Method.java:578)
> 	at com.sun.javatest.regtest.agent.MainActionHelper$AgentVMRunnable.run(MainActionHelper.java:312)
> 	at java.base/java.lang.Thread.run(Thread.java:1623)
> ```
> 
> I suspect that the failing condition is one that I added to the test a long time ago so I'll be taking a look.

Aww, that is bad timing. I pushed a change yesterday that broke SA, and I only pushed a fix 2 hours ago. It should be good now, in case you want to try it again.

Thank you for your effort to review and test this change!

Roman

-------------

PR: https://git.openjdk.org/jdk/pull/10907


More information about the serviceability-dev mailing list