Quantcast

[9] Review Request: 8152683 Deadlock when resuming from sleep with different monitor setup

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[9] Review Request: 8152683 Deadlock when resuming from sleep with different monitor setup

Sergey Bylokhov
Hello,
Please review the fix for jdk9.

Bug: https://bugs.openjdk.java.net/browse/JDK-8152683
Webrev can be found at: http://cr.openjdk.java.net/~serb/8152683/webrev.00/

In the fix we update the mainDisplayId when the list of displays are updated on Appkit thread, instead of reading it each time from the different threads. No new issues were found by jck and jtreg tests.

The fix was contributed by Karl von Randow <[hidden email]>:
[1] http://mail.openjdk.java.net/pipermail/2d-dev/2017-January/008087.html

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [9] Review Request: 8152683 Deadlock when resuming from sleep with different monitor setup

Vadim Pakhnushev
Looks good.

Vadim

On 05.03.2017 21:20, Sergey Bylokhov wrote:

> Hello,
> Please review the fix for jdk9.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8152683
> Webrev can be found at: http://cr.openjdk.java.net/~serb/8152683/webrev.00/
>
> In the fix we update the mainDisplayId when the list of displays are updated on Appkit thread, instead of reading it each time from the different threads. No new issues were found by jck and jtreg tests.
>
> The fix was contributed by Karl von Randow <[hidden email]>:
> [1] http://mail.openjdk.java.net/pipermail/2d-dev/2017-January/008087.html
>

Loading...