Reviewed-by: Lionel Landwerlin <lionel.g.landwer...@intel.com>

On 27/12/17 20:58, Jason Ekstrand wrote:
Some older versions of the Vulkan driver didn't properly tag dynamic
state as needing to be captured.  Also, this prevents crashes when
looking at dumps on older kernels.
---
  src/intel/tools/gen_batch_decoder.c | 5 +++++
  1 file changed, 5 insertions(+)

diff --git a/src/intel/tools/gen_batch_decoder.c 
b/src/intel/tools/gen_batch_decoder.c
index e8dd19b..f09b833 100644
--- a/src/intel/tools/gen_batch_decoder.c
+++ b/src/intel/tools/gen_batch_decoder.c
@@ -582,6 +582,11 @@ decode_dynamic_state_pointers(struct gen_batch_decode_ctx 
*ctx,
                                const char *struct_type, const uint32_t *p,
                                int count)
  {
+   if (ctx->dynamic_base.map == NULL) {
+      fprintf(ctx->fp, "  dynamic %s state unavailable\n", struct_type);
+      return;
+   }
+
     struct gen_group *inst = gen_spec_find_instruction(ctx->spec, p);
     struct gen_group *state = gen_spec_find_struct(ctx->spec, struct_type);


_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to