JDK-8219370 : NMT: Move synchronization primitives from mtInternal to mtSynchronizer
Type:Sub-task
Component:hotspot
Sub-Component:runtime
Affected Version:8,11,13
Priority:P4
Status:Resolved
Resolution:Fixed
Submitted:2019-02-19
Updated:2020-02-19
Resolved:2019-02-20
The Version table provides details related to the release that this issue/RFE will be addressed.
Unresolved : Release in which this issue/RFE will be addressed. Resolved: Release in which this issue/RFE has been resolved. Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.
Create separate tracking category for synchronization primitives, e.g. mutex, monitor and semaphore, etc.
Comments
The risk of backporting JDK-8174231 seems unjustified for backporting this one.
19-02-2020
Removing jdk8u-fix-request as there is still no response to my comments on this from August 2019:
https://mail.openjdk.java.net/pipermail/jdk8u-dev/2019-August/010101.html
In short, JDK-8208499 & JDK-8174231 should be considered first.
18-02-2020
Fix Request (8u)
I would like to backport this patch to 8u.
This patch aggregated synchronization primitives under a new memory category, which improves NMT usability.
The patch does not apply cleanly, due to:
1) PlatformMonitor is not in 8u code base
2) PlatformEvent and PlatformParker have yet been factored out to Posix for compliant OSs
3) semaphore is defined as StackObj in 8u
4) Memory type declaration is not compatible
Webrev: http://cr.openjdk.java.net/~zgu/JDK-8219370-8u/webrev.00/index.html (Reviewed)
20-08-2019
Fix Request (11u)
I would like to backport this patch to 11u.
This patch aggregated synchronization primitives under a new memory category, which improves NMT usability.
The patch does not apply cleanly, due to:
1) PlatformMonitor no longer in 11u code base
2) JDK-8210246 changed how new memory category is defined, and it does not look like in 11u backport list.
Webrev: http://cr.openjdk.java.net/~zgu/JDK-8219370-11u/webrev.00/index.html