From bb0969e6a5c2bddcd0499d81c0c241003f8b07ef Mon Sep 17 00:00:00 2001 From: Chris Wilson Date: Tue, 14 May 2013 15:22:59 +0100 Subject: [PATCH] sna: Assert that the mapping is released before closing the GEM handle Signed-off-by: Chris Wilson --- src/sna/kgem.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/sna/kgem.c b/src/sna/kgem.c index 8b096c50..ef8f0363 100644 --- a/src/sna/kgem.c +++ b/src/sna/kgem.c @@ -1499,6 +1499,7 @@ static void kgem_bo_free(struct kgem *kgem, struct kgem_bo *bo) if (bo->map) kgem_bo_release_map(kgem, bo); assert(list_is_empty(&bo->vma)); + assert(bo->map == NULL); _list_del(&bo->list); _list_del(&bo->request); @@ -2182,6 +2183,7 @@ static void kgem_commit(struct kgem *kgem) kgem_retire(kgem); assert(list_is_empty(&rq->buffers)); + assert(rq->bo->map == NULL); gem_close(kgem->fd, rq->bo->handle); kgem_cleanup_cache(kgem); } else { @@ -5796,7 +5798,7 @@ kgem_replace_bo(struct kgem *kgem, return NULL; dst = __kgem_bo_alloc(handle, size); - if (dst== NULL) { + if (dst == NULL) { gem_close(kgem->fd, handle); return NULL; }