[Mesa-dev] [PATCH v2 15/15] i965: Print size of validation and relocation lists in INTEL_DEBUG=flush

Kenneth Graunke kenneth at whitecape.org
Wed Sep 13 20:54:17 UTC 2017


It's nice to have this information.  While we're at it, tweak the
formatting to try and vertically align numbers in the common case.
---
 src/mesa/drivers/dri/i965/intel_batchbuffer.c | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/intel_batchbuffer.c b/src/mesa/drivers/dri/i965/intel_batchbuffer.c
index fddc84fcf9b..b6ce8174d57 100644
--- a/src/mesa/drivers/dri/i965/intel_batchbuffer.c
+++ b/src/mesa/drivers/dri/i965/intel_batchbuffer.c
@@ -886,10 +886,15 @@ _intel_batchbuffer_flush_fence(struct brw_context *brw,
    if (unlikely(INTEL_DEBUG & (DEBUG_BATCH | DEBUG_SUBMIT))) {
       int bytes_for_commands = 4 * USED_BATCH(brw->batch);
       int bytes_for_state = brw->batch.state_used;
-      fprintf(stderr, "%s:%d: Batchbuffer flush with %4db (%0.1f%%) (pkt) + "
-              "%4db (%0.1f%%) (state)\n", file, line,
+      fprintf(stderr, "%19s:%-3d: Batchbuffer flush with %5db (%0.1f%%) (pkt),"
+              " %5db (%0.1f%%) (state), %4d BOs (%0.1fMb aperture),"
+              " %4d batch relocs, %4d state relocs\n", file, line,
               bytes_for_commands, 100.0f * bytes_for_commands / BATCH_SZ,
-              bytes_for_state, 100.0f * bytes_for_state / STATE_SZ);
+              bytes_for_state, 100.0f * bytes_for_state / STATE_SZ,
+              brw->batch.exec_count,
+              (float) brw->batch.aperture_space / (1024 * 1024),
+              brw->batch.batch_relocs.reloc_count,
+              brw->batch.state_relocs.reloc_count);
    }
 
    brw_finish_batch(brw);
-- 
2.14.1



More information about the mesa-dev mailing list