RFR: 8317600: VtableStubs::stub_containing() table load not ordered wrt to stores

[Thomas Schatzl]

On Mon, 9 Oct 2023 23:03:25 GMT, Dean Long <dlong at openjdk.org> wrote:

>> Hi all,
>> 
>>   please review this change that makes sure that `Vtablestubs::stub_containing()` (and `::vtable_stub_do()` I noticed while implementing this) use proper memory fences to correctly observe the next pointers in the hash table buckets.
>> 
>> I assume that this memory ordering issue never occurs in the wild, there is probably always some additional memory fencing between the addition of the element to the given bucket to the iteration. However it is still the more correct code (I believe) to explicitly enforce memory ordering.
>> 
>> Testing: gha, class unloading stress test
>> 
>> Thanks,
>>   Thomas
>
> OK, I mis-read Atomic:load as meaning load_acquire for some reason.

Thanks @dean-long @vnkozlov for your reviews

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

PR Comment: https://git.openjdk.org/jdk/pull/16056#issuecomment-1766991438