[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