[SCSI] zfcp: Remove scsi_cmnd->serial_number from debug traces
authorChristof Schmitt <christof.schmitt@de.ibm.com>
Tue, 28 Sep 2010 08:11:06 +0000 (10:11 +0200)
committerJames Bottomley <James.Bottomley@suse.de>
Thu, 7 Oct 2010 22:25:26 +0000 (17:25 -0500)
With the change that drivers have to explicitly request the serial
number for SCSI commands, this field should not be part of the zfcp
traces. It is not worth the effort to request the serial number only
for tracing purposes, so simply remove this field from the debug
traces.

Reviewed-by: Swen Schillig <swen@vnet.ibm.com>
Signed-off-by: Christof Schmitt <christof.schmitt@de.ibm.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
drivers/s390/scsi/zfcp_dbf.c
drivers/s390/scsi/zfcp_dbf.h

index 2224caa8b92d9faf0199f45c49f3faebc14c706b..2cdd6b28ff7fd1f006663277301f5cbc0e49271a 100644 (file)
@@ -154,7 +154,6 @@ void _zfcp_dbf_hba_fsf_response(const char *tag2, int level,
                scsi_cmnd = (struct scsi_cmnd *)fsf_req->data;
                if (scsi_cmnd) {
                        response->u.fcp.cmnd = (unsigned long)scsi_cmnd;
-                       response->u.fcp.serial = scsi_cmnd->serial_number;
                        response->u.fcp.data_dir =
                                qtcb->bottom.io.data_direction;
                }
@@ -330,7 +329,6 @@ static void zfcp_dbf_hba_view_response(char **p,
                        break;
                zfcp_dbf_out(p, "data_direction", "0x%04x", r->u.fcp.data_dir);
                zfcp_dbf_out(p, "scsi_cmnd", "0x%0Lx", r->u.fcp.cmnd);
-               zfcp_dbf_out(p, "scsi_serial", "0x%016Lx", r->u.fcp.serial);
                *p += sprintf(*p, "\n");
                break;
 
@@ -881,7 +879,6 @@ void _zfcp_dbf_scsi(const char *tag, const char *tag2, int level,
                                }
                                rec->scsi_result = scsi_cmnd->result;
                                rec->scsi_cmnd = (unsigned long)scsi_cmnd;
-                               rec->scsi_serial = scsi_cmnd->serial_number;
                                memcpy(rec->scsi_opcode, scsi_cmnd->cmnd,
                                        min((int)scsi_cmnd->cmd_len,
                                                ZFCP_DBF_SCSI_OPCODE));
@@ -950,7 +947,6 @@ static int zfcp_dbf_scsi_view_format(debug_info_t *id, struct debug_view *view,
        zfcp_dbf_out(&p, "scsi_lun", "0x%08x", r->scsi_lun);
        zfcp_dbf_out(&p, "scsi_result", "0x%08x", r->scsi_result);
        zfcp_dbf_out(&p, "scsi_cmnd", "0x%0Lx", r->scsi_cmnd);
-       zfcp_dbf_out(&p, "scsi_serial", "0x%016Lx", r->scsi_serial);
        zfcp_dbf_outd(&p, "scsi_opcode", r->scsi_opcode, ZFCP_DBF_SCSI_OPCODE,
                      0, ZFCP_DBF_SCSI_OPCODE);
        zfcp_dbf_out(&p, "scsi_retries", "0x%02x", r->scsi_retries);
index 6a48c197b45d3fb72582d6c2f9f611d0542e0a4f..04081b1b62b4b1568ecad4fa0c05b0e917f36bb6 100644 (file)
@@ -110,7 +110,6 @@ struct zfcp_dbf_hba_record_response {
        union {
                struct {
                        u64 cmnd;
-                       u64 serial;
                        u32 data_dir;
                } fcp;
                struct {
@@ -206,7 +205,6 @@ struct zfcp_dbf_scsi_record {
        u32 scsi_lun;
        u32 scsi_result;
        u64 scsi_cmnd;
-       u64 scsi_serial;
 #define ZFCP_DBF_SCSI_OPCODE   16
        u8 scsi_opcode[ZFCP_DBF_SCSI_OPCODE];
        u8 scsi_retries;