mbox series

[v2,0/5] libsas: Some logging tidy-up

Message ID 1542277232-135789-1-git-send-email-john.garry@huawei.com (mailing list archive)
Headers show
Series libsas: Some logging tidy-up | expand

Message

John Garry Nov. 15, 2018, 10:20 a.m. UTC
This patchset presents some logging tidy-up, namely removing the printk
wrappers and using pr_XXX() instead.

In addition, some log levels are revised, as previous levels (generally
debug) were too low.

Finally a referece to "pci" is removed from a log, and unreferenced dump
files are deleted.

Differences v1->v2:
- Reunited strings to a single line
- fixed "sas: sas:" prefix in messages

John Garry (5):
  scsi: libsas: Delete sas_dump.{c, h}
  scsi: libsas: Use pr_fmt(fmt)
  scsi: libsas: Drop sas_printk()
  scsi: libsas: Drop SAS_DPRINTK() and revise logs levels
  scsi: libsas: Remove pcidev reference

 drivers/scsi/libsas/Makefile        |   3 +-
 drivers/scsi/libsas/sas_ata.c       |  26 ++--
 drivers/scsi/libsas/sas_discover.c  |  33 +++--
 drivers/scsi/libsas/sas_dump.c      |  63 ----------
 drivers/scsi/libsas/sas_dump.h      |  29 -----
 drivers/scsi/libsas/sas_event.c     |   1 -
 drivers/scsi/libsas/sas_expander.c  | 236 +++++++++++++++++-------------------
 drivers/scsi/libsas/sas_init.c      |  10 +-
 drivers/scsi/libsas/sas_internal.h  |  16 ++-
 drivers/scsi/libsas/sas_phy.c       |   8 +-
 drivers/scsi/libsas/sas_port.c      |  23 ++--
 drivers/scsi/libsas/sas_scsi_host.c | 123 +++++++++----------
 drivers/scsi/libsas/sas_task.c      |  10 +-
 13 files changed, 233 insertions(+), 348 deletions(-)
 delete mode 100644 drivers/scsi/libsas/sas_dump.c
 delete mode 100644 drivers/scsi/libsas/sas_dump.h

Comments

Joe Perches Nov. 15, 2018, 12:34 p.m. UTC | #1
On Thu, 2018-11-15 at 18:20 +0800, John Garry wrote:
> This patchset presents some logging tidy-up, namely removing the printk
> wrappers and using pr_XXX() instead.
> 
> In addition, some log levels are revised, as previous levels (generally
> debug) were too low.

Seems sensible, thanks.
Martin K. Petersen Nov. 15, 2018, 7:40 p.m. UTC | #2
John,

> This patchset presents some logging tidy-up, namely removing the
> printk wrappers and using pr_XXX() instead.
>
> In addition, some log levels are revised, as previous levels
> (generally debug) were too low.
>
> Finally a referece to "pci" is removed from a log, and unreferenced
> dump files are deleted.

Applied to 4.21/scsi-queue. Thanks!