Integrated: 8317433: Async UL: Only grab lock once when write():ing

Johan Sjölen jsjolen at openjdk.org
Fri Oct 13 13:24:45 UTC 2023


On Tue, 3 Oct 2023 14:31:43 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

> Hi,
> 
> The writer thread used to await for `_data_available` to become true through using the async log lock as a condition variable. When data is available it'd let go of the lock and then call `AsyncLogWriter::write()`, which would immediately try to acquire that same lock. We can remove this second instance by moving the work needed to be done under lock to `run`.
> 
> Thank you for considering this.

This pull request has now been integrated.

Changeset: 45b7aeda
Author:    Johan Sjölen <jsjolen at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/45b7aedabf3ca3d53be6c4bde4fddb584a5accca
Stats:     44 lines in 2 files changed: 17 ins; 23 del; 4 mod

8317433: Async UL: Only grab lock once when write():ing

Reviewed-by: dholmes, xliu

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

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


More information about the hotspot-runtime-dev mailing list