radv: do not print useless descriptors info in hang reports
authorSamuel Pitoiset <samuel.pitoiset@gmail.com>
Tue, 15 Oct 2019 12:49:38 +0000 (14:49 +0200)
committerSamuel Pitoiset <samuel.pitoiset@gmail.com>
Mon, 21 Oct 2019 06:50:39 +0000 (08:50 +0200)
This information has never been useful. All descriptors are
already dumped with colors etc, and it's more useful.

Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
src/amd/vulkan/radv_debug.c

index 3d7265551604eeadf72bd236a3de3b3c3aef1731..5bba93ac805477986107090cfcba4a40a491994d 100644 (file)
@@ -145,37 +145,6 @@ radv_dump_debug_registers(struct radv_device *device, FILE *f)
        fprintf(f, "\n");
 }
 
-static const char *
-radv_get_descriptor_name(enum VkDescriptorType type)
-{
-       switch (type) {
-       case VK_DESCRIPTOR_TYPE_SAMPLER:
-               return "SAMPLER";
-       case VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER:
-               return "COMBINED_IMAGE_SAMPLER";
-       case VK_DESCRIPTOR_TYPE_SAMPLED_IMAGE:
-               return "SAMPLED_IMAGE";
-       case VK_DESCRIPTOR_TYPE_STORAGE_IMAGE:
-               return "STORAGE_IMAGE";
-       case VK_DESCRIPTOR_TYPE_UNIFORM_TEXEL_BUFFER:
-               return "UNIFORM_TEXEL_BUFFER";
-       case VK_DESCRIPTOR_TYPE_STORAGE_TEXEL_BUFFER:
-               return "STORAGE_TEXEL_BUFFER";
-       case VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER:
-               return "UNIFORM_BUFFER";
-       case VK_DESCRIPTOR_TYPE_STORAGE_BUFFER:
-               return "STORAGE_BUFFER";
-       case VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER_DYNAMIC:
-               return "UNIFORM_BUFFER_DYNAMIC";
-       case VK_DESCRIPTOR_TYPE_STORAGE_BUFFER_DYNAMIC:
-               return "STORAGE_BUFFER_DYNAMIC";
-       case VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT:
-               return "INPUT_ATTACHMENT";
-       default:
-               return "UNKNOWN";
-       }
-}
-
 static void
 radv_dump_buffer_descriptor(enum chip_class chip_class, const uint32_t *desc,
                            FILE *f)
@@ -234,52 +203,10 @@ radv_dump_descriptor_set(enum chip_class chip_class,
                return;
        layout = set->layout;
 
-       fprintf(f, "** descriptor set (%d) **\n", id);
-       fprintf(f, "va: 0x%"PRIx64"\n", set->va);
-       fprintf(f, "size: %d\n", set->size);
-       fprintf(f, "mapped_ptr:\n");
-
-       for (i = 0; i < set->size / 4; i++) {
-               fprintf(f, "\t[0x%x] = 0x%08x\n", i, set->mapped_ptr[i]);
-       }
-       fprintf(f, "\n");
-
-       fprintf(f, "\t*** layout ***\n");
-       fprintf(f, "\tbinding_count: %d\n", layout->binding_count);
-       fprintf(f, "\tsize: %d\n", layout->size);
-       fprintf(f, "\tshader_stages: %x\n", layout->shader_stages);
-       fprintf(f, "\tdynamic_shader_stages: %x\n",
-               layout->dynamic_shader_stages);
-       fprintf(f, "\tbuffer_count: %d\n", layout->buffer_count);
-       fprintf(f, "\tdynamic_offset_count: %d\n",
-               layout->dynamic_offset_count);
-       fprintf(f, "\n");
-
        for (i = 0; i < set->layout->binding_count; i++) {
                uint32_t *desc =
                        set->mapped_ptr + layout->binding[i].offset / 4;
 
-               fprintf(f, "\t\t**** binding layout (%d) ****\n", i);
-               fprintf(f, "\t\ttype: %s\n",
-                       radv_get_descriptor_name(layout->binding[i].type));
-               fprintf(f, "\t\tarray_size: %d\n",
-                       layout->binding[i].array_size);
-               fprintf(f, "\t\toffset: %d\n",
-                       layout->binding[i].offset);
-               fprintf(f, "\t\tbuffer_offset: %d\n",
-                       layout->binding[i].buffer_offset);
-               fprintf(f, "\t\tdynamic_offset_offset: %d\n",
-                       layout->binding[i].dynamic_offset_offset);
-               fprintf(f, "\t\tdynamic_offset_count: %d\n",
-                       layout->binding[i].dynamic_offset_count);
-               fprintf(f, "\t\tsize: %d\n",
-                       layout->binding[i].size);
-               fprintf(f, "\t\timmutable_samplers_offset: %d\n",
-                       layout->binding[i].immutable_samplers_offset);
-               fprintf(f, "\t\timmutable_samplers_equal: %d\n",
-                       layout->binding[i].immutable_samplers_equal);
-               fprintf(f, "\n");
-
                switch (layout->binding[i].type) {
                case VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER:
                case VK_DESCRIPTOR_TYPE_STORAGE_BUFFER:
@@ -319,7 +246,7 @@ radv_dump_descriptors(struct radv_pipeline *pipeline, FILE *f)
        uint64_t *ptr = (uint64_t *)device->trace_id_ptr;
        int i;
 
-       fprintf(f, "List of descriptors:\n");
+       fprintf(f, "Descriptors:\n");
        for (i = 0; i < MAX_SETS; i++) {
                struct radv_descriptor_set *set =
                        (struct radv_descriptor_set *)ptr[i + 3];