RFR: 8280468: Crashes in getConfigColormap, getConfigVisualId, XVisualIDFromVisual on Linux [v2]

Maxim Kartashev duke at openjdk.java.net
Fri Apr 8 06:49:12 UTC 2022


On Thu, 7 Apr 2022 23:05:04 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

>> Maxim Kartashev has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Addressed PR comments
>>   
>>   1. Got rid of X11GraphicsDevice.configLock in favour of the AWT lock.
>>   2. Reduced the use of the screen number (and, consequently, AWT lock
>>   contention) in XCanvasPeer.
>
> src/java.desktop/unix/classes/sun/awt/X11GraphicsDevice.java line 161:
> 
>> 159:         XToolkit.awtLock();
>> 160:         try {
>> 161:             if (configs == null) {
> 
> To make the diff less distruptive you can move the ssyncronisation to the getConfigurations and getDefaultConfiguration where we have kind of DCL.

Sure thing. Done.

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

PR: https://git.openjdk.java.net/jdk/pull/7182



More information about the client-libs-dev mailing list