[Mesa-dev] [PATCH] anv: use devinfo for number of thread/eu

[Manolova, Plamena]

Reviewed-by: Plamena Manolova <plamena.manolova at intel.com>

On Wed, Jun 28, 2017 at 3:24 PM, Lionel Landwerlin <
lionel.g.landwerlin at intel.com> wrote:

> It turns out Gen9LP has fewer threads per EU (6 vs 7).
>
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
> ---
>  src/intel/vulkan/anv_device.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c
> index 504a757571f..3f761226484 100644
> --- a/src/intel/vulkan/anv_device.c
> +++ b/src/intel/vulkan/anv_device.c
> @@ -359,8 +359,9 @@ anv_physical_device_init(struct anv_physical_device
> *device,
>
>     if (device->info.is_cherryview &&
>         device->subslice_total > 0 && device->eu_total > 0) {
> -      /* Logical CS threads = EUs per subslice * 7 threads per EU */
> -      uint32_t max_cs_threads = device->eu_total / device->subslice_total
> * 7;
> +      /* Logical CS threads = EUs per subslice * num threads per EU */
> +      uint32_t max_cs_threads =
> +         device->eu_total / device->subslice_total *
> device->info.num_thread_per_eu;
>
>        /* Fuse configurations may give more threads than expected, never
> less. */
>        if (max_cs_threads > device->info.max_cs_threads)
> --
> 2.13.2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170628/626d7793/attachment.html>