linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCHv3 00/38] scsi logging update
@ 2014-09-29 11:58 Hannes Reinecke
  2014-09-29 11:58 ` [PATCH 01/38] Remove scsi_cmd_print_sense_hdr() Hannes Reinecke
                   ` (37 more replies)
  0 siblings, 38 replies; 52+ messages in thread
From: Hannes Reinecke @ 2014-09-29 11:58 UTC (permalink / raw)
  To: James Bottomley
  Cc: Christoph Hellwig, linux-scsi, Robert Elliott, Hannes Reinecke

Hi all,

here's the third version of my scsi logging updates.
Main (and most important) difference to the previous
patchset is that the stacksize does not increase when
printing SCSI CDBs, so the objection from hch should
be resolved with this.

To achieve this I've split up vprintk_emit() into two
functions, the original vprintk_emit() which formats
the string into an on-stack buffer and printk_emit_string()
which just ships out a pre-formatted string.
With that printk_emit_string() doesn't need to allocate
a buffer on stack, and I can use this reduced stack
usage to provide my own on-stack buffer, thereby using
the same stack size as the original version.

Unfortunately I've had to modify kernel/printk/printk.c,
but then printk_emit_string() looks as if it'd be useful
for others, too, so with a bit of luck I'll get away with it.

I've been using the (not yet released) patchset from
Steven Rostedt for moving seq_buf into lib; but he promised
me to send it upstream shortly. If that poses a problem it's
fairly straightforward to move it to scnprintf() etc.

And I've included tons of suggestions from Robert Elliott;
thanks for this.

As usual, comments and reviews are welcome.

Hannes Reinecke (36):
  Remove scsi_cmd_print_sense_hdr()
  sd: Remove scsi_print_sense() in sd_done()
  aha152x: Debug output update and whitespace cleanup
  scsi: introduce sdev_prefix_printk()
  scsi: Use sdev as argument for sense code printing
  acornscsi: use scsi_print_command()
  fas216: Update logging messages
  53c700: remove scsi_print_sense() usage
  scsi: stop decoding if scsi_normalize_sense() fails
  scsi: do not decode sense extras
  scsi: use 'bool' as return value for scsi_normalize_sense()
  scsi: remove scsi_print_status()
  Implement scsi_opcode_sa_name
  scsi: merge print_opcode_name()
  scsi: consolidate opcode lookup in scsi_opcode_sa_name()
  scsi: remove last argument from print_opcode_name()
  scsi: Remove scsi_print_command when calling abort
  scsi: separate out scsi_(host|driver)byte_string()
  sd: Cleanup logging
  scsi: simplify scsi_log_(send|completion)
  scsi: fixup logging messages in scsi_error.c
  scsi: use shost argument in scsi_eh_prt_fail_stats
  scsi_trace: add tracepoint for completion
  tracing: Add trace_seq_buffer_ptr() helper function
  Rearrange buffer formatting in printk()
  Externalize string buffer for printk
  Introduce dev_printk_string() and dev_printk_header()
  scsi: Use real functions for logging
  scsi: log request tag for scmd_printk()
  scsi: use external buffer for command logging
  libata: use __scsi_print_command()
  scsi: Remove command pointer argument from logging messages
  scsi: use seq_buf for formatting sense buffer
  scsi: use seq_buf for formatting scsi_print_result()
  scsi: Conditionally compile in constants.c
  scsi_error: document scsi_try_to_abort_cmd

