diff options
author | Andrew Vasquez <andrew.vasquez@qlogic.com> | 2010-05-04 22:01:27 (GMT) |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2010-05-16 22:21:54 (GMT) |
commit | b7d2280c153b33fc60f1a89406d2329137a8b61c (patch) | |
tree | 1b579072959644f7dd520f0dd6bbbec2c981266a /drivers/scsi/qla2xxx/qla_init.c | |
parent | 5ff1d58410ffb160b388d622ef0c6a0411a05559 (diff) | |
download | linux-b7d2280c153b33fc60f1a89406d2329137a8b61c.tar.xz |
[SCSI] qla2xxx: Cleanup FCP-command-status processing debug statements.
Migrate to a consistent set of debug entries during status-IOCB
handling:
* group CS_TIMEOUT handling with CS_PORT_UNAVAILABLE and the like
(more regrouping of common behaviour).
* drop CS_DATA_OVERRUN handling as it now falls into the
'default' case (still returns DID_ERROR).
* consolidate CS_RESET and CS_ABORTED handling, as we the only
functional difference was a printk() (still returns DID_RESET).
* dropped all the earlier inconsistent [qla_]printk()s sprinkled
throught the needlessly large case-statement. Failure case
I/Os are now logged with a 'standard' format:
<command failure details>
<command generic details>
so, for example a dropped-frame is logged as:
qla2xxx 0000:13:00.0: scsi(16:0:0) Dropped frame(s) detected (4000 of 4000 bytes).
qla2xxx 0000:13:00.0: scsi(16:0:0) FCP command status: 0x15-0x18 (70018) \
oxid=78 ser=76 cdb=280000 len=4000 rsp_info=0 resid=0 fw_resid=4000
which should now convey all relevant information.
Signed-off-by: Giridhar Malavali <giridhar.malavali@qlogic.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/scsi/qla2xxx/qla_init.c')
0 files changed, 0 insertions, 0 deletions