Re: [PATCH v3] bus: mhi: host: Add tracing support

From: kernel test robot
Date: Sat Nov 11 2023 - 00:07:15 EST

Next message: syzbot: "Re: [syzbot] [PATCH] test 305230142ae0"
Previous message: Andrii Nakryiko: "Re: [PATCH perf] perf: get_perf_callchain return NULL for crosstask"
In reply to: Krishna chaitanya chundru: "[PATCH v3] bus: mhi: host: Add tracing support"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Krishna,

kernel test robot noticed the following build errors:

[auto build test ERROR on 3006adf3be79cde4d14b1800b963b82b6e5572e0]

url: https://github.com/intel-lab-lkp/linux/commits/Krishna-chaitanya-chundru/bus-mhi-host-Add-tracing-support/20231111-101955
base: 3006adf3be79cde4d14b1800b963b82b6e5572e0
patch link: https://lore.kernel.org/r/20231111-ftrace_support-v3-1-f358d2911a74%40quicinc.com
patch subject: [PATCH v3] bus: mhi: host: Add tracing support
config: sh-allmodconfig (https://download.01.org/0day-ci/archive/20231111/202311111237.zTm5NBLH-lkp@xxxxxxxxx/config)
compiler: sh4-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231111/202311111237.zTm5NBLH-lkp@xxxxxxxxx/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202311111237.zTm5NBLH-lkp@xxxxxxxxx/

All errors (new ones prefixed by >>):

In file included from include/trace/define_trace.h:102,
from drivers/bus/mhi/host/trace.h:224,
from drivers/bus/mhi/host/init.c:24:
drivers/bus/mhi/host/./trace.h: In function 'trace_event_raw_event_mhi_process_ctrl_ev_ring':
>> drivers/bus/mhi/host/../common.h:144:58: error: invalid type argument of '->' (have 'u64' {aka 'long long unsigned int'})
144 | #define MHI_TRE_GET_DWORD(tre, word) le32_to_cpu((tre)->dword[(word)])
| ^~
include/trace/trace_events.h:402:11: note: in definition of macro 'DECLARE_EVENT_CLASS'
402 | { assign; } \
| ^~~~~~
include/trace/trace_events.h:44:30: note: in expansion of macro 'PARAMS'
44 | PARAMS(assign), \
| ^~~~~~
drivers/bus/mhi/host/./trace.h:123:1: note: in expansion of macro 'TRACE_EVENT'
123 | TRACE_EVENT(mhi_process_ctrl_ev_ring,
| ^~~~~~~~~~~
drivers/bus/mhi/host/./trace.h:138:9: note: in expansion of macro 'TP_fast_assign'
138 | TP_fast_assign(
| ^~~~~~~~~~~~~~
include/linux/compiler_types.h:413:9: note: in expansion of macro '__compiletime_assert'
413 | __compiletime_assert(condition, msg, prefix, suffix)
| ^~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:425:9: note: in expansion of macro '_compiletime_assert'
425 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ^~~~~~~~~~~~~~~~~~~
include/linux/build_bug.h:39:37: note: in expansion of macro 'compiletime_assert'
39 | #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
| ^~~~~~~~~~~~~~~~~~
include/linux/bitfield.h:71:17: note: in expansion of macro 'BUILD_BUG_ON_MSG'
71 | BUILD_BUG_ON_MSG(__bf_cast_unsigned(_mask, _mask) > \
| ^~~~~~~~~~~~~~~~
include/linux/bitfield.h:61:43: note: in expansion of macro '__unsigned_scalar_typeof'
61 | #define __bf_cast_unsigned(type, x) ((__unsigned_scalar_typeof(type))(x))
| ^~~~~~~~~~~~~~~~~~~~~~~~
include/linux/bitfield.h:72:34: note: in expansion of macro '__bf_cast_unsigned'
72 | __bf_cast_unsigned(_reg, ~0ull), \
| ^~~~~~~~~~~~~~~~~~
include/linux/bitfield.h:154:17: note: in expansion of macro '__BF_FIELD_CHECK'
154 | __BF_FIELD_CHECK(_mask, _reg, 0U, "FIELD_GET: "); \
| ^~~~~~~~~~~~~~~~
drivers/bus/mhi/host/../common.h:159:41: note: in expansion of macro 'FIELD_GET'
159 | #define MHI_TRE_GET_EV_STATE(tre) FIELD_GET(GENMASK(31, 24), (MHI_TRE_GET_DWORD(tre, 0)))
| ^~~~~~~~~
include/linux/byteorder/generic.h:89:21: note: in expansion of macro '__le32_to_cpu'
89 | #define le32_to_cpu __le32_to_cpu
| ^~~~~~~~~~~~~
drivers/bus/mhi/host/../common.h:159:69: note: in expansion of macro 'MHI_TRE_GET_DWORD'
159 | #define MHI_TRE_GET_EV_STATE(tre) FIELD_GET(GENMASK(31, 24), (MHI_TRE_GET_DWORD(tre, 0)))
| ^~~~~~~~~~~~~~~~~
drivers/bus/mhi/host/./trace.h:144:34: note: in expansion of macro 'MHI_TRE_GET_EV_STATE'
144 | __entry->state = MHI_TRE_GET_EV_STATE(rp);
| ^~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/host/../common.h:144:58: error: invalid type argument of '->' (have 'u64' {aka 'long long unsigned int'})
144 | #define MHI_TRE_GET_DWORD(tre, word) le32_to_cpu((tre)->dword[(word)])
| ^~
include/trace/trace_events.h:402:11: note: in definition of macro 'DECLARE_EVENT_CLASS'
402 | { assign; } \
| ^~~~~~
include/trace/trace_events.h:44:30: note: in expansion of macro 'PARAMS'
44 | PARAMS(assign), \
| ^~~~~~
drivers/bus/mhi/host/./trace.h:123:1: note: in expansion of macro 'TRACE_EVENT'
123 | TRACE_EVENT(mhi_process_ctrl_ev_ring,
| ^~~~~~~~~~~
drivers/bus/mhi/host/./trace.h:138:9: note: in expansion of macro 'TP_fast_assign'
138 | TP_fast_assign(
| ^~~~~~~~~~~~~~
drivers/bus/mhi/host/../common.h:159:41: note: in expansion of macro 'FIELD_GET'
159 | #define MHI_TRE_GET_EV_STATE(tre) FIELD_GET(GENMASK(31, 24), (MHI_TRE_GET_DWORD(tre, 0)))
| ^~~~~~~~~
include/linux/byteorder/generic.h:89:21: note: in expansion of macro '__le32_to_cpu'
89 | #define le32_to_cpu __le32_to_cpu
| ^~~~~~~~~~~~~
drivers/bus/mhi/host/../common.h:159:69: note: in expansion of macro 'MHI_TRE_GET_DWORD'
159 | #define MHI_TRE_GET_EV_STATE(tre) FIELD_GET(GENMASK(31, 24), (MHI_TRE_GET_DWORD(tre, 0)))
| ^~~~~~~~~~~~~~~~~
drivers/bus/mhi/host/./trace.h:144:34: note: in expansion of macro 'MHI_TRE_GET_EV_STATE'
144 | __entry->state = MHI_TRE_GET_EV_STATE(rp);
| ^~~~~~~~~~~~~~~~~~~~
In file included from include/trace/define_trace.h:103:
drivers/bus/mhi/host/./trace.h: In function 'perf_trace_mhi_process_ctrl_ev_ring':
>> drivers/bus/mhi/host/../common.h:144:58: error: invalid type argument of '->' (have 'u64' {aka 'long long unsigned int'})
144 | #define MHI_TRE_GET_DWORD(tre, word) le32_to_cpu((tre)->dword[(word)])
| ^~
include/trace/perf.h:51:11: note: in definition of macro 'DECLARE_EVENT_CLASS'
51 | { assign; } \
| ^~~~~~
include/trace/trace_events.h:44:30: note: in expansion of macro 'PARAMS'
44 | PARAMS(assign), \
| ^~~~~~
drivers/bus/mhi/host/./trace.h:123:1: note: in expansion of macro 'TRACE_EVENT'
123 | TRACE_EVENT(mhi_process_ctrl_ev_ring,
| ^~~~~~~~~~~
drivers/bus/mhi/host/./trace.h:138:9: note: in expansion of macro 'TP_fast_assign'
138 | TP_fast_assign(
| ^~~~~~~~~~~~~~
include/linux/compiler_types.h:413:9: note: in expansion of macro '__compiletime_assert'
413 | __compiletime_assert(condition, msg, prefix, suffix)
| ^~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:425:9: note: in expansion of macro '_compiletime_assert'
425 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ^~~~~~~~~~~~~~~~~~~
include/linux/build_bug.h:39:37: note: in expansion of macro 'compiletime_assert'
39 | #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
| ^~~~~~~~~~~~~~~~~~
include/linux/bitfield.h:71:17: note: in expansion of macro 'BUILD_BUG_ON_MSG'
71 | BUILD_BUG_ON_MSG(__bf_cast_unsigned(_mask, _mask) > \
| ^~~~~~~~~~~~~~~~
include/linux/bitfield.h:61:43: note: in expansion of macro '__unsigned_scalar_typeof'
61 | #define __bf_cast_unsigned(type, x) ((__unsigned_scalar_typeof(type))(x))
| ^~~~~~~~~~~~~~~~~~~~~~~~
include/linux/bitfield.h:72:34: note: in expansion of macro '__bf_cast_unsigned'
72 | __bf_cast_unsigned(_reg, ~0ull), \
| ^~~~~~~~~~~~~~~~~~
include/linux/bitfield.h:154:17: note: in expansion of macro '__BF_FIELD_CHECK'
154 | __BF_FIELD_CHECK(_mask, _reg, 0U, "FIELD_GET: "); \
| ^~~~~~~~~~~~~~~~
drivers/bus/mhi/host/../common.h:159:41: note: in expansion of macro 'FIELD_GET'
159 | #define MHI_TRE_GET_EV_STATE(tre) FIELD_GET(GENMASK(31, 24), (MHI_TRE_GET_DWORD(tre, 0)))
| ^~~~~~~~~
include/linux/byteorder/generic.h:89:21: note: in expansion of macro '__le32_to_cpu'
89 | #define le32_to_cpu __le32_to_cpu
| ^~~~~~~~~~~~~
drivers/bus/mhi/host/../common.h:159:69: note: in expansion of macro 'MHI_TRE_GET_DWORD'
159 | #define MHI_TRE_GET_EV_STATE(tre) FIELD_GET(GENMASK(31, 24), (MHI_TRE_GET_DWORD(tre, 0)))
| ^~~~~~~~~~~~~~~~~
drivers/bus/mhi/host/./trace.h:144:34: note: in expansion of macro 'MHI_TRE_GET_EV_STATE'
144 | __entry->state = MHI_TRE_GET_EV_STATE(rp);
| ^~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/host/../common.h:144:58: error: invalid type argument of '->' (have 'u64' {aka 'long long unsigned int'})
144 | #define MHI_TRE_GET_DWORD(tre, word) le32_to_cpu((tre)->dword[(word)])
| ^~
include/trace/perf.h:51:11: note: in definition of macro 'DECLARE_EVENT_CLASS'
51 | { assign; } \
| ^~~~~~
include/trace/trace_events.h:44:30: note: in expansion of macro 'PARAMS'
44 | PARAMS(assign), \
| ^~~~~~
drivers/bus/mhi/host/./trace.h:123:1: note: in expansion of macro 'TRACE_EVENT'
123 | TRACE_EVENT(mhi_process_ctrl_ev_ring,
| ^~~~~~~~~~~
drivers/bus/mhi/host/./trace.h:138:9: note: in expansion of macro 'TP_fast_assign'
138 | TP_fast_assign(
| ^~~~~~~~~~~~~~
drivers/bus/mhi/host/../common.h:159:41: note: in expansion of macro 'FIELD_GET'
159 | #define MHI_TRE_GET_EV_STATE(tre) FIELD_GET(GENMASK(31, 24), (MHI_TRE_GET_DWORD(tre, 0)))
| ^~~~~~~~~
include/linux/byteorder/generic.h:89:21: note: in expansion of macro '__le32_to_cpu'
89 | #define le32_to_cpu __le32_to_cpu
| ^~~~~~~~~~~~~
drivers/bus/mhi/host/../common.h:159:69: note: in expansion of macro 'MHI_TRE_GET_DWORD'
159 | #define MHI_TRE_GET_EV_STATE(tre) FIELD_GET(GENMASK(31, 24), (MHI_TRE_GET_DWORD(tre, 0)))
| ^~~~~~~~~~~~~~~~~
drivers/bus/mhi/host/./trace.h:144:34: note: in expansion of macro 'MHI_TRE_GET_EV_STATE'
144 | __entry->state = MHI_TRE_GET_EV_STATE(rp);
| ^~~~~~~~~~~~~~~~~~~~

vim +144 drivers/bus/mhi/host/../common.h

8485149c7a84bb Manivannan Sadhasivam 2022-03-01 122
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 123 /* Channel reset command */
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 124 #define MHI_TRE_CMD_RESET_PTR 0
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 125 #define MHI_TRE_CMD_RESET_DWORD0 0
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 126 #define MHI_TRE_CMD_RESET_DWORD1(chid) cpu_to_le32(FIELD_PREP(GENMASK(31, 24), chid) | \
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 127 FIELD_PREP(GENMASK(23, 16), \
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 128 MHI_CMD_RESET_CHAN))
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 129
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 130 /* Channel stop command */
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 131 #define MHI_TRE_CMD_STOP_PTR 0
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 132 #define MHI_TRE_CMD_STOP_DWORD0 0
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 133 #define MHI_TRE_CMD_STOP_DWORD1(chid) cpu_to_le32(FIELD_PREP(GENMASK(31, 24), chid) | \
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 134 FIELD_PREP(GENMASK(23, 16), \
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 135 MHI_CMD_STOP_CHAN))
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 136
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 137 /* Channel start command */
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 138 #define MHI_TRE_CMD_START_PTR 0
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 139 #define MHI_TRE_CMD_START_DWORD0 0
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 140 #define MHI_TRE_CMD_START_DWORD1(chid) cpu_to_le32(FIELD_PREP(GENMASK(31, 24), chid) | \
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 141 FIELD_PREP(GENMASK(23, 16), \
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 142 MHI_CMD_START_CHAN))
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 143
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 @144 #define MHI_TRE_GET_DWORD(tre, word) le32_to_cpu((tre)->dword[(word)])
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 145 #define MHI_TRE_GET_CMD_CHID(tre) FIELD_GET(GENMASK(31, 24), MHI_TRE_GET_DWORD(tre, 1))
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 146 #define MHI_TRE_GET_CMD_TYPE(tre) FIELD_GET(GENMASK(23, 16), MHI_TRE_GET_DWORD(tre, 1))
8485149c7a84bb Manivannan Sadhasivam 2022-03-01 147

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

Next message: syzbot: "Re: [syzbot] [PATCH] test 305230142ae0"
Previous message: Andrii Nakryiko: "Re: [PATCH perf] perf: get_perf_callchain return NULL for crosstask"
In reply to: Krishna chaitanya chundru: "[PATCH v3] bus: mhi: host: Add tracing support"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]