RFR: 8324881: ObjectSynchronizer::inflate(Thread* current...) is invoked for non-current thread [v3]

[Daniel D. Daugherty]

On Thu, 1 Feb 2024 06:19:46 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Axel Boldt-Christmas has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   More restrictive API
>
> src/hotspot/share/runtime/synchronizer.cpp line 449:
> 
>> 447: 
>> 448: // Handle notifications when synchronizing on value based classes
>> 449: void ObjectSynchronizer::handle_sync_on_value_based_class(Handle obj, JavaThread* locking_thread) {
> 
> Can we add a comment (or even repeat the assertion) that locking thread is either the current thread or a suspended thread, please. This method performs a  number of actions that are not safe to make on an arbitrary thread.

I breezed right by this function... where did that come from?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17626#discussion_r1474537884