[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Integrated: 8223312: Utilize handshakes instead of is_thread_fully_suspended
From:       Robbin Ehn <rehn () openjdk ! java ! net>
Date:       2020-10-22 15:23:27
Message-ID: CK7eJPg50FGpX77ZJdGo38Qww5SvogaP20l-BeQuBmE=.f7372792-3655-4110-b530-5d249f425a6b () github ! com
[Download RAW message or body]

On Mon, 19 Oct 2020 09:59:34 GMT, Robbin Ehn <rehn@openjdk.org> wrote:

> The main point of this change-set is to make it easier to implement S/R on top of \
> handshakes. Which is a prerequisite for removing _suspend_flag (which duplicates \
> the handshake functionality). But we also remove some complicated S/R methods.
> 
> We basically just put in everything in the handshake closure, so the diff just \
> looks much worse than what it is. 
> TraceSuspendDebugBits have an ifdef, but in both cases it now just returns.
> But I was unsure if I should remove now or when is_ext_suspend_completed() is \
> removed. 
> Passes multiple t1-5 runs, locally it passes many jck:vm/nsk_jvmti/nsk_jdi/jdk-jdi \
> runs.

This pull request has now been integrated.

Changeset: 4634dbef
Author:    Robbin Ehn <rehn@openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/4634dbef
Stats:     611 lines in 6 files changed: 174 ins; 376 del; 61 mod

8223312: Utilize handshakes instead of is_thread_fully_suspended

Reviewed-by: dholmes, rrich, dcubed, eosterlund

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

PR: https://git.openjdk.java.net/jdk/pull/729


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic