[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-serviceability-dev
Subject: Integrated: 8287812: Cleanup JDWP agent GetEnv initialization
From: Chris Plummer <cjplummer () openjdk ! org>
Date: 2022-12-14 19:42:12
Message-ID: bdgsHCdD0y6RLPgp-HYfPlRDj0WBkW1a5mKKouG_zvA=.5cc179ef-b6f5-4f33-a7c4-d3598bc4607c () github ! com
[Download RAW message or body]
On Thu, 8 Dec 2022 22:40:57 GMT, Chris Plummer <cjplummer@openjdk.org> wrote:
> 3 things to cleanup in this area:
>
> (1) The JDWP agent uses `JNI GetEnv(JVMTI_VERSION_1)` to get a JVMTI environment. \
> If `GetEnv` fails the JDWP agent prints this:
> `ERROR: JDWP unable to access JVMTI Version 1 (0x30010000), is your J2SE a 1.5 or \
> newer version? JNIEnv's GetEnv() returned -3`
> The text "is your J2SE a 1.5 or newer version?" dates from JDK 5 when JVMTI was \
> introduced and doesn't make sense now.
> (2) `JVMTI_VERSION_1` suggests that the JDWP agent is looking for a JVMTI v1 \
> environment when it really wants the latest. `GetEnv` should request \
> `JVMTI_VERSION` so that it always requests the current version.
> (3) There is some outdated compatibility checking between runtime and compile time \
> versions of JVMTI that date back to the 1.0, 1.1, and 1.2 era, and are no longer \
> needed.
This pull request has now been integrated.
Changeset: ccb94acc
Author: Chris Plummer <cjplummer@openjdk.org>
URL: https://git.openjdk.org/jdk/commit/ccb94acc442767a7047756806c8dc7ecacd8bae9
Stats: 77 lines in 2 files changed: 11 ins; 61 del; 5 mod
8287812: Cleanup JDWP agent GetEnv initialization
Reviewed-by: alanb, sspitsyn
-------------
PR: https://git.openjdk.org/jdk/pull/11602
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic