public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Kei Tokunaga <tokunaga.keiich@jp.fujitsu.com>
To: linux-scsi@vger.kernel.org,
	James Bottomley <James.Bottomley@suse.de>,
	Ingo Molnar <mingo@redhat.com>,
	Steven Rostedt <rostedt@goodmis.org>,
	Frederic Weisbecker <fweisbec@gmail.com>
Cc: Tomohiro Kusumi <kusumi.tomohiro@jp.fujitsu.com>,
	lkml <linux-kernel@vger.kernel.org>,
	Li Zefan <lizf@cn.fujitsu.com>,
	Xiao Guangrong <xiaoguangrong@cn.fujitsu.com>,
	Kei Tokunaga <tokunaga.keiich@jp.fujitsu.com>
Subject: [PATCH 0/2] scsi: ftrace based scsi tracing feature
Date: Wed, 20 Jan 2010 15:42:55 +0900	[thread overview]
Message-ID: <4B56A5EF.40703@jp.fujitsu.com> (raw)

Hi,

We'd like to propose this patchset (ftrace based scsi tracing)
as the first attempt for SCSI tracing, starting with the minimum
trace points.

Tracing gives us an idea how far SCSI processing goes, for
example, even in a case that a SCSI IRQ handling is blocked by
other drivers that disable interrupts for a long time.  Also,
trace could be used for debugging purpose in development similar
to how USB developers use usbmon for collecting submitted and
completed URBs for debugging.  In a mission critical area, there
is sometimes a case that we have to analyze a problem at customers
without taking a memory dump as much as possible.  There is also
a case to solve a problem which reproducibility is very low (but
we can not just leave the problem) so collecting additional data
afterwards is difficult.

In this version, it prints host#, channel, id, lun, and CDB (and
a value of 'result' if it's called in scsi_done().)  All fields
of CDB are printed in hex, and some of them are printed in ascii
as well.  That makes the outputs be more user-friendly.  For
example, retrieving 'lba' and 'transfer length (txlen)' data from
CDB of "SCSI3 block command READ/WRITE," and printing them in the
friendly format, users can easily correlate the info from SCSI
layer with block layer.  (All the conversions to ascii that requires
relatively large costs are done in TP_printk(), making the overheads
at storing trace be minimum.)

CDB has various lengths for its size and that made us end up
creating __print_hex().

This patchset applies to 2.6.33-rc4.

 ([PATCH 0/2] scsi: ftrace based scsi tracing feature)
  [PATCH 1/2] scsi: add __print_hex() to ftrace
  [PATCH 2/2] scsi: add scsi trace core function and put trace points

Thanks,
Kei





                 reply	other threads:[~2010-01-20  6:43 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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=4B56A5EF.40703@jp.fujitsu.com \
    --to=tokunaga.keiich@jp.fujitsu.com \
    --cc=James.Bottomley@suse.de \
    --cc=fweisbec@gmail.com \
    --cc=kusumi.tomohiro@jp.fujitsu.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=lizf@cn.fujitsu.com \
    --cc=mingo@redhat.com \
    --cc=rostedt@goodmis.org \
    --cc=xiaoguangrong@cn.fujitsu.com \
    /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