Steven Rostedt (Red Hat) (2):
  tracing: Create seq_buf layer in trace_seq
  seq_buf: Move the seq_buf code to lib/

 arch/x86/kvm/mmutrace.h              |   2 +-
 drivers/ata/libata-eh.c              |  12 +-
 drivers/base/core.c                  |  24 +
 drivers/firmware/efi/cper.c          |   2 +-
 drivers/scsi/53c700.c                |  13 +-
 drivers/scsi/Makefile                |   6 +-
 drivers/scsi/NCR5380.c               |   5 +-
 drivers/scsi/aha152x.c               | 994 ++++++++---------------------------
 drivers/scsi/arm/acornscsi.c         |  12 +-
 drivers/scsi/arm/fas216.c            |  41 +-
 drivers/scsi/atari_NCR5380.c         |   3 +-
 drivers/scsi/ch.c                    |  11 +-
 drivers/scsi/constants.c             | 533 ++++---------------
 drivers/scsi/hosts.c                 |   4 +-
 drivers/scsi/osst.c                  |   8 +-
 drivers/scsi/ps3rom.c                |   4 -
 drivers/scsi/scsi.c                  |  45 +-
 drivers/scsi/scsi_error.c            | 151 +++---
 drivers/scsi/scsi_ioctl.c            |   2 +-
 drivers/scsi/scsi_lib.c              |  22 +-
 drivers/scsi/scsi_logging.c          | 308 +++++++++++
 drivers/scsi/scsi_trace.c            |  16 +
 drivers/scsi/sd.c                    |  52 +-
 drivers/scsi/sd.h                    |   7 +-
 drivers/scsi/sg.c                    |   6 +-
 drivers/scsi/sr.h                    |   3 +-
 drivers/scsi/sr_ioctl.c              |  19 +-
 drivers/scsi/st.c                    |   9 +-
 drivers/scsi/stex.c                  |   9 +-
 drivers/scsi/storvsc_drv.c           |   3 +-
 drivers/scsi/sun3_NCR5380.c          |   3 +-
 include/linux/device.h               |  12 +
 include/linux/printk.h               |   5 +
 include/linux/seq_buf.h              |  58 ++
 include/linux/trace_seq.h            |  10 +-
 include/scsi/scsi.h                  |   3 +
 include/scsi/scsi_dbg.h              |  71 ++-
 include/scsi/scsi_device.h           |  19 +-
 include/scsi/scsi_eh.h               |  14 +-
 include/trace/events/scsi.h          |  31 ++
 kernel/printk/printk.c               |  47 +-
 kernel/trace/trace.c                 |  39 +-
 kernel/trace/trace_events.c          |   6 +-
 kernel/trace/trace_functions_graph.c |   6 +-
 kernel/trace/trace_seq.c             | 184 ++++---
 lib/Makefile                         |   2 +-
 lib/seq_buf.c                        | 348 ++++++++++++
 lib/trace_seq.c                      | 303 +++++++++++
 48 files changed, 1862 insertions(+), 1625 deletions(-)
 create mode 100644 drivers/scsi/scsi_logging.c
 create mode 100644 include/linux/seq_buf.h
 create mode 100644 lib/seq_buf.c
 create mode 100644 lib/trace_seq.c

-- 
1.8.5.2


^ permalink raw reply	[flat|nested] 52+ messages in thread

end of thread, other threads:[~2014-09-30 19:37 UTC | newest]

