public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
From: Anastasia Kovaleva <a.kovaleva@yadro.com>
To: <martin.petersen@oracle.com>, <bootc@bootc.net>,
	<bostroesser@gmail.com>, <michael.christie@oracle.com>
Cc: <linux-scsi@vger.kernel.org>, <target-devel@vger.kernel.org>,
	<linux@yadro.com>
Subject: [PATCH 00/10] Improve SCSI target logs
Date: Fri, 29 Nov 2024 18:30:46 +0300	[thread overview]
Message-ID: <20241129153056.6985-1-a.kovaleva@yadro.com> (raw)

This patch series address two main issues of the current logs in SCSI
target (discussed in [1]):

  1. Absence of a common prefix for the subsystem to make it possible to
     filter them out to a separate file.
  2. Absence of an I-T-L nexus in a log message to understand what is
     that log for.

Patches 2 - 6 of the series are automated. So, we also included fixes
[PATCH 04] of other style errors and warnings that checkpatch can fix.
We've been using these logs for almost an year and didn't encounter any
kind of problems.

Code style of all patches was made by git clang-format with these
additional parameters:

+AllowAllArgumentsOnNextLine: false
+ColumnLimit: 100
+PenaltyBreakOpenParenthesis: 12000
+PenaltyBreakBeforeFirstCallParameter: 12000
+PenaltyBreakString: 12000

In general, logs present in the following format:
  target <file_prefix>: General log Message
  target <file_prefix> (<iqn_initiator> -> <RTPI>): Session Log Message
  target <file_prefix> (<iqn_initiator> -> <RTPI>/<MappedLun>): Command Log Message
  target <file_prefix> <fabric_name>_tpg[<tpg_tag>](<tpg_rtpi>/<LUN>): Lun Log Message

Examples:
  target sbp: initiator already logged-in
  target core (iqn.2001-09.com.example:foobar-1 -> 1): ABORT_TASK: Sending TMR_TASK_DOES_NOT_EXIST for ref_tag: 2281488
  target core (iqn.2001-09.com.example:foobar-1 -> 1/99): Unknown PERSISTENT_RESERVE_OUT service action: 0x11
  target core iSCSI_tpg[1](1/99): Unable to extract alua_tg_pt_status

[1] - https://lore.kernel.org/target-devel/ZF0MiCRW8HWm8YYj@yadro.com/

