drm/i915: Limit the number of loops for reading a split 64bit register
authorChris Wilson <chris@chris-wilson.co.uk>
Tue, 8 Sep 2015 13:17:13 +0000 (14:17 +0100)
committerJani Nikula <jani.nikula@intel.com>
Wed, 9 Sep 2015 08:08:10 +0000 (11:08 +0300)
commitacd29f7b22262d9e848393b9b6ae13eb42d22514
tree859d4579042980dd075e4b70cdf6c0caf6dbe764
parente85376cbd23299e4d40bace0cb5ab867b270fdbd
drm/i915: Limit the number of loops for reading a split 64bit register

In I915_READ64_2x32 we attempt to read a 64bit register using 2 32bit
reads. Due to the nature of the registers we try to read in this manner,
they may increment between the two instruction (e.g. a timestamp
counter). To keep the result accurate, we repeat the read if we detect
an overflow (i.e. the upper value varies). However, some hardware is just
plain flaky and may endless loop as the the upper 32bits are not stable.
Just give up after a couple of tries and report whatever we read last.

v2: Use the most recent values when erring out on an unstable register.

Reported-by: russianneuromancer@ya.ru
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91906
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: MichaƂ Winiarski <michal.winiarski@intel.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Cc: stable@vger.kernel.org
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
drivers/gpu/drm/i915/i915_drv.h