diff options
author | Joe Eykholt <jeykholt@cisco.com> | 2009-08-25 17:01:39 -0400 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2009-09-10 13:07:45 -0400 |
commit | cdbe6dfece038e75214735d56cd06319b9a6df46 (patch) | |
tree | e4356d4feee535ad1456a8852a610598710a4877 /drivers | |
parent | b84c7962653e4d04065d2603f0e1424ee0f455ae (diff) |
[SCSI] libfc: rport debug messages were printing pointer values
Don't print large negative decimal numbers for frame pointers in
the debug messages from fc_rport_error(). Just print 0 if its a
frame pointer, and print the error numbers as positive.
Signed-off-by: Joe Eykholt <jeykholt@cisco.com>
Signed-off-by: Robert Love <robert.w.love@intel.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/scsi/libfc/fc_rport.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/scsi/libfc/fc_rport.c b/drivers/scsi/libfc/fc_rport.c index cbf6c9f233ca..6ecf36d4708c 100644 --- a/drivers/scsi/libfc/fc_rport.c +++ b/drivers/scsi/libfc/fc_rport.c | |||
@@ -436,7 +436,8 @@ static void fc_rport_timeout(struct work_struct *work) | |||
436 | static void fc_rport_error(struct fc_rport_priv *rdata, struct fc_frame *fp) | 436 | static void fc_rport_error(struct fc_rport_priv *rdata, struct fc_frame *fp) |
437 | { | 437 | { |
438 | FC_RPORT_DBG(rdata, "Error %ld in state %s, retries %d\n", | 438 | FC_RPORT_DBG(rdata, "Error %ld in state %s, retries %d\n", |
439 | PTR_ERR(fp), fc_rport_state(rdata), rdata->retries); | 439 | IS_ERR(fp) ? -PTR_ERR(fp) : 0, |
440 | fc_rport_state(rdata), rdata->retries); | ||
440 | 441 | ||
441 | switch (rdata->rp_state) { | 442 | switch (rdata->rp_state) { |
442 | case RPORT_ST_PLOGI: | 443 | case RPORT_ST_PLOGI: |