[jdk17] RFR: 8268602: a couple runtime/os tests don't check exit code [v2]

Igor Ignatyev iignatyev at openjdk.java.net
Fri Jun 11 12:45:17 UTC 2021


> 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

Igor Ignatyev has updated the pull request incrementally with one additional commit since the last revision:

  check exit code first

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

Changes:
  - all: https://git.openjdk.java.net/jdk17/pull/25/files
  - new: https://git.openjdk.java.net/jdk17/pull/25/files/abaabc29..1c3cae32

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk17&pr=25&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk17&pr=25&range=00-01

  Stats: 4 lines in 2 files changed: 2 ins; 2 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk17/pull/25.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk17 pull/25/head:pull/25

PR: https://git.openjdk.java.net/jdk17/pull/25


More information about the hotspot-runtime-dev mailing list