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

10 bpc ximage color channels - fix xdummy? #1576

Open
totaam opened this issue Jul 12, 2017 · 2 comments
Open

10 bpc ximage color channels - fix xdummy? #1576

totaam opened this issue Jul 12, 2017 · 2 comments

Comments

@totaam
Copy link
Collaborator

totaam commented Jul 12, 2017

Issue migrated from trac ticket # 1576

component: server | priority: major

2017-07-12 12:54:49: antoine created the issue


Follow up from #1309.
When running a 30 bit xdummy virtual display, the colors look correct.
But when running against an existing display powered by the nvidia driver... the red and blue channels are inverted.
My guess is that dummy needs fixing - no idea how.
It would be nice to have another driver to compare.

I have used the following code to dump the XImage data in both cases, and the output is identical... so it doesn't look like a case of our code missing a switch:

--- xpra/x11/bindings/ximage.pyx	(revision 16291)
+++ xpra/x11/bindings/ximage.pyx	(working copy)
@@ -311,6 +311,9 @@
             else:
                 self.pixel_format = BGRA
         elif self.depth==30:
+            #log.warn("30bpp, byte_order=%i, bitmap_bit_order=%i, bits_per_pixel=%s", image.byte_order, image.bitmap_bit_order, image.bits_per_pixel)
+            #log.warn("pad=%i, unit=%i, format=%i", image.bitmap_pad, image.bitmap_unit, image.format)
+            #log.warn("masks: red=%#x, green=%#x, blue=%#x", image.red_mask, image.green_mask, image.blue_mask)
             self.bytesperpixel = 4
             if image.byte_order==MSBFirst:
                 self.pixel_format = R210
@totaam
Copy link
Collaborator Author

totaam commented Oct 25, 2017

2017-10-25 11:34:17: antoine changed status from new to assigned

@totaam
Copy link
Collaborator Author

totaam commented Oct 25, 2017

2017-10-25 11:34:17: antoine commented


re-scheduling since #1577 is waiting for an upstream change

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

No branches or pull requests

1 participant