From: Imre Deak Date: Wed, 31 Aug 2016 16:13:07 +0000 (+0300) Subject: drm/i915/bdw: sseu: Fix sseu status parsing X-Git-Tag: v4.9.8~1110^2~27^2~91 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=795b38b3928d0ad5a6b2a530b30916cc76f4b667;p=platform%2Fkernel%2Flinux-rpi3.git drm/i915/bdw: sseu: Fix sseu status parsing Currently when checking for fused off EUs we may ignore the EU count in an enabled slice if there is any disabled slice preceding the enabled one (with a lower slice ID). Perhaps this can't happen in reality, but there is no reason to have this assumption built-in, the code is clearer without it. Reviewed-by: Robert Bragg Reviewed-by: Ben Widawsky Tested-by: Ben Widawsky Signed-off-by: Imre Deak Link: http://patchwork.freedesktop.org/patch/msgid/1472659987-10417-8-git-send-email-imre.deak@intel.com --- diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c index 413989a..7be8321 100644 --- a/drivers/gpu/drm/i915/i915_debugfs.c +++ b/drivers/gpu/drm/i915/i915_debugfs.c @@ -5095,7 +5095,7 @@ static void broadwell_sseu_device_status(struct drm_i915_private *dev_priv, sseu_subslice_total(sseu); /* subtract fused off EU(s) from enabled slice(s) */ - for (s = 0; s < hweight8(sseu->slice_mask); s++) { + for (s = 0; s < fls(sseu->slice_mask); s++) { u8 subslice_7eu = INTEL_INFO(dev_priv)->sseu.subslice_7eu[s];