[PATCH vkd3d v2] tests: Use only one heap for textures in the d3d12 shader runner.

[Zebediah Figura]

The d3d12 specification forbids simultaneously binding more than one heap of a
given type.

Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com>
---
Tested with 227604; I can reproduce the error without this patch applied and can
confirm this patch fixes it.

v2: Fix the crosstest build.

 tests/shader_runner.c       |  2 +-
 tests/shader_runner.h       |  3 +++
 tests/shader_runner_d3d12.c | 24 ++++++++++++++++++------
 3 files changed, 22 insertions(+), 7 deletions(-)

diff --git a/tests/shader_runner.c b/tests/shader_runner.c
index 1808a581b..8cfdf052c 100644
--- a/tests/shader_runner.c
+++ b/tests/shader_runner.c
@@ -61,7 +61,7 @@ typedef int HRESULT;
 #include "vkd3d_test.h"
 #include "shader_runner.h"
 
-static void VKD3D_NORETURN VKD3D_PRINTF_FUNC(1, 2) fatal_error(const char *format, ...)
+void fatal_error(const char *format, ...)
 {
     va_list args;
 
diff --git a/tests/shader_runner.h b/tests/shader_runner.h
index 822a863f1..6958191d9 100644
--- a/tests/shader_runner.h
+++ b/tests/shader_runner.h
@@ -21,6 +21,7 @@
 #include "vkd3d_d3dcommon.h"
 #include "vkd3d_d3d12.h"
 #include "vkd3d_dxgiformat.h"
+#include "vkd3d_common.h"
 #include "utils.h"
 
 enum shader_model
@@ -89,6 +90,8 @@ struct shader_runner_ops
     void (*probe_vec4)(struct shader_context *context, const RECT *rect, const struct vec4 *v, unsigned int ulps);
 };
 
+void fatal_error(const char *format, ...) VKD3D_NORETURN VKD3D_PRINTF_FUNC(1, 2);
+
 void run_shader_tests(struct shader_context *context, int argc, char **argv, const struct shader_runner_ops *ops);
 
 #ifdef _WIN32
diff --git a/tests/shader_runner_d3d12.c b/tests/shader_runner_d3d12.c
index 00bb9e414..571efeacd 100644
--- a/tests/shader_runner_d3d12.c
+++ b/tests/shader_runner_d3d12.c
@@ -29,7 +29,6 @@ struct d3d12_texture
     struct texture t;
 
     D3D12_DESCRIPTOR_RANGE descriptor_range;
-    ID3D12DescriptorHeap *heap;
     ID3D12Resource *resource;
     unsigned int root_index;
 };
@@ -44,6 +43,8 @@ struct d3d12_shader_context
     struct shader_context c;
 
     struct test_context test_context;
+
+    ID3D12DescriptorHeap *heap;
 };
 
 static struct d3d12_shader_context *d3d12_shader_context(struct shader_context *c)
@@ -77,6 +78,8 @@ static ID3D10Blob *compile_shader(const char *source, enum shader_model shader_m
     return blob;
 }
 
+#define MAX_RESOURCE_DESCRIPTORS 256
+
 static struct texture *d3d12_runner_create_texture(struct shader_context *c, const struct texture_params *params)
 {
     struct d3d12_shader_context *context = d3d12_shader_context(c);
@@ -85,11 +88,17 @@ static struct texture *d3d12_runner_create_texture(struct shader_context *c, con
     D3D12_SUBRESOURCE_DATA resource_data;
     struct d3d12_texture *texture;
 
+    if (!context->heap)
+        context->heap = create_gpu_descriptor_heap(device,
+                D3D12_DESCRIPTOR_HEAP_TYPE_CBV_SRV_UAV, MAX_RESOURCE_DESCRIPTORS);
+
+    if (params->slot >= MAX_RESOURCE_DESCRIPTORS)
+        fatal_error("Resource slot %u is too high; please increase MAX_RESOURCE_DESCRIPTORS.\n", params->slot);
+
     texture = calloc(1, sizeof(*texture));
 
     texture->t.slot = params->slot;
 
-    texture->heap = create_gpu_descriptor_heap(device, D3D12_DESCRIPTOR_HEAP_TYPE_CBV_SRV_UAV, 1);
     texture->resource = create_default_texture(device, params->width, params->height,
             params->format, 0, D3D12_RESOURCE_STATE_COPY_DEST);
     resource_data.pData = params->data;
@@ -99,7 +108,7 @@ static struct texture *d3d12_runner_create_texture(struct shader_context *c, con
     transition_resource_state(test_context->list, texture->resource, D3D12_RESOURCE_STATE_COPY_DEST,
             D3D12_RESOURCE_STATE_NON_PIXEL_SHADER_RESOURCE | D3D12_RESOURCE_STATE_PIXEL_SHADER_RESOURCE);
     ID3D12Device_CreateShaderResourceView(device, texture->resource,
-            NULL, get_cpu_descriptor_handle(test_context, texture->heap, 0));
+            NULL, get_cpu_descriptor_handle(test_context, context->heap, params->slot));
 
     return &texture->t;
 }
@@ -108,7 +117,6 @@ static void d3d12_runner_destroy_texture(struct shader_context *c, struct textur
 {
     struct d3d12_texture *texture = d3d12_texture(t);
 
-    ID3D12DescriptorHeap_Release(texture->heap);
     ID3D12Resource_Release(texture->resource);
     free(texture);
 }
@@ -214,7 +222,7 @@ static void d3d12_runner_draw_quad(struct shader_context *c)
         struct d3d12_texture *texture = d3d12_texture(context->c.textures[i]);
 
         ID3D12GraphicsCommandList_SetGraphicsRootDescriptorTable(command_list, texture->root_index,
-                get_gpu_descriptor_handle(test_context, texture->heap, 0));
+                get_gpu_descriptor_handle(test_context, context->heap, texture->t.slot));
     }
 
     ID3D12GraphicsCommandList_OMSetRenderTargets(command_list, 1, &test_context->rtv, false, NULL);
@@ -266,12 +274,16 @@ void run_shader_tests_d3d12(int argc, char **argv)
         .no_root_signature = true,
         .rt_format = DXGI_FORMAT_R32G32B32A32_FLOAT,
     };
-    struct d3d12_shader_context context;
+    struct d3d12_shader_context context = {0};
 
     parse_args(argc, argv);
     enable_d3d12_debug_layer(argc, argv);
     init_adapter_info();
     init_test_context(&context.test_context, &desc);
+
     run_shader_tests(&context.c, argc, argv, &d3d12_runner_ops);
+
+    if (context.heap)
+        ID3D12DescriptorHeap_Release(context.heap);
     destroy_test_context(&context.test_context);
 }
-- 
2.35.1

[Henri Verbeet]

Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com>