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

[Leo Korinth]

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?

This pull request has now been integrated.

Changeset: 9d441e3e
Author:    Leo Korinth <lkorinth at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/9d441e3e56e733d64dbadde6854761e4545c1c70
Stats:     44 lines in 1 file changed: 32 ins; 4 del; 8 mod

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

Reviewed-by: tschatzl, kbarrett

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

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