trace: don't trace resource_destroy
authorMarek Olšák <marek.olsak@amd.com>
Sun, 16 Apr 2017 18:09:03 +0000 (20:09 +0200)
committerMarek Olšák <marek.olsak@amd.com>
Tue, 25 Apr 2017 20:39:31 +0000 (22:39 +0200)
due to the lack of pipe_resource wrapping, we can get this call from inside
of driver calls, which would try to lock an already-locked mutex.

Reviewed-by: Nicolai Hähnle <nicolai.haehnle@amd.com>
src/gallium/drivers/trace/tr_screen.c

index 0fa8d0f449f2a3f8486da6a92d7467d952b86b5a..e56434c5bda61370cc02ae29eb1741896c7ff8a9 100644 (file)
@@ -391,13 +391,10 @@ trace_screen_resource_destroy(struct pipe_screen *_screen,
    struct trace_screen *tr_scr = trace_screen(_screen);
    struct pipe_screen *screen = tr_scr->screen;
 
-   trace_dump_call_begin("pipe_screen", "resource_destroy");
-
-   trace_dump_arg(ptr, screen);
-   trace_dump_arg(ptr, resource);
-
-   trace_dump_call_end();
-
+   /* Don't trace this, because due to the lack of pipe_resource wrapping,
+    * we can get this call from inside of driver calls, which would try
+    * to lock an already-locked mutex.
+    */
    screen->resource_destroy(screen, resource);
 }