RFR: 8295851: Do not use ttyLock in BytecodeTracer::trace [v3]

Coleen Phillimore coleenp at openjdk.org
Fri Jun 20 13:29:42 UTC 2025


> This didn't need ttyLock for synchronization, the code only needs to see if the method changes so it can print the method name before the bytecodes, like:
> 
> [490166] static void java.lang.String.<clinit>()
> [490166]       13    18  putstatic 613 <java/lang/String.CASE_INSENSITIVE_ORDER:Ljava/util/Comparator;>
> [490166]       14    21  return
> 
> [490166] static void java.lang.System.<clinit>()
> [490166]       15     0  invokestatic 471 <java/lang/System.registerNatives()V>
> [490166]       16     3  aconst_null
> [490166]       17     4  putstatic 474 <java/lang/System.in:Ljava/io/InputStream;>
> 
> Verified manually and added some parallelism to the test, and fixed trace() to initialize is_linked(), which it always is.
> Also ran tier1-4.

Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:

  Put back include osthread.hpp

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/25915/files
  - new: https://git.openjdk.org/jdk/pull/25915/files/0e4421be..5cdc5e11

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

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/25915.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/25915/head:pull/25915

PR: https://git.openjdk.org/jdk/pull/25915


More information about the hotspot-runtime-dev mailing list