zfcp fixes and cleanups
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest22 patches 3 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/22 | zfcp: replace zfcp_qdio_sbale_count by sg_nentsdrivers/ drivers/ | +1 | 0 replies | |
| 2/22 | zfcp: use setup_timer instead of init_timerdrivers/ | +2 | 0 replies | |
| 3/22 | zfcp: Remove unneeded linux/miscdevice.h includedrivers/ | +0 | 0 replies | |
| 4/22 | zfcp: convert bool-definitions to use 'true' instead of '1'drivers/ drivers/ | +2 | 0 replies | |
| 5/22 | zfcp: fix queuecommand for scsi_eh commands when DIX enableddrivers/ | +2 | 0 replies | |
| 6/22 | zfcp: add handling for FCP_RESID_OVER to the fcp ingress pathdrivers/ | +5 | 0 replies | |
| 7/22 | zfcp: fix capping of unsuccessful GPN_FT SAN response trace recordsdrivers/ drivers/ | +11 | 0 replies | |
| 8/22 | zfcp: fix passing fsf_req to SCSI trace on TMF to correlate with HBAdrivers/ drivers/ | +8 | 0 replies | |
| 9/22 | zfcp: fix missing trace records for early returns in TMF eh handlersdrivers/ | +6 | 0 replies | |
| 10/22 | zfcp: fix payload with full FCP_RSP IU in SCSI trace recordsdrivers/ | +17 | 0 replies | |
| 11/22 | zfcp: trace HBA FSF response by default on dismiss or timedout late responsedrivers/ | +5 | 0 replies | |
| 12/22 | zfcp: trace high part of "new" 64 bit SCSI LUNdrivers/ drivers/ | +4 | 0 replies | |
| 13/22 | zfcp: more fitting constant for fc_ct_hdr.ct_reason on port scan responsedrivers/ | +2 | 0 replies | |
| 14/22 | zfcp: clarify that we don't need "link" test on failed open portdrivers/ | +3 | 0 replies | |
| 15/22 | zfcp: use common code fcp_cmnd and fcp_resp with union in fsf_qtcb_bottom_iodrivers/ drivers/ drivers/ +1 more | +17 | 0 replies | |
| 16/22 | zfcp: use endianness conversions with common FC(P) struct fieldsdrivers/ drivers/ drivers/ +1 more | +43 | 0 replies | |
| 17/22 | zfcp: fix kernel doc comment typos for struct zfcp_dbf_scsidrivers/ | +3 | 0 replies | |
| 18/22 | zfcp: clean up redundant code with fall through in link down SRB switch casedrivers/ | +0 | 0 replies | |
| 19/22 | zfcp: clean up no longer existent prototype from zfcp API headerdrivers/ | +0 | 0 replies | |
| 20/22 | zfcp: clean up a member of struct zfcp_qdio that was assigned but never useddrivers/ drivers/ | +0 | 0 replies | |
| 21/22 | zfcp: clean up unnecessary module_param_named() with no_auto_port_rescandrivers/ | +1 | 0 replies | |
| 22/22 | zfcp: early returns for traces disabled via leveldrivers/ | +46 | 0 replies |
Reviews trailers per patch
18 of 22 patches have a Reviewed-by or Acked-by.
| Patch | Benjamin Block | Steffen Maier | Jens Remus |
|---|---|---|---|
| 1/22 | |||
| 2/22 | |||
| 3/22 | |||
| 4/22 | |||
| 5/22 | R-b | ||
| 6/22 | R-b | ||
| 7/22 | R-b | ||
| 8/22 | R-b | ||
| 9/22 | R-b | ||
| 10/22 | R-b | ||
| 11/22 | R-b | ||
| 12/22 | R-b | R-b | |
| 13/22 | R-b | ||
| 14/22 | R-b | ||
| 15/22 | R-b | ||
| 16/22 | R-b | ||
| 17/22 | R-b | ||
| 18/22 | R-b | ||
| 19/22 | R-b | ||
| 20/22 | R-b | ||
| 21/22 | R-b | ||
| 22/22 | 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 [PATCH 00/22] zfcp fixes and cleanups
Hi all,
here is a series of (important) fixes and some additional cleanups for the zfcp driver. Our fixes are marked for stable accordingly.
Patches 01 - 04 are cleanups and external patches (also cleanups) that we
& 13 - 22 had queued for quite some time now
Patches 05 - 12 are driver fixes that are all marked for stable as well
The set applies to the fixes branch of James' scsi.git.
Beste Grüße / Best regards,
- Benjamin Block
Benjamin Block (2):
zfcp: convert bool-definitions to use 'true' instead of '1'
zfcp: add handling for FCP_RESID_OVER to the fcp ingress path
Corentin Labbe (1):
zfcp: Remove unneeded linux/miscdevice.h include
LABBE Corentin (1):
zfcp: replace zfcp_qdio_sbale_count by sg_nents
Lukáš Korenčik (1):
zfcp: use setup_timer instead of init_timer
Martin Peschke (4):
zfcp: clean up redundant code with fall through in link down SRB
switch case
zfcp: clean up a member of struct zfcp_qdio that was assigned but
never used
zfcp: clean up unnecessary module_param_named() with
no_auto_port_rescan
zfcp: early returns for traces disabled via level
Steffen Maier (13):
zfcp: fix queuecommand for scsi_eh commands when DIX enabled
zfcp: fix capping of unsuccessful GPN_FT SAN response trace records
zfcp: fix passing fsf_req to SCSI trace on TMF to correlate with HBA
zfcp: fix missing trace records for early returns in TMF eh handlers
zfcp: fix payload with full FCP_RSP IU in SCSI trace records
zfcp: trace HBA FSF response by default on dismiss or timedout late
response
zfcp: trace high part of "new" 64 bit SCSI LUN
zfcp: more fitting constant for fc_ct_hdr.ct_reason on port scan
response
zfcp: clarify that we don't need "link" test on failed open port
zfcp: use common code fcp_cmnd and fcp_resp with union in
fsf_qtcb_bottom_io
zfcp: use endianness conversions with common FC(P) struct fields
zfcp: fix kernel doc comment typos for struct zfcp_dbf_scsi
zfcp: clean up no longer existent prototype from zfcp API header
drivers/s390/scsi/zfcp_aux.c | 1 - drivers/s390/scsi/zfcp_dbf.c | 95 +++++++++++++++++++++++++++++++++++-------- drivers/s390/scsi/zfcp_dbf.h | 25 ++++++++---- drivers/s390/scsi/zfcp_erp.c | 5 +-- drivers/s390/scsi/zfcp_ext.h | 1 - drivers/s390/scsi/zfcp_fc.c | 52 +++++++++++------------ drivers/s390/scsi/zfcp_fc.h | 25 ++++++++---- drivers/s390/scsi/zfcp_fsf.c | 35 ++++++++-------- drivers/s390/scsi/zfcp_fsf.h | 12 ++++-- drivers/s390/scsi/zfcp_qdio.c | 2 +- drivers/s390/scsi/zfcp_qdio.h | 17 -------- drivers/s390/scsi/zfcp_scsi.c | 18 ++++---- 12 files changed, 178 insertions(+), 110 deletions(-)
--
2.11.2