[Intel-gfx] [PATCH] drm/i915: Taint the kernel on dumping the GEM ftrace buffer
kbuild test robot
lkp at intel.com
Sat Nov 9 14:33:31 UTC 2019
Hi Chris,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on drm-intel/for-linux-next]
[also build test ERROR on next-20191108]
[cannot apply to v5.4-rc6]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
url: https://github.com/0day-ci/linux/commits/Chris-Wilson/drm-i915-Taint-the-kernel-on-dumping-the-GEM-ftrace-buffer/20191109-171107
base: git://anongit.freedesktop.org/drm-intel for-linux-next
config: i386-randconfig-b003-201944 (attached as .config)
compiler: gcc-7 (Debian 7.4.0-14) 7.4.0
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp at intel.com>
All error/warnings (new ones prefixed by >>):
In file included from drivers/gpu/drm/i915/gt/intel_engine_types.h:20:0,
from drivers/gpu/drm/i915/gt/intel_gt.h:9,
from drivers/gpu/drm/i915/selftests/igt_flush_test.c:7:
drivers/gpu/drm/i915/selftests/igt_flush_test.c: In function 'igt_flush_test':
>> drivers/gpu/drm/i915/i915_gem.h:72:30: error: implicit declaration of function 'set_taint_for_CI'; did you mean 'add_taint_for_CI'? [-Werror=implicit-function-declaration]
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
>> drivers/gpu/drm/i915/selftests/igt_flush_test.c:28:3: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
>> drivers/gpu/drm/i915/i915_gem.h:72:47: error: 'TAIN_WARN' undeclared (first use in this function); did you mean 'TAINT_WARN'?
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
>> drivers/gpu/drm/i915/selftests/igt_flush_test.c:28:3: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
drivers/gpu/drm/i915/i915_gem.h:72:47: note: each undeclared identifier is reported only once for each function it appears in
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
>> drivers/gpu/drm/i915/selftests/igt_flush_test.c:28:3: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
cc1: all warnings being treated as errors
--
In file included from drivers/gpu/drm/i915/gt/intel_engine_types.h:20:0,
from drivers/gpu/drm/i915/gt/intel_context_types.h:17,
from drivers/gpu/drm/i915/gem/i915_gem_context_types.h:20,
from drivers/gpu/drm/i915/gem/i915_gem_context.h:10,
from drivers/gpu/drm/i915/gt/intel_context.c:7:
drivers/gpu/drm/i915/gt/selftest_context.c: In function '__live_active_context':
>> drivers/gpu/drm/i915/i915_gem.h:72:30: error: implicit declaration of function 'set_taint_for_CI'; did you mean 'add_taint_for_CI'? [-Werror=implicit-function-declaration]
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
>> drivers/gpu/drm/i915/gt/selftest_context.c:275:3: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
>> drivers/gpu/drm/i915/i915_gem.h:72:47: error: 'TAIN_WARN' undeclared (first use in this function); did you mean 'TAINT_WARN'?
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
>> drivers/gpu/drm/i915/gt/selftest_context.c:275:3: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
drivers/gpu/drm/i915/i915_gem.h:72:47: note: each undeclared identifier is reported only once for each function it appears in
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
>> drivers/gpu/drm/i915/gt/selftest_context.c:275:3: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
cc1: all warnings being treated as errors
--
In file included from drivers/gpu/drm/i915/gt/intel_engine_types.h:20:0,
from drivers/gpu/drm/i915/gt/intel_context_types.h:17,
from drivers/gpu/drm/i915/gem/i915_gem_context_types.h:20,
from drivers/gpu/drm/i915/gem/i915_gem_context.h:10,
from drivers/gpu/drm/i915/gt/intel_lrc.c:136:
drivers/gpu/drm/i915/gt/selftest_lrc.c: In function 'live_sanitycheck':
>> drivers/gpu/drm/i915/i915_gem.h:72:30: error: implicit declaration of function 'set_taint_for_CI'; did you mean 'add_taint_for_CI'? [-Werror=implicit-function-declaration]
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
>> drivers/gpu/drm/i915/gt/selftest_lrc.c:84:4: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
>> drivers/gpu/drm/i915/i915_gem.h:72:47: error: 'TAIN_WARN' undeclared (first use in this function); did you mean 'TAINT_WARN'?
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
>> drivers/gpu/drm/i915/gt/selftest_lrc.c:84:4: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
drivers/gpu/drm/i915/i915_gem.h:72:47: note: each undeclared identifier is reported only once for each function it appears in
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
>> drivers/gpu/drm/i915/gt/selftest_lrc.c:84:4: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
drivers/gpu/drm/i915/gt/selftest_lrc.c: In function 'slice_semaphore_queue':
>> drivers/gpu/drm/i915/i915_gem.h:72:47: error: 'TAIN_WARN' undeclared (first use in this function); did you mean 'TAINT_WARN'?
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
drivers/gpu/drm/i915/gt/selftest_lrc.c:416:3: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
drivers/gpu/drm/i915/gt/selftest_lrc.c: In function 'live_timeslice_queue':
>> drivers/gpu/drm/i915/i915_gem.h:72:47: error: 'TAIN_WARN' undeclared (first use in this function); did you mean 'TAINT_WARN'?
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
drivers/gpu/drm/i915/gt/selftest_lrc.c:614:4: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
drivers/gpu/drm/i915/gt/selftest_lrc.c: In function 'live_busywait_preempt':
>> drivers/gpu/drm/i915/i915_gem.h:72:47: error: 'TAIN_WARN' undeclared (first use in this function); did you mean 'TAINT_WARN'?
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
drivers/gpu/drm/i915/gt/selftest_lrc.c:794:4: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
drivers/gpu/drm/i915/gt/selftest_lrc.c: In function 'live_preempt':
>> drivers/gpu/drm/i915/i915_gem.h:72:47: error: 'TAIN_WARN' undeclared (first use in this function); did you mean 'TAINT_WARN'?
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
drivers/gpu/drm/i915/gt/selftest_lrc.c:895:4: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
drivers/gpu/drm/i915/gt/selftest_lrc.c: In function 'live_late_preempt':
>> drivers/gpu/drm/i915/i915_gem.h:72:47: error: 'TAIN_WARN' undeclared (first use in this function); did you mean 'TAINT_WARN'?
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
drivers/gpu/drm/i915/gt/selftest_lrc.c:1013:4: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
drivers/gpu/drm/i915/gt/selftest_lrc.c: In function 'live_preempt_cancel':
>> drivers/gpu/drm/i915/i915_gem.h:72:47: error: 'TAIN_WARN' undeclared (first use in this function); did you mean 'TAINT_WARN'?
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
drivers/gpu/drm/i915/gt/selftest_lrc.c:1485:2: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
drivers/gpu/drm/i915/gt/selftest_lrc.c: In function 'live_preempt_hang':
>> drivers/gpu/drm/i915/i915_gem.h:72:47: error: 'TAIN_WARN' undeclared (first use in this function); did you mean 'TAINT_WARN'?
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
drivers/gpu/drm/i915/gt/selftest_lrc.c:1967:4: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
drivers/gpu/drm/i915/gt/selftest_lrc.c: In function 'nop_virtual_engine':
>> drivers/gpu/drm/i915/i915_gem.h:72:47: error: 'TAIN_WARN' undeclared (first use in this function); did you mean 'TAINT_WARN'?
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
drivers/gpu/drm/i915/gt/selftest_lrc.c:2476:5: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
drivers/gpu/drm/i915/gt/selftest_lrc.c: In function 'mask_virtual_engine':
>> drivers/gpu/drm/i915/i915_gem.h:72:47: error: 'TAIN_WARN' undeclared (first use in this function); did you mean 'TAINT_WARN'?
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
drivers/gpu/drm/i915/gt/selftest_lrc.c:2619:4: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
cc1: all warnings being treated as errors
--
In file included from drivers/gpu/drm/i915/gt/intel_engine_types.h:20:0,
from drivers/gpu/drm/i915/gt/intel_context_types.h:17,
from drivers/gpu/drm/i915/i915_request.h:31,
from drivers/gpu/drm/i915/i915_active.h:13,
from drivers/gpu/drm/i915/display/intel_frontbuffer.h:30,
from drivers/gpu/drm/i915/i915_drv.h:71,
from drivers/gpu/drm/i915/display/intel_display_types.h:46,
from drivers/gpu/drm/i915/gt/intel_reset.c:10:
drivers/gpu/drm/i915/gt/selftest_hangcheck.c: In function 'active_request_put':
>> drivers/gpu/drm/i915/i915_gem.h:72:30: error: implicit declaration of function 'set_taint_for_CI'; did you mean 'add_taint_for_CI'? [-Werror=implicit-function-declaration]
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
>> drivers/gpu/drm/i915/gt/selftest_hangcheck.c:685:3: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
>> drivers/gpu/drm/i915/i915_gem.h:72:47: error: 'TAIN_WARN' undeclared (first use in this function); did you mean 'TAINT_WARN'?
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
>> drivers/gpu/drm/i915/gt/selftest_hangcheck.c:685:3: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
drivers/gpu/drm/i915/i915_gem.h:72:47: note: each undeclared identifier is reported only once for each function it appears in
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
>> drivers/gpu/drm/i915/gt/selftest_hangcheck.c:685:3: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
drivers/gpu/drm/i915/gt/selftest_hangcheck.c: In function '__igt_reset_engines':
>> drivers/gpu/drm/i915/i915_gem.h:72:47: error: 'TAIN_WARN' undeclared (first use in this function); did you mean 'TAINT_WARN'?
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
drivers/gpu/drm/i915/gt/selftest_hangcheck.c:881:6: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
drivers/gpu/drm/i915/gt/selftest_hangcheck.c: In function 'igt_reset_queue':
>> drivers/gpu/drm/i915/i915_gem.h:72:47: error: 'TAIN_WARN' undeclared (first use in this function); did you mean 'TAINT_WARN'?
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
drivers/gpu/drm/i915/gt/selftest_hangcheck.c:1422:5: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
cc1: all warnings being treated as errors
--
In file included from drivers/gpu/drm/i915/gt/intel_engine_types.h:20:0,
from drivers/gpu/drm/i915/gt/intel_context_types.h:17,
from drivers/gpu/drm/i915/gem/i915_gem_context_types.h:20,
from drivers/gpu/drm/i915/gem/i915_gem_context.h:10,
from drivers/gpu/drm/i915/i915_request.c:32:
drivers/gpu/drm/i915/selftests/i915_request.c: In function '__igt_breadcrumbs_smoketest':
>> drivers/gpu/drm/i915/i915_gem.h:72:30: error: implicit declaration of function 'set_taint_for_CI'; did you mean 'add_taint_for_CI'? [-Werror=implicit-function-declaration]
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
>> drivers/gpu/drm/i915/selftests/i915_request.c:375:4: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
>> drivers/gpu/drm/i915/i915_gem.h:72:47: error: 'TAIN_WARN' undeclared (first use in this function); did you mean 'TAINT_WARN'?
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
>> drivers/gpu/drm/i915/selftests/i915_request.c:375:4: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
drivers/gpu/drm/i915/i915_gem.h:72:47: note: each undeclared identifier is reported only once for each function it appears in
do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
^
>> drivers/gpu/drm/i915/selftests/i915_request.c:375:4: note: in expansion of macro 'GEM_TRACE_DUMP'
GEM_TRACE_DUMP();
^~~~~~~~~~~~~~
cc1: all warnings being treated as errors
vim +72 drivers/gpu/drm/i915/i915_gem.h
64
65 #if IS_ENABLED(CONFIG_DRM_I915_TRACE_GEM)
66 #define GEM_TRACE(...) trace_printk(__VA_ARGS__)
67 #define GEM_TRACE_ERR(...) do { \
68 pr_err(__VA_ARGS__); \
69 trace_printk(__VA_ARGS__); \
70 } while (0)
71 #define GEM_TRACE_DUMP() \
> 72 do { ftrace_dump(DUMP_ALL); set_taint_for_CI(TAIN_WARN); } while (0)
73 #define GEM_TRACE_DUMP_ON(expr) \
74 do { if (expr) GEM_TRACE_DUMP(); } while (0)
75 #else
76 #define GEM_TRACE(...) do { } while (0)
77 #define GEM_TRACE_ERR(...) do { } while (0)
78 #define GEM_TRACE_DUMP() do { } while (0)
79 #define GEM_TRACE_DUMP_ON(expr) BUILD_BUG_ON_INVALID(expr)
80 #endif
81
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation
-------------- next part --------------
A non-text attachment was scrubbed...
Name: .config.gz
Type: application/gzip
Size: 32060 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/intel-gfx/attachments/20191109/19f5683b/attachment-0001.gz>
More information about the Intel-gfx
mailing list