drm/i915: Ensure wa_init_finish() is called for ctx workaround list
authorMatt Roper <matthew.d.roper@intel.com>
Thu, 26 Aug 2021 03:35:59 +0000 (20:35 -0700)
committerMatt Roper <matthew.d.roper@intel.com>
Fri, 27 Aug 2021 04:08:14 +0000 (21:08 -0700)
commitba3d8257f2d94ab227af880e3e40868c80ad8d93
treefc0f2bceca1f6f1dc88969b0e9f0e872cd2b70ee
parentf3ede209d44d71636890a78fa89c5b1c83340320
drm/i915: Ensure wa_init_finish() is called for ctx workaround list

A recent restructuring of our context workaround list initialization
added an early return for non-render engines; this caused us to
potentially miss the wa_init_finish() call at the end of the function.
The mistake is pretty harmless --- the only impact is that non-render
engines on graphics version 12.50+ platforms we don't trim down the
workaround list to reclaim some memory, and we don't print the usual
"Initialized 1 context workaround" message in dmesg.  Let's change the
early return to a jump down to the wa_init_finish() call at the bottom
of the function.

Reported-by: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Fixes: 9e9dfd080201 ("drm/i915/dg2: Maintain backward-compatible nested batch behavior")
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210826033559.1209020-1-matthew.d.roper@intel.com
drivers/gpu/drm/i915/gt/intel_workarounds.c