RFR: 8275375: [REDO] JDK-8271949 dumppath in -XX:FlightRecorderOptions does not affect [v3]
Erik Gahlin
egahlin at openjdk.java.net
Mon Oct 25 16:53:08 UTC 2021
On Wed, 20 Oct 2021 00:13:38 GMT, Yasumasa Suenaga <ysuenaga at openjdk.org> wrote:
>> This PR is for redo-ing [JDK-8271949](https://bugs.openjdk.java.net/browse/JDK-8271949).
>> I changed to use `SecuritySupport` at `Options::setDumpPath` as @egahlin mentioned. Please see change both Options.java and SecuritySupport.java .
>>
>> I've tested this change with all jdk/jfr tests on Linux x64, and they works fine.
>> @egahlin @mgronlun Can you run Mach5 tests with this change?
>
> Yasumasa Suenaga has updated the pull request incrementally with one additional commit since the last revision:
>
> Propagate IOException at setDumpPath() to the caller
When I run "java -XX:FlightRecorderOptions:dumppath=incorrect" there seems to be a missing line break between "Cannot write JFR emergency dump to koko" and "Error occurred during initialization of VM"
When I try "$ jcmd <pid> JFR.configure" I see "Repository path: N/A", which is perhaps OK. Then I run "jcmd <pid> JFR.configure dumppath=/directory/somewhere" and it prints "Dump path: /directory/somewhere", but when I run JFR.configure again, it still prints "Repository path: N/A"
-------------
PR: https://git.openjdk.java.net/jdk/pull/6000
More information about the hotspot-jfr-dev
mailing list