<div dir="ltr">This should go into 18.2 which means also back-porting the header update which shouldn't be a big deal.  I have a feeling DXVK will add a hard requirement on v3 fairly shortly so it would be good if it works in released mesa.<br></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Sep 10, 2018 at 1:44 PM Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Mon, Sep 10, 2018 at 1:43 PM Bas Nieuwenhuizen <<a href="mailto:bas@basnieuwenhuizen.nl" target="_blank">bas@basnieuwenhuizen.nl</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I tried, but I can't find from your driver what the HW does, so for<br>
this review the same assumption that the CTS tests pass for you.<br></blockquote><div><br></div><div>Our hardware does the DX thing which is also what the spec says.  Yes, we pass the tests.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Reviewed-by: Bas Nieuwenhuizen <<a href="mailto:bas@basnieuwenhuizen.nl" target="_blank">bas@basnieuwenhuizen.nl</a>><br></blockquote><div><br></div><div>Thanks!<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Mon, Sep 10, 2018 at 7:08 PM Jason Ekstrand <<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>> wrote:<br>
><br>
> Cc: Bas Nieuwenhuizen <<a href="mailto:bas@basnieuwenhuizen.nl" target="_blank">bas@basnieuwenhuizen.nl</a>><br>
> ---<br>
>  src/intel/vulkan/anv_device.c      | 8 ++++++++<br>
>  src/intel/vulkan/anv_extensions.py | 2 +-<br>
>  2 files changed, 9 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c<br>
> index 7ab8543300b..44855dae128 100644<br>
> --- a/src/intel/vulkan/anv_device.c<br>
> +++ b/src/intel/vulkan/anv_device.c<br>
> @@ -934,6 +934,14 @@ void anv_GetPhysicalDeviceFeatures2(<br>
>           break;<br>
>        }<br>
><br>
> +      case VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_VERTEX_ATTRIBUTE_DIVISOR_FEATURES_EXT: {<br>
> +         VkPhysicalDeviceVertexAttributeDivisorFeaturesEXT *features =<br>
> +            (VkPhysicalDeviceVertexAttributeDivisorFeaturesEXT *)ext;<br>
> +         features->vertexAttributeInstanceRateDivisor = VK_TRUE;<br>
> +         features->vertexAttributeInstanceRateZeroDivisor = VK_TRUE;<br>
> +         break;<br>
> +      }<br>
> +<br>
>        default:<br>
>           anv_debug_ignored_stype(ext->sType);<br>
>           break;<br>
> diff --git a/src/intel/vulkan/anv_extensions.py b/src/intel/vulkan/anv_extensions.py<br>
> index a21aee5a001..951505a854e 100644<br>
> --- a/src/intel/vulkan/anv_extensions.py<br>
> +++ b/src/intel/vulkan/anv_extensions.py<br>
> @@ -122,7 +122,7 @@ EXTENSIONS = [<br>
>                'device->has_context_priority'),<br>
>      Extension('VK_EXT_shader_viewport_index_layer',       1, True),<br>
>      Extension('VK_EXT_shader_stencil_export',             1, 'device->info.gen >= 9'),<br>
> -    Extension('VK_EXT_vertex_attribute_divisor',          2, True),<br>
> +    Extension('VK_EXT_vertex_attribute_divisor',          3, True),<br>
>      Extension('VK_EXT_post_depth_coverage',               1, 'device->info.gen >= 9'),<br>
>      Extension('VK_EXT_sampler_filter_minmax',             1, 'device->info.gen >= 9'),<br>
>  ]<br>
> --<br>
> 2.17.1<br>
><br>
</blockquote></div></div>
</blockquote></div>