<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Mar 30, 2017 at 11:27 AM, Chad Versace <span dir="ltr"><<a href="mailto:chadversary@chromium.org" target="_blank">chadversary@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">On Mon 13 Mar 2017, Jason Ekstrand wrote:<br>
> There's really nothing for us to do here.  So long as the user doesn't<br>
> set any crazy environment variables such as INTEL_VK_HIZ=false, all of<br>
> the compression formats etc. should "just work" at least for opaque<br>
> handle types.<br>
<br>
</span>I think the commit message should go with the opaque fd commit. This<br>
patch's commit message should say something like,<br>
<br>
  Turn it on. Trivially correct. Don't support any VkExternalMemoryHandleTypes yet.<br></blockquote><div><br></div><div>Good call.  I wrote:<br><br>    This is the trivial implementation that just exposes the extension<br>    string but exposes zero external handle types.<br><br></div><div>I moved the other comment to the external_memory_fd commit.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
but in real sentences ;)<br>
<div><div class="gmail-h5"><br>
> ---<br>
>  src/intel/vulkan/anv_device.c           | 6 +++++-<br>
>  src/intel/vulkan/anv_<wbr>entrypoints_gen.py | 1 +<br>
>  2 files changed, 6 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/src/intel/vulkan/anv_device.<wbr>c b/src/intel/vulkan/anv_device.<wbr>c<br>
> index f92a313..385a806 100644<br>
> --- a/src/intel/vulkan/anv_device.<wbr>c<br>
> +++ b/src/intel/vulkan/anv_device.<wbr>c<br>
> @@ -314,7 +314,11 @@ static const VkExtensionProperties device_extensions[] = {<br>
>     {<br>
>        .extensionName = VK_KHR_DESCRIPTOR_UPDATE_<wbr>TEMPLATE_EXTENSION_NAME,<br>
>        .specVersion = 1,<br>
> -   }<br>
> +   },<br>
> +   {<br>
> +      .extensionName = VK_KHX_EXTERNAL_MEMORY_<wbr>EXTENSION_NAME,<br>
> +      .specVersion = 1,<br>
> +   },<br>
>  };<br>
><br>
>  static void *<br>
> diff --git a/src/intel/vulkan/anv_<wbr>entrypoints_gen.py b/src/intel/vulkan/anv_<wbr>entrypoints_gen.py<br>
> index 2c084ae..e8cdfb7 100644<br>
> --- a/src/intel/vulkan/anv_<wbr>entrypoints_gen.py<br>
> +++ b/src/intel/vulkan/anv_<wbr>entrypoints_gen.py<br>
> @@ -39,6 +39,7 @@ supported_extensions = [<br>
>     'VK_KHR_wayland_surface',<br>
>     'VK_KHR_xcb_surface',<br>
>     'VK_KHR_xlib_surface',<br>
> +   'VK_KHX_external_memory',<br>
>     'VK_KHX_external_memory_<wbr>capabilities',<br>
>  ]<br>
><br>
> --<br>
> 2.5.0.400.gff86faf<br>
><br>
</div></div>> ______________________________<wbr>_________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">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/<wbr>mailman/listinfo/mesa-dev</a><br>
</blockquote></div><br></div></div>