RFR: 8336384: AbstractQueuedSynchronizer.acquire should cancel acquire when failing due to a LinkageError or other errors [v2]

David Holmes dholmes at openjdk.org
Mon Aug 12 23:42:07 UTC 2024


On Mon, 12 Aug 2024 16:11:46 GMT, Viktor Klang <vklang at openjdk.org> wrote:

>> 8336384: AbstractQueuedSynchronizer.acquire should cancel acquire when failing due to a LinkageError or other errors
>
> Viktor Klang has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Catching both Error and RuntimeException

It has been a while since I knew this code reasonably well so perhaps I have just forgotten this difference between AQS and built-in monitors, but it seems that a Condition.await can return by throwing an exception without re-acquiring the associated synchronizer. Or is that handled at a higher-level?

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

PR Review: https://git.openjdk.org/jdk/pull/20548#pullrequestreview-2234191725


More information about the core-libs-dev mailing list