JDK-8232355 : Two obsolete flags have the wrong obsolete version in 8u
  • Type: Bug
  • Component: hotspot
  • Sub-Component: runtime
  • Affected Version: 8u222
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2019-10-16
  • Updated: 2020-01-28
  • Resolved: 2020-01-16
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.
Other
openjdk8u252 b01Fixed
Related Reports
Relates :  
JDK-8046598 - Scalable Native Memory Tracking development
Relates :  
JDK-8032490 - Remove -XX:+-UseOldInlining
Relates :  
JDK-8035585 - Restore UseOldInlining in jdk8u
Description
UseOldInlining and AutoShutdownNMT have the wrong obsolete version jdk9.
And UseOldInlining should be removed in c2_globals.hpp

cmd:
java��-XX:-AutoShutdownNMT��-version

output:
OpenJDK 64-Bit Server VM warning: ignoring option AutoShutdownNMT; support was removed in 9.0
openjdk version "1.8.0_212"
OpenJDK Runtime Environment (build 1.8.0_212-b00)
OpenJDK 64-Bit Server VM (build 25.212-b00, mixed mode)
Comments
URL: https://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/rev/8c3a44b7ecfc User: andrew Date: 2020-01-28 03:31:16 +0000
28-01-2020
URL: https://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/8c3a44b7ecfc User: ddong Date: 2020-01-16 04:10:13 +0000
16-01-2020
Looks like the values weren't updated in the backports of JDK-8046598 & JDK-8032490, and then JDK-8035585 tried to work around this by reintroducing the dead -XX:+UseOldInlining without any associated behaviour.
06-01-2020
Webrev: http://cr.openjdk.java.net/~ddong/8232355/hotspot.00/ Review thread: https://mail.openjdk.java.net/pipermail/jdk8u-dev/2019-October/010454.html Review approval: https://mail.openjdk.java.net/pipermail/jdk8u-dev/2019-October/010463.html
04-12-2019
yes, there should use the version in which they were actually made obsolete. AutoShutdownNMT was made obsolete in http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/833b0f92429a the version is 8u40 UseOldInlining was made obsolete in http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/1419657ed891 the version is 8u20 The webrev has been updated.
16-10-2019
! { "UseOldInlining", JDK_Version::jdk_update(8, 242), JDK_Version::jdk(10) }, ! { "AutoShutdownNMT", JDK_Version::jdk_update(8, 242), JDK_Version::jdk(10) }, The version used should be the version in which they were actually made obsolete. That is 8u40 for AutoShutdownNMT. Didn't check UseOldInlining.
16-10-2019
webrev: http://cr.openjdk.java.net/~ddong/8232355/hotspot.00/ mail: https://mail.openjdk.java.net/pipermail/jdk8u-dev/2019-October/010454.html
16-10-2019