RFR: 8353190: Use "/native" Run Option for TestAvailableProcessors Execution [v2]
Oleksii Sylichenko
duke at openjdk.org
Tue Apr 22 19:51:24 UTC 2025
> Currently, the test executes the program using `test.nativepath`, but it relies on path resolution. I propose following standard conventions in this case and running the test with the `/native` option instead.
>
> With this change:
> - Path resolution is no longer required.
> - If `nativepath` is not specified at runtime, a standard error message will be printed to the console.
>
> This improves consistency and simplifies the test execution process.
>
> I kindly ask the original author of the test @swesonga and those who reviewed the original pull request #17576 for this test to take a look at this PR: @jdksjolen and @dholmes
Oleksii Sylichenko has updated the pull request incrementally with one additional commit since the last revision:
8353190: remove unnecessary option `--enable-native-access=ALL-UNNAMED`
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/23803/files
- new: https://git.openjdk.org/jdk/pull/23803/files/13a0160b..0d002f22
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=23803&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=23803&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.org/jdk/pull/23803.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/23803/head:pull/23803
PR: https://git.openjdk.org/jdk/pull/23803
More information about the hotspot-runtime-dev
mailing list