Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixing flipped screen problem #35

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

doccaz
Copy link

@doccaz doccaz commented Feb 8, 2019

I noticed that the original spice5-html had this issue as well, and it was fixed in this patch:
https://cgit.freedesktop.org/~pgrunt/spice-html5/commit/?h=topdown

... which if I understood correctly, is just implementing a "topdown" (e.g. invert the coordinates for drawing) on a particular packet type/color depth, SPICE_SURFACE_FMT_32_xRGB.

Looking through the sources for the eyeOS web client, I found this in lib/images/bitmap.js:

                if(!topdown) {
                        ret = wdi.RasterOperation.flip(ret);
                }

Looking further up, we find the part of the code that deals with 32-bit color packets. It's called wdi.SpiceBitmapFmt.SPICE_BITMAP_FMT_32BIT here.
To fix it, I added a " topdown = true" statement. I've tested with a few of my own VMs that were experiencing this bug, especially in text mode. So far none have regressed.

@doccaz
Copy link
Author

doccaz commented Feb 8, 2019

This seems to fix the following open issues: #22 and #33 .

@pgurenko
Copy link

pgurenko commented Jun 2, 2019

@doccaz, as of now, the fix helps when desktop is showing initially:

image

but after next mouse over, weird flips still happen:

image

It looks like guys from flexVDI are trying to address the same issue with flag received from server: flexVDI@6d491df#diff-fbf4b1476b4fa5b149be11ff1ebd0625

But even with their revision, I have pretty much the same problem - flipped redrawing on mouse move.

It could be the spice guest tools qxl driver-related problem though which is installed on the VM.

@pgurenko
Copy link

pgurenko commented Jun 2, 2019

Fixed in FlexVDI fork (flexVDI#1): there, topdown is a flag from server and no need to explicitly setting it anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants