drm/i915: Don't taint when using fault injection
authorMichał Winiarski <michal.winiarski@intel.com>
Mon, 6 Jul 2020 14:41:07 +0000 (16:41 +0200)
committerChris Wilson <chris@chris-wilson.co.uk>
Mon, 6 Jul 2020 18:21:07 +0000 (19:21 +0100)
It is not really unexpected to hit wedge on init this way.
We're already downgrading error printk when running with fault injection,
let's use the same approach for CI tainting.

v2: Don't check fault inject in trace dump (Chris)

Signed-off-by: Michał Winiarski <michal.winiarski@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Petri Latvala <petri.latvala@intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Link: https://patchwork.freedesktop.org/patch/msgid/20200706144107.204821-3-michal@hardline.pl
drivers/gpu/drm/i915/i915_utils.c

index 01a3d3c..4c305d8 100644 (file)
@@ -53,7 +53,10 @@ void add_taint_for_CI(struct drm_i915_private *i915, unsigned int taint)
 {
        __i915_printk(i915, KERN_NOTICE, "CI tainted:%#x by %pS\n",
                      taint, (void *)_RET_IP_);
-       __add_taint_for_CI(taint);
+
+       /* Failures that occur during fault injection testing are expected */
+       if (!i915_error_injected())
+               __add_taint_for_CI(taint);
 }
 
 #if IS_ENABLED(CONFIG_DRM_I915_DEBUG)