On Thu, 1 Jun 2023 12:37:20 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> This is the implementation for JEP 451. There are two parts to this: >> >> 1. A multi-line warning is printed when a JVM TI or Java agent is loaded >> into a running VM. For JVM TI, the message is printed to stderr from >> JvmtiAgent::load. For Java agents, it is printed to System.err (as that may >> be redirected) in the JPLIS (j.l.instrumentation) implementation. This part >> includes an update to the JVM TI spec and API docs to require the warning. >> >> 2. If running with -Djdk.instrument.traceUsage or >> -Djdk.instrument.traceUsage=true, the calls to the Instrumentation API print >> a trace message and stack trace. >> >> Testing: tier1-6 > > Alan Bateman has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brought in by the merge/rebase. The pull request contains 19 additional > commits since the last revision: > > - Split is_loaded, typos in comments > - Merge > - Add impl note to document the XX option > - Cleanup > - Merge > - Allow for warning to be skipped when same agent loaded a second/subsequent > time > - Merge > - Tweak javadoc, update test to use more test infra > - Merge > - Merge > - ... and 9 more: https://git.openjdk.org/jdk/compare/25f0fc82...cbd9bb1d Except for the possible addition of some comments regarding what is meant by "loaded", the changes look good. ------------- Marked as reviewed by cjplummer (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/13899#pullrequestreview-1455815464