RFR: 8346952 : GetGraphicsStressTest.java fails: Native resources unavailable [v3]

[Sergey Bylokhov]

On Tue, 10 Jun 2025 11:32:58 GMT, Anass Baya <abaya at openjdk.org> wrote:

>> check how the _SetFocusableWindow is implemented below.
>
> Hello @mrserb,
> 
>> I think the only thing you should do is to replace the usage of JNI_GET_PDATA by the JNI_CHECK_PEER_GOTO and add a label "ret" at the end so default value of "result" will be returned.
> 
> Yes, In the latest changes, I followed that approach to ensure that the peer is not yet destroyed in _GetScreenImOn() as explain in my latest comment. So if you agree I will remove the lock.

Yes, the new lock is not needed. You can verify this by adding delays/Sleep calls into the methods and confirming that they wait for each other.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25619#discussion_r2138797641