<div dir="ltr"><div dir="ltr"><div dir="ltr">Hi All,<div><br></div><div>Can we get some feedback on this patch please?</div><div><br></div><div>At the moment, we can't compile vulkan using libxcb < 1.13 due to the use of the RandR lease mechanism. Dave Airlie (cc'd) did some work previously to allow things to be compiled with libxcb 1.11/1.12, mainly by ifdef'ing certain code with<span style="color:rgb(80,0,80)"> HAVE_DRI3_MODIFIERS so that it would compile, but without the extra features that xcb 1.13+ provides. See commit </span><font color="#500050">7aeef2d4efdc809a698e "dri3: allow building against older xcb (v3)"</font><span style="color:rgb(80,0,80)"> for that.</span></div><div><br></div><div>The patch (originally provided by Maxime) appears to solve the new build issue, and in much the same way as Dave Airlie did with his previous patch. </div><div><br></div><div>Note: If we "must have" xcb 1.13, then that's fine. But if that is the case, then we need to update mesa's build deps so that this is explicit (ie: effectively reverting Dave Airlie's commits on the issue).</div><div><br></div></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, 25 Sep 2018 at 02:04, Eric Engestrom <<a href="mailto:eric.engestrom@intel.com" target="_blank">eric.engestrom@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+Cc Keith, Jason & Daniel, who know this code best.<br>
<br>
On Monday, 2018-09-24 08:46:22 +1000, Stuart Young wrote:<br>
> From: Maxime <<a href="mailto:berillions@gmail.com" target="_blank">berillions@gmail.com</a>><br>
> <br>
> Since the Randr lease code was added, compiling against libxcb 1.12 no<br>
> longer works.<br>
> <br>
> CC: <a href="mailto:mesa-stable@lists.freedesktop.org" target="_blank">mesa-stable@lists.freedesktop.org</a><br>
> Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=108024" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=108024</a><br>
> Fixes: 7ab1fffcd2a504024b16e408de329f7a94553ecc<br>
<br>
Thanks for the `Fixes:` tag, it's really useful :)<br>
We usually also add the commit title as well, to help humans know what it's<br>
about without having to look it up (makes no difference to scripts though):<br>
Fixes: 7ab1fffcd2a504024b16 "vulkan: Add EXT_acquire_xlib_display [v5]"<br>
<br>
> Tested-By: Maxime <<a href="mailto:berillions@gmail.com" target="_blank">berillions@gmail.com</a>><br>
> <br>
> ---<br>
>  src/vulkan/wsi/wsi_common_display.c | 2 ++<br>
>  1 file changed, 2 insertions(+)<br>
> <br>
> diff --git a/src/vulkan/wsi/wsi_common_display.c b/src/vulkan/wsi/wsi_common_display.c<br>
> index e99b0465ec..e23d2983c9 100644<br>
> --- a/src/vulkan/wsi/wsi_common_display.c<br>
> +++ b/src/vulkan/wsi/wsi_common_display.c<br>
> @@ -2304,6 +2304,7 @@ wsi_acquire_xlib_display(VkPhysicalDevice physical_device,<br>
>     if (!crtc)<br>
>        return VK_ERROR_INITIALIZATION_FAILED;<br>
>  <br>
> +#ifdef HAVE_DRI3_MODIFIERS<br>
>     xcb_randr_lease_t lease = xcb_generate_id(connection);<br>
>     xcb_randr_create_lease_cookie_t cl_c =<br>
>        xcb_randr_create_lease(connection, root, lease, 1, 1,<br>
> @@ -2324,6 +2325,7 @@ wsi_acquire_xlib_display(VkPhysicalDevice physical_device,<br>
>        return VK_ERROR_INITIALIZATION_FAILED;<br>
>  <br>
>     wsi->fd = fd;<br>
> +#endif<br>
>  <br>
>     return VK_SUCCESS;<br>
>  }<br>
> -- <br>
> 2.11.0<br>
> <br>
> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="m_-5766202629444056255gmail_signature" data-smartmail="gmail_signature">Stuart Young (aka Cefiar)</div>