RFR: 8320652: ThreadInfo.isInNative needs to be updated to say what executing native code means [v3]

Alan Bateman alanb at openjdk.org
Fri Dec 1 12:16:29 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.

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 seven additional commits since the last revision:

 - Extend description
 - Merge
 - 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/5e2cc189..4836a9de

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=16791&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=16791&range=01-02

  Stats: 16832 lines in 625 files changed: 12312 ins; 2774 del; 1746 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