Re: [Mesa-dev] [PATCH 2/2] vulkan/wsi/x11: correct DRI3 version in comment

2018-03-13 Thread Daniel Stone
Hi Emil,

On 13 March 2018 at 11:40, Emil Velikov  wrote:
> During development the version was bumped, yet the comment did not get
> an update.
>
> Cc: Daniel Stone 
> Fixes: c80c08e2260 ("vulkan/wsi/x11: Add support for DRI3 v1.2")
> Signed-off-by: Emil Velikov 

These are both:
Reviewed-by: Daniel Stone 

> Mildly related: the DRI3 (and DRI2) version query implementation is
> broken. In a sense that the server can return version greater than the
> client one.
>
> I can set a fix for that, although it might regress some broken (yet
> barely working) programs.
>
> From the DRI3 proto, with some emphasis from your truly:
>
> The client sends the highest supported version to the server
> and the server sends the highest version it supports, but _no_
> _higher_ than the _requested version_.

Given there's only been 1.0 until a week or two ago, it shouldn't
cause any issues, you'd hope. Seems like a reasonable patch to me.

Cheers,
Daniel
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev


[Mesa-dev] [PATCH 2/2] vulkan/wsi/x11: correct DRI3 version in comment

2018-03-13 Thread Emil Velikov
From: Emil Velikov 

During development the version was bumped, yet the comment did not get
an update.

Cc: Daniel Stone 
Fixes: c80c08e2260 ("vulkan/wsi/x11: Add support for DRI3 v1.2")
Signed-off-by: Emil Velikov 
---
Mildly related: the DRI3 (and DRI2) version query implementation is
broken. In a sense that the server can return version greater than the
client one.

I can set a fix for that, although it might regress some broken (yet
barely working) programs.

From the DRI3 proto, with some emphasis from your truly:

The client sends the highest supported version to the server
and the server sends the highest version it supports, but _no_
_higher_ than the _requested version_.
---
 src/vulkan/wsi/wsi_common_x11.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/vulkan/wsi/wsi_common_x11.c b/src/vulkan/wsi/wsi_common_x11.c
index e7a7da19285..0667aa1dbc9 100644
--- a/src/vulkan/wsi/wsi_common_x11.c
+++ b/src/vulkan/wsi/wsi_common_x11.c
@@ -1047,7 +1047,7 @@ x11_image_init(VkDevice device_h, struct x11_swapchain 
*chain,
image->pixmap = xcb_generate_id(chain->conn);
 
if (image->base.drm_modifier != DRM_FORMAT_MOD_INVALID) {
-  /* If the image has a modifier, we must have DRI3 v1.1. */
+  /* If the image has a modifier, we must have DRI3 v1.2. */
   assert(chain->has_dri3_modifiers);
 
   cookie =
-- 
2.16.0

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