RFR: 8366905: Store AdapterBlob pointer in AdapterHandlerEntry [v3]

[Andrew Dinn]

On Mon, 22 Sep 2025 10:38:36 GMT, Andrew Dinn <adinn at openjdk.org> wrote:

>> Ashutosh Mehra has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Add missing ResourceMark
>>   
>>   Signed-off-by: Ashutosh Mehra <asmehra at redhat.com>
>
> src/hotspot/share/runtime/sharedRuntime.hpp line 736:
> 
>> 734:   }
>> 735: 
>> 736:   address get_i2c_entry()                  const { return _adapter_blob != nullptr ? _adapter_blob->i2c_entry() : nullptr; }
> 
> Should these getters assert `_adapter_blob != nullptr`?
> I think we should only ever have an AdapterHandlerEntry with a null blob on ZERO and in that case an attempt to use the entry would be invalid.

n.b. I guess that would require modifying `print_adapter_on` to check the adapter blob and avoid calling the entry lookup methods/printing the entry address details. It normally gets called when we have a known entry address as a starting point to look up an adapter handle but it is also called from `print_adapter_handler_info` when we may have a handler with a null blob.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27101#discussion_r2367607731