xf86-video-intel: src/sna/sna_accel.c src/sna/sna_dri.c src/sna/sna_render.c
Chris Wilson
ickle at kemper.freedesktop.org
Tue Nov 20 14:11:54 PST 2012
src/sna/sna_accel.c | 17 ++++++++++++++++-
src/sna/sna_dri.c | 2 ++
src/sna/sna_render.c | 8 ++++++--
3 files changed, 24 insertions(+), 3 deletions(-)
New commits:
commit 03fb9ded43f9bf8b73f99d5b3a8dc592fe22b523
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date: Tue Nov 20 21:13:07 2012 +0000
sna: Assert that we do not create a proxy with existent GPU damage
References: http://bugs.freedesktop.org/show_bug.cgi?id=47597
Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
diff --git a/src/sna/sna_accel.c b/src/sna/sna_accel.c
index 040a14d..dfd7d26 100644
--- a/src/sna/sna_accel.c
+++ b/src/sna/sna_accel.c
@@ -660,6 +660,7 @@ bool sna_pixmap_attach_to_bo(PixmapPtr pixmap, struct kgem_bo *bo)
return false;
priv->gpu_bo = kgem_bo_reference(bo);
+ assert(priv->gpu_bo->proxy == NULL);
sna_damage_all(&priv->gpu_damage,
pixmap->drawable.width,
pixmap->drawable.height);
@@ -1468,6 +1469,7 @@ _sna_pixmap_move_to_cpu(PixmapPtr pixmap, unsigned int flags)
}
pixmap->devKind = priv->gpu_bo->pitch;
+ assert(priv->gpu_bo->proxy == NULL);
sna_damage_all(&priv->gpu_damage,
pixmap->drawable.width,
pixmap->drawable.height);
@@ -1523,6 +1525,7 @@ skip_inplace_map:
priv->mapped = true;
pixmap->devKind = priv->gpu_bo->pitch;
if (flags & MOVE_WRITE) {
+ assert(priv->gpu_bo->proxy == NULL);
sna_damage_all(&priv->gpu_damage,
pixmap->drawable.width,
pixmap->drawable.height);
@@ -2555,6 +2558,7 @@ done:
box_inplace(pixmap, &r.extents)) {
DBG(("%s: large operation on undamaged, promoting to full GPU\n",
__FUNCTION__));
+ assert(priv->gpu_bo->proxy == NULL);
sna_damage_all(&priv->gpu_damage,
pixmap->drawable.width,
pixmap->drawable.height);
@@ -3014,6 +3018,7 @@ sna_pixmap_move_to_gpu(PixmapPtr pixmap, unsigned flags)
if (flags & MOVE_WRITE && priv->gpu_bo && priv->gpu_bo->proxy) {
DBG(("%s: discarding cached upload buffer\n", __FUNCTION__));
+ assert(priv->gpu_damage == NULL);
kgem_bo_destroy(&sna->kgem, priv->gpu_bo);
priv->gpu_bo = NULL;
}
@@ -3066,6 +3071,7 @@ sna_pixmap_move_to_gpu(PixmapPtr pixmap, unsigned flags)
* synchronisation that takes the most time. This is
* mitigated by avoiding fallbacks in the first place.
*/
+ assert(priv->gpu_bo->proxy == NULL);
sna_damage_all(&priv->gpu_damage,
pixmap->drawable.width,
pixmap->drawable.height);
@@ -3078,6 +3084,7 @@ sna_pixmap_move_to_gpu(PixmapPtr pixmap, unsigned flags)
if (priv->gpu_bo->proxy) {
DBG(("%s: reusing cached upload\n", __FUNCTION__));
assert((flags & MOVE_WRITE) == 0);
+ assert(priv->gpu_damage == NULL);
return priv;
}
@@ -3144,6 +3151,7 @@ sna_pixmap_move_to_gpu(PixmapPtr pixmap, unsigned flags)
/* For large bo, try to keep only a single copy around */
if (priv->create & KGEM_CAN_CREATE_LARGE) {
+ assert(priv->gpu_bo->proxy == NULL);
sna_damage_all(&priv->gpu_damage,
pixmap->drawable.width,
pixmap->drawable.height);
@@ -3540,6 +3548,7 @@ sna_put_zpixmap_blt(DrawablePtr drawable, GCPtr gc, RegionPtr region,
else
sna_damage_subtract(&priv->cpu_damage, region);
if (priv->cpu_damage == NULL) {
+ assert(priv->gpu_bo->proxy == NULL);
sna_damage_all(&priv->gpu_damage,
pixmap->drawable.width,
pixmap->drawable.height);
@@ -3599,6 +3608,7 @@ sna_put_zpixmap_blt(DrawablePtr drawable, GCPtr gc, RegionPtr region,
}
}
assert(priv->cpu_damage == NULL);
+ assert(priv->gpu_bo->proxy == NULL);
sna_damage_all(&priv->gpu_damage,
pixmap->drawable.width,
pixmap->drawable.height);
@@ -4644,8 +4654,10 @@ sna_copy_boxes(DrawablePtr src, DrawablePtr dst, GCPtr gc,
if (n == 1 &&
tmp->drawable.width == src_pixmap->drawable.width &&
- tmp->drawable.height == src_pixmap->drawable.height)
+ tmp->drawable.height == src_pixmap->drawable.height) {
+ assert(src_priv->gpu_damage);
kgem_proxy_bo_attach(src_bo, &src_priv->gpu_bo);
+ }
if (!sna->render.copy_boxes(sna, alu,
tmp, src_bo, dx, dy,
@@ -4703,6 +4715,7 @@ sna_copy_boxes(DrawablePtr src, DrawablePtr dst, GCPtr gc,
dst_priv->cpu = false;
if (damage) {
+ assert(dst_priv->gpu_bo->proxy == NULL);
if (replaces) {
sna_damage_destroy(&dst_priv->cpu_damage);
sna_damage_all(&dst_priv->gpu_damage,
@@ -9484,6 +9497,7 @@ sna_poly_fill_rect_blt(DrawablePtr drawable,
r.y2 - r.y1 == pixmap->drawable.height) {
struct sna_pixmap *priv = sna_pixmap(pixmap);
if (bo == priv->gpu_bo) {
+ assert(priv->gpu_bo->proxy == NULL);
sna_damage_all(&priv->gpu_damage,
pixmap->drawable.width,
pixmap->drawable.height);
@@ -11925,6 +11939,7 @@ sna_poly_fill_rect(DrawablePtr draw, GCPtr gc, int n, xRectangle *rect)
box_inplace(pixmap, ®ion.extents))) {
DBG(("%s: promoting to full GPU\n", __FUNCTION__));
if (priv->gpu_bo) {
+ assert(priv->gpu_bo->proxy == NULL);
sna_damage_all(&priv->gpu_damage,
pixmap->drawable.width,
pixmap->drawable.height);
diff --git a/src/sna/sna_dri.c b/src/sna/sna_dri.c
index eb7c835..82d9485 100644
--- a/src/sna/sna_dri.c
+++ b/src/sna/sna_dri.c
@@ -175,6 +175,7 @@ static struct kgem_bo *sna_pixmap_set_dri(struct sna *sna,
}
assert(priv->cpu_damage == NULL);
+ assert(priv->gpu_bo->proxy == NULL);
if (priv->flush++)
return priv->gpu_bo;
@@ -460,6 +461,7 @@ static void set_bo(PixmapPtr pixmap, struct kgem_bo *bo)
assert(pixmap->drawable.width * pixmap->drawable.bitsPerPixel <= 8*bo->pitch);
assert(pixmap->drawable.height * bo->pitch <= kgem_bo_size(bo));
+ assert(bo->proxy == NULL);
/* Post damage on the new front buffer so that listeners, such
* as DisplayLink know take a copy and shove it over the USB,
diff --git a/src/sna/sna_render.c b/src/sna/sna_render.c
index 34c795b..f5586cb 100644
--- a/src/sna/sna_render.c
+++ b/src/sna/sna_render.c
@@ -515,8 +515,10 @@ static struct kgem_bo *upload(struct sna *sna,
if (priv &&
pixmap->usage_hint == 0 &&
channel->width == pixmap->drawable.width &&
- channel->height == pixmap->drawable.height)
+ channel->height == pixmap->drawable.height) {
+ assert(priv->gpu_damage == NULL);
kgem_proxy_bo_attach(bo, &priv->gpu_bo);
+ }
}
return bo;
@@ -1147,8 +1149,10 @@ sna_render_picture_extract(struct sna *sna,
box.x2 - box.x1 == pixmap->drawable.width &&
box.y2 - box.y1 == pixmap->drawable.height) {
struct sna_pixmap *priv = sna_pixmap(pixmap);
- if (priv)
+ if (priv) {
+ assert(priv->gpu_damage == NULL);
kgem_proxy_bo_attach(bo, &priv->gpu_bo);
+ }
}
}
}
More information about the xorg-commit
mailing list