On Mon, 7 Nov 2022 20:40:33 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:

>> This patch moves the acquisition of the boot class loader lock out of the 
>> JVM and into the Java function.
>> Tested with tier1-4, and jvmti and jdi tests locally.
>
> Coleen Phillimore has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   really revert the file

> We might not need this. appendClasspath is thread safe so it's okay for 
> several
> agents calling appendToSystemClassLoaderSearch at around the same time.
> I don't think it needs to sycnrhonize with anything else.

I'm thinking about a good place where to place a comment about this.
Probably, it can be placed before the method 
`appendToClassPathForInstrumentation`.

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

PR: https://git.openjdk.org/jdk/pull/11023

Reply via email to