summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChris Wilson <chris@chris-wilson.co.uk>2013-07-20 20:04:00 +0100
committerChris Wilson <chris@chris-wilson.co.uk>2013-07-20 20:04:59 +0100
commitb27b1654145c62493614dd7545dcd9fe6936750e (patch)
tree4619bb5af2da32b62137ce764beb7249749e520e
parentf39d9ed0929f83e65dde8da8f03fae6ba8264e4a (diff)
sna: Relax assertion that CPU syncs are performed on reffed objects
Along some paths, we use a CPU mapping on a temporary object that never gets a user ref - so the assertion is bogus. A lesson I keep forgetting. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
-rw-r--r--src/sna/kgem.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/src/sna/kgem.c b/src/sna/kgem.c
index c309caed..863ab8a2 100644
--- a/src/sna/kgem.c
+++ b/src/sna/kgem.c
@@ -5190,10 +5190,8 @@ void kgem_bo_sync__cpu(struct kgem *kgem, struct kgem_bo *bo)
/* SHM pixmaps use proxies for subpage offsets */
assert(!bo->purged);
- assert(bo->refcnt);
while (bo->proxy)
bo = bo->proxy;
- assert(bo->refcnt);
assert(!bo->purged);
if (bo->domain != DOMAIN_CPU || FORCE_MMAP_SYNC & (1 << DOMAIN_CPU)) {