Windows guest display is inverted vertically

Bug #1836361 reported by Dariusz Gadomski on 2019-07-12
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
spice-html5 (Debian)
Unknown
Unknown
spice-html5 (Ubuntu)
Status tracked in Eoan
Xenial
Medium
Dariusz Gadomski
Bionic
Medium
Dariusz Gadomski
Cosmic
Medium
Dariusz Gadomski
Disco
Medium
Dariusz Gadomski
Eoan
Medium
Dariusz Gadomski

Bug Description

[Impact]

 * When running a Windows VM using the QXL driver from [1] the display is inverted and therefore almost unusable.

[Test Case]

 * Launch a Windows VM with a QXL display (I used Win2k16 running under libvirt).
 * Start websockify
     sudo apt install python3-websockify && websockify 5959 localhost:5900
 * Run a HTTP server to spice-html5:
   cd /usr/share/spice-html5 && python -m SimpleHTTPServer
 * Open a browser and go to http://localhost:8000
 * Enter
   host: localhost
   port: 5959
 * Click start

Expected behaviour:
Guest UI is displayed correctly.

Actual behaviour:
Guest UI gets inverted on refresh.

[Regression Potential]

 * This component may be used with openstack and it's possible that there are workarounds in place, so they may start working incorrectly.

[Other Info]

 * Part of this fixed is included cosmic and onwards (spice-html5 0.1.7-3).

[1] https://fedorapeople.org/groups/virt/virtio-win/direct-downloads/latest-virtio/virtio-win.iso

Changed in spice-html5 (Ubuntu):
importance: Undecided → Medium
tags: added: sts
Dariusz Gadomski (dgadomski) wrote :

Adding an example screenshot.

description: updated
Changed in spice-html5 (Ubuntu):
assignee: nobody → Dariusz Gadomski (dgadomski)
Changed in spice-html5 (Ubuntu Bionic):
assignee: nobody → Dariusz Gadomski (dgadomski)
Changed in spice-html5 (Ubuntu Cosmic):
assignee: nobody → Dariusz Gadomski (dgadomski)
Changed in spice-html5 (Ubuntu Disco):
assignee: nobody → Dariusz Gadomski (dgadomski)
Dariusz Gadomski (dgadomski) wrote :

SRU proposal for Eoan.

Changed in spice-html5 (Ubuntu Bionic):
importance: Undecided → Medium
Changed in spice-html5 (Ubuntu Cosmic):
importance: Undecided → Medium
Changed in spice-html5 (Ubuntu Disco):
importance: Undecided → Medium
Dariusz Gadomski (dgadomski) wrote :

SRU proposal for disco.

Dariusz Gadomski (dgadomski) wrote :

SRU proposal for cosmic.

Dariusz Gadomski (dgadomski) wrote :

SRU proposal for bionic.

The attachment "eoan_spice-html5_0.1.7-3ubuntu1.debdiff" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

tags: added: patch
Dan Streetman (ddstreet) on 2019-07-17
tags: added: sts-sponsor
Dan Streetman (ddstreet) wrote :

marked cosmic wontfix because it is EOL today.

Changed in spice-html5 (Ubuntu Cosmic):
status: New → Won't Fix
Changed in spice-html5 (Ubuntu Eoan):
status: New → In Progress
Changed in spice-html5 (Ubuntu Disco):
status: New → In Progress
Changed in spice-html5 (Ubuntu Bionic):
status: New → In Progress
Dan Streetman (ddstreet) wrote :

one minor correction on version numbering; you set the disco and eoan versions to:

eoan: 0.1.7-3ubuntu1
disco: 0.1.7-3ubuntu1.19.04.1

The current version in both is 0.1.7-3, so the new eoan version is correct, but the version numbering across releases should always be increasing - meaning, the eoan version should be higher than the disco version. So to do that, I changed the disco version to:

0.1.7-3ubuntu0.19.04.1

Dan Streetman (ddstreet) wrote :

uploaded to b, d, e

@dgadomski does Xenial need patching as well?

tags: added: sts-sponsor-ddstreet
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package spice-html5 - 0.1.7-3ubuntu1

---------------
spice-html5 (0.1.7-3ubuntu1) eoan; urgency=medium

  * d/p/fix-windows-spice-upside-down-bitmaps.patch:
    Handle non topdown bitmaps (LP: #1836361)

 -- Dariusz Gadomski <email address hidden> Fri, 12 Jul 2019 16:24:36 +0200

Changed in spice-html5 (Ubuntu Eoan):
status: In Progress → Fix Released
Dariusz Gadomski (dgadomski) wrote :

@ddstreet: I've just checked and Xenial was also affected, attaching debdiff.

Dan Streetman (ddstreet) wrote :

uploaded for xenial, thanks!

Changed in spice-html5 (Ubuntu Xenial):
status: New → In Progress
importance: Undecided → Medium
assignee: nobody → Dariusz Gadomski (dgadomski)

Hello Dariusz, or anyone else affected,

Accepted spice-html5 into bionic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/spice-html5/0.1.7-2ubuntu1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-bionic to verification-done-bionic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-bionic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in spice-html5 (Ubuntu Bionic):
status: In Progress → Fix Committed
tags: added: verification-needed verification-needed-bionic
Dariusz Gadomski (dgadomski) wrote :

I confirm bionic-proposed version (0.1.7-2ubuntu1) works as expected. Nothing looks flipped. Screenshot attached.

tags: added: verification-done-bionic
removed: verification-needed-bionic
Brian Murray (brian-murray) wrote :

Hello Dariusz, or anyone else affected,

Accepted spice-html5 into disco-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/spice-html5/0.1.7-3ubuntu0.19.04.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-disco to verification-done-disco. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-disco. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in spice-html5 (Ubuntu Disco):
status: In Progress → Fix Committed
tags: added: verification-needed-disco
Brian Murray (brian-murray) wrote :

Hello Dariusz, or anyone else affected,

Accepted spice-html5 into xenial-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/spice-html5/0.1.4-1ubuntu1.16.04.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-xenial to verification-done-xenial. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-xenial. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in spice-html5 (Ubuntu Xenial):
status: In Progress → Fix Committed
tags: added: verification-needed-xenial
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package spice-html5 - 0.1.7-2ubuntu1

---------------
spice-html5 (0.1.7-2ubuntu1) bionic; urgency=medium

  * d/p/fix-windows-spice-upside-down.patch
    d/p/fix-windows-spice-upside-down-bitmaps.patch
    Fix window spice upside down display (LP: #1836361)

 -- Dariusz Gadomski <email address hidden> Fri, 12 Jul 2019 17:39:24 +0200

Changed in spice-html5 (Ubuntu Bionic):
status: Fix Committed → Fix Released

The verification of the Stable Release Update for spice-html5 has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Dan Streetman (ddstreet) wrote :

@dgadomski I know you're out currently, but please don't forget to verify this for disco and xenial once you get back. Thanks!

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.