| commit | 56f6e0a7e7b09adb553339f9075696e918b96587 | [log] [tgz] |
|---|---|---|
| author | Chris Wilson <chris@chris-wilson.co.uk> | Thu Jan 05 15:30:20 2017 +0000 |
| committer | Chris Wilson <chris@chris-wilson.co.uk> | Thu Jan 05 15:34:41 2017 +0000 |
| tree | e3f05bda48b83163900272c097d9d6cd24704164 | |
| parent | d51dafaf07bf78ff0db4f2c238287dd9ccf42c08 [diff] [blame] |
drm/i915: Assert that we do create the deferred context In order to convince static analyzers that the allocation function returns an error or sets ce->state, assert that it is set afterwards. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Link: http://patchwork.freedesktop.org/patch/msgid/20170105153023.30575-2-chris@chris-wilson.co.uk
diff --git a/drivers/gpu/drm/i915/intel_lrc.c b/drivers/gpu/drm/i915/intel_lrc.c index 2279788..51ecb39 100644 --- a/drivers/gpu/drm/i915/intel_lrc.c +++ b/drivers/gpu/drm/i915/intel_lrc.c
@@ -781,6 +781,7 @@ static int execlists_context_pin(struct intel_engine_cs *engine, if (ret) goto err; } + GEM_BUG_ON(!ce->state); flags = PIN_GLOBAL; if (ctx->ggtt_offset_bias)