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

List:       openjdk-serviceability-dev
Subject:    Re: RFR: 8249627: Degrade Thread.suspend and Thread.resume [v3]
From:       Alan Bateman <alanb () openjdk ! org>
Date:       2022-09-27 11:40:30
Message-ID: Xk2Pw2LHXKS22OYcjJ13HZ1zTpxcqwwA7ZdENN-7dkU=.985cd33f-5771-47e4-97fe-245459c5b6a7 () github ! com
[Download RAW message or body]

On Sat, 24 Sep 2022 06:50:45 GMT, Alan Bateman <alanb@openjdk.org> wrote:

> > Degrade Thread.suspend/resume to throw UOE unconditionally.
> > 
> > Another step in the removal of this deadlock prone mis-feature from the \
> > user-facing API. Thread.suspend/resume have been deprecated since JDK 1.2 (1998) \
> > and terminally deprecated since Java 14. ThreadGroup.suspend/resume were degraded \
> > to throw UOE in Java 19. As of Java 19, Thread.suspend/resume continues to work \
> > for platform threads but throws UOE for virtual threads. The next step is to \
> > degrade both methods to throw UOE for all threads. A corpus search of 19M classes \
> > in 113k JAR files found only 22 classes using these methods so this change is \
> > unlikely to be disruptive. 
> > The change requires some minor adjustments to the JVM TI and JDWP specifications, \
> > and a minor update to the JDI docs. 
> > Leonid Mesnik is working on [PR10351](https://github.com/openjdk/jdk/pull/10351) \
> > to remove/replace the last few usages of Thread.suspend/resume from the hotspot \
> > tests (most of these can use JVMTI SuspendThread/ResumeThread).
> 
> Alan Bateman has updated the pull request with a new target base due to a merge or \
> a rebase. The pull request now contains 11 commits: 
> - Remove tests from exclude list
> - Merge
> - Update DegradedMethodsThrowUOE to test Thread.stop
> - Merge
> - Update modifyThread and modifyThreadGroup targets
> - Merge
> - Fix typo in test comment
> - Keep link to threadPrimitiveDeprecation.html
> - Merge
> - Tests
> - ... and 1 more: https://git.openjdk.org/jdk/compare/543851db...6aa38889

Wakeup bot

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

PR: https://git.openjdk.org/jdk/pull/10324


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