On Mon, 25 Nov 2024 13:41:51 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
> These test-only changes look OK to me. Is it intentional that only one of the > test definitions of `SleepALot` has been updated to remove `/othervm` instead > of both? There are a few tests that override the default timeout with othervm/timeout=NNN. I didn't change them but if there aren't other settings set then I think jtreg will allow it (in these tests, the only reason to fork a new VM is because of VM options (mostly) and a few that need other configuration). So I'll leave it for now, but thanks for bringing it up. ------------- PR Comment: https://git.openjdk.org/jdk/pull/22357#issuecomment-2498109794
