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

[David Holmes]

On Mon, 12 Aug 2024 21:37:45 GMT, Doug Lea <dl at openjdk.org> wrote:

>> src/java.base/share/classes/java/util/concurrent/locks/AbstractQueuedLongSynchronizer.java line 381:
>> 
>>> 379:                     else
>>> 380:                         break;
>>> 381:                 } catch (Error | RuntimeException ex) {
>> 
>> @DougLea @AlanBateman Changed to Error | RuntimeException here and for AQS
>
> lgtm!

`catch (Throwable ex)` would be consistent with the similar block at line 331. Though I'm unclear how that compiles without the method declaring `throws Throwable` ??

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20548#discussion_r1714470560