From c9a22bcb93db27e200a184c1f7b9bf0f288815c7 Mon Sep 17 00:00:00 2001 From: Hao Li Date: Fri, 8 Jan 2021 22:56:05 +0000 Subject: [PATCH] Disable timestamp query extension on Metal Currently it's not clear how to convert timestamps to nanoseconds on Metal, disable timestamp query extension on Metal until we know how to do that. WebGPU issue: https://github.com/gpuweb/gpuweb/issues/1325 Bug: dawn:434 Change-Id: If63bd5fe9dccc1fff16a42cddf63e9d8265e65c5 Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/37020 Commit-Queue: Austin Eng Reviewed-by: Corentin Wallez Reviewed-by: Austin Eng --- src/dawn_native/metal/BackendMTL.mm | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/dawn_native/metal/BackendMTL.mm b/src/dawn_native/metal/BackendMTL.mm index ada2c6d681..0637514119 100644 --- a/src/dawn_native/metal/BackendMTL.mm +++ b/src/dawn_native/metal/BackendMTL.mm @@ -221,7 +221,10 @@ namespace dawn_native { namespace metal { if ([*mDevice supportsFamily:MTLGPUFamilyMac2] || [*mDevice supportsFamily:MTLGPUFamilyApple5]) { mSupportedExtensions.EnableExtension(Extension::PipelineStatisticsQuery); - mSupportedExtensions.EnableExtension(Extension::TimestampQuery); + + // TODO(hao.x.li@intel.com): Not enable timestamp query here becuase it's not + // clear how to convert timestamps to nanoseconds on Metal. + // See https://github.com/gpuweb/gpuweb/issues/1325 } }