On 19 February 2018 at 10:54, Iago Toral Quiroga <ito...@igalia.com> wrote:
> The Vulkan working group has recently clarified that this is the intended
> behavior and accepted spec language changes to make this clear.
> ---
>  src/intel/vulkan/anv_entrypoints_gen.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/intel/vulkan/anv_entrypoints_gen.py 
> b/src/intel/vulkan/anv_entrypoints_gen.py
> index c5a654f19b2..34ffedb1165 100644
> --- a/src/intel/vulkan/anv_entrypoints_gen.py
> +++ b/src/intel/vulkan/anv_entrypoints_gen.py
> @@ -225,7 +225,7 @@ anv_entrypoint_is_enabled(int index, uint32_t 
> core_version,
>        return ${e.core_version.c_vk_version()} <= core_version;
>     % elif e.extension:
>        % if e.extension.type == 'instance':
> -      return instance->${e.extension.name[3:]};
> +      return !device && instance->${e.extension.name[3:]};


Out of curiosity - is this documented in the loader <> ICD interface spec?
Haven't looked recently: perhaps we should bump the version advertised?

Might also be applicable for the (return NULL) fixes from Jason.

HTH
Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to