RFR: 8266056: runtime/stringtable/StringTableCleaningTest.java failed with "RuntimeException: Missing Callback in [10, 11]" [v2]

[David Holmes]

On Fri, 30 Apr 2021 13:08:15 GMT, Per Liden <pliden at openjdk.org> wrote:

>> This patch addresses two issues with the StringTableCleaningTest test.
>> 
>> 1) The test doesn't fully take into account that a concurrent GC might not complete (it can abort if the VM wants to terminate). findCallback() should return -1 if no callbacks are found, instead of failing the test.
>> 
>> 2) The regexp patterns used for ZGC needs to be more exact. The end pattern now also matches aborted GCs, which it shouldn't. This leads to "Garbage Collection (*) Aborted" being incorrectly matched as a GC end.
>> 
>> This patch also makes sure the output from the test VM ends up in the test log, otherwise it's impossible to tell what went wrong if this test fails in the future.
>> 
>> This patch also removes the test from the ProblemList.
>> 
>> Testing:
>> * Passed Tier1-3.
>> * Manual testing of StringTableCleaningTest with various GCs.
>
> Per Liden has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Review comments from David

I had assumed you wanted to see the output even on success. :)

Update is fine.

Thanks,
David

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

Marked as reviewed by dholmes (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/3815