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

List:       openjdk-serviceability-dev
Subject:    Integrated: 8264663: Update test SuspendWithCurrentThread.java to verify that suspend doesn't exit u
From:       Leonid Mesnik <lmesnik () openjdk ! java ! net>
Date:       2021-04-26 19:52:37
Message-ID: 6IjORIMxH9GR2wV35viKUkrH-14xeV-qPWQPsI1GOLQ=.3a6532bf-7f80-448e-b775-8ff7b8bbe106 () github ! com
[Download RAW message or body]

On Fri, 23 Apr 2021 21:07:56 GMT, Leonid Mesnik <lmesnik@openjdk.org> wrote:

> Test test/hotspot/jtreg/serviceability/jvmti/SuspendWithCurrentThread/SuspendWithCurrentThread.java
>  doesn't check that thread stops in SuspendThreadList(...).
> 
> Actually, before https://bugs.openjdk.java.net/browse/JDK-8257831 the thread didn't \
> suspend itself but only get a request to be suspended.  So it continued to execute \
> and stopped a little bit later. 
> Such behavior is a violation of spec which says " If the calling thread is \
> specified in the request_list array, this function will not return until some other \
> thread resumes it." 
> While the bug is fixed it is still useful to verify correct behavior.  If the fix \
> is backported without JDK-8257831 test should start failing.

This pull request has now been integrated.

Changeset: b5c63513
Author:    Leonid Mesnik <lmesnik@openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/b5c63513
Stats:     8 lines in 1 file changed: 7 ins; 0 del; 1 mod

8264663: Update test SuspendWithCurrentThread.java to verify that suspend doesn't \
exit until resumed

Reviewed-by: dcubed, sspitsyn

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

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


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