[14] RFR (XS): 8226409: Enable argument profiling for sun.misc.Unsafe.put*/get*

[Vladimir Kozlov]

Changes are good and trivial.

But can you explain why do we care about sun.misc.Unsafe in current JDK? All should use jdk.internal.misc.Unsafe.
May be this issue is good motivation to switch to jdk.internal.misc.Unsafe?

Thanks,
Vladimir

On 7/3/19 2:01 PM, Vladimir Ivanov wrote:
> http://cr.openjdk.java.net/~vlivanov/8226409/webrev.00/
> https://bugs.openjdk.java.net/browse/JDK-8226409
> 
> JDK-8181211 [1] enabled argument profiling for jdk.internal.misc.Unsafe to recuperate performance regression when 
> off-heap/on-heap accesses are treated as mixed accesses (due to recent bug fixes in that area).
> 
> Profiling data allows to speculatively optimize for on-heap or off-heap scenario and avoid memory barriers around unsafe 
> accesses which are mandatory in mixed case.
> 
> Unfortunately, it doesn't help sun.misc.Unsafe at all.
> 
> sun.misc.Unsafe delegates to jdk.internal.misc.Unsafe and profiling data is mostly useless there since it is almost 
> always polluted.
> 
> Proposed fix is to enable argument profiling for sun.misc.Unsafe as well.
> 
> Testing: hs-precheckin-comp, tier1-4
> 
> Thanks!
> 
> Best regards,
> Vladimir Ivanov
> 
> [1] https://bugs.openjdk.java.net/browse/JDK-8181211