[Glamor] [PATCH 28/34] glamor: Fix ignoring the ALU during SetSpans().

Alex Deucher alexdeucher at gmail.com
Fri Feb 28 10:02:23 PST 2014


From: Eric Anholt <eric at anholt.net>

Ported from Eric's xserver glamor tree.

Signed-off-by: Eric Anholt <eric at anholt.net>
Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
---
 src/glamor_setspans.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/glamor_setspans.c b/src/glamor_setspans.c
index b1abfee..a9709e6 100644
--- a/src/glamor_setspans.c
+++ b/src/glamor_setspans.c
@@ -48,7 +48,11 @@ _glamor_set_spans(DrawablePtr drawable, GCPtr gc, char *src,
 		goto fail;
 	}
 
-	/* XXX Shall we set alu here? */
+	if (gc->alu != GXcopy) {
+	    glamor_fallback("SetSpans with non-copy ALU.\n");
+	    goto fail;
+	}
+
 	if (!glamor_set_planemask(dest_pixmap, gc->planemask))
 		goto fail;
 
-- 
1.8.3.1



More information about the Glamor mailing list