Alexandre Courbot
2014-Oct-27  09:11 UTC
[Nouveau] [PATCH] drm: warn when moving a pinned object
Pinned BOs are supposed to remain in their current location until
unpinned. Display a warning for the supposedly-erroneous case where we
are trying to move such objects.
Signed-off-by: Alexandre Courbot <acourbot at nvidia.com>
---
 drm/nouveau_bo.c | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/drm/nouveau_bo.c b/drm/nouveau_bo.c
index 3d474ac03f88..9a8adeec80cd 100644
--- a/drm/nouveau_bo.c
+++ b/drm/nouveau_bo.c
@@ -323,8 +323,8 @@ nouveau_bo_pin(struct nouveau_bo *nvbo, uint32_t memtype)
 		goto out;
 	}
 
-	if (nvbo->pin_refcnt++)
-		goto out;
+	if (nvbo->pin_refcnt)
+		goto ref_inc;
 
 	nouveau_bo_placement_set(nvbo, memtype, 0);
 
@@ -341,6 +341,10 @@ nouveau_bo_pin(struct nouveau_bo *nvbo, uint32_t memtype)
 			break;
 		}
 	}
+
+ref_inc:
+	nvbo->pin_refcnt++;
+
 out:
 	ttm_bo_unreserve(bo);
 	return ret;
@@ -1184,6 +1188,9 @@ nouveau_bo_move(struct ttm_buffer_object *bo, bool evict,
bool intr,
 	struct nouveau_drm_tile *new_tile = NULL;
 	int ret = 0;
 
+	if (nvbo->pin_refcnt)
+		NV_WARN(drm, "Moving pinned object %p!\n", nvbo);
+
 	if (drm->device.info.family < NV_DEVICE_INFO_V0_TESLA) {
 		ret = nouveau_bo_vm_bind(bo, new_mem, &new_tile);
 		if (ret)
-- 
2.1.2
Maybe Matching Threads
- [RFC PATCH 6/7] drm/nouveau: Support marking buffers for explicit sync
- [PATCH] drm/nouveau: Make use of TTM busy_placements.
- [resend PATCH] drm/nouveau: unpin buffers before releasing to prevent lockdep warnings
- [PATCH 3/3, resend with fixed to field] drm/nouveau: unpin buffers before releasing to prevent lockdep warnings
- [PATCH v2 06/18] drm/nouveau: use embedded gem object
