[Mesa-dev] [PATCH 06/16] anv: Predicate 48bit support on gen >= 8

[Nanley Chery]

On Thu, May 18, 2017 at 02:00:53PM -0700, Jason Ekstrand wrote:
> This doesn't matter right now since it only affects whether or not we
> set the kernel bit but, if we ever do anything else based on it, we'll
> want it to be correct per-gen.
> ---
>  src/intel/vulkan/anv_device.c | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)

This patch is
Reviewed-by: Nanley Chery <nanley.g.chery at intel.com>

> 
> diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c
> index 50486b6..b80715f 100644
> --- a/src/intel/vulkan/anv_device.c
> +++ b/src/intel/vulkan/anv_device.c
> @@ -225,7 +225,12 @@ anv_physical_device_init(struct anv_physical_device *device,
>        goto fail;
>     }
>  
> -   device->supports_48bit_addresses = anv_gem_supports_48b_addresses(fd);
> +   /* The kernel query only tells us whether or not the kernel supports the
> +    * EXEC_OBJECT_SUPPORTS_48B_ADDRESS flag and not whether or not the
> +    * hardware has actual 48bit address support.
> +    */
> +   device->supports_48bit_addresses =
> +      (device->info.gen >= 8) && anv_gem_supports_48b_addresses(fd);
>  
>     result = anv_compute_heap_size(fd, &device->heap_size);
>     if (result != VK_SUCCESS)
> -- 
> 2.5.0.400.gff86faf
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev