Hi Adam, Adding in serviceability-dev as you've made changes in that area too.
Will take a closer look at the changes soon. David ----- On 18/07/2019 2:05 am, Adam Farley8 wrote:
Hey All, Reviewers and sponsors requested to inspect the following. I've re-written the code change, as discussed with David Holes in emails last week, and now the webrev changes do this: - Cause the VM to shut down with a relevant error message if one or more of the sun.boot.library.path paths is too long for the system. - Apply similar error-producing code to the (legacy?) code in linker_md.c. - Allow the numerical parameter for split_path to indicate anything we plan to add to the path once split, allowing for more accurate path length detection. - Add an optional parameter to the os::split_path function that specifies where the paths came from, for a better error message. Bug: https://bugs.openjdk.java.net/browse/JDK-8227021 New Webrev: http://cr.openjdk.java.net/~afarley/8227021.1/webrev/ Best Regards Adam Farley IBM Runtimes Unless stated otherwise above: IBM United Kingdom Limited - Registered in England and Wales with number 741598. Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU
