Conor McCarthy : tests: Add root signature tests for descriptor register range collisions.
Alexandre Julliard
julliard at winehq.org
Wed Oct 6 15:51:13 CDT 2021
Module: vkd3d
Branch: master
Commit: 39ec44f345d39144a4f895f2dfb7f844e1530fad
URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=39ec44f345d39144a4f895f2dfb7f844e1530fad
Author: Conor McCarthy <cmccarthy at codeweavers.com>
Date: Wed Oct 6 18:43:26 2021 +0200
tests: Add root signature tests for descriptor register range collisions.
Signed-off-by: Conor McCarthy <cmccarthy at codeweavers.com>
Signed-off-by: Henri Verbeet <hverbeet at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
tests/d3d12.c | 50 ++++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 50 insertions(+)
diff --git a/tests/d3d12.c b/tests/d3d12.c
index 009f833..0b02e10 100644
--- a/tests/d3d12.c
+++ b/tests/d3d12.c
@@ -584,6 +584,21 @@ static bool is_shader_float64_supported(ID3D12Device *device)
return options.DoublePrecisionFloatShaderOps;
}
+static D3D12_RESOURCE_BINDING_TIER get_resource_binding_tier(ID3D12Device *device)
+{
+ D3D12_FEATURE_DATA_D3D12_OPTIONS options;
+ HRESULT hr;
+
+ if (FAILED(hr = ID3D12Device_CheckFeatureSupport(device,
+ D3D12_FEATURE_D3D12_OPTIONS, &options, sizeof(options))))
+ {
+ trace("Failed to check feature support, hr %#x.\n", hr);
+ return D3D12_RESOURCE_BINDING_TIER_1;
+ }
+
+ return options.ResourceBindingTier;
+}
+
#define create_cb_root_signature(a, b, c, e) create_cb_root_signature_(__LINE__, a, b, c, e)
static ID3D12RootSignature *create_cb_root_signature_(unsigned int line,
ID3D12Device *device, unsigned int reg_idx, D3D12_SHADER_VISIBILITY shader_visibility,
@@ -2590,6 +2605,7 @@ static void test_create_root_signature(void)
{
D3D12_ROOT_SIGNATURE_DESC root_signature_desc;
D3D12_DESCRIPTOR_RANGE descriptor_ranges[2];
+ D3D12_RESOURCE_BINDING_TIER binding_tier;
D3D12_ROOT_PARAMETER root_parameters[3];
ID3D12RootSignature *root_signature;
ID3D12Device *device, *tmp_device;
@@ -2602,6 +2618,11 @@ static void test_create_root_signature(void)
return;
}
+ /* Tier 2 is required for unbounded SRVs and Tier > 2 for unbounded CBVs and UAVs
+ * due to the need for partial binding support. It is also required for overlapping
+ * ranges of different types. */
+ binding_tier = get_resource_binding_tier(device);
+
/* descriptor table */
descriptor_ranges[0].RangeType = D3D12_DESCRIPTOR_RANGE_TYPE_SRV;
descriptor_ranges[0].NumDescriptors = 1;
@@ -2637,6 +2658,35 @@ static void test_create_root_signature(void)
refcount = ID3D12RootSignature_Release(root_signature);
ok(!refcount, "ID3D12RootSignature has %u references left.\n", (unsigned int)refcount);
+ /* Overlapping ranges but unique register indices. */
+ descriptor_ranges[0].NumDescriptors = 8;
+ descriptor_ranges[1].RangeType = D3D12_DESCRIPTOR_RANGE_TYPE_SRV;
+ descriptor_ranges[1].NumDescriptors = 2;
+ descriptor_ranges[1].BaseShaderRegister = 8;
+ descriptor_ranges[1].RegisterSpace = 0;
+ descriptor_ranges[1].OffsetInDescriptorsFromTableStart = 7;
+ root_parameters[0].DescriptorTable.NumDescriptorRanges = 2;
+ hr = create_root_signature(device, &root_signature_desc, &root_signature);
+ ok(hr == S_OK, "Failed to create root signature, hr %#x.\n", hr);
+ ID3D12RootSignature_Release(root_signature);
+
+ /* Separate ranges with ambiguous register indices. */
+ descriptor_ranges[1].BaseShaderRegister = 7;
+ descriptor_ranges[1].OffsetInDescriptorsFromTableStart = 8;
+ hr = create_root_signature(device, &root_signature_desc, &root_signature);
+ todo ok(hr == E_INVALIDARG, "Got unexpected hr %#x.\n", hr);
+ if (SUCCEEDED(hr))
+ ID3D12RootSignature_Release(root_signature);
+
+ /* Identical ranges and register indices but different type. */
+ descriptor_ranges[1] = descriptor_ranges[0];
+ descriptor_ranges[1].RangeType = D3D12_DESCRIPTOR_RANGE_TYPE_CBV;
+ hr = create_root_signature(device, &root_signature_desc, &root_signature);
+ ok(hr == S_OK || (binding_tier <= D3D12_RESOURCE_BINDING_TIER_2 && (hr == E_FAIL || hr == E_INVALIDARG)),
+ "Got unexpected hr %#x.\n", hr);
+ if (SUCCEEDED(hr))
+ ID3D12RootSignature_Release(root_signature);
+
/* sampler and SRV in the same descriptor table */
descriptor_ranges[1].RangeType = D3D12_DESCRIPTOR_RANGE_TYPE_SAMPLER;
descriptor_ranges[1].NumDescriptors = 1;
More information about the wine-cvs
mailing list