RFR: JDK-8256367: [windows] Better logging for some system calls
Yasumasa Suenaga
ysuenaga at openjdk.java.net
Mon Nov 16 06:32:53 UTC 2020
On Mon, 16 Nov 2020 06:17:05 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
>> LGTM. Should the assert in os::split_reserved_memory() be changed to guarantee so it will stop the product build VM?
>
>> LGTM. Should the assert in os::split_reserved_memory() be changed to guarantee so it will stop the product build VM?
>
> Thanks Ioi. I'd rather not make it a guarantee - I find those a bad way to communicate errors to the end user. Ideally, the calling function should decide if it is worth continuing, but that would involve fixing ReservedSpace.(first|last)_part and was out of scope for that little fix.
Looks good, but should all `log_info()` should be replaced to `log_warning()`? I wonder why the failure would be reported as info level originally.
-------------
PR: https://git.openjdk.java.net/jdk/pull/1214
More information about the hotspot-runtime-dev
mailing list