Integrated: 8263017: Read barriers are missing in nmethod printing code

[Vladimir Ivanov]

On Wed, 10 Mar 2021 17:59:30 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

> OOP accesses from nmethod printing code miss read barriers. 
> It causes sporadic crashes when the code stumbles upon a stale OOP.
> 
> When `nmethod::print_nmethod` is invoked, the corresponding nmethod is already installed in the code cache and there was a safepoint check in between (see `ThreadToNativeFromVM` usage in `CompileBroker::invoke_compiler_on_method`).
>  
> The fix adds missing read barriers in `oop_Relocation::oop_value()` and introduces `oop_Relocation::raw_oop_value()` for cases when read barriers aren't needed.
> 
> Along the way, there are some cleanups applied (e.g, `nmethod::print_[oops|metadata]` is superseded by `nmethod::print_recorded_[oops|metadata]`).
> 
> Testing:
> - [x] failing test w/ `-XX:+UseZGC` `-XX:ZCollectionInterval=0.01` `-XX:ZFragmentationLimit=0`
> - [x] hs-tier1 - hs-tier4
> 
> Thanks!

This pull request has now been integrated.

Changeset: 0b10c6b4
Author:    Vladimir Ivanov <vlivanov at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/0b10c6b4
Stats:     2 lines in 1 file changed: 2 ins; 0 del; 0 mod

8263017: Read barriers are missing in nmethod printing code

Reviewed-by: eosterlund, kvn

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

PR: https://git.openjdk.java.net/jdk/pull/2919