On Thursday, October 6, 2016 1:48:04 AM PDT Philipp Zabel wrote:
> According to chapters 16.5. (Timestamp Queries) and 30.2 (Limits) of the
> Vulkan Specification 1.0.29, the .limits.timestampPeriod field returned
> by vkGetPhysicalDeviceProperties is measured in nanoseconds, not in
> seconds.
> 
> Signed-off-by: Philipp Zabel <philipp.za...@gmail.com>
> ---
>  src/intel/vulkan/anv_device.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c
> index f786ebe..c7b9979 100644
> --- a/src/intel/vulkan/anv_device.c
> +++ b/src/intel/vulkan/anv_device.c
> @@ -572,7 +572,7 @@ void anv_GetPhysicalDeviceProperties(
>        .storageImageSampleCounts                 = VK_SAMPLE_COUNT_1_BIT,
>        .maxSampleMaskWords                       = 1,
>        .timestampComputeAndGraphics              = false,
> -      .timestampPeriod                          = time_stamp_base / (1000 * 
> 1000 * 1000),
> +      .timestampPeriod                          = time_stamp_base,
>        .maxClipDistances                         = 0 /* FIXME */,
>        .maxCullDistances                         = 0 /* FIXME */,
>        .maxCombinedClipAndCullDistances          = 0 /* FIXME */,
> 

Yep, definitely supposed to be nanoseconds.  Thanks for the fix!

Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

and pushed:

To ssh://git.freedesktop.org/git/mesa/mesa
   88428fb..0408d50  master -> master

(hope you don't mind, Jason!)

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to