[jdk17] Integrated: 8268602: a couple runtime/os tests don't check exit code
Igor Ignatyev
iignatyev at openjdk.java.net
Fri Jun 11 12:45:18 UTC 2021
On Fri, 11 Jun 2021 09:51:24 GMT, Igor Ignatyev <iignatyev at openjdk.org> wrote:
> Hi all,
>
> could you please review this trivial and test-only patch that adds checks of exit code to two `runtime/os` tests?
> from JBS:
>> `runtime/os/TestUseCpuAllocPath.java` and `AvailableProcessors.java` spawn new JVMs but don't check their exit code which might lead to both type-I and type-II errors
>
> testing: ``runtime/os` on `{linux,windows,macosx}-x64`
>
> Thanks,
> -- Igor
This pull request has now been integrated.
Changeset: cce8da2c
Author: Igor Ignatyev <iignatyev at openjdk.org>
URL: https://git.openjdk.java.net/jdk17/commit/cce8da2c60812873b63de4e65bc992bc6d825fe1
Stats: 4 lines in 2 files changed: 2 ins; 0 del; 2 mod
8268602: a couple runtime/os tests don't check exit code
Reviewed-by: dholmes
-------------
PR: https://git.openjdk.java.net/jdk17/pull/25
More information about the hotspot-runtime-dev
mailing list