[SCSI] libfc: Remove the FC_EM_DBG macro
authorRobert Love <robert.w.love@intel.com>
Thu, 30 Jul 2009 00:05:05 +0000 (17:05 -0700)
committerJames Bottomley <James.Bottomley@suse.de>
Sat, 22 Aug 2009 22:52:08 +0000 (17:52 -0500)
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>
drivers/scsi/libfc/fc_exch.c
include/scsi/libfc.h

index f1fa2b196e98fa07a51ec9e272ca2df1f313e9af..3ad7f88e7ae30857a521ae8438459cc7df5caa79 100644 (file)
@@ -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);
        }
 }
@@ -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);
 }
 
@@ -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;
        }
index b381b1ca9aecbbeb263f4381f52853d91e0e9b47..f1bde91f98a2fa960f62a652903af7aed3fec8aa 100644 (file)
@@ -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,      \