RFR: 8353683: [REDO] j.u.l.Handler classes create deadlock risk via synchronized publish() method
Daniel Fuchs
dfuchs at openjdk.org
Thu Apr 10 14:38:33 UTC 2025
On Tue, 8 Apr 2025 11:03:39 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).
>
> This is a redo of https://github.com/openjdk/jdk/pull/23491 with a tiny additional edit to remove an unwanted `<p>` element in the public JavaDoc (see https://github.com/openjdk/jdk/pull/24504/commits/772a03e0ca08da9412b85ceb8de5adf4fb52d2ac ).
OK I ran tier1 and tier2 + logging test and got no failures. @david-beaumont can you integrate and let me know?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/24504#issuecomment-2793857309
More information about the core-libs-dev
mailing list