summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChris Wilson <chris@chris-wilson.co.uk>2014-07-16 07:19:03 +0100
committerChris Wilson <chris@chris-wilson.co.uk>2014-07-16 07:19:03 +0100
commit34ada63118680bf8ad2ef737fd4a8edfd8246e37 (patch)
treea6dea74238aecfff674595b0f404389cb6614ad9
parent4422b1854209a2cdf023c1fbc60834fde2fbc0c7 (diff)
sna/gen2+: Remove assertion on draw->type for fills
I overlooked the tiling-fill path passing down a temporary DrawableRec into the fill_boxes callback - invalidating the assertion. Fixes regression from commit 43176b9bfafe389c4b9ed676f4e50e3b06f858c4 Author: Chris Wilson <chris@chris-wilson.co.uk> Date: Mon Jun 30 21:01:11 2014 +0100 sna/dri2: Pass around the correct DrawableRec for sampling from the foriegn bo Reported-by: Jiri Slaby <jirislaby@gmail.com> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70461#c62 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
-rw-r--r--src/sna/gen2_render.c1
-rw-r--r--src/sna/gen3_render.c1
-rw-r--r--src/sna/gen4_render.c1
-rw-r--r--src/sna/gen5_render.c1
-rw-r--r--src/sna/gen6_render.c1
-rw-r--r--src/sna/gen7_render.c1
-rw-r--r--src/sna/gen8_render.c1
-rw-r--r--src/sna/sna_render.c1
8 files changed, 0 insertions, 8 deletions
diff --git a/src/sna/gen2_render.c b/src/sna/gen2_render.c
index 4cc8dcd5..379079df 100644
--- a/src/sna/gen2_render.c
+++ b/src/sna/gen2_render.c
@@ -2821,7 +2821,6 @@ gen2_render_fill_boxes(struct sna *sna,
memset(&tmp, 0, sizeof(tmp));
tmp.op = op;
- assert(dst->type == DRAWABLE_PIXMAP);
tmp.dst.pixmap = (PixmapPtr)dst;
tmp.dst.width = dst->width;
tmp.dst.height = dst->height;
diff --git a/src/sna/gen3_render.c b/src/sna/gen3_render.c
index 8897f52d..bb18dd11 100644
--- a/src/sna/gen3_render.c
+++ b/src/sna/gen3_render.c
@@ -5927,7 +5927,6 @@ gen3_render_fill_boxes(struct sna *sna,
__FUNCTION__, op, (int)format, pixel));
tmp.op = op;
- assert(dst->type == DRAWABLE_PIXMAP);
tmp.dst.pixmap = (PixmapPtr)dst;
tmp.dst.width = dst->width;
tmp.dst.height = dst->height;
diff --git a/src/sna/gen4_render.c b/src/sna/gen4_render.c
index 0bef20fc..32ad8688 100644
--- a/src/sna/gen4_render.c
+++ b/src/sna/gen4_render.c
@@ -2732,7 +2732,6 @@ gen4_render_fill_boxes(struct sna *sna,
tmp.op = op;
- assert(dst->type == DRAWABLE_PIXMAP);
tmp.dst.pixmap = (PixmapPtr)dst;
tmp.dst.width = dst->width;
tmp.dst.height = dst->height;
diff --git a/src/sna/gen5_render.c b/src/sna/gen5_render.c
index 965e60fc..93bc0d21 100644
--- a/src/sna/gen5_render.c
+++ b/src/sna/gen5_render.c
@@ -2728,7 +2728,6 @@ gen5_render_fill_boxes(struct sna *sna,
tmp.op = op;
- assert(dst->type == DRAWABLE_PIXMAP);
tmp.dst.pixmap = (PixmapPtr)dst;
tmp.dst.width = dst->width;
tmp.dst.height = dst->height;
diff --git a/src/sna/gen6_render.c b/src/sna/gen6_render.c
index ab6e296c..4ee786bd 100644
--- a/src/sna/gen6_render.c
+++ b/src/sna/gen6_render.c
@@ -3097,7 +3097,6 @@ gen6_render_fill_boxes(struct sna *sna,
__FUNCTION__, pixel, n,
box[0].x1, box[0].y1, box[0].x2, box[0].y2));
- assert(dst->type == DRAWABLE_PIXMAP);
tmp.dst.pixmap = (PixmapPtr)dst;
tmp.dst.width = dst->width;
tmp.dst.height = dst->height;
diff --git a/src/sna/gen7_render.c b/src/sna/gen7_render.c
index b1faac44..59b954d3 100644
--- a/src/sna/gen7_render.c
+++ b/src/sna/gen7_render.c
@@ -3331,7 +3331,6 @@ gen7_render_fill_boxes(struct sna *sna,
__FUNCTION__, pixel, n,
box[0].x1, box[0].y1, box[0].x2, box[0].y2));
- assert(dst->type == DRAWABLE_PIXMAP);
tmp.dst.pixmap = (PixmapPtr)dst;
tmp.dst.width = dst->width;
tmp.dst.height = dst->height;
diff --git a/src/sna/gen8_render.c b/src/sna/gen8_render.c
index 2eb44a9a..826675ce 100644
--- a/src/sna/gen8_render.c
+++ b/src/sna/gen8_render.c
@@ -3161,7 +3161,6 @@ gen8_render_fill_boxes(struct sna *sna,
__FUNCTION__, pixel, n,
box[0].x1, box[0].y1, box[0].x2, box[0].y2));
- assert(dst->type == DRAWABLE_PIXMAP);
tmp.dst.pixmap = (PixmapPtr)dst;
tmp.dst.width = dst->width;
tmp.dst.height = dst->height;
diff --git a/src/sna/sna_render.c b/src/sna/sna_render.c
index 89be5a58..02001025 100644
--- a/src/sna/sna_render.c
+++ b/src/sna/sna_render.c
@@ -2181,7 +2181,6 @@ sna_render_copy_boxes__overlap(struct sna *sna, uint8_t alu,
pixman_box16_t stack_boxes[64], *boxes = stack_boxes;
int num_boxes, i;
- assert(draw->type == DRAWABLE_PIXMAP);
DBG(("%s: pixmap=%ld, handle=%d, %d x [(%d, %d), (%d, %d)], dst=(%d, %d), src=(%d, %d)\n",
__FUNCTION__, draw->serialNumber, bo->handle,
n, extents->x1, extents->y1, extents->x2, extents->y2,