RFR: 8283223: gc/stringdedup/TestStringDeduplicationFullGC.java#Parallel failed with "RuntimeException: String verification failed"

[Thomas Schatzl]

On Fri, 15 Mar 2024 14:44:03 GMT, Leo Korinth <lkorinth at openjdk.org> wrote:

> I have tried to reproduce the problem by adding load to the system but I have failed. Thomas have reproduced similar problems by adding some fairly large delays in the string deduplication service thread (https://bugs.openjdk.org/browse/JDK-8275684). In this patch I added the `systemLoad()` method and the `minMax()` method so that we can log the load when the problem appears. If the failure reappears (I can see no resent failures). We can use those methods to make the test pass on too high load.
> 
> `getSystemLoadAverage()` is a bit blunt and gives an average over one minute, but ought to work if we eventually would like to silence test failures on over-commissioned machines.
> 
> Do you think this is okay, or should I change the test to directly skip the testcase instead of adding the load log on failure, maybe that is better?

Lgtm. Thanks for adding diagnosis code.

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

Marked as reviewed by tschatzl (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/18327#pullrequestreview-1945782519