Re: [PATCH 6/6] dxgi/tests: Add test for frame latency events using D3D12.

[Philip Rebohle]

Actually this started failing on Windows 10 now for some reason, I'll 
investigate.

Am 15.04.20 um 17:23 schrieb Philip Rebohle:
> Signed-off-by: Philip Rebohle
>
> dlls/dxgi/tests/dxgi.c | 122 +++++++++++++++++++++++++++++++++++++++++
>   1 file changed, 122 insertions(+)
>
> diff --git a/dlls/dxgi/tests/dxgi.c b/dlls/dxgi/tests/dxgi.c
> index 3887982634..b016c35938 100644
> --- a/dlls/dxgi/tests/dxgi.c
> +++ b/dlls/dxgi/tests/dxgi.c
> @@ -6188,6 +6188,127 @@ static void test_factory_check_feature_support(void)
>       ok(!ref_count, "Factory has %u references left.\n", ref_count);
>   }
>
> +static void test_frame_latency_event(IUnknown *device, BOOL is_d3d12)
> +{
> - DXGI_SWAP_CHAIN_DESC1 swapchain_desc;
> - IDXGISwapChain2 *swapchain2;
> - IDXGISwapChain1 *swapchain1;
> - IDXGIFactory2 *factory2;
> - IDXGIFactory *factory;
> - UINT frame_latency;
> - DWORD wait_result;
> - ULONG ref_count;
> - unsigned int i;
> - HANDLE event;
> - HWND window;
> - HRESULT hr;
> -
> - get_factory(device, is_d3d12, &factory);
> -
> - hr = IDXGIFactory_QueryInterface(factory, &IID_IDXGIFactory2, (void**)&factory2);
> - IDXGIFactory_Release(factory);
> -
> - if (FAILED(hr))
> - {
> -
>    win_skip("IDXGIFactory2 not available.\n");
> -
>    return;
> - }
> -
> - window = CreateWindowA("static", "dxgi_test", 0, 640, 480, 0, 0, 0, 0, 0, 0);
> -
> - swapchain_desc.Width = 640;
> - swapchain_desc.Height = 480;
> - swapchain_desc.Format = DXGI_FORMAT_R8G8B8A8_UNORM;
> - swapchain_desc.Stereo = FALSE;
> - swapchain_desc.SampleDesc.Count = 1;
> - swapchain_desc.SampleDesc.Quality = 0;
> - swapchain_desc.BufferUsage = DXGI_USAGE_RENDER_TARGET_OUTPUT;
> - swapchain_desc.BufferCount = 2;
> - swapchain_desc.Scaling = DXGI_SCALING_STRETCH;
> - swapchain_desc.SwapEffect = is_d3d12 ? DXGI_SWAP_EFFECT_FLIP_DISCARD : DXGI_SWAP_EFFECT_DISCARD;
> - swapchain_desc.AlphaMode = DXGI_ALPHA_MODE_UNSPECIFIED;
> - swapchain_desc.Flags = 0;
> -
> - hr = IDXGIFactory2_CreateSwapChainForHwnd(factory2, device,
> -
>        window, &swapchain_desc, NULL, NULL, &swapchain1);
> - ok(hr == S_OK, "Failed to create swap chain, hr %#x.\n", hr);
> -
> - hr = IDXGISwapChain1_QueryInterface(swapchain1, &IID_IDXGISwapChain2, (void**)&swapchain2);
> - IDXGISwapChain1_Release(swapchain1);
> -
> - if (FAILED(hr))
> - {
> -
>    win_skip("IDXGISwapChain2 not available.\n");
> -
>    IDXGIFactory2_Release(factory2);
> -
>    return;
> - }
> -
> - /* test swap chain without waitable object */
> - frame_latency = 0xdeadbeef;
> - hr = IDXGISwapChain2_GetMaximumFrameLatency(swapchain2, &frame_latency);
> - ok(hr == DXGI_ERROR_INVALID_CALL, "Got unexpected hr %#x.\n", hr);
> - ok(frame_latency == 0xdeadbeef, "Got unexpected frame latency %#x.\n", frame_latency);
> - hr = IDXGISwapChain2_SetMaximumFrameLatency(swapchain2, 1);
> - ok(hr == DXGI_ERROR_INVALID_CALL, "Got unexpected hr %#x.\n", hr);
> - event = IDXGISwapChain2_GetFrameLatencyWaitableObject(swapchain2);
> - ok(!event, "Got unexpected event %p.\n", event);
> -
> - ref_count = IDXGISwapChain2_Release(swapchain2);
> - ok(!ref_count, "Swap chain has %u references left.\n", ref_count);
> -
> - /* test swap chain with waitable object */
> - swapchain_desc.Flags = DXGI_SWAP_CHAIN_FLAG_FRAME_LATENCY_WAITABLE_OBJECT;
> -
> - hr = IDXGIFactory2_CreateSwapChainForHwnd(factory2, device,
> -
>        window, &swapchain_desc, NULL, NULL, &swapchain1);
> - ok(hr == S_OK, "Failed to create swap chain, hr %#x.\n", hr);
> - hr = IDXGISwapChain1_QueryInterface(swapchain1, &IID_IDXGISwapChain2, (void**)&swapchain2);
> - ok(hr == S_OK, "Got unexpected hr %#x.\n", hr);
> - IDXGISwapChain1_Release(swapchain1);
> -
> - event = IDXGISwapChain2_GetFrameLatencyWaitableObject(swapchain2);
> - ok(!!event, "Got unexpected event %p.\n", event);
> -
> - /* manual-reset event */
> - wait_result = WaitForSingleObject(event, 0);
> - ok(!wait_result, "Got unexpected wait result %#x.\n", wait_result);
> - wait_result = WaitForSingleObject(event, 0);
> - ok(!wait_result, "Got unexpected wait result %#x.\n", wait_result);
> -
> - hr = IDXGISwapChain2_GetMaximumFrameLatency(swapchain2, &frame_latency);
> - ok(hr == S_OK, "Got unexpected hr %#x.\n", hr);
> - ok(frame_latency == 1, "Got unexpected frame latency %#x.\n", frame_latency);
> -
> - hr = IDXGISwapChain2_SetMaximumFrameLatency(swapchain2, 0);
> - ok(hr == DXGI_ERROR_INVALID_CALL, "Got unexpected hr %#x.\n", hr);
> - hr = IDXGISwapChain2_GetMaximumFrameLatency(swapchain2, &frame_latency);
> - ok(hr == S_OK, "Got unexpected hr %#x.\n", hr);
> - ok(frame_latency == 1, "Got unexpected frame latency %#x.\n", frame_latency);
> -
> - hr = IDXGISwapChain2_SetMaximumFrameLatency(swapchain2, 2);
> - ok(hr == S_OK, "Got unexpected hr %#x.\n", hr);
> - hr = IDXGISwapChain2_GetMaximumFrameLatency(swapchain2, &frame_latency);
> - ok(hr == S_OK, "Got unexpected hr %#x.\n", hr);
> - ok(frame_latency == 2, "Got unexpected frame latency %#x.\n", frame_latency);
> -
> - for (i = 0; i < 5; i++)
> - {
> -
>    hr = IDXGISwapChain2_Present(swapchain2, 0, 0);
> -
>    ok(hr == S_OK, "Present failed with hr %#x.\n", hr);
> - }
> -
> - wait_result = WaitForSingleObject(event, 1000);
> - ok(!wait_result, "Got unexpected wait result %#x.\n", wait_result);
> -
> - ref_count = IDXGISwapChain2_Release(swapchain2);
> - ok(!ref_count, "Swap chain has %u references left.\n", ref_count);
> -
> - DestroyWindow(window);
> -
> - ref_count = IDXGIFactory2_Release(factory2);
> - ok(ref_count == !is_d3d12, "Factory has %u references left.\n", ref_count);
>
> +}
> - static void run_on_d3d10(void (*test_func)(IUnknown *device, BOOL is_d3d12))
> {
>     IDXGIDevice *device;
>
> @@ -6325,6 +6446,7 @@ START_TEST(dxgi)
>       run_on_d3d12(test_swapchain_formats);
>       run_on_d3d12(test_output_ownership);
>       run_on_d3d12(test_cursor_clipping);
> -
>
> run_on_d3d12(test_frame_latency_event);
>
> FreeLibrary(d3d12_module);
> }