Integrated: 8295851: Do not use ttyLock in BytecodeTracer::trace

[Coleen Phillimore]

On Fri, 20 Jun 2025 11:50:49 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

> 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.

This pull request has now been integrated.

Changeset: 4edf791a
Author:    Coleen Phillimore <coleenp at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/4edf791aecd432ecde00652acfaabddf136f4ca7
Stats:     67 lines in 2 files changed: 37 ins; 12 del; 18 mod

8295851: Do not use ttyLock in BytecodeTracer::trace

Reviewed-by: dholmes, matsaave

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

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