[PATCH vkd3d 1/5] vkd3d: Return S_OK from ID3D12CommandQueue::GetClockCalibration().

Henri Verbeet hverbeet at gmail.com
Thu Jan 13 10:18:32 CST 2022


On Mon, 10 Jan 2022 at 15:02, Conor McCarthy <cmccarthy at codeweavers.com> wrote:
> @@ -6225,10 +6225,14 @@ static HRESULT STDMETHODCALLTYPE d3d12_command_queue_GetTimestampFrequency(ID3D1
>  static HRESULT STDMETHODCALLTYPE d3d12_command_queue_GetClockCalibration(ID3D12CommandQueue *iface,
>          UINT64 *gpu_timestamp, UINT64 *cpu_timestamp)
>  {
> -    FIXME("iface %p, gpu_timestamp %p, cpu_timestamp %p stub!\n",
> +    TRACE("iface %p, gpu_timestamp %p, cpu_timestamp %p.\n",
>              iface, gpu_timestamp, cpu_timestamp);
>
> -    return E_NOTIMPL;
> +    WARN("Setting timestamps to zero.\n");
> +    *gpu_timestamp = 0;
> +    *cpu_timestamp = 0;
> +
> +    return S_OK;
>  }
>
Always returning 0 here (without FIXME) seems like something that may
end up being problematic in hard to diagnose ways at some point.
Couldn't we use VK_EXT_calibrated_timestamps here, or failing that,
build something on top of vkCmdWriteTimestamp()?



More information about the wine-devel mailing list