drm/i915: drop unnecessary local variable to suppress build warning
authorJani Nikula <jani.nikula@intel.com>
Sun, 11 Aug 2013 09:44:02 +0000 (12:44 +0300)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 22 Aug 2013 11:31:38 +0000 (13:31 +0200)
Although I could not reproduce this (different compiler version,
perhaps), reportedly we get:

drivers/gpu/drm/i915/i915_irq.c:1943:27: warning: ‘score’ may be used
uninitialized in this function [-Wuninitialized]

Drop the 'score' variable altogether as it's not really needed.

Reported-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_irq.c

index 06659a72d0cf8d47d962233843648ad57c55ed6e..28d57477aa4202c0be661b9aedef1fe39eadcef8 100644 (file)
@@ -1905,8 +1905,6 @@ static void i915_hangcheck_elapsed(unsigned long data)
                                } else
                                        busy = false;
                        } else {
-                               int score;
-
                                /* We always increment the hangcheck score
                                 * if the ring is busy and still processing
                                 * the same request, so that no single request
@@ -1927,20 +1925,18 @@ static void i915_hangcheck_elapsed(unsigned long data)
 
                                switch (ring->hangcheck.action) {
                                case HANGCHECK_WAIT:
-                                       score = 0;
                                        break;
                                case HANGCHECK_ACTIVE:
-                                       score = BUSY;
+                                       ring->hangcheck.score += BUSY;
                                        break;
                                case HANGCHECK_KICK:
-                                       score = KICK;
+                                       ring->hangcheck.score += KICK;
                                        break;
                                case HANGCHECK_HUNG:
-                                       score = HUNG;
+                                       ring->hangcheck.score += HUNG;
                                        stuck[i] = true;
                                        break;
                                }
-                               ring->hangcheck.score += score;
                        }
                } else {
                        /* Gradually reduce the count so that we catch DoS