[Spice-devel] [common PATCH 0/8 v3] Fix colors on BE clients, when using image compression
Christophe Fergeau
cfergeau at redhat.com
Mon Oct 26 06:18:37 PDT 2015
Hey,
This series looks fine except for a few small questions. Would be great
if you could test these 16bpp code paths though.
Thanks,
Christophe
On Thu, Oct 22, 2015 at 05:02:46PM +0200, Lukas Venhoda wrote:
> When using image compression on BE client, the color order was wrong.
> (BGRA instead of ARGB)
>
> Theese patches try to fix this issue in most of the image compressions.
> Only those compressions I could test are changed.
>
> For most of the fixes, new macros for color order were implemented.
> Theese macros changed according to the endianness of the machine it runs on.
> Original implementation is in QEMU in file qemu-pixman.h
>
> Not all PIXMAN_[a-z] were changed. I don't know what half of them do, nor how to test them.
> When I tryed changing them all, it broke my fixes, so I left them as is.
> If someone finds a problem with them, I can always patch them later.
>
> ---
> Changes since v2:
> - Added jpeg compression support
> - Fixed alpha clearing when using x8r8g8b8 colors
> - Fixed typos and imrpoved commit logs
>
> Changes since v1:
> - Added more image compressions (lz, glz, lz4)
> - Fixed some typos in canvas_util.c
> - Cleaned up pixman-utils.h using the new macros
> ---
>
> Lukas Venhoda (8):
> pixman_utils: Add macros for color byte ordering
> pixman_utils: Use PIXMAN_LE_ constants in spice_bitmap_try_as_pixman()
> ppc: Added supprt for bigendian color byte order
> ppc: Fix colors on ppc when using QUIC
> ppc: Fix colors on ppc when using LZ
> ppc: Fix colors on ppc when using LZ4
> ppc: Fix colors on ppc when using jpeg
> ppc: Fix alpha state checking and clearing on BE machines
>
> common/canvas_base.c | 34 ++++++++++++++++++++--------------
> common/canvas_utils.c | 14 ++++++++++++++
> common/pixman_utils.c | 18 +++---------------
> common/pixman_utils.h | 10 ++++++++++
> 4 files changed, 47 insertions(+), 29 deletions(-)
>
> --
> 2.4.3
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20151026/a45a07de/attachment.sig>
More information about the Spice-devel
mailing list