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

[Andrew Brygin]

Hi Anton,

  the change looks fine to me.

Thanks,
Andrew

On 5/25/2015 3:46 PM, Alexander Zvegintsev wrote:
> 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.
>>
>