diff options
| author | Chris Wilson <chris@chris-wilson.co.uk> | 2020-06-19 16:19:38 +0100 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2020-06-19 20:00:18 +0100 |
| commit | 5a15550e56cc57bb5b953b0f6629c6805f4724fe (patch) | |
| tree | 39c55564f3ef480485fc528c07fe2d7e20bb0320 /drivers/gpu/drm/i915/gt/intel_rps.c | |
| parent | bf3c508375069c9e0b57d8453f99ec26e95b6548 (diff) | |
drm/i915/gt: Initialise rps timestamp
Smatch warns that we may iterate over an empty array of gt->engines[].
One hopes that this is impossible, but nevertheless we can simply
appease smatch by initialising the timestamp to zero before we starting
probing the busy-time from the engines.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200619151938.21740-1-chris@chris-wilson.co.uk
Diffstat (limited to 'drivers/gpu/drm/i915/gt/intel_rps.c')
| -rw-r--r-- | drivers/gpu/drm/i915/gt/intel_rps.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/gpu/drm/i915/gt/intel_rps.c b/drivers/gpu/drm/i915/gt/intel_rps.c index bdece932592b..296391deeb94 100644 --- a/drivers/gpu/drm/i915/gt/intel_rps.c +++ b/drivers/gpu/drm/i915/gt/intel_rps.c @@ -51,10 +51,11 @@ static void rps_timer(struct timer_list *t) { struct intel_rps *rps = from_timer(rps, t, timer); struct intel_engine_cs *engine; + ktime_t dt, last, timestamp; enum intel_engine_id id; s64 max_busy[3] = {}; - ktime_t dt, timestamp, last; + timestamp = 0; for_each_engine(engine, rps_to_gt(rps), id) { s64 busy; int i; @@ -69,7 +70,6 @@ static void rps_timer(struct timer_list *t) swap(busy, max_busy[i]); } } - last = rps->pm_timestamp; rps->pm_timestamp = timestamp; |