JDK-8307067 : remove broken EnableThreadSMRExtraValidityChecks option
  • Type: Sub-task
  • Component: hotspot
  • Sub-Component: runtime
  • Affected Version: 17,21
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • OS: generic
  • CPU: generic
  • Submitted: 2023-04-27
  • Updated: 2023-07-25
  • Resolved: 2023-05-13
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 17 JDK 21
17.0.9-oracleFixed 21 b23Fixed
Description
The EnableThreadSMRExtraValidityChecks diagnostic option
is broken and should be removed.

If -XX:-EnableThreadSMRExtraValidityChecks is specified the
ThreadsListHandle::cv_internal_thread_to_JavaThread() function
can return an unprotected JavaThread* and use of that JavaThread*
can result in crashes.

This was discussed during the reviews for the following fix:

    JDK-8305670 Performance regression in LockSupport.unpark with lots of idle threads
Comments
Fix request [17u] I backport this for parity with 17.0.9-oracle. Low risk, only remove unused code. Flag can be removed as it is diagnostic. Clean backport. SAP nighlty testing passed.
18-07-2023
A pull request was submitted for review. URL: https://git.openjdk.org/jdk17u-dev/pull/1586 Date: 2023-07-17 07:45:17 +0000
17-07-2023
Changeset: 5e26e64c Author: Daniel D. Daugherty <dcubed@openjdk.org> Date: 2023-05-13 14:39:58 +0000 URL: https://git.openjdk.org/jdk/commit/5e26e64cbfa4baf81024efeb57768458e59971f5
13-05-2023
A pull request was submitted for review. URL: https://git.openjdk.org/jdk/pull/13704 Date: 2023-04-27 22:17:30 +0000
28-04-2023