Attachment 82197 Details for Bug 66663 – patch that fixes the accelerated case

[patch] patch that fixes the accelerated case

0001-Always-go-through-DFS-UTS-when-byte-swapping-is-need.patch (text/plain), 1.08 KB, created by Mark Kettenis on 2013-07-08 18:27:22 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Mark Kettenis

Created: 2013-07-08 18:27:22 UTC

Size: 1.08 KB

patch

obsolete

>From 8a2e1cead5fde2e6954384a6f40478dc70006215 Mon Sep 17 00:00:00 2001
>From: Mark Kettenis <kettenis@openbsd.org>
>Date: Sun, 7 Jul 2013 13:44:13 +0200
>Subject: [PATCH] Always go through DFS/UTS when byte swapping is needed.
>
>Before commit ef9bfb262db7004bef3704e5d914687e50d3fca4 and
>e5bd99faa3b6629a55168386d5dfa936ee4e97ae, byte swapping for the front buffer
>used to be done in hardware.  Now that this no longer happens we need to let
>DFS/UTS ihandle the byte swapping.
>
>See https://bugs.freedesktop.org/show_bug.cgi?id=66663 .
>
>Signed-off-by: Mark Kettenis <kettenis@openbsd.org>
>---
> src/radeon_exa.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/src/radeon_exa.c b/src/radeon_exa.c
>index fcafb68..cf368d5 100644
>--- a/src/radeon_exa.c
>+++ b/src/radeon_exa.c
>@@ -172,7 +172,7 @@ Bool RADEONPrepareAccess_CS(PixmapPtr pPix, int index)
> 
> #if X_BYTE_ORDER == X_BIG_ENDIAN
>     /* May need to handle byte swapping in DownloadFrom/UploadToScreen */
>-    if (can_fail && pPix->drawable.bitsPerPixel > 8)
>+    if (pPix->drawable.bitsPerPixel > 8)
> 	return FALSE;
> #endif
> 
>-- 
>1.8.3.1
>

Actions: View | Splinter Review

Attachments on bug 66663: 82134 | 82197

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.