From: kernel test robot <lkp@intel.com>
To: Kassey Li <quic_yingangl@quicinc.com>,
rostedt@goodmis.org, James.Bottomley@hansenpartnership.com,
martin.petersen@oracle.com, mathieu.desnoyers@efficios.com,
linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org,
linux-scsi@vger.kernel.org
Cc: oe-kbuild-all@lists.linux.dev, quic_yingangl@quicinc.com
Subject: Re: [PATCH v2] scsi: trace: change the rtn log in string
Date: Thu, 15 May 2025 07:05:35 +0800 [thread overview]
Message-ID: <202505150658.uk9oRnKr-lkp@intel.com> (raw)
In-Reply-To: <20250514074456.450006-1-quic_yingangl@quicinc.com>
Hi Kassey,
kernel test robot noticed the following build errors:
[auto build test ERROR on trace/for-next]
[also build test ERROR on jejb-scsi/for-next mkp-scsi/for-next linus/master v6.15-rc6 next-20250514]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Kassey-Li/scsi-trace-change-the-rtn-log-in-string/20250514-155034
base: https://git.kernel.org/pub/scm/linux/kernel/git/trace/linux-trace for-next
patch link: https://lore.kernel.org/r/20250514074456.450006-1-quic_yingangl%40quicinc.com
patch subject: [PATCH v2] scsi: trace: change the rtn log in string
config: i386-buildonly-randconfig-001-20250515 (https://download.01.org/0day-ci/archive/20250515/202505150658.uk9oRnKr-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-12) 11.3.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250515/202505150658.uk9oRnKr-lkp@intel.com/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@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202505150658.uk9oRnKr-lkp@intel.com/
All errors (new ones prefixed by >>):
In file included from include/trace/define_trace.h:119,
from include/trace/events/scsi.h:355,
from drivers/scsi/scsi.c:73:
include/trace/events/scsi.h: In function 'trace_raw_output_scsi_dispatch_cmd_error':
>> include/trace/events/scsi.h:250:36: error: 'rtn' undeclared (first use in this function)
250 | __print_symbolic(rtn, { SCSI_MLQUEUE_HOST_BUSY, "HOST_BUSY" },
| ^~~
include/trace/trace_events.h:219:34: note: in definition of macro 'DECLARE_EVENT_CLASS'
219 | trace_event_printf(iter, print); \
| ^~~~~
include/trace/trace_events.h:45:30: note: in expansion of macro 'PARAMS'
45 | PARAMS(print)); \
| ^~~~~~
include/trace/events/scsi.h:203:1: note: in expansion of macro 'TRACE_EVENT'
203 | TRACE_EVENT(scsi_dispatch_cmd_error,
| ^~~~~~~~~~~
include/trace/events/scsi.h:241:9: note: in expansion of macro 'TP_printk'
241 | TP_printk("host_no=%u channel=%u id=%u lun=%u data_sgl=%u prot_sgl=%u" \
| ^~~~~~~~~
include/trace/events/scsi.h:250:19: note: in expansion of macro '__print_symbolic'
250 | __print_symbolic(rtn, { SCSI_MLQUEUE_HOST_BUSY, "HOST_BUSY" },
| ^~~~~~~~~~~~~~~~
include/trace/events/scsi.h:250:36: note: each undeclared identifier is reported only once for each function it appears in
250 | __print_symbolic(rtn, { SCSI_MLQUEUE_HOST_BUSY, "HOST_BUSY" },
| ^~~
include/trace/trace_events.h:219:34: note: in definition of macro 'DECLARE_EVENT_CLASS'
219 | trace_event_printf(iter, print); \
| ^~~~~
include/trace/trace_events.h:45:30: note: in expansion of macro 'PARAMS'
45 | PARAMS(print)); \
| ^~~~~~
include/trace/events/scsi.h:203:1: note: in expansion of macro 'TRACE_EVENT'
203 | TRACE_EVENT(scsi_dispatch_cmd_error,
| ^~~~~~~~~~~
include/trace/events/scsi.h:241:9: note: in expansion of macro 'TP_printk'
241 | TP_printk("host_no=%u channel=%u id=%u lun=%u data_sgl=%u prot_sgl=%u" \
| ^~~~~~~~~
include/trace/events/scsi.h:250:19: note: in expansion of macro '__print_symbolic'
250 | __print_symbolic(rtn, { SCSI_MLQUEUE_HOST_BUSY, "HOST_BUSY" },
| ^~~~~~~~~~~~~~~~
vim +/rtn +250 include/trace/events/scsi.h
204
205 TP_PROTO(struct scsi_cmnd *cmd, int rtn),
206
207 TP_ARGS(cmd, rtn),
208
209 TP_STRUCT__entry(
210 __field( unsigned int, host_no )
211 __field( unsigned int, channel )
212 __field( unsigned int, id )
213 __field( unsigned int, lun )
214 __field( int, rtn )
215 __field( unsigned int, opcode )
216 __field( unsigned int, cmd_len )
217 __field( int, driver_tag)
218 __field( int, scheduler_tag)
219 __field( unsigned int, data_sglen )
220 __field( unsigned int, prot_sglen )
221 __field( unsigned char, prot_op )
222 __dynamic_array(unsigned char, cmnd, cmd->cmd_len)
223 ),
224
225 TP_fast_assign(
226 __entry->host_no = cmd->device->host->host_no;
227 __entry->channel = cmd->device->channel;
228 __entry->id = cmd->device->id;
229 __entry->lun = cmd->device->lun;
230 __entry->rtn = rtn;
231 __entry->opcode = cmd->cmnd[0];
232 __entry->cmd_len = cmd->cmd_len;
233 __entry->driver_tag = scsi_cmd_to_rq(cmd)->tag;
234 __entry->scheduler_tag = scsi_cmd_to_rq(cmd)->internal_tag;
235 __entry->data_sglen = scsi_sg_count(cmd);
236 __entry->prot_sglen = scsi_prot_sg_count(cmd);
237 __entry->prot_op = scsi_get_prot_op(cmd);
238 memcpy(__get_dynamic_array(cmnd), cmd->cmnd, cmd->cmd_len);
239 ),
240
241 TP_printk("host_no=%u channel=%u id=%u lun=%u data_sgl=%u prot_sgl=%u" \
242 " prot_op=%s driver_tag=%d scheduler_tag=%d cmnd=(%s %s raw=%s)" \
243 " rtn=%s",
244 __entry->host_no, __entry->channel, __entry->id,
245 __entry->lun, __entry->data_sglen, __entry->prot_sglen,
246 show_prot_op_name(__entry->prot_op), __entry->driver_tag,
247 __entry->scheduler_tag, show_opcode_name(__entry->opcode),
248 __parse_cdb(__get_dynamic_array(cmnd), __entry->cmd_len),
249 __print_hex(__get_dynamic_array(cmnd), __entry->cmd_len),
> 250 __print_symbolic(rtn, { SCSI_MLQUEUE_HOST_BUSY, "HOST_BUSY" },
251 { SCSI_MLQUEUE_DEVICE_BUSY, "DEVICE_BUSY" },
252 { SCSI_MLQUEUE_EH_RETRY, "EH_RETRY" },
253 { SCSI_MLQUEUE_TARGET_BUSY, "TARGET_BUSY" })
254 )
255 );
256
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
prev parent reply other threads:[~2025-05-14 23:06 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-14 7:44 [PATCH v2] scsi: trace: change the rtn log in string Kassey Li
2025-05-14 13:09 ` Steven Rostedt
2025-05-19 9:40 ` Kassey Li quic
2025-05-14 13:39 ` James Bottomley
2025-05-19 9:37 ` Kassey Li quic
2025-05-14 23:05 ` kernel test robot [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202505150658.uk9oRnKr-lkp@intel.com \
--to=lkp@intel.com \
--cc=James.Bottomley@hansenpartnership.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=linux-trace-kernel@vger.kernel.org \
--cc=martin.petersen@oracle.com \
--cc=mathieu.desnoyers@efficios.com \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=quic_yingangl@quicinc.com \
--cc=rostedt@goodmis.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox