On Thu, 6 Feb 2025 12:07:57 GMT, David Beaumont <d...@openjdk.org> wrote:
> 8349206: 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). test/jdk/java/util/logging/LoggingDeadlock5.java line 31: > 29: * java.logging > 30: * @compile -XDignore.symbol.file LoggingDeadlock5.java > 31: * @run main/othervm/timeout=10 LoggingDeadlock5 I would not specify any /timeout value here, and rather let jtreg deal with that. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/23491#discussion_r1953221796