Thread overview: 52+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-09-29 11:58 [PATCHv3 00/38] scsi logging update Hannes Reinecke
2014-09-29 11:58 ` [PATCH 01/38] Remove scsi_cmd_print_sense_hdr() Hannes Reinecke
2014-09-29 11:58 ` [PATCH 02/38] sd: Remove scsi_print_sense() in sd_done() Hannes Reinecke
2014-09-29 11:58 ` [PATCH 03/38] aha152x: Debug output update and whitespace cleanup Hannes Reinecke
2014-09-29 11:58 ` [PATCH 04/38] scsi: introduce sdev_prefix_printk() Hannes Reinecke
2014-09-29 11:58 ` [PATCH 05/38] scsi: Use sdev as argument for sense code printing Hannes Reinecke
2014-09-29 11:58 ` [PATCH 06/38] acornscsi: use scsi_print_command() Hannes Reinecke
2014-09-29 11:58 ` [PATCH 07/38] fas216: Update logging messages Hannes Reinecke
2014-09-29 11:58 ` [PATCH 08/38] 53c700: remove scsi_print_sense() usage Hannes Reinecke
2014-09-29 11:58 ` [PATCH 09/38] scsi: stop decoding if scsi_normalize_sense() fails Hannes Reinecke
2014-09-29 11:58 ` [PATCH 10/38] scsi: do not decode sense extras Hannes Reinecke
2014-09-29 11:58 ` [PATCH 11/38] scsi: use 'bool' as return value for scsi_normalize_sense() Hannes Reinecke
2014-09-29 11:58 ` [PATCH 12/38] scsi: remove scsi_print_status() Hannes Reinecke
2014-09-29 11:58 ` [PATCH 13/38] Implement scsi_opcode_sa_name Hannes Reinecke
2014-09-29 19:29   ` Douglas Gilbert
2014-09-30  5:50     ` Hannes Reinecke
2014-09-29 11:58 ` [PATCH 14/38] scsi: merge print_opcode_name() Hannes Reinecke
2014-09-29 11:58 ` [PATCH 15/38] scsi: consolidate opcode lookup in scsi_opcode_sa_name() Hannes Reinecke
2014-09-29 11:58 ` [PATCH 16/38] scsi: remove last argument from print_opcode_name() Hannes Reinecke
2014-09-29 11:58 ` [PATCH 17/38] scsi: Remove scsi_print_command when calling abort Hannes Reinecke
2014-09-29 11:58 ` [PATCH 18/38] scsi: separate out scsi_(host|driver)byte_string() Hannes Reinecke
2014-09-29 11:58 ` [PATCH 19/38] sd: Cleanup logging Hannes Reinecke
2014-09-29 11:58 ` [PATCH 20/38] scsi: simplify scsi_log_(send|completion) Hannes Reinecke
2014-09-29 11:58 ` [PATCH 21/38] scsi: fixup logging messages in scsi_error.c Hannes Reinecke
2014-09-29 11:58 ` [PATCH 22/38] scsi: use shost argument in scsi_eh_prt_fail_stats Hannes Reinecke
2014-09-29 11:58 ` [PATCH 23/38] scsi_trace: add tracepoint for completion Hannes Reinecke
2014-09-29 11:58 ` [PATCH 24/38] tracing: Add trace_seq_buffer_ptr() helper function Hannes Reinecke
2014-09-29 11:58 ` [PATCH 25/38] tracing: Create seq_buf layer in trace_seq Hannes Reinecke
2014-09-29 11:58 ` [PATCH 26/38] seq_buf: Move the seq_buf code to lib/ Hannes Reinecke
2014-09-29 14:04   ` Steven Rostedt
2014-09-29 14:08     ` Hannes Reinecke
2014-09-29 11:58 ` [PATCH 27/38] Rearrange buffer formatting in printk() Hannes Reinecke
2014-09-30 16:16   ` Petr Mladek
2014-09-30 19:37     ` Petr Mladek
2014-09-29 11:58 ` [PATCH 28/38] Externalize string buffer for printk Hannes Reinecke
2014-09-30 16:39   ` Petr Mladek
2014-09-30 18:52     ` Hannes Reinecke
2014-09-29 11:58 ` [PATCH 29/38] Introduce dev_printk_string() and dev_printk_header() Hannes Reinecke
2014-09-29 16:58   ` Greg Kroah-Hartman
2014-09-30  5:48     ` Hannes Reinecke
2014-09-29 11:58 ` [PATCH 30/38] scsi: Use real functions for logging Hannes Reinecke
2014-09-29 11:59 ` [PATCH 31/38] scsi: log request tag for scmd_printk() Hannes Reinecke
2014-09-29 11:59 ` [PATCH 32/38] scsi: use external buffer for command logging Hannes Reinecke
2014-09-29 11:59 ` [PATCH 33/38] libata: use __scsi_print_command() Hannes Reinecke
2014-09-29 14:06   ` Tejun Heo
2014-09-29 14:10     ` Hannes Reinecke
2014-09-29 14:11       ` Tejun Heo
2014-09-29 11:59 ` [PATCH 34/38] scsi: Remove command pointer argument from logging messages Hannes Reinecke
2014-09-29 11:59 ` [PATCH 35/38] scsi: use seq_buf for formatting sense buffer Hannes Reinecke
2014-09-29 11:59 ` [PATCH 36/38] scsi: use seq_buf for formatting scsi_print_result() Hannes Reinecke
2014-09-29 11:59 ` [PATCH 37/38] scsi: Conditionally compile in constants.c Hannes Reinecke
2014-09-29 11:59 ` [PATCH 38/38] scsi_error: document scsi_try_to_abort_cmd Hannes Reinecke

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).