RFR(s): 8252414: Redundant suspend check when determining if a java thread is safe
Robbin Ehn
robbin.ehn at oracle.com
Thu Aug 27 09:07:00 UTC 2020
Thanks David!
/Robbin
On 2020-08-27 10:07, David Holmes wrote:
> Hi Robbin,
>
> On 27/08/2020 5:21 pm, Robbin Ehn wrote:
>> Hi all, please review.
>>
>> In 8221207 - "Redo JDK-8218446 - SuspendAtExit hangs" we fixed so a
>> thread is always blocked when suspended.
>>
>> And added this nice assert.
>> int JavaThread::java_suspend_self() {
>> assert(thread_state() == _thread_blocked, "wrong state for
>> java_suspend_self()");
>>
>> When checking if a thread is safepoint/handshake safe there no need to
>> look at ext suspend flag anymore, since the thread is blocked.
>
> That seems sound to me. :)
>
> Thanks,
> David
> -----
>
>
>> Bug:
>> https://bugs.openjdk.java.net/browse/JDK-8252414
>> Code:
>> http://cr.openjdk.java.net/~rehn/8252414/webrev/index.html
>>
>> Passes t1-5
>>
>> Thanks, Robbin
More information about the hotspot-runtime-dev
mailing list