RFR: 8314830: runtime/ErrorHandling/ tests ignore external VM flags

David Holmes dholmes at openjdk.org
Mon Sep 18 01:57:42 UTC 2023


On Fri, 15 Sep 2023 17:58:43 GMT, Mikhailo Seledtsov <mseledtsov at openjdk.org> wrote:

> 8314830: runtime/ErrorHandling/ tests ignore external VM flags

I agree with the reasoning here, except for VeryEarlyAssert.

Thanks.

test/hotspot/jtreg/runtime/ErrorHandling/VeryEarlyAssertTest.java line 52:

> 50: 
> 51: 
> 52:     ProcessBuilder pb = ProcessTools.createTestJvm("-version");

I don't think this is necessary. IIUC this test triggers an assertion failure during the static initialization that occurs in debug.o which will happen when libjvm is loaded, well before any argument processing could occur, or have any impact.

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

Changes requested by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/15767#pullrequestreview-1630197266
PR Review Comment: https://git.openjdk.org/jdk/pull/15767#discussion_r1328193835


More information about the hotspot-runtime-dev mailing list