Commit 7a73e671 authored by Chris Wilson's avatar Chris Wilson

drm/i915: Taint the kernel on dumping the GEM ftrace buffer

As the ftrace buffer is single shot, once dumped it will not update. As
such, it only provides information for the first bug and all subsequent
bugs are noise. The goal of CI is to have zero bugs, so taint the kernel
causing CI to reboot the machine; fix the bug and move on.
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Reviewed-by: default avatarJoonas Lahtinen <joonas.lahtinen@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20191110185806.17413-13-chris@chris-wilson.co.uk
parent 1af65515
...@@ -68,9 +68,10 @@ struct drm_i915_private; ...@@ -68,9 +68,10 @@ struct drm_i915_private;
pr_err(__VA_ARGS__); \ pr_err(__VA_ARGS__); \
trace_printk(__VA_ARGS__); \ trace_printk(__VA_ARGS__); \
} while (0) } while (0)
#define GEM_TRACE_DUMP() ftrace_dump(DUMP_ALL) #define GEM_TRACE_DUMP() \
do { ftrace_dump(DUMP_ALL); add_taint_for_CI(TAINT_WARN); } while (0)
#define GEM_TRACE_DUMP_ON(expr) \ #define GEM_TRACE_DUMP_ON(expr) \
do { if (expr) ftrace_dump(DUMP_ALL); } while (0) do { if (expr) GEM_TRACE_DUMP(); } while (0)
#else #else
#define GEM_TRACE(...) do { } while (0) #define GEM_TRACE(...) do { } while (0)
#define GEM_TRACE_ERR(...) do { } while (0) #define GEM_TRACE_ERR(...) do { } while (0)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment