Skip to content

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