From 4c766c7959192b455ae752a7759ffc159e0524a6 Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Thu, 21 Aug 2014 10:50:31 -0700 Subject: [PATCH] i965: Issue performance warnings when copying the program cache BO. MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit We don't really want unnecessary buffer copying, so it'd be nice to know when it's happening. v2: Drop stall warnings when doing a read-only CPU mapping of the cache BO. The GPU also uses it in a read-only fashion, so there won't be any stalls, even though the buffer is busy. (Thanks to Chris Wilson for catching this mistake.) Signed-off-by: Kenneth Graunke Reviewed-by: Kristian Høgsberg [v1] --- src/mesa/drivers/dri/i965/brw_state_cache.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_state_cache.c b/src/mesa/drivers/dri/i965/brw_state_cache.c index 62e03b12984..f4cae8f560b 100644 --- a/src/mesa/drivers/dri/i965/brw_state_cache.c +++ b/src/mesa/drivers/dri/i965/brw_state_cache.c @@ -241,6 +241,8 @@ brw_upload_item_data(struct brw_cache *cache, struct brw_cache_item *item, const void *data) { + struct brw_context *brw = cache->brw; + /* Allocate space in the cache BO for our new program. */ if (cache->next_offset + item->size > cache->bo->size) { uint32_t new_size = cache->bo->size * 2; @@ -255,6 +257,7 @@ brw_upload_item_data(struct brw_cache *cache, * recreate it. */ if (cache->bo_used_by_gpu) { + perf_debug("Copying busy program cache buffer.\n"); brw_cache_new_bo(cache, cache->bo->size); } -- 2.30.2