RFR: 8257967: JFR: Events for loaded agents [v15]

[Markus Grönlund]

On Sat, 1 Apr 2023 03:47:26 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:

>> Markus Grönlund has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   fixes
>
> src/hotspot/share/prims/agentList.cpp line 204:
> 
>> 202: 
>> 203: // Invokes Agent_OnAttach for agents loaded dynamically during runtime.
>> 204: jint AgentList::load_agent(const char* agent_name, const char* absParam,
> 
> I feel that it is better to keep the original function name "load_agent_library". As you listed there two kinds of agents: Java and Native. The function name give a hint it is native agent. Also, it is better to avoid changes that aren't really necessary.

I changed the names because I found it very hard to understand what the old names represented: "AgentLibrary" vs "Library"? "add_init_agent" vs "add_instrumentation_agent", or even "add_loaded_agent"? Also a bit confusing that "load_agent_library" would also include statically linked agents - no library is loaded there.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/12923#discussion_r1155930115