Anastasia Kovaleva (10):
  target: core: Improve SCSI target logs
  target: Use new log wrappers
  target: Fix quoted strings splitting
  target: Fix some obvious style problems
  target: Add log prefix to all tcm files
  target: Remove old prefixes from log messages
  target: core: Extend CaW logging
  target: core: Use extended logs where possible
  target: Use __func__ in logs
  target: iscsi: Improve a log message

 drivers/target/iscsi/cxgbit/cxgbit_cm.c       | 191 +++---
 drivers/target/iscsi/cxgbit/cxgbit_ddp.c      |  19 +-
 drivers/target/iscsi/cxgbit/cxgbit_main.c     |  47 +-
 drivers/target/iscsi/cxgbit/cxgbit_target.c   | 115 ++--
 drivers/target/iscsi/iscsi_target.c           | 559 +++++++--------
 drivers/target/iscsi/iscsi_target_auth.c      | 123 ++--
 drivers/target/iscsi/iscsi_target_configfs.c  |  91 +--
 .../target/iscsi/iscsi_target_datain_values.c |  20 +-
 drivers/target/iscsi/iscsi_target_device.c    |   4 +-
 drivers/target/iscsi/iscsi_target_erl0.c      | 149 ++--
 drivers/target/iscsi/iscsi_target_erl1.c      | 158 ++---
 drivers/target/iscsi/iscsi_target_erl2.c      |  60 +-
 drivers/target/iscsi/iscsi_target_login.c     | 157 ++---
 drivers/target/iscsi/iscsi_target_nego.c      | 187 +++---
 .../target/iscsi/iscsi_target_nodeattrib.c    |  94 ++-
 .../target/iscsi/iscsi_target_parameters.c    | 354 ++++------
 .../target/iscsi/iscsi_target_seq_pdu_list.c  |  86 +--
 drivers/target/iscsi/iscsi_target_stat.c      |   2 +
 drivers/target/iscsi/iscsi_target_tmr.c       | 121 ++--
 drivers/target/iscsi/iscsi_target_tpg.c       | 172 +++--
 drivers/target/iscsi/iscsi_target_transport.c |   6 +-
 drivers/target/iscsi/iscsi_target_util.c      | 133 ++--
 drivers/target/loopback/tcm_loop.c            | 114 ++--
 drivers/target/sbp/sbp_target.c               | 156 +++--
 drivers/target/target_core_alua.c             | 192 +++---
 drivers/target/target_core_configfs.c         | 533 ++++++---------
 drivers/target/target_core_device.c           |  99 +--
 drivers/target/target_core_fabric_configfs.c  |  65 +-
 drivers/target/target_core_fabric_lib.c       |  27 +-
 drivers/target/target_core_file.c             | 126 ++--
 drivers/target/target_core_hba.c              |  15 +-
 drivers/target/target_core_iblock.c           |  96 ++-
 drivers/target/target_core_pr.c               | 635 ++++++++----------
 drivers/target/target_core_pscsi.c            | 160 ++---
 drivers/target/target_core_rd.c               |  87 ++-
 drivers/target/target_core_sbc.c              | 159 ++---
 drivers/target/target_core_spc.c              |  32 +-
 drivers/target/target_core_stat.c             |   4 +-
 drivers/target/target_core_tmr.c              |  57 +-
 drivers/target/target_core_tpg.c              |  81 +--
 drivers/target/target_core_transport.c        | 226 +++----
 drivers/target/target_core_ua.c               |  27 +-
 drivers/target/target_core_user.c             | 257 ++++---
 drivers/target/target_core_xcopy.c            | 228 +++----
 drivers/target/tcm_fc/tcm_fc.h                |   2 +-
 drivers/target/tcm_fc/tfc_cmd.c               |  47 +-
 drivers/target/tcm_fc/tfc_conf.c              |  18 +-
 drivers/target/tcm_fc/tfc_io.c                |  22 +-
 drivers/target/tcm_fc/tfc_sess.c              |   7 +-
 drivers/target/tcm_remote/tcm_remote.c        |  41 +-
 include/target/target_core_base.h             |  92 +++
 51 files changed, 2847 insertions(+), 3606 deletions(-)

-- 
2.40.3


             reply	other threads:[~2024-11-29 15:40 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-11-29 15:30 Anastasia Kovaleva [this message]
2024-11-29 15:30 ` [PATCH 01/10] target: core: Improve SCSI target logs Anastasia Kovaleva
2024-11-29 15:30 ` [PATCH 02/10] target: Use new log wrappers Anastasia Kovaleva
2024-11-29 15:30 ` [PATCH 03/10] target: Fix quoted strings splitting Anastasia Kovaleva
2024-11-29 15:30 ` [PATCH 04/10] target: Fix some obvious style problems Anastasia Kovaleva
2024-11-29 15:30 ` [PATCH 05/10] target: Add log prefix to all tcm files Anastasia Kovaleva
2024-11-29 15:30 ` [PATCH 06/10] target: Remove old prefixes from log messages Anastasia Kovaleva
2024-11-29 15:30 ` [PATCH 07/10] target: core: Extend CaW logging Anastasia Kovaleva
2024-11-29 15:30 ` [PATCH 08/10] target: core: Use extended logs where possible Anastasia Kovaleva
2024-11-29 21:05   ` kernel test robot
2024-11-29 22:08   ` kernel test robot
2024-11-29 15:30 ` [PATCH 09/10] target: Use __func__ in logs Anastasia Kovaleva
2024-11-29 15:30 ` [PATCH 10/10] target: iscsi: Improve a log message Anastasia Kovaleva

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=20241129153056.6985-1-a.kovaleva@yadro.com \
    --to=a.kovaleva@yadro.com \
    --cc=bootc@bootc.net \
    --cc=bostroesser@gmail.com \
    --cc=linux-scsi@vger.kernel.org \
    --cc=linux@yadro.com \
    --cc=martin.petersen@oracle.com \
    --cc=michael.christie@oracle.com \
    --cc=target-devel@vger.kernel.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