xf86-video-intel: src/sna/kgem.c

Chris Wilson ickle at kemper.freedesktop.org
Sat Jul 20 12:05:18 PDT 2013


 src/sna/kgem.c |    2 --
 1 file changed, 2 deletions(-)

New commits:
commit b27b1654145c62493614dd7545dcd9fe6936750e
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date:   Sat Jul 20 20:04:00 2013 +0100

    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 at chris-wilson.co.uk>

diff --git a/src/sna/kgem.c b/src/sna/kgem.c
index c309cae..863ab8a 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)) {


More information about the xorg-commit mailing list