=?UTF-8?Q?J=C3=B3zef=20Kucia=20?=: vkd3d: Validate RTV format for inactive render targets in pipeline state desc.
Alexandre Julliard
julliard at winehq.org
Mon Dec 17 12:29:00 CST 2018
Module: vkd3d
Branch: master
Commit: 5bb045c8f914e3380bea10ba9eb82f0d3a1d2ff1
URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=5bb045c8f914e3380bea10ba9eb82f0d3a1d2ff1
Author: Józef Kucia <jkucia at codeweavers.com>
Date: Thu Dec 13 10:28:39 2018 +0100
vkd3d: Validate RTV format for inactive render targets in pipeline state desc.
Signed-off-by: Józef Kucia <jkucia at codeweavers.com>
Signed-off-by: Henri Verbeet <hverbeet at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
libs/vkd3d/state.c | 9 +++++++++
tests/d3d12.c | 7 +++++++
2 files changed, 16 insertions(+)
diff --git a/libs/vkd3d/state.c b/libs/vkd3d/state.c
index 9c2d3ac..b2f177c 100644
--- a/libs/vkd3d/state.c
+++ b/libs/vkd3d/state.c
@@ -1945,6 +1945,15 @@ static HRESULT d3d12_pipeline_state_init_graphics(struct d3d12_pipeline_state *s
memset(&input_signature, 0, sizeof(input_signature));
+ for (i = desc->NumRenderTargets; i < ARRAY_SIZE(desc->RTVFormats); ++i)
+ {
+ if (desc->RTVFormats[i] != DXGI_FORMAT_UNKNOWN)
+ {
+ WARN("Format must be set to DXGI_FORMAT_UNKNOWN for inactive render targets.\n");
+ return E_INVALIDARG;
+ }
+ }
+
if (!(root_signature = unsafe_impl_from_ID3D12RootSignature(desc->pRootSignature)))
{
WARN("Root signature is NULL.\n");
diff --git a/tests/d3d12.c b/tests/d3d12.c
index 420d751..b96ca43 100644
--- a/tests/d3d12.c
+++ b/tests/d3d12.c
@@ -2491,6 +2491,13 @@ static void test_create_graphics_pipeline_state(void)
ok(hr == S_OK, "Got unexpected hr %#x.\n", hr);
ID3D12PipelineState_Release(pipeline_state);
+ /* Inactive render targets formats must be set to DXGI_FORMAT_UNKNOWN. */
+ init_pipeline_state_desc(&pso_desc, root_signature, DXGI_FORMAT_R8G8B8A8_UNORM, NULL, NULL, NULL);
+ pso_desc.RTVFormats[1] = DXGI_FORMAT_R8G8B8A8_UNORM;
+ hr = ID3D12Device_CreateGraphicsPipelineState(device, &pso_desc,
+ &IID_ID3D12PipelineState, (void **)&pipeline_state);
+ ok(hr == E_INVALIDARG, "Got unexpected hr %#x.\n", hr);
+
refcount = ID3D12RootSignature_Release(root_signature);
ok(!refcount, "ID3D12RootSignature has %u references left.\n", (unsigned int)refcount);
refcount = ID3D12Device_Release(device);
More information about the wine-cvs
mailing list