broadcom/vc5: Print the buffer name in simulator overflow checks.
authorEric Anholt <eric@anholt.net>
Fri, 12 Jan 2018 01:22:58 +0000 (17:22 -0800)
committerEric Anholt <eric@anholt.net>
Sat, 13 Jan 2018 05:57:30 +0000 (21:57 -0800)
Revealed that I was writing past the TSDA, not the Z buffer as I expected.

src/gallium/drivers/vc5/vc5_simulator.c

index d3ab2bc539df26a62ae255ea95c0442643076587..93e8d44e30ecc5632fae05c92bd902be76c74c66 100644 (file)
@@ -257,8 +257,10 @@ vc5_simulator_unpin_bos(int fd, struct vc5_job *job)
                 struct vc5_simulator_bo *sim_bo =
                         vc5_get_simulator_bo(file, bo->handle);
 
-                assert(*(uint32_t *)(sim_bo->vaddr +
-                                     sim_bo->size) == BO_SENTINEL);
+                if (*(uint32_t *)(sim_bo->vaddr +
+                                  sim_bo->size) != BO_SENTINEL) {
+                        fprintf(stderr, "Buffer overflow in %s\n", bo->name);
+                }
 
                 vc5_bo_map(bo);
                 memcpy(bo->map, sim_bo->vaddr, bo->size);