RFR: 8272345: macos doesn't check `os::set_boot_path()` result

David Holmes dholmes at openjdk.java.net
Thu Aug 12 01:08:24 UTC 2021


On Wed, 11 Aug 2021 22:23:42 GMT, Igor Ignatyev <iignatyev at openjdk.org> wrote:

> Hi all,
> 
> could you please review this small fix?
> 
> from JBS:
>> we don't check `os::set_boot_path()` return value on macos, which leads to a crash/assert down the road in case we weren't able to set BCP. on all other OSes, we terminate JVM w/ "Failed setting boot class path." error message.
> 
> testing:
> - [x] tier1
> - [x] `runtime/cds/appcds/MoveJDKTest.java`
> 
> 
> Thanks,
> -- Igor

Hi Igor,

Functional fix looks good. One suggestion with the test.

Thanks,
David

test/hotspot/jtreg/runtime/cds/appcds/MoveJDKTest.java line 30:

> 28:  * @bug 8272345
> 29:  * @requires vm.cds
> 30:  * @requires os.family == "linux" | os.family == "mac"

The test probably works everywhere but Windows if the issue is only symbolic links. If problems were reported on AIX or generic BSD then we could address that then.

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

Marked as reviewed by dholmes (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/5094


More information about the hotspot-runtime-dev mailing list