Re: [RFC PATCH 10/10] scsi/trace: Use scsi_print_command trace point instead of printk
From: Hannes Reinecke
Date: Thu Aug 28 2014 - 08:15:12 EST
On 08/28/2014 08:19 AM, Yoshihiro YUNOMAE wrote:
Hi Hannes,I would go with the wording in constants.c, but set in double quotes
like "Test Unit Ready" to avoid parsing errors later on.
I tried to remove duplicated decoder of SCSI command, but the
output format of it in constants.c is different from it in traceevents.
I have two questions for it.
constants: Test Unit Ready
=> Which of "XXX_YYY_ZZZ" and "Xxx Yyy Zzz" should the kernel output
strings? This difference comes from difference of implementation.
The decoder in traceevents are using macro. So, it cannot define
separated words. On the other hand, the decoder in constants are using
constant string array table. So, it can define separated words.
(Ex2)Yes, I would prefer this. Again, we should be setting the strings in
double quotes irrespective if there are spaces in the resulting
string or not.
constants: Set limits(12)
=> Should we merge those decoder?
I understand we use the decoder of constants, but we need to solveAs mentioned previously, my aim is to convert the logging system in
these problems. Would you give me your comments?
a) convert all lone 'printk' statements into dev_printk() variants
and ensure everything is printed in one statement to avoid
linebreaks in the resulting logging message.
This will eliminate the immediate problem we're having, namely
that debugging is near to impossible under high load.
The patchset is nearing completion, and I will be posting it
later this week.
b) Convert scsi_trace to use the functions from constants.c to
avoid code duplication and update scsi_trace to log sense codes.
c) Remove all logging functions in the hot path (ie
SCSI_LOG_MLQUEUE and SCSI_LOG_MLCOMPLETE) and replace them
with trace events.
b) is what you're working on, right?
For 'c)' the problem is that we should retain the existing
functionality with scsi_logging_level here, at least initially.
So we cannot just replace it, as this would leave SCSI_LOG_ML
a dummy without any real functionality.
I'll be posting my current patchset for review shortly.
Dr. Hannes Reinecke zSeries & Storage
hare@xxxxxxx +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 NÃrnberg
GF: J. Hawn, J. Guild, F. ImendÃrffer, HRB 16746 (AG NÃrnberg)
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/