Bug 74939

Created attachment 94010 [details]
Example of app with the driver before 94d0d14

Created attachment 94011 [details]
Example of app with the driver after 94d0d14

Created attachment 94013 [details]
Example of app with the driver after 94d0d14 (with Gnome3 window decoration)

Created attachment 94463 [details] [review]
Fix ?

Does this helps ? Can you try different combination ?

(In reply to comment #4)
> +#if X_BYTE_ORDER == X_BIG_ENDIAN
> +    buf[0] = pix_a;
> +    buf[1] = pix_b;
> +    buf[2] = pix_g;
> +    buf[3] = pix_r;
> +#else
>      buf[0] = pix_r;
>      buf[1] = pix_g;
>      buf[2] = pix_b;
>      buf[3] = pix_a;
> +#endif

These values are floats, so their order is not subject to endianness. If this change helps, the extraction of pix_r/g/b/a from the packed fg value should be fixed instead.

(In reply to comment #4)
> Created attachment 94463 [details] [review] [review]
> Fix ?
> 
> Does this helps ? Can you try different combination ?

Unfortunately, this change, neither different combinations, helped.

Created attachment 94587 [details] [review]
possible fix

It's not correct to swap the values late, they are mangled in various ways before. They need to be swapped right at extraction. Does the attached patch help?

(In reply to comment #8)
> It's not correct to swap the values late, they are mangled in various ways
> before. They need to be swapped right at extraction. Does the attached patch
> help?

This doesn't fix the issue either. I have tried something similar as well before.

Thanks.

Do you have Xorg log with the faulty commit ?

Created attachment 94679 [details] [review]
Fix constant upload

Ok attach patch do fix the issue can you test on your config that it works for you too.

(In reply to comment #11)
> Created attachment 94679 [details] [review] [review]
> Fix constant upload
> 
> Ok attach patch do fix the issue can you test on your config that it works
> for you too.

This patch fixed the issue on my ppc64 setup as well.

Thank you!

Pushed upstream closing