Mesa (master): i965: Don't map/ unmap in brw_print_program_cache on LLC platforms.

Kenneth Graunke kwg at kemper.freedesktop.org
Wed Jan 18 05:55:48 UTC 2017


Module: Mesa
Branch: master
Commit: aa291c3ba9b1062d219276cef12b1b7c5380b423
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=aa291c3ba9b1062d219276cef12b1b7c5380b423

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Fri Nov 11 18:05:14 2016 -0800

i965: Don't map/unmap in brw_print_program_cache on LLC platforms.

We have a persistent mapping.  Don't map it a second time or try to
unmap it.  Just use the pointer.

This most likely would wreak havoc except that this code is unused
(it's only called from an if (0) debug block).

Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Eduardo Lima Mitev <elima at igalia.com>

---

 src/mesa/drivers/dri/i965/brw_program_cache.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_program_cache.c b/src/mesa/drivers/dri/i965/brw_program_cache.c
index 8939fb1..44d9994 100644
--- a/src/mesa/drivers/dri/i965/brw_program_cache.c
+++ b/src/mesa/drivers/dri/i965/brw_program_cache.c
@@ -514,7 +514,8 @@ brw_print_program_cache(struct brw_context *brw)
    const struct brw_cache *cache = &brw->cache;
    struct brw_cache_item *item;
 
-   drm_intel_bo_map(cache->bo, false);
+   if (!brw->has_llc)
+      drm_intel_bo_map(cache->bo, false);
 
    for (unsigned i = 0; i < cache->size; i++) {
       for (item = cache->items[i]; item; item = item->next) {
@@ -524,5 +525,6 @@ brw_print_program_cache(struct brw_context *brw)
       }
    }
 
-   drm_intel_bo_unmap(cache->bo);
+   if (!brw->has_llc)
+      drm_intel_bo_unmap(cache->bo);
 }




More information about the mesa-commit mailing list