On Sat, Apr 29, 2017 at 9:18 AM, Emil Velikov <[email protected]> wrote:
> On 18 April 2017 at 05:00, Ilia Mirkin <[email protected]> wrote:
>> val_bool and val_int are in a union. val_bool gets the first byte, which
>> happens to work on LE when setting via the int, but breaks on BE. By
>> setting the value properly, we are able to use DRI3 on BE architectures.
>> Tested by running glxgears with a NV34 in a G5 PPC.
>>
>> Signed-off-by: Ilia Mirkin <[email protected]>
>> Cc: [email protected]
>> ---
>>  src/gallium/auxiliary/pipe-loader/pipe_loader_drm.c | 8 ++++----
>>  src/gallium/targets/pipe-loader/pipe_i915.c         | 2 +-
>>  src/gallium/targets/pipe-loader/pipe_msm.c          | 2 +-
>>  src/gallium/targets/pipe-loader/pipe_nouveau.c      | 2 +-
>>  src/gallium/targets/pipe-loader/pipe_r300.c         | 2 +-
>>  src/gallium/targets/pipe-loader/pipe_r600.c         | 2 +-
>>  src/gallium/targets/pipe-loader/pipe_radeonsi.c     | 2 +-
>>  7 files changed, 10 insertions(+), 10 deletions(-)
>>
> A while ago I've managed to half the duplication, but we still have some ;-)
>
> With the vmwgfx hunk squashed:
> Reviewed-by: Emil Velikov <[email protected]>

Thanks. I totally forgot about this... will try to push it out today.
Feel free to do it for me if you like.

  -ilia
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to