diff options
author | Damien Lespiau <damien.lespiau@intel.com> | 2013-12-16 11:28:39 +0000 |
---|---|---|
committer | Damien Lespiau <damien.lespiau@intel.com> | 2014-01-06 13:40:21 +0000 |
commit | 1d2ef9553ae4b738595676b7554f93d70acd78a0 (patch) | |
tree | 97d2848c4a3572a2d91746438377372821a190bf /tools/intel_error_decode.c | |
parent | 01eeab94482de8c8fa687ffcfa95e5d32720909d (diff) |
intel_error_decode: Factor out common decoding code
4 pieces of code were looking very similar. Let's factor out a common
function in the not so unlikely case we need to tweak that code.
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Diffstat (limited to 'tools/intel_error_decode.c')
-rw-r--r-- | tools/intel_error_decode.c | 52 |
1 files changed, 21 insertions, 31 deletions
diff --git a/tools/intel_error_decode.c b/tools/intel_error_decode.c index a22ec7d4..529ec54a 100644 --- a/tools/intel_error_decode.c +++ b/tools/intel_error_decode.c @@ -39,6 +39,7 @@ */ #define _GNU_SOURCE +#include <stdbool.h> #include <stdio.h> #include <stdlib.h> #include <stdarg.h> @@ -319,6 +320,19 @@ static void print_batch(int is_batch, const char *ring_name, uint32_t gtt_offset } } +static void decode(struct drm_intel_decode *ctx, bool is_batch, + const char *ring_name, uint32_t gtt_offset, uint32_t *data, + int *count) +{ + if (!*count) + return; + + print_batch(is_batch, ring_name, gtt_offset); + drm_intel_decode_set_batch_pointer(ctx, data, gtt_offset, *count); + drm_intel_decode(ctx); + *count = 0; +} + static void read_data_file(FILE *file) { @@ -350,14 +364,8 @@ read_data_file(FILE *file) matched = sscanf(dashes, "--- gtt_offset = 0x%08x\n", &new_gtt_offset); if (matched == 1) { - if (count) { - print_batch(is_batch, ring_name, gtt_offset); - drm_intel_decode_set_batch_pointer(decode_ctx, - data, gtt_offset, - count); - drm_intel_decode(decode_ctx); - count = 0; - } + decode(decode_ctx, is_batch, ring_name, + gtt_offset, data, &count); gtt_offset = new_gtt_offset; is_batch = 1; free(ring_name); @@ -368,14 +376,8 @@ read_data_file(FILE *file) matched = sscanf(dashes, "--- ringbuffer = 0x%08x\n", &new_gtt_offset); if (matched == 1) { - if (count) { - print_batch(is_batch, ring_name, gtt_offset); - drm_intel_decode_set_batch_pointer(decode_ctx, - data, gtt_offset, - count); - drm_intel_decode(decode_ctx); - count = 0; - } + decode(decode_ctx, is_batch, ring_name, + gtt_offset, data, &count); gtt_offset = new_gtt_offset; is_batch = 0; free(ring_name); @@ -389,14 +391,8 @@ read_data_file(FILE *file) unsigned int reg; /* display reg section is after the ringbuffers, don't mix them */ - if (count) { - print_batch(is_batch, ring_name, gtt_offset); - drm_intel_decode_set_batch_pointer(decode_ctx, - data, gtt_offset, - count); - drm_intel_decode(decode_ctx); - count = 0; - } + decode(decode_ctx, is_batch, ring_name, gtt_offset, + data, &count); printf("%s", line); @@ -464,13 +460,7 @@ read_data_file(FILE *file) data[count-1] = value; } - if (count) { - print_batch(is_batch, ring_name, gtt_offset); - drm_intel_decode_set_batch_pointer(decode_ctx, - data, gtt_offset, - count); - drm_intel_decode(decode_ctx); - } + decode(decode_ctx, is_batch, ring_name, gtt_offset, data, &count); free(data); free(line); |