Skip to content

Commit

Permalink
[SCSI] libfc: Remove the FC_EM_DBG macro
Browse files Browse the repository at this point in the history
Currently there is a 1:1 relationship between the lport
and exchange manager. This macro takes an EM as an argument
and determines the lport from it. However, later patches
will use an EM list per lport, so we will no longer have
this 1:1 relationship- this macro must change.

The FC_EM_DBG macro is rarely used. There are four callers,
two can use FC_LPORT_DBG instead and two can be removed
since they're not necessary. This patch makes those changes
and removes the macro.

Signed-off-by: Robert Love <robert.w.love@intel.com>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
  • Loading branch information
Robert Love authored and James Bottomley committed Aug 22, 2009
1 parent 9631609 commit d459b7e
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 13 deletions.
13 changes: 6 additions & 7 deletions drivers/scsi/libfc/fc_exch.c
Original file line number Diff line number Diff line change
Expand Up @@ -1129,7 +1129,7 @@ static void fc_exch_recv_req(struct fc_lport *lp, struct fc_exch_mgr *mp,
lp->tt.lport_recv(lp, sp, fp);
fc_exch_release(ep); /* release from lookup */
} else {
FC_EM_DBG(mp, "exch/seq lookup failed: reject %x\n", reject);
FC_LPORT_DBG(lp, "exch/seq lookup failed: reject %x\n", reject);
fc_frame_free(fp);
}
}
Expand Down Expand Up @@ -1235,13 +1235,12 @@ static void fc_exch_recv_resp(struct fc_exch_mgr *mp, struct fc_frame *fp)
struct fc_seq *sp;

sp = fc_seq_lookup_orig(mp, fp); /* doesn't hold sequence */
if (!sp) {

if (!sp)
atomic_inc(&mp->stats.xid_not_found);
FC_EM_DBG(mp, "seq lookup failed\n");
} else {
else
atomic_inc(&mp->stats.non_bls_resp);
FC_EM_DBG(mp, "non-BLS response to sequence");
}

fc_frame_free(fp);
}

Expand Down Expand Up @@ -1950,7 +1949,7 @@ void fc_exch_recv(struct fc_lport *lp, struct fc_exch_mgr *mp,
fc_exch_recv_req(lp, mp, fp);
break;
default:
FC_EM_DBG(mp, "dropping invalid frame (eof %x)", fr_eof(fp));
FC_LPORT_DBG(lp, "dropping invalid frame (eof %x)", fr_eof(fp));
fc_frame_free(fp);
break;
}
Expand Down
6 changes: 0 additions & 6 deletions include/scsi/libfc.h
Original file line number Diff line number Diff line change
Expand Up @@ -88,12 +88,6 @@ do { \
(pkt)->lp->host->host_no, \
pkt->rport->port_id, ##args))

#define FC_EM_DBG(em, fmt, args...) \
FC_CHECK_LOGGING(FC_EM_LOGGING, \
printk(KERN_INFO "host%u: em: " fmt, \
(em)->lp->host->host_no, \
##args))

#define FC_EXCH_DBG(exch, fmt, args...) \
FC_CHECK_LOGGING(FC_EXCH_LOGGING, \
printk(KERN_INFO "host%u: xid %4x: " fmt, \
Expand Down

0 comments on commit d459b7e

Please sign in to comment.