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

List:       openjdk-serviceability-dev
Subject:    Re: RFR: 8264663: Update test SuspendWithCurrentThread.java to verify that suspend doesn't exit unti
From:       Leonid Mesnik <lmesnik () openjdk ! java ! net>
Date:       2021-04-24 1:59:28
Message-ID: uFBGMiOXlwSznctcz--ZvJxmh9RrVEP4G6ujV_agAjg=.35b7b496-24d0-4490-bc9e-fd5769cf0d92 () 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.

Dan, Serguei. Thank you for review. I've verified that test build and passed on \
linux/win/mac x64 and linux-aarch64.

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

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


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