Integrated: 8324129: C2: Remove some ttyLocker usages in preparation for JDK-8306767

Emanuel Peter epeter at openjdk.org
Mon Jan 22 10:44:44 UTC 2024


On Thu, 18 Jan 2024 15:17:27 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

> I'm removing some instances of `ttyLocker`. Instead of locking, I first put all the output on a `stringStream`, and then print this stream all at once, which is atomic.
> 
> Removing the `ttyLocker` is nice, because it means we have less interference with other locking mechanisms, such as the `extra_data_lock` cases I have to introduce with [JDK-8306767](https://bugs.openjdk.org/browse/JDK-8306767).
> 
> @coleenp wished that I do this separately, so I filed this RFE here.

This pull request has now been integrated.

Changeset: c84af493
Author:    Emanuel Peter <epeter at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/c84af4938647efbc2d6c94efef748446bf6d50b4
Stats:     73 lines in 11 files changed: 14 ins; 2 del; 57 mod

8324129: C2: Remove some ttyLocker usages in preparation for JDK-8306767

Reviewed-by: kvn, vlivanov, dholmes

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

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


More information about the hotspot-dev mailing list