RFR: 8305578: X11GraphicsDevice.pGetBounds() is slow in remote X11 sessions [v6]
Sergey Bylokhov
serb at openjdk.org
Wed May 24 02:54:07 UTC 2023
On Thu, 4 May 2023 08:06:11 GMT, Maxim Kartashev <mkartashev at openjdk.org> wrote:
>> Getting bounds of a `GraphicsDevice` and insets of a screen are fairly frequent operations. When working over the network (remote X session), those can take several *seconds* to complete, introducing repeated freezes even when typing in an editor feels only slightly sluggish.
>>
>> On the other hand, neither bounds nor insets change very often - if at all - during the lifetime of an application. So caching their values seems like a natural solution to the problem. The caches need to be reset only when there's a possibility of change in the screens configuration.
>>
>> A similar patch has been living in [JetBrains Runtime](https://github.com/JetBrains/JetBrainsRuntime/) for well over a year.
>
> Maxim Kartashev has updated the pull request incrementally with one additional commit since the last revision:
>
> Double-checked locking for insets and less synchronization for bounds
The latest version looks fine.
-------------
Marked as reviewed by serb (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/13346#pullrequestreview-1440858185
More information about the client-libs-dev
mailing list