]> git.baikalelectronics.ru Git - kernel.git/commit
drm/i915: Prevent unbounded wm results in g4x_compute_wm()
authorChris Wilson <chris@chris-wilson.co.uk>
Tue, 7 Nov 2017 14:03:38 +0000 (14:03 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Tue, 7 Nov 2017 17:50:32 +0000 (17:50 +0000)
commit23dbfc89751a4a10f587631c58162b4c7e3bf0c8
tree492e14afda47c3da74b3be8c31cb0190eb13a52d
parent890e05377a23620babbc0dc7cf1fd0b6a853011d
drm/i915: Prevent unbounded wm results in g4x_compute_wm()

Smatch warns of

drivers/gpu/drm/i915/intel_pm.c:1161 g4x_compute_wm() warn: signedness bug returning '(-33554430)'

which is a result of it believing that wm may be INT_MAX following
g4x_tlb_miss_wa(). Just declaring g4x_tlb_miss_wa() as returning an
unsigned integer is not sufficient, we need to tell smatch that wm itself
is unsigned for it to not worry. So mark up the locals we expect to be
non-negative, and so silence smatch.

v2: Mark up vlv_compute_wm_level() as unsigned similarly.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Cc: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com> #v1
Link: https://patchwork.freedesktop.org/patch/msgid/20171107140338.13748-1-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/intel_pm.c