RFR: 8320652: ThreadInfo.isInNative needs to be updated to say what executing native code means [v2]
Alan Bateman
alanb at openjdk.org
Mon Nov 27 20:31:23 UTC 2023
> This is a docs only change to j.l.management.ThreadInfo::isInNative.
>
> The method currently specifies that it tests if the thread is "executing native code via the Java Native Interface (JNI)". It would be clearer to say that it tests if the thread is executing a native method, and expand it to include native code invoked using the new foreign linker APIs. For now, I've left out the detail of a downcall handle created with the "critical" linker option.
>
> The existing javadoc has wording to try to clarify that "native code" does not include runtime or compiled code. I went through a few iterations to work this in but all attempts just invite more questions and nit picking. To keep it simple, this sentence is dropped so that the method description is focused on the cases where the method returns true.
Alan Bateman has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:
- Change link to downcallHandle
- Merge
- Simplify wording
- Merge
- Initial commit
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/16791/files
- new: https://git.openjdk.org/jdk/pull/16791/files/93ac45a6..5e2cc189
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=16791&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=16791&range=00-01
Stats: 3366 lines in 72 files changed: 2233 ins; 606 del; 527 mod
Patch: https://git.openjdk.org/jdk/pull/16791.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/16791/head:pull/16791
PR: https://git.openjdk.org/jdk/pull/16791
More information about the serviceability-dev
mailing list