RFR (S) 8213137: Remove static initialization of monitor/mutex instances
Thomas Schatzl
thomas.schatzl at oracle.com
Mon Nov 5 13:05:23 UTC 2018
Hi,
On Mon, 2018-11-05 at 11:43 +1000, David Holmes wrote:
> This impacts GC, compiler, runtime and serviceability.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8213137
> webrev: http://cr.openjdk.java.net/~dholmes/8213137/webrev/
>
> To facilitate changes to the Mutex/Monitor code we need to ensure
> there are no statically initialized Monitor/Mutex instances in the VM
> - as the constructors may rely on VM initialization that has not
> happened when C++ initializers execute.
>
> There were 6 locally defined "lock" members of classes that were
> statically initialized, and these are across all areas of the VM:
>
> - Decoder::_shared_decoder_lock
> - DCmdFactory::_dcmdFactory_lock
> - JfrThreadSampler::_transition_block_lock
> - NMethodSweeper::_stat_lock
> - ThreadsSMRSupport::_delete_lock
> - CodeCacheUnloadingTask::_lock
>
> The last is actually now unused and so is just deleted. The rest
Removal of the CodeCacheUnloadingTask lock looks good. I also looked
through the other changes which seems simple enough to me too, but
should obviously be looked at from people from the respective areas
too.
Some nits:
Maybe the spacing in mutexLocker.cpp:325 for the JfrThreadSampler_lock
could be fixed though.
Also the assert in decoder.cpp:94 seems to be as superfluous to me as
in the other lines where it has been removed.
Thanks,
Thomas
More information about the hotspot-dev
mailing list