scsi logging update (the boring part)
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v8 Latest27 patches 11 repliesIn review
- v7 posting 2/227 patches 5 repliesSuperseded
- v7 posting 1/226 patches 3 repliesSuperseded
Earlier versions · 6
| Version | Posted | Patches | Replies | Status |
|---|---|---|---|---|
| v6 | 26 patches | 2 replies | Superseded | |
| v5 | 24 patches | 23 replies | Superseded | |
| v4 | 23 patches | 14 replies | Superseded | |
| v3 | 38 patches | 13 replies | Applied | |
| v2 | 20 patches | 28 replies | Superseded | |
| v1 | 22 patches | 40 replies | Superseded |
Patches in v8 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/27 | Remove scsi_cmd_print_sense_hdr()drivers/ include/ | +0 | 0 replies | |
| 2/27 | sd: Remove scsi_print_sense() in sd_done()drivers/ | +0 | 0 replies | |
| 3/27 | aha152x: Debug output update and whitespace cleanupdrivers/ | +224 | 0 replies | |
| 4/27 | scsi: introduce sdev_prefix_printk()drivers/ drivers/ drivers/ +3 more | +17 | 0 replies | |
| 5/27 | scsi: Use sdev as argument for sense code printingdrivers/ drivers/ drivers/ +13 more | +99 | 0 replies | |
| 6/27 | acornscsi: use scsi_print_command()drivers/ | +6 | 0 replies | |
| 7/27 | fas216: Return DID_ERROR for incomplete data transferdrivers/ | +1 | 0 replies | |
| 8/27 | fas216: Update logging messagesdrivers/ | +11 | 0 replies | |
| 9/27 | 53c700: remove scsi_print_sense() usagedrivers/ | +2 | 0 replies | |
| 10/27 | scsi: stop decoding if scsi_normalize_sense() failsdrivers/ | +16 | 0 replies | |
| 11/27 | scsi: do not decode sense extrasdrivers/ | +0 | 0 replies | |
| 12/27 | scsi: use 'bool' as return value for scsi_normalize_sense()drivers/ drivers/ include/ | +16 | 0 replies | |
| 13/27 | scsi: remove scsi_print_status()drivers/ include/ | +0 | 0 replies | |
| 14/27 | Implement scsi_opcode_sa_namedrivers/ | +53 | 0 replies | |
| 15/27 | scsi: merge print_opcode_name()drivers/ | +33 | 0 replies | |
| 16/27 | scsi: consolidate opcode lookup in scsi_opcode_sa_name()drivers/ | +22 | 0 replies | |
| 17/27 | scsi: repurpose the last argument from print_opcode_name()drivers/ drivers/ drivers/ +2 more | +27 | 0 replies | |
| 18/27 | scsi: Remove scsi_print_command when calling abortdrivers/ drivers/ drivers/ +4 more | +11 | 0 replies | |
| 19/27 | scsi: separate out scsi_(host|driver)byte_string()drivers/ include/ | +46 | 0 replies | |
| 20/27 | Remove scsi_show_result()drivers/ drivers/ drivers/ +1 more | +28 | 0 replies | |
| 21/27 | scsi: simplify scsi_log_(send|completion)drivers/ drivers/ drivers/ +1 more | +56 | 0 replies | |
| 22/27 | scsi: fixup logging messages in scsi_error.cdrivers/ drivers/ | +39 | 0 replies | |
| 23/27 | scsi: use shost argument in scsi_eh_prt_fail_statsdrivers/ | +1 | 0 replies | |
| 24/27 | scsi_error: document scsi_try_to_abort_cmddrivers/ | +21 | 0 replies | |
| 25/27 | scsi: check for correct return code in scsi_eh_abort_cmds()drivers/ | +1 | 2 replies | |
| 26/27 | scsi: correct return values for .eh_abort_handler implementationsdrivers/ drivers/ drivers/ +1 more | +7 | 2 replies | |
| 27/27 | scsi: ratelimit I/O error messagesdrivers/ | +20 | 5 replies |
Reviews trailers per patch
26 of 27 patches have a Reviewed-by or Acked-by.
| Patch | Robert Elliott | Christoph Hellwig | Yoshihiro Yunomae |
|---|---|---|---|
| 1/27 | R-b | R-b | |
| 2/27 | R-b | R-b | |
| 3/27 | R-b | R-b | |
| 4/27 | R-b | R-b | |
| 5/27 | R-b | R-b | |
| 6/27 | R-b | R-b | |
| 7/27 | R-b | R-b | |
| 8/27 | R-b | R-b | |
| 9/27 | R-b | R-b | |
| 10/27 | R-b | R-b | |
| 11/27 | R-b | R-b | |
| 12/27 | R-b | R-b | R-b |
| 13/27 | R-b | R-b | |
| 14/27 | R-b | R-b | |
| 15/27 | R-b | R-b | |
| 16/27 | R-b | ||
| 17/27 | R-b | R-b | |
| 18/27 | R-b | R-b | |
| 19/27 | R-b | R-b | |
| 20/27 | R-b | R-b | |
| 21/27 | R-b | R-b | |
| 22/27 | R-b | R-b | |
| 23/27 | R-b | R-b | |
| 24/27 | R-b | R-b | |
| 25/27 | |||
| 26/27 | R-b R-b | ||
| 27/27 | T-b R-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.
Cover letter [PATCHv8 00/27] scsi logging update (the boring part)
here is now the eigth iteration of my scsi logging update. With this I've updated the patch to ratelimit I/O error messages as suggested by Robert Elliott.
Additionally I've updated it to core-for-3.19 and included the missing hunks for ufs as requested by Christoph Hellwig.
Hannes Reinecke (27):
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: Return DID_ERROR for incomplete data transfer
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: repurpose the last argument from print_opcode_name()
scsi: Remove scsi_print_command when calling abort
scsi: separate out scsi_(host|driver)byte_string() Remove scsi_show_result()
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_error: document scsi_try_to_abort_cmd
scsi: check for correct return code in scsi_eh_abort_cmds()
scsi: correct return values for .eh_abort_handler implementations
scsi: ratelimit I/O error messages
drivers/scsi/53c700.c | 13 +- drivers/scsi/NCR5380.c | 7 +- drivers/scsi/aha152x.c | 994 +++++++++----------------------------- drivers/scsi/aha1740.c | 2 +- drivers/scsi/arm/acornscsi.c | 12 +- drivers/scsi/arm/fas216.c | 39 +- drivers/scsi/atari_NCR5380.c | 3 +- drivers/scsi/ch.c | 29 +- drivers/scsi/constants.c | 578 +++++++++------------- drivers/scsi/esas2r/esas2r_main.c | 2 +- drivers/scsi/hosts.c | 4 +- drivers/scsi/megaraid.c | 8 +- drivers/scsi/osst.c | 8 +- drivers/scsi/ps3rom.c | 4 - drivers/scsi/scsi.c | 45 +- drivers/scsi/scsi_error.c | 123 ++--- drivers/scsi/scsi_ioctl.c | 2 +- drivers/scsi/scsi_lib.c | 29 +- drivers/scsi/sd.c | 60 +-- drivers/scsi/sd.h | 6 +- drivers/scsi/sg.c | 6 +- drivers/scsi/sr.h | 3 +- drivers/scsi/sr_ioctl.c | 10 +- drivers/scsi/st.c | 9 +- drivers/scsi/stex.c | 9 +- drivers/scsi/storvsc_drv.c | 3 +- drivers/scsi/sun3_NCR5380.c | 3 +- drivers/scsi/ufs/ufshcd.c | 8 +- include/scsi/scsi_dbg.h | 28 +- include/scsi/scsi_device.h | 9 + include/scsi/scsi_eh.h | 14 +- 31 files changed, 703 insertions(+), 1367 deletions(-)
Signature · 1 line
1.8.5.2