<AWT Dev> [9] Review request for 8078606: Deadlock in awt clipboard

[Alexander Zvegintsev]

Hello Anton,

the fix looks fine to me.

Thanks,

Alexander.

On 05/25/2015 02:04 PM, Anton Nashatyrev wrote:
> Hello,
>
>     any volunteers?
>
> Thanks!
> Anton.
>
> On 08.05.2015 19:08, Anton Nashatyrev wrote:
>> Hello,
>>     could you please review the following fix:
>>
>> fix: http://cr.openjdk.java.net/~anashaty/8078606/webrev.00/ 
>> <http://cr.openjdk.java.net/%7Eanashaty/8078606/webrev.00/>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8078606
>>
>>     Problem: XClipboard sometimes deadlocked
>>
>>     Reason: XClipboard.registerClipboardViewerChecked acquires 
>> XCliboard lock first, then tries to acquire AWTLock (within the 
>> XWindow.getXAWTRootWindow() call)
>>
>>     Fix: reverse the order of acquiring locks. Additionally eliminate 
>> AWTLock when accessing XRootWindow singleton.
>>
>> Thanks!
>> Anton.
>