From d80837467488f6bef2e53b3d26a8d6fd4e46ea37 Mon Sep 17 00:00:00 2001 From: Paulo Zanoni Date: Fri, 24 Jan 2020 14:41:25 -0800 Subject: [PATCH] intel/device: bdw_gt1 actually has 6 eus per subslice 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 Reviewed-by: Lionel Landwerlin Reviewed-by: Kenneth Graunke Signed-off-by: Paulo Zanoni Part-of: (cherry picked from commit aa78801f0a6cfeaf3d16b4333239c0b862f73c10) --- .pick_status.json | 2 +- src/intel/dev/gen_device_info.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.pick_status.json b/.pick_status.json index 091b6165126..4e124ddb4dc 100644 --- a/.pick_status.json +++ b/.pick_status.json @@ -400,7 +400,7 @@ "description": "intel/device: bdw_gt1 actually has 6 eus per subslice", "nominated": true, "nomination_type": 0, - "resolution": 0, + "resolution": 1, "master_sha": null, "because_sha": null }, diff --git a/src/intel/dev/gen_device_info.c b/src/intel/dev/gen_device_info.c index 7a22476916b..ef444016e37 100644 --- a/src/intel/dev/gen_device_info.c +++ b/src/intel/dev/gen_device_info.c @@ -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 = {