[Mesa-dev] [PATCH] vulkan/wsi: clarify the severity of lack of DRI3

[Jason Ekstrand]

Sure

Acked-by: Jason ekstrand <jason at jlekstrand.net>

On Wed, Jan 18, 2017 at 2:49 PM, Andres Rodriguez <andresx7 at gmail.com>
wrote:

> The current message sounds like a small warning, clarify that it can
> result in lack of presentation support and application crashes.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98263
> Signed-off-by: Andres Rodriguez <andresx7 at gmail.com>
> ---
>  src/vulkan/wsi/wsi_common_x11.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/src/vulkan/wsi/wsi_common_x11.c b/src/vulkan/wsi/wsi_common_
> x11.c
> index 158446c..7479f74 100644
> --- a/src/vulkan/wsi/wsi_common_x11.c
> +++ b/src/vulkan/wsi/wsi_common_x11.c
> @@ -265,7 +265,8 @@ VkBool32 wsi_get_physical_device_xcb_
> presentation_support(
>        return false;
>
>     if (!wsi_conn->has_dri3) {
> -      fprintf(stderr, "vulkan: No DRI3 support\n");
> +      fprintf(stderr, "vulkan: No DRI3 support detected - required for
> presentation\n");
> +      fprintf(stderr, "Note: Buggy applications may crash, please report
> to vendor\n");
>        return false;
>     }
>
> @@ -313,7 +314,8 @@ x11_surface_get_support(VkIcdSurfaceBase *icd_surface,
>        return VK_ERROR_OUT_OF_HOST_MEMORY;
>
>     if (!wsi_conn->has_dri3) {
> -      fprintf(stderr, "vulkan: No DRI3 support\n");
> +      fprintf(stderr, "vulkan: No DRI3 support detected - required for
> presentation\n");
> +      fprintf(stderr, "Note: Buggy applications may crash, please report
> to vendor\n");
>        *pSupported = false;
>        return VK_SUCCESS;
>     }
> --
> 2.9.3
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170118/9a50f092/attachment.html>