RFR: 8217717: ZGC: Broken oop map in C1 load barrier stub

[Per Liden]

On 1/24/19 12:05 PM, Per Liden wrote:
> The C1 load barrier stub for ZGC can emit a patching area for a lea 
> instruction. However, the call into the VM to do the patching at this 
> site is not associated with a proper oop map. As a result, we can end up 
> with a stable oop in the base pointer register if a GC safepoint happens 

s/stable/stale/ ;)

> when we enter the VM to patch.
> 
> This problem is not super easy to reproduce by default, but is a serious 
> bug when it happens. However, it is fairly easy to reproduce when also 
> injecting a small delay in the ZNMethodTable::register_nmethod().
> 
> The proposed solution is to never allow patching to happen inside the 
> load barrier stub. Instead, if patching is needed, we emit the lea 
> instruction as part of the load preceding the load barrier. That will 
> automatically generate the proper oop map for the patch site.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8217717
> Webrev: http://cr.openjdk.java.net/~pliden/8217717/webrev.0
> 
> Testing: Manual testing and inspection of generated code. Passes 
> gc-test-suite. Currently running tier1-3 on all platforms, and tier1-7 
> on Linux/x86.
> 
> /Per