[Mesa-dev] [PATCH 2/2] radv: allow to force enable vsync with RADV_VSYNC=1

[Jason Ekstrand]

Why not just do this in a way that common to the two drivers? Forcing FIFO 
and MAILBOX both seem useful. For that matter, why not just 
MESA_VK_WSI_PRESENT_MODE=FIFO/mailbox/immediate?

On April 8, 2019 13:39:20 Samuel Pitoiset <samuel.pitoiset at gmail.com> wrote:

> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107391
> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
> ---
> src/amd/vulkan/radv_wsi.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/src/amd/vulkan/radv_wsi.c b/src/amd/vulkan/radv_wsi.c
> index 2fd07447668..7651043b2b2 100644
> --- a/src/amd/vulkan/radv_wsi.c
> +++ b/src/amd/vulkan/radv_wsi.c
> @@ -38,11 +38,13 @@ radv_wsi_proc_addr(VkPhysicalDevice physicalDevice, 
> const char *pName)
> VkResult
> radv_init_wsi(struct radv_physical_device *physical_device)
> {
> +	bool vsync = !!env_var_as_unsigned("RADV_VSYNC", 0);
> +
> 	return wsi_device_init(&physical_device->wsi_device,
> 			       radv_physical_device_to_handle(physical_device),
> 			       radv_wsi_proc_addr,
> 			       &physical_device->instance->alloc,
> -			       physical_device->master_fd, false);
> +			       physical_device->master_fd, vsync);
> }
>
> void
> --
> 2.21.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev