[jdk17] Integrated: 8267952: async logging supports to dynamically change tags and decorators

Xin Liu xliu at openjdk.java.net
Mon Jun 28 18:21:10 UTC 2021


On Wed, 23 Jun 2021 20:18:32 GMT, Xin Liu <xliu at openjdk.org> wrote:

> This patch fixed intermittent crashes of gtest/AsyncLogGtest.java. The root cause 
> is that previous flush() can't guarantee flush all pending messages in AsyncLog
> buffer. This patch implements flush() using a synchronizaton token and waits for
> completion. This approach isn't MT-safe but it can serialize all flush() calls in
> a thread. 
> 
> Two concurrent tests are appended to cover the hazard cases.
> This patch also comments LogConfiguration::configure_output() MT-safety.

This pull request has now been integrated.

Changeset: 09bb8468
Author:    Xin Liu <xliu at openjdk.org>
URL:       https://git.openjdk.java.net/jdk17/commit/09bb84685fd959157857cf7739aedddd4a5e0cee
Stats:     198 lines in 9 files changed: 156 ins; 13 del; 29 mod

8267952: async logging supports to dynamically change tags and decorators

Reviewed-by: dholmes, phh

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

PR: https://git.openjdk.java.net/jdk17/pull/130


More information about the hotspot-runtime-dev mailing list