[OpenJDK 2D-Dev] RFR: 8252133: The java/awt/GraphicsDevice/DisplayModes/CycleDMImage.java fails if metal pipeline is active

Sergey Bylokhov serb at openjdk.java.net
Thu Oct 8 07:14:53 UTC 2020


This bug easily reproduced by the test in question on the dual video card systems
when the metal pipeline is active. But it is possible to reproduce it in the OGL
pipeline as well, but it is required some additional steps.

Problem description:
   Our CGraphicsEnvironment maintains the list of active graphics devices. The one
important feature of this CGraphicsEnvironment is to invalidate the old devices and
map them to the new devices. For example, if the user got a reference to the device,
and this device was removed then this reference will refer to the main screen.

The problem in the current implementation arise when the system has two video cards:
  1 The user get some GraphicsDevice
  2 The user sets the full-screen window for this device
  3 The user change screen resolution for this device
  4 The resolution of the screen is not changed ->> BUG.

The problem is that somewhere after step 1 or 2 and before step 3 the macOS decided
to switch to the discrete video card, but it does not report the old device(integrated VC)
as removed, because actually no screens were removed.

Since it was not reported as removed we did not invalidate it and did not map it to the
new device ->> request to change the screen resolution at step 3 send to some non existed
deviceID.

As a fix I suggest to change this current logic:
  - Invalidate devices reported by macOS as removed
  - Initialize the main screen
  - Initialize all NEW screens

To this logic:
  - Ignore devices reported by the macOS as removed
  - Initialize the main screen
  - Initialize all NEW screens
  - Check that the main device is in the list of all NEW devices
  - Invalidate all OLD devices which are not in the list of NEW devices

The old review request:
https://mail.openjdk.java.net/pipermail/2d-dev/2020-August/011011.html

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

Commit messages:
 - Merge branch 'master' into JDK-8252133
 - Update CycleDMImage.java
 - Merge branch 'master' into JDK-8252133
 - 8252133: The java/awt/GraphicsDevice/DisplayModes/CycleDMImage.java fails if metal pipeline is active

Changes: https://git.openjdk.java.net/jdk/pull/554/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=554&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8252133
  Stats: 104 lines in 3 files changed: 47 ins; 26 del; 31 mod
  Patch: https://git.openjdk.java.net/jdk/pull/554.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/554/head:pull/554

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


More information about the 2d-dev mailing list