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