intel/device: bdw_gt1 actually has 6 eus per subslice
authorPaulo Zanoni <paulo.r.zanoni@intel.com>
Fri, 24 Jan 2020 22:41:25 +0000 (14:41 -0800)
committerMarge Bot <eric+marge@anholt.net>
Tue, 3 Mar 2020 00:36:10 +0000 (00:36 +0000)
Found by inspection, I'm not aware of any bugs caused by this typo.

According to Lionel, it seems we only use this to generate masks
of available EUs for perfromance queries, and it's only used when we
can't query the fused parts of the GPU through DRM_IOCTL_I915_QUERY.
So this patch should help for the corner case where the Kernel is too
old to support the query ioctl.

v2: improve commit message, cc stable (Lionel).

Cc: mesa-stable@lists.freedesktop.org
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4006>

src/intel/dev/gen_device_info.c

index 7a22476916bf1b26b59131f5a5ae37ca1952804d..ef444016e3785d93f64bbfc01f5dc2c9c622804c 100644 (file)
@@ -434,7 +434,7 @@ static const struct gen_device_info gen_device_info_bdw_gt1 = {
    .is_broadwell = true,
    .num_slices = 1,
    .num_subslices = { 2, },
-   .num_eu_per_subslice = 8,
+   .num_eu_per_subslice = 6,
    .l3_banks = 2,
    .max_cs_threads = 42,
    .urb = {