RFR: 8353683: [REDO] j.u.l.Handler classes create deadlock risk via synchronized publish() method

Chen Liang liach at openjdk.org
Tue Apr 8 16:19:18 UTC 2025


On Tue, 8 Apr 2025 11:00:27 GMT, David Beaumont <duke at openjdk.org> wrote:

> 8353683: j.u.l.Handler classes create deadlock risk via synchronized publish() method.
> 
> 1. Remove synchronization of calls to publish() in Handlers in java.util.logging package.
> 2. Add explanatory comments to various affected methods.
> 3. Add a test to ensure deadlocks no longer occur.
> 
> Note that this change does not address issue in MemoryHandler (see JDK-8349208).

src/java.logging/share/classes/java/util/logging/Handler.java line 46:

> 44:  * {@code Handler} class.
> 45:  *
> 46:  * <h2><a id=threadSafety>Thread Safety and Deadlock Risk in Handlers</a></h2>

FYI you can add the id attribute to the h2 tag without declaring a separate a tag.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24504#discussion_r2033559665


More information about the core-libs-dev mailing list