RFR: 8337396: Cleanup usage of ExternalAddess [v5]

[Vladimir Ivanov]

On Fri, 2 Aug 2024 15:56:05 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> `ExternalAddess` should be used only for data load. For calls (and jump) instructions we should use `RuntimeAddress` which uses `runtime_call_Relocation`.
>> 
>> I found few places where `ExternalAddess` is used incorrectly and fixed them.
>> 
>> I also added code to print "hottest" (most referenced) `ExternalAddess` addresses in global table to move them into static global tables which will be introduced by [JDK-8334691](https://bugs.openjdk.org/browse/JDK-8334691) and [JDK-8337519](https://bugs.openjdk.org/browse/JDK-8337519).
>> 
>> Here is current output from debug VM on MacBook M1 (Aarch64):
>> 
>> External addresses table: 6 entries, 324 accesses
>> 0:      158 0x00000001082de0f0 : extn: vmClasses::_klasses+480
>> 1:       84 0x00000001082ddf20 : extn: vmClasses::_klasses+16
>> 2:       40 0x00000001082c4790 : extn: SharedRuntime::_partial_subtype_ctr
>> 3:       24 0x00000001082bdb04 : extn: JvmtiExport::_should_notify_object_alloc
>> 4:       18 0x0000000118384080 : stub: forward exception
>> 
>> 
>> on MacOS-x64:
>> 
>> External addresses table: 143 entries, 44405 accesses
>> 0:    11766 0x00000001047922a0 : extn: CompressedOops::_narrow_oop
>> 1:    11002 0x0000000104474384 : 'should not reach here'
>> 2:     9672 0x0000000104581a90 : extn: ClassLoader::file_name_for_class_name(char const*, int)::class_suffix+882068
>> 3:     2447 0x0000000104508005 : extn: ClassLoader::file_name_for_class_name(char const*, int)::class_suffix+383753
>> 4:     1916 0x000000010458188e : extn: ClassLoader::file_name_for_class_name(char const*, int)::class_suffix+881554
>> 
>> 
>> and on linux-x64:
>> 
>> External addresses table: 143 entries, 77297 accesses
>> 0:    22334 0x00007f35d5b9c000 : ''
>> 1:    19789 0x00007f35d55eea1f : 'should not reach here'
>> 2:    18366 0x00007f35d5747bb8 : 'MacroAssembler::decode_heap_oop: heap base corrupted?'
>> 3:     5036 0x00007f35d56e4d40 : 'uncommon trap returned which should never happen'
>> 4:     3643 0x00007f35d57479f8 : 'MacroAssembler::encode_heap_oop: heap base corrupted?'
>> 
>> 
>> Few points about difference in output:
>> 1. aarch64 does not use `ExternalAddess` or any relocation for messages (strings).
>> 2. `stub: forward exception` corresponds to `StubRoutines::forward_exception_entry()` for which C2 generates tail-call from [C2's stubs](https://github.com/openjdk/jdk/blob/master/src/hotspot/share/opto/generateOptoStub.cpp#L258C48-L258C87). It is difficult to convert it to `RuntimeAddress` because how relocation for constants in C2 are handled.
>> 3...
>
> Vladimir Kozlov has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Remove PrintNMethodStatistics not needed check in ExternalsRecorder::print_statistics

Looks good.

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

Marked as reviewed by vlivanov (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/20412#pullrequestreview-2216158126