]> git.karo-electronics.de Git - karo-tx-linux.git/blobdiff - drivers/infiniband/hw/cxgb4/cm.c
Merge branch 'x86-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel...
[karo-tx-linux.git] / drivers / infiniband / hw / cxgb4 / cm.c
index 03a1b0e64fc3e8d44616a21cb13cf219be926a55..b6fe45924c6ed5a3a835c89f00b9265a940e57ef 100644 (file)
@@ -101,7 +101,7 @@ MODULE_PARM_DESC(enable_tcp_window_scaling,
 
 int c4iw_debug;
 module_param(c4iw_debug, int, 0644);
-MODULE_PARM_DESC(c4iw_debug, "Enable debug logging (default=0)");
+MODULE_PARM_DESC(c4iw_debug, "obsolete");
 
 static int peer2peer = 1;
 module_param(peer2peer, int, 0644);
@@ -180,7 +180,7 @@ static void ref_qp(struct c4iw_ep *ep)
 
 static void start_ep_timer(struct c4iw_ep *ep)
 {
-       PDBG("%s ep %p\n", __func__, ep);
+       pr_debug("%s ep %p\n", __func__, ep);
        if (timer_pending(&ep->timer)) {
                pr_err("%s timer already started! ep %p\n",
                       __func__, ep);
@@ -196,7 +196,7 @@ static void start_ep_timer(struct c4iw_ep *ep)
 
 static int stop_ep_timer(struct c4iw_ep *ep)
 {
-       PDBG("%s ep %p stopping\n", __func__, ep);
+       pr_debug("%s ep %p stopping\n", __func__, ep);
        del_timer_sync(&ep->timer);
        if (!test_and_set_bit(TIMEOUT, &ep->com.flags)) {
                c4iw_put_ep(&ep->com);
@@ -212,7 +212,7 @@ static int c4iw_l2t_send(struct c4iw_rdev *rdev, struct sk_buff *skb,
 
        if (c4iw_fatal_error(rdev)) {
                kfree_skb(skb);
-               PDBG("%s - device in error state - dropping\n", __func__);
+               pr_debug("%s - device in error state - dropping\n", __func__);
                return -EIO;
        }
        error = cxgb4_l2t_send(rdev->lldi.ports[0], skb, l2e);
@@ -229,7 +229,7 @@ int c4iw_ofld_send(struct c4iw_rdev *rdev, struct sk_buff *skb)
 
        if (c4iw_fatal_error(rdev)) {
                kfree_skb(skb);
-               PDBG("%s - device in error state - dropping\n", __func__);
+               pr_debug("%s - device in error state - dropping\n", __func__);
                return -EIO;
        }
        error = cxgb4_ofld_send(rdev->lldi.ports[0], skb);
@@ -263,10 +263,10 @@ static void set_emss(struct c4iw_ep *ep, u16 opt)
        if (ep->emss < 128)
                ep->emss = 128;
        if (ep->emss & 7)
-               PDBG("Warning: misaligned mtu idx %u mss %u emss=%u\n",
-                    TCPOPT_MSS_G(opt), ep->mss, ep->emss);
-       PDBG("%s mss_idx %u mss %u emss=%u\n", __func__, TCPOPT_MSS_G(opt),
-            ep->mss, ep->emss);
+               pr_debug("Warning: misaligned mtu idx %u mss %u emss=%u\n",
+                        TCPOPT_MSS_G(opt), ep->mss, ep->emss);
+       pr_debug("%s mss_idx %u mss %u emss=%u\n", __func__, TCPOPT_MSS_G(opt),
+                ep->mss, ep->emss);
 }
 
 static enum c4iw_ep_state state_read(struct c4iw_ep_common *epc)
@@ -287,7 +287,7 @@ static void __state_set(struct c4iw_ep_common *epc, enum c4iw_ep_state new)
 static void state_set(struct c4iw_ep_common *epc, enum c4iw_ep_state new)
 {
        mutex_lock(&epc->mutex);
-       PDBG("%s - %s -> %s\n", __func__, states[epc->state], states[new]);
+       pr_debug("%s - %s -> %s\n", __func__, states[epc->state], states[new]);
        __state_set(epc, new);
        mutex_unlock(&epc->mutex);
        return;
@@ -322,7 +322,7 @@ static void *alloc_ep(int size, gfp_t gfp)
                mutex_init(&epc->mutex);
                c4iw_init_wr_wait(&epc->wr_wait);
        }
-       PDBG("%s alloc ep %p\n", __func__, epc);
+       pr_debug("%s alloc ep %p\n", __func__, epc);
        return epc;
 }
 
@@ -384,7 +384,7 @@ void _c4iw_free_ep(struct kref *kref)
        struct c4iw_ep *ep;
 
        ep = container_of(kref, struct c4iw_ep, com.kref);
-       PDBG("%s ep %p state %s\n", __func__, ep, states[ep->com.state]);
+       pr_debug("%s ep %p state %s\n", __func__, ep, states[ep->com.state]);
        if (test_bit(QP_REFERENCED, &ep->com.flags))
                deref_qp(ep);
        if (test_bit(RELEASE_RESOURCES, &ep->com.flags)) {
@@ -467,7 +467,7 @@ static struct net_device *get_real_dev(struct net_device *egress_dev)
 
 static void arp_failure_discard(void *handle, struct sk_buff *skb)
 {
-       pr_err(MOD "ARP failure\n");
+       pr_err("ARP failure\n");
        kfree_skb(skb);
 }
 
@@ -528,7 +528,7 @@ static void pass_accept_rpl_arp_failure(void *handle, struct sk_buff *skb)
 {
        struct c4iw_ep *ep = handle;
 
-       pr_err(MOD "ARP failure during accept - tid %u -dropping connection\n",
+       pr_err("ARP failure during accept - tid %u - dropping connection\n",
               ep->hwtid);
 
        __state_set(&ep->com, DEAD);
@@ -542,7 +542,7 @@ static void act_open_req_arp_failure(void *handle, struct sk_buff *skb)
 {
        struct c4iw_ep *ep = handle;
 
-       printk(KERN_ERR MOD "ARP failure during connect\n");
+       pr_err("ARP failure during connect\n");
        connect_reply_upcall(ep, -EHOSTUNREACH);
        __state_set(&ep->com, DEAD);
        if (ep->com.remote_addr.ss_family == AF_INET6) {
@@ -567,7 +567,7 @@ static void abort_arp_failure(void *handle, struct sk_buff *skb)
        struct c4iw_rdev *rdev = &ep->com.dev->rdev;
        struct cpl_abort_req *req = cplhdr(skb);
 
-       PDBG("%s rdev %p\n", __func__, rdev);
+       pr_debug("%s rdev %p\n", __func__, rdev);
        req->cmd = CPL_ABORT_NO_RST;
        ret = c4iw_ofld_send(rdev, skb);
        if (ret) {
@@ -642,7 +642,7 @@ static int send_halfclose(struct c4iw_ep *ep)
        struct sk_buff *skb = skb_dequeue(&ep->com.ep_skb_list);
        u32 wrlen = roundup(sizeof(struct cpl_close_con_req), 16);
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        if (WARN_ON(!skb))
                return -ENOMEM;
 
@@ -657,7 +657,7 @@ static int send_abort(struct c4iw_ep *ep)
        u32 wrlen = roundup(sizeof(struct cpl_abort_req), 16);
        struct sk_buff *req_skb = skb_dequeue(&ep->com.ep_skb_list);
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        if (WARN_ON(!req_skb))
                return -ENOMEM;
 
@@ -720,12 +720,11 @@ static int send_connect(struct c4iw_ep *ep)
                        roundup(sizev4, 16) :
                        roundup(sizev6, 16);
 
-       PDBG("%s ep %p atid %u\n", __func__, ep, ep->atid);
+       pr_debug("%s ep %p atid %u\n", __func__, ep, ep->atid);
 
        skb = get_skb(NULL, wrlen, GFP_KERNEL);
        if (!skb) {
-               printk(KERN_ERR MOD "%s - failed to alloc skb.\n",
-                      __func__);
+               pr_err("%s - failed to alloc skb\n", __func__);
                return -ENOMEM;
        }
        set_wr_txq(skb, CPL_PRIORITY_SETUP, ep->ctrlq_idx);
@@ -822,13 +821,13 @@ static int send_connect(struct c4iw_ep *ep)
                                t5req->params =
                                          cpu_to_be64(FILTER_TUPLE_V(params));
                                t5req->rsvd = cpu_to_be32(isn);
-                               PDBG("%s snd_isn %u\n", __func__, t5req->rsvd);
+                       pr_debug("%s snd_isn %u\n", __func__, t5req->rsvd);
                                t5req->opt2 = cpu_to_be32(opt2);
                        } else {
                                t6req->params =
                                          cpu_to_be64(FILTER_TUPLE_V(params));
                                t6req->rsvd = cpu_to_be32(isn);
-                               PDBG("%s snd_isn %u\n", __func__, t6req->rsvd);
+                       pr_debug("%s snd_isn %u\n", __func__, t6req->rsvd);
                                t6req->opt2 = cpu_to_be32(opt2);
                        }
                }
@@ -877,13 +876,13 @@ static int send_connect(struct c4iw_ep *ep)
                                t5req6->params =
                                            cpu_to_be64(FILTER_TUPLE_V(params));
                                t5req6->rsvd = cpu_to_be32(isn);
-                               PDBG("%s snd_isn %u\n", __func__, t5req6->rsvd);
+                       pr_debug("%s snd_isn %u\n", __func__, t5req6->rsvd);
                                t5req6->opt2 = cpu_to_be32(opt2);
                        } else {
                                t6req6->params =
                                            cpu_to_be64(FILTER_TUPLE_V(params));
                                t6req6->rsvd = cpu_to_be32(isn);
-                               PDBG("%s snd_isn %u\n", __func__, t6req6->rsvd);
+                       pr_debug("%s snd_isn %u\n", __func__, t6req6->rsvd);
                                t6req6->opt2 = cpu_to_be32(opt2);
                        }
 
@@ -907,7 +906,8 @@ static int send_mpa_req(struct c4iw_ep *ep, struct sk_buff *skb,
        struct mpa_message *mpa;
        struct mpa_v2_conn_params mpa_v2_params;
 
-       PDBG("%s ep %p tid %u pd_len %d\n", __func__, ep, ep->hwtid, ep->plen);
+       pr_debug("%s ep %p tid %u pd_len %d\n",
+                __func__, ep, ep->hwtid, ep->plen);
 
        BUG_ON(skb_cloned(skb));
 
@@ -961,8 +961,8 @@ static int send_mpa_req(struct c4iw_ep *ep, struct sk_buff *skb,
        if (mpa_rev_to_use == 2) {
                mpa->private_data_size = htons(ntohs(mpa->private_data_size) +
                                               sizeof (struct mpa_v2_conn_params));
-               PDBG("%s initiator ird %u ord %u\n", __func__, ep->ird,
-                    ep->ord);
+               pr_debug("%s initiator ird %u ord %u\n", __func__, ep->ird,
+                        ep->ord);
                mpa_v2_params.ird = htons((u16)ep->ird);
                mpa_v2_params.ord = htons((u16)ep->ord);
 
@@ -1014,7 +1014,8 @@ static int send_mpa_reject(struct c4iw_ep *ep, const void *pdata, u8 plen)
        struct sk_buff *skb;
        struct mpa_v2_conn_params mpa_v2_params;
 
-       PDBG("%s ep %p tid %u pd_len %d\n", __func__, ep, ep->hwtid, ep->plen);
+       pr_debug("%s ep %p tid %u pd_len %d\n",
+                __func__, ep, ep->hwtid, ep->plen);
 
        mpalen = sizeof(*mpa) + plen;
        if (ep->mpa_attr.version == 2 && ep->mpa_attr.enhanced_rdma_conn)
@@ -1023,7 +1024,7 @@ static int send_mpa_reject(struct c4iw_ep *ep, const void *pdata, u8 plen)
 
        skb = get_skb(NULL, wrlen, GFP_KERNEL);
        if (!skb) {
-               printk(KERN_ERR MOD "%s - cannot alloc skb!\n", __func__);
+               pr_err("%s - cannot alloc skb!\n", __func__);
                return -ENOMEM;
        }
        set_wr_txq(skb, CPL_PRIORITY_DATA, ep->txq_idx);
@@ -1094,7 +1095,8 @@ static int send_mpa_reply(struct c4iw_ep *ep, const void *pdata, u8 plen)
        struct sk_buff *skb;
        struct mpa_v2_conn_params mpa_v2_params;
 
-       PDBG("%s ep %p tid %u pd_len %d\n", __func__, ep, ep->hwtid, ep->plen);
+       pr_debug("%s ep %p tid %u pd_len %d\n",
+                __func__, ep, ep->hwtid, ep->plen);
 
        mpalen = sizeof(*mpa) + plen;
        if (ep->mpa_attr.version == 2 && ep->mpa_attr.enhanced_rdma_conn)
@@ -1103,7 +1105,7 @@ static int send_mpa_reply(struct c4iw_ep *ep, const void *pdata, u8 plen)
 
        skb = get_skb(NULL, wrlen, GFP_KERNEL);
        if (!skb) {
-               printk(KERN_ERR MOD "%s - cannot alloc skb!\n", __func__);
+               pr_err("%s - cannot alloc skb!\n", __func__);
                return -ENOMEM;
        }
        set_wr_txq(skb, CPL_PRIORITY_DATA, ep->txq_idx);
@@ -1185,8 +1187,8 @@ static int act_establish(struct c4iw_dev *dev, struct sk_buff *skb)
 
        ep = lookup_atid(t, atid);
 
-       PDBG("%s ep %p tid %u snd_isn %u rcv_isn %u\n", __func__, ep, tid,
-            be32_to_cpu(req->snd_isn), be32_to_cpu(req->rcv_isn));
+       pr_debug("%s ep %p tid %u snd_isn %u rcv_isn %u\n", __func__, ep, tid,
+                be32_to_cpu(req->snd_isn), be32_to_cpu(req->rcv_isn));
 
        mutex_lock(&ep->com.mutex);
        dst_confirm(ep->dst);
@@ -1229,13 +1231,13 @@ static void close_complete_upcall(struct c4iw_ep *ep, int status)
 {
        struct iw_cm_event event;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        memset(&event, 0, sizeof(event));
        event.event = IW_CM_EVENT_CLOSE;
        event.status = status;
        if (ep->com.cm_id) {
-               PDBG("close complete delivered ep %p cm_id %p tid %u\n",
-                    ep, ep->com.cm_id, ep->hwtid);
+               pr_debug("close complete delivered ep %p cm_id %p tid %u\n",
+                        ep, ep->com.cm_id, ep->hwtid);
                ep->com.cm_id->event_handler(ep->com.cm_id, &event);
                deref_cm_id(&ep->com);
                set_bit(CLOSE_UPCALL, &ep->com.history);
@@ -1246,12 +1248,12 @@ static void peer_close_upcall(struct c4iw_ep *ep)
 {
        struct iw_cm_event event;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        memset(&event, 0, sizeof(event));
        event.event = IW_CM_EVENT_DISCONNECT;
        if (ep->com.cm_id) {
-               PDBG("peer close delivered ep %p cm_id %p tid %u\n",
-                    ep, ep->com.cm_id, ep->hwtid);
+               pr_debug("peer close delivered ep %p cm_id %p tid %u\n",
+                        ep, ep->com.cm_id, ep->hwtid);
                ep->com.cm_id->event_handler(ep->com.cm_id, &event);
                set_bit(DISCONN_UPCALL, &ep->com.history);
        }
@@ -1261,13 +1263,13 @@ static void peer_abort_upcall(struct c4iw_ep *ep)
 {
        struct iw_cm_event event;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        memset(&event, 0, sizeof(event));
        event.event = IW_CM_EVENT_CLOSE;
        event.status = -ECONNRESET;
        if (ep->com.cm_id) {
-               PDBG("abort delivered ep %p cm_id %p tid %u\n", ep,
-                    ep->com.cm_id, ep->hwtid);
+               pr_debug("abort delivered ep %p cm_id %p tid %u\n", ep,
+                        ep->com.cm_id, ep->hwtid);
                ep->com.cm_id->event_handler(ep->com.cm_id, &event);
                deref_cm_id(&ep->com);
                set_bit(ABORT_UPCALL, &ep->com.history);
@@ -1278,7 +1280,8 @@ static void connect_reply_upcall(struct c4iw_ep *ep, int status)
 {
        struct iw_cm_event event;
 
-       PDBG("%s ep %p tid %u status %d\n", __func__, ep, ep->hwtid, status);
+       pr_debug("%s ep %p tid %u status %d\n",
+                __func__, ep, ep->hwtid, status);
        memset(&event, 0, sizeof(event));
        event.event = IW_CM_EVENT_CONNECT_REPLY;
        event.status = status;
@@ -1307,8 +1310,8 @@ static void connect_reply_upcall(struct c4iw_ep *ep, int status)
                }
        }
 
-       PDBG("%s ep %p tid %u status %d\n", __func__, ep,
-            ep->hwtid, status);
+       pr_debug("%s ep %p tid %u status %d\n", __func__, ep,
+                ep->hwtid, status);
        set_bit(CONN_RPL_UPCALL, &ep->com.history);
        ep->com.cm_id->event_handler(ep->com.cm_id, &event);
 
@@ -1321,7 +1324,7 @@ static int connect_request_upcall(struct c4iw_ep *ep)
        struct iw_cm_event event;
        int ret;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        memset(&event, 0, sizeof(event));
        event.event = IW_CM_EVENT_CONNECT_REQUEST;
        memcpy(&event.local_addr, &ep->com.local_addr,
@@ -1358,13 +1361,13 @@ static void established_upcall(struct c4iw_ep *ep)
 {
        struct iw_cm_event event;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        memset(&event, 0, sizeof(event));
        event.event = IW_CM_EVENT_ESTABLISHED;
        event.ird = ep->ord;
        event.ord = ep->ird;
        if (ep->com.cm_id) {
-               PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+               pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
                ep->com.cm_id->event_handler(ep->com.cm_id, &event);
                set_bit(ESTAB_UPCALL, &ep->com.history);
        }
@@ -1376,10 +1379,11 @@ static int update_rx_credits(struct c4iw_ep *ep, u32 credits)
        u32 wrlen = roundup(sizeof(struct cpl_rx_data_ack), 16);
        u32 credit_dack;
 
-       PDBG("%s ep %p tid %u credits %u\n", __func__, ep, ep->hwtid, credits);
+       pr_debug("%s ep %p tid %u credits %u\n",
+                __func__, ep, ep->hwtid, credits);
        skb = get_skb(NULL, wrlen, GFP_KERNEL);
        if (!skb) {
-               printk(KERN_ERR MOD "update_rx_credits - cannot alloc skb!\n");
+               pr_err("update_rx_credits - cannot alloc skb!\n");
                return 0;
        }
 
@@ -1427,7 +1431,7 @@ static int process_mpa_reply(struct c4iw_ep *ep, struct sk_buff *skb)
        int err;
        int disconnect = 0;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
 
        /*
         * If we get more than the supported amount of private data
@@ -1454,8 +1458,8 @@ static int process_mpa_reply(struct c4iw_ep *ep, struct sk_buff *skb)
 
        /* Validate MPA header. */
        if (mpa->revision > mpa_rev) {
-               printk(KERN_ERR MOD "%s MPA version mismatch. Local = %d,"
-                      " Received = %d\n", __func__, mpa_rev, mpa->revision);
+               pr_err("%s MPA version mismatch. Local = %d, Received = %d\n",
+                      __func__, mpa_rev, mpa->revision);
                err = -EPROTO;
                goto err_stop_timer;
        }
@@ -1525,8 +1529,9 @@ static int process_mpa_reply(struct c4iw_ep *ep, struct sk_buff *skb)
                                MPA_V2_IRD_ORD_MASK;
                        resp_ord = ntohs(mpa_v2_params->ord) &
                                MPA_V2_IRD_ORD_MASK;
-                       PDBG("%s responder ird %u ord %u ep ird %u ord %u\n",
-                            __func__, resp_ird, resp_ord, ep->ird, ep->ord);
+                       pr_debug("%s responder ird %u ord %u ep ird %u ord %u\n",
+                                __func__,
+                                resp_ird, resp_ord, ep->ird, ep->ord);
 
                        /*
                         * This is a double-check. Ideally, below checks are
@@ -1570,12 +1575,11 @@ static int process_mpa_reply(struct c4iw_ep *ep, struct sk_buff *skb)
                if (peer2peer)
                        ep->mpa_attr.p2p_type = p2p_type;
 
-       PDBG("%s - crc_enabled=%d, recv_marker_enabled=%d, "
-            "xmit_marker_enabled=%d, version=%d p2p_type=%d local-p2p_type = "
-            "%d\n", __func__, ep->mpa_attr.crc_enabled,
-            ep->mpa_attr.recv_marker_enabled,
-            ep->mpa_attr.xmit_marker_enabled, ep->mpa_attr.version,
-            ep->mpa_attr.p2p_type, p2p_type);
+       pr_debug("%s - crc_enabled=%d, recv_marker_enabled=%d, xmit_marker_enabled=%d, version=%d p2p_type=%d local-p2p_type = %d\n",
+                __func__, ep->mpa_attr.crc_enabled,
+                ep->mpa_attr.recv_marker_enabled,
+                ep->mpa_attr.xmit_marker_enabled, ep->mpa_attr.version,
+                ep->mpa_attr.p2p_type, p2p_type);
 
        /*
         * If responder's RTR does not match with that of initiator, assign
@@ -1610,7 +1614,7 @@ static int process_mpa_reply(struct c4iw_ep *ep, struct sk_buff *skb)
         * supports, generate TERM message
         */
        if (rtr_mismatch) {
-               printk(KERN_ERR "%s: RTR mismatch, sending TERM\n", __func__);
+               pr_err("%s: RTR mismatch, sending TERM\n", __func__);
                attrs.layer_etype = LAYER_MPA | DDP_LLP;
                attrs.ecode = MPA_NOMATCH_RTR;
                attrs.next_state = C4IW_QP_STATE_TERMINATE;
@@ -1629,8 +1633,7 @@ static int process_mpa_reply(struct c4iw_ep *ep, struct sk_buff *skb)
         * initiator ORD.
         */
        if (insuff_ird) {
-               printk(KERN_ERR "%s: Insufficient IRD, sending TERM\n",
-                               __func__);
+               pr_err("%s: Insufficient IRD, sending TERM\n", __func__);
                attrs.layer_etype = LAYER_MPA | DDP_LLP;
                attrs.ecode = MPA_INSUFF_IRD;
                attrs.next_state = C4IW_QP_STATE_TERMINATE;
@@ -1669,7 +1672,7 @@ static int process_mpa_request(struct c4iw_ep *ep, struct sk_buff *skb)
        struct mpa_v2_conn_params *mpa_v2_params;
        u16 plen;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
 
        /*
         * If we get more than the supported amount of private data
@@ -1678,7 +1681,7 @@ static int process_mpa_request(struct c4iw_ep *ep, struct sk_buff *skb)
        if (ep->mpa_pkt_len + skb->len > sizeof(ep->mpa_pkt))
                goto err_stop_timer;
 
-       PDBG("%s enter (%s line %u)\n", __func__, __FILE__, __LINE__);
+       pr_debug("%s enter (%s line %u)\n", __func__, __FILE__, __LINE__);
 
        /*
         * Copy the new data into our accumulation buffer.
@@ -1694,15 +1697,15 @@ static int process_mpa_request(struct c4iw_ep *ep, struct sk_buff *skb)
        if (ep->mpa_pkt_len < sizeof(*mpa))
                return 0;
 
-       PDBG("%s enter (%s line %u)\n", __func__, __FILE__, __LINE__);
+       pr_debug("%s enter (%s line %u)\n", __func__, __FILE__, __LINE__);
        mpa = (struct mpa_message *) ep->mpa_pkt;
 
        /*
         * Validate MPA Header.
         */
        if (mpa->revision > mpa_rev) {
-               printk(KERN_ERR MOD "%s MPA version mismatch. Local = %d,"
-                      " Received = %d\n", __func__, mpa_rev, mpa->revision);
+               pr_err("%s MPA version mismatch. Local = %d, Received = %d\n",
+                      __func__, mpa_rev, mpa->revision);
                goto err_stop_timer;
        }
 
@@ -1757,8 +1760,8 @@ static int process_mpa_request(struct c4iw_ep *ep, struct sk_buff *skb)
                                MPA_V2_IRD_ORD_MASK;
                        ep->ord = min_t(u32, ep->ord,
                                        cur_max_read_depth(ep->com.dev));
-                       PDBG("%s initiator ird %u ord %u\n", __func__, ep->ird,
-                            ep->ord);
+                       pr_debug("%s initiator ird %u ord %u\n",
+                                __func__, ep->ird, ep->ord);
                        if (ntohs(mpa_v2_params->ird) & MPA_V2_PEER2PEER_MODEL)
                                if (peer2peer) {
                                        if (ntohs(mpa_v2_params->ord) &
@@ -1775,11 +1778,11 @@ static int process_mpa_request(struct c4iw_ep *ep, struct sk_buff *skb)
                if (peer2peer)
                        ep->mpa_attr.p2p_type = p2p_type;
 
-       PDBG("%s - crc_enabled=%d, recv_marker_enabled=%d, "
-            "xmit_marker_enabled=%d, version=%d p2p_type=%d\n", __func__,
-            ep->mpa_attr.crc_enabled, ep->mpa_attr.recv_marker_enabled,
-            ep->mpa_attr.xmit_marker_enabled, ep->mpa_attr.version,
-            ep->mpa_attr.p2p_type);
+       pr_debug("%s - crc_enabled=%d, recv_marker_enabled=%d, xmit_marker_enabled=%d, version=%d p2p_type=%d\n",
+                __func__,
+                ep->mpa_attr.crc_enabled, ep->mpa_attr.recv_marker_enabled,
+                ep->mpa_attr.xmit_marker_enabled, ep->mpa_attr.version,
+                ep->mpa_attr.p2p_type);
 
        __state_set(&ep->com, MPA_REQ_RCVD);
 
@@ -1815,7 +1818,7 @@ static int rx_data(struct c4iw_dev *dev, struct sk_buff *skb)
        ep = get_ep_from_tid(dev, tid);
        if (!ep)
                return 0;
-       PDBG("%s ep %p tid %u dlen %u\n", __func__, ep, ep->hwtid, dlen);
+       pr_debug("%s ep %p tid %u dlen %u\n", __func__, ep, ep->hwtid, dlen);
        skb_pull(skb, sizeof(*hdr));
        skb_trim(skb, dlen);
        mutex_lock(&ep->com.mutex);
@@ -1866,10 +1869,10 @@ static int abort_rpl(struct c4iw_dev *dev, struct sk_buff *skb)
 
        ep = get_ep_from_tid(dev, tid);
        if (!ep) {
-               printk(KERN_WARNING MOD "Abort rpl to freed endpoint\n");
+               pr_warn("Abort rpl to freed endpoint\n");
                return 0;
        }
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        mutex_lock(&ep->com.mutex);
        switch (ep->com.state) {
        case ABORTING:
@@ -1878,8 +1881,7 @@ static int abort_rpl(struct c4iw_dev *dev, struct sk_buff *skb)
                release = 1;
                break;
        default:
-               printk(KERN_ERR "%s ep %p state %d\n",
-                    __func__, ep, ep->com.state);
+               pr_err("%s ep %p state %d\n", __func__, ep, ep->com.state);
                break;
        }
        mutex_unlock(&ep->com.mutex);
@@ -1995,7 +1997,8 @@ static void set_tcp_window(struct c4iw_ep *ep, struct port_info *pi)
 {
        ep->snd_win = snd_win;
        ep->rcv_win = rcv_win;
-       PDBG("%s snd_win %d rcv_win %d\n", __func__, ep->snd_win, ep->rcv_win);
+       pr_debug("%s snd_win %d rcv_win %d\n",
+                __func__, ep->snd_win, ep->rcv_win);
 }
 
 #define ACT_OPEN_RETRY_COUNT 2
@@ -2100,7 +2103,7 @@ static int c4iw_reconnect(struct c4iw_ep *ep)
        int iptype;
        __u8 *ra;
 
-       PDBG("%s qp %p cm_id %p\n", __func__, ep->com.qp, ep->com.cm_id);
+       pr_debug("%s qp %p cm_id %p\n", __func__, ep->com.qp, ep->com.cm_id);
        init_timer(&ep->timer);
        c4iw_init_wr_wait(&ep->com.wr_wait);
 
@@ -2124,7 +2127,7 @@ static int c4iw_reconnect(struct c4iw_ep *ep)
         */
        ep->atid = cxgb4_alloc_atid(ep->com.dev->rdev.lldi.tids, ep);
        if (ep->atid == -1) {
-               pr_err("%s - cannot alloc atid.\n", __func__);
+               pr_err("%s - cannot alloc atid\n", __func__);
                err = -ENOMEM;
                goto fail2;
        }
@@ -2151,7 +2154,7 @@ static int c4iw_reconnect(struct c4iw_ep *ep)
                ra = (__u8 *)&raddr6->sin6_addr;
        }
        if (!ep->dst) {
-               pr_err("%s - cannot find route.\n", __func__);
+               pr_err("%s - cannot find route\n", __func__);
                err = -EHOSTUNREACH;
                goto fail3;
        }
@@ -2159,13 +2162,13 @@ static int c4iw_reconnect(struct c4iw_ep *ep)
                        ep->com.dev->rdev.lldi.adapter_type,
                        ep->com.cm_id->tos);
        if (err) {
-               pr_err("%s - cannot alloc l2e.\n", __func__);
+               pr_err("%s - cannot alloc l2e\n", __func__);
                goto fail4;
        }
 
-       PDBG("%s txq_idx %u tx_chan %u smac_idx %u rss_qid %u l2t_idx %u\n",
-            __func__, ep->txq_idx, ep->tx_chan, ep->smac_idx, ep->rss_qid,
-            ep->l2t->idx);
+       pr_debug("%s txq_idx %u tx_chan %u smac_idx %u rss_qid %u l2t_idx %u\n",
+                __func__, ep->txq_idx, ep->tx_chan, ep->smac_idx, ep->rss_qid,
+                ep->l2t->idx);
 
        state_set(&ep->com, CONNECTING);
        ep->tos = ep->com.cm_id->tos;
@@ -2215,12 +2218,12 @@ static int act_open_rpl(struct c4iw_dev *dev, struct sk_buff *skb)
        la6 = (struct sockaddr_in6 *)&ep->com.local_addr;
        ra6 = (struct sockaddr_in6 *)&ep->com.remote_addr;
 
-       PDBG("%s ep %p atid %u status %u errno %d\n", __func__, ep, atid,
-            status, status2errno(status));
+       pr_debug("%s ep %p atid %u status %u errno %d\n", __func__, ep, atid,
+                status, status2errno(status));
 
        if (cxgb_is_neg_adv(status)) {
-               PDBG("%s Connection problems for atid %u status %u (%s)\n",
-                    __func__, atid, status, neg_adv_str(status));
+               pr_debug("%s Connection problems for atid %u status %u (%s)\n",
+                        __func__, atid, status, neg_adv_str(status));
                ep->stats.connect_neg_adv++;
                mutex_lock(&dev->rdev.stats.lock);
                dev->rdev.stats.neg_adv++;
@@ -2315,11 +2318,11 @@ static int pass_open_rpl(struct c4iw_dev *dev, struct sk_buff *skb)
        struct c4iw_listen_ep *ep = get_ep_from_stid(dev, stid);
 
        if (!ep) {
-               PDBG("%s stid %d lookup failure!\n", __func__, stid);
+               pr_debug("%s stid %d lookup failure!\n", __func__, stid);
                goto out;
        }
-       PDBG("%s ep %p status %d error %d\n", __func__, ep,
-            rpl->status, status2errno(rpl->status));
+       pr_debug("%s ep %p status %d error %d\n", __func__, ep,
+                rpl->status, status2errno(rpl->status));
        c4iw_wake_up(&ep->com.wr_wait, status2errno(rpl->status));
        c4iw_put_ep(&ep->com);
 out:
@@ -2332,7 +2335,7 @@ static int close_listsrv_rpl(struct c4iw_dev *dev, struct sk_buff *skb)
        unsigned int stid = GET_TID(rpl);
        struct c4iw_listen_ep *ep = get_ep_from_stid(dev, stid);
 
-       PDBG("%s ep %p\n", __func__, ep);
+       pr_debug("%s ep %p\n", __func__, ep);
        c4iw_wake_up(&ep->com.wr_wait, status2errno(rpl->status));
        c4iw_put_ep(&ep->com);
        return 0;
@@ -2350,7 +2353,7 @@ static int accept_cr(struct c4iw_ep *ep, struct sk_buff *skb,
        int win;
        enum chip_type adapter_type = ep->com.dev->rdev.lldi.adapter_type;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        BUG_ON(skb_cloned(skb));
 
        skb_get(skb);
@@ -2421,7 +2424,7 @@ static int accept_cr(struct c4iw_ep *ep, struct sk_buff *skb,
                if (peer2peer)
                        isn += 4;
                rpl5->iss = cpu_to_be32(isn);
-               PDBG("%s iss %u\n", __func__, be32_to_cpu(rpl5->iss));
+               pr_debug("%s iss %u\n", __func__, be32_to_cpu(rpl5->iss));
        }
 
        rpl->opt0 = cpu_to_be64(opt0);
@@ -2434,7 +2437,7 @@ static int accept_cr(struct c4iw_ep *ep, struct sk_buff *skb,
 
 static void reject_cr(struct c4iw_dev *dev, u32 hwtid, struct sk_buff *skb)
 {
-       PDBG("%s c4iw_dev %p tid %u\n", __func__, dev, hwtid);
+       pr_debug("%s c4iw_dev %p tid %u\n", __func__, dev, hwtid);
        BUG_ON(skb_cloned(skb));
        skb_trim(skb, sizeof(struct cpl_tid_release));
        release_tid(&dev->rdev, hwtid, skb);
@@ -2460,12 +2463,13 @@ static int pass_accept_req(struct c4iw_dev *dev, struct sk_buff *skb)
 
        parent_ep = (struct c4iw_ep *)get_ep_from_stid(dev, stid);
        if (!parent_ep) {
-               PDBG("%s connect request on invalid stid %d\n", __func__, stid);
+               pr_debug("%s connect request on invalid stid %d\n",
+                        __func__, stid);
                goto reject;
        }
 
        if (state_read(&parent_ep->com) != LISTEN) {
-               PDBG("%s - listening ep not in LISTEN\n", __func__);
+               pr_debug("%s - listening ep not in LISTEN\n", __func__);
                goto reject;
        }
 
@@ -2474,18 +2478,18 @@ static int pass_accept_req(struct c4iw_dev *dev, struct sk_buff *skb)
 
        /* Find output route */
        if (iptype == 4)  {
-               PDBG("%s parent ep %p hwtid %u laddr %pI4 raddr %pI4 lport %d rport %d peer_mss %d\n"
-                    , __func__, parent_ep, hwtid,
-                    local_ip, peer_ip, ntohs(local_port),
-                    ntohs(peer_port), peer_mss);
+               pr_debug("%s parent ep %p hwtid %u laddr %pI4 raddr %pI4 lport %d rport %d peer_mss %d\n"
+                        , __func__, parent_ep, hwtid,
+                        local_ip, peer_ip, ntohs(local_port),
+                        ntohs(peer_port), peer_mss);
                dst = cxgb_find_route(&dev->rdev.lldi, get_real_dev,
                                      *(__be32 *)local_ip, *(__be32 *)peer_ip,
                                      local_port, peer_port, tos);
        } else {
-               PDBG("%s parent ep %p hwtid %u laddr %pI6 raddr %pI6 lport %d rport %d peer_mss %d\n"
-                    , __func__, parent_ep, hwtid,
-                    local_ip, peer_ip, ntohs(local_port),
-                    ntohs(peer_port), peer_mss);
+               pr_debug("%s parent ep %p hwtid %u laddr %pI6 raddr %pI6 lport %d rport %d peer_mss %d\n"
+                        , __func__, parent_ep, hwtid,
+                        local_ip, peer_ip, ntohs(local_port),
+                        ntohs(peer_port), peer_mss);
                dst = cxgb_find_route6(&dev->rdev.lldi, get_real_dev,
                                local_ip, peer_ip, local_port, peer_port,
                                PASS_OPEN_TOS_G(ntohl(req->tos_stid)),
@@ -2493,15 +2497,13 @@ static int pass_accept_req(struct c4iw_dev *dev, struct sk_buff *skb)
                                 &parent_ep->com.local_addr)->sin6_scope_id);
        }
        if (!dst) {
-               printk(KERN_ERR MOD "%s - failed to find dst entry!\n",
-                      __func__);
+               pr_err("%s - failed to find dst entry!\n", __func__);
                goto reject;
        }
 
        child_ep = alloc_ep(sizeof(*child_ep), GFP_KERNEL);
        if (!child_ep) {
-               printk(KERN_ERR MOD "%s - failed to allocate ep entry!\n",
-                      __func__);
+               pr_err("%s - failed to allocate ep entry!\n", __func__);
                dst_release(dst);
                goto reject;
        }
@@ -2509,8 +2511,7 @@ static int pass_accept_req(struct c4iw_dev *dev, struct sk_buff *skb)
        err = import_ep(child_ep, iptype, peer_ip, dst, dev, false,
                        parent_ep->com.dev->rdev.lldi.adapter_type, tos);
        if (err) {
-               printk(KERN_ERR MOD "%s - failed to allocate l2t entry!\n",
-                      __func__);
+               pr_err("%s - failed to allocate l2t entry!\n", __func__);
                dst_release(dst);
                kfree(child_ep);
                goto reject;
@@ -2571,8 +2572,8 @@ static int pass_accept_req(struct c4iw_dev *dev, struct sk_buff *skb)
        child_ep->dst = dst;
        child_ep->hwtid = hwtid;
 
-       PDBG("%s tx_chan %u smac_idx %u rss_qid %u\n", __func__,
-            child_ep->tx_chan, child_ep->smac_idx, child_ep->rss_qid);
+       pr_debug("%s tx_chan %u smac_idx %u rss_qid %u\n", __func__,
+                child_ep->tx_chan, child_ep->smac_idx, child_ep->rss_qid);
 
        init_timer(&child_ep->timer);
        cxgb4_insert_tid(t, child_ep, hwtid);
@@ -2607,12 +2608,12 @@ static int pass_establish(struct c4iw_dev *dev, struct sk_buff *skb)
        int ret;
 
        ep = get_ep_from_tid(dev, tid);
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        ep->snd_seq = be32_to_cpu(req->snd_isn);
        ep->rcv_seq = be32_to_cpu(req->rcv_isn);
 
-       PDBG("%s ep %p hwtid %u tcp_opt 0x%02x\n", __func__, ep, tid,
-            ntohs(req->tcp_opt));
+       pr_debug("%s ep %p hwtid %u tcp_opt 0x%02x\n", __func__, ep, tid,
+                ntohs(req->tcp_opt));
 
        set_emss(ep, ntohs(req->tcp_opt));
 
@@ -2644,7 +2645,7 @@ static int peer_close(struct c4iw_dev *dev, struct sk_buff *skb)
        if (!ep)
                return 0;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        dst_confirm(ep->dst);
 
        set_bit(PEER_CLOSE, &ep->com.history);
@@ -2666,12 +2667,12 @@ static int peer_close(struct c4iw_dev *dev, struct sk_buff *skb)
                 * in rdma connection migration (see c4iw_accept_cr()).
                 */
                __state_set(&ep->com, CLOSING);
-               PDBG("waking up ep %p tid %u\n", ep, ep->hwtid);
+               pr_debug("waking up ep %p tid %u\n", ep, ep->hwtid);
                c4iw_wake_up(&ep->com.wr_wait, -ECONNRESET);
                break;
        case MPA_REP_SENT:
                __state_set(&ep->com, CLOSING);
-               PDBG("waking up ep %p tid %u\n", ep, ep->hwtid);
+               pr_debug("waking up ep %p tid %u\n", ep, ep->hwtid);
                c4iw_wake_up(&ep->com.wr_wait, -ECONNRESET);
                break;
        case FPDU_MODE:
@@ -2735,17 +2736,17 @@ static int peer_abort(struct c4iw_dev *dev, struct sk_buff *skb)
                return 0;
 
        if (cxgb_is_neg_adv(req->status)) {
-               PDBG("%s Negative advice on abort- tid %u status %d (%s)\n",
-                    __func__, ep->hwtid, req->status,
-                    neg_adv_str(req->status));
+               pr_debug("%s Negative advice on abort- tid %u status %d (%s)\n",
+                        __func__, ep->hwtid, req->status,
+                        neg_adv_str(req->status));
                ep->stats.abort_neg_adv++;
                mutex_lock(&dev->rdev.stats.lock);
                dev->rdev.stats.neg_adv++;
                mutex_unlock(&dev->rdev.stats.lock);
                goto deref_ep;
        }
-       PDBG("%s ep %p tid %u state %u\n", __func__, ep, ep->hwtid,
-            ep->com.state);
+       pr_debug("%s ep %p tid %u state %u\n", __func__, ep, ep->hwtid,
+                ep->com.state);
        set_bit(PEER_ABORT, &ep->com.history);
 
        /*
@@ -2777,8 +2778,8 @@ static int peer_abort(struct c4iw_dev *dev, struct sk_buff *skb)
                         * do some housekeeping so as to re-initiate the
                         * connection
                         */
-                       PDBG("%s: mpa_rev=%d. Retrying with mpav1\n", __func__,
-                            mpa_rev);
+                       pr_debug("%s: mpa_rev=%d. Retrying with mpav1\n",
+                                __func__, mpa_rev);
                        ep->retry_with_mpa_v1 = 1;
                }
                break;
@@ -2797,16 +2798,14 @@ static int peer_abort(struct c4iw_dev *dev, struct sk_buff *skb)
                                     ep->com.qp, C4IW_QP_ATTR_NEXT_STATE,
                                     &attrs, 1);
                        if (ret)
-                               printk(KERN_ERR MOD
-                                      "%s - qp <- error failed!\n",
-                                      __func__);
+                               pr_err("%s - qp <- error failed!\n", __func__);
                }
                peer_abort_upcall(ep);
                break;
        case ABORTING:
                break;
        case DEAD:
-               PDBG("%s PEER_ABORT IN DEAD STATE!!!!\n", __func__);
+               pr_debug("%s PEER_ABORT IN DEAD STATE!!!!\n", __func__);
                mutex_unlock(&ep->com.mutex);
                goto deref_ep;
        default:
@@ -2870,7 +2869,7 @@ static int close_con_rpl(struct c4iw_dev *dev, struct sk_buff *skb)
        if (!ep)
                return 0;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        BUG_ON(!ep);
 
        /* The cm_id may be null if we failed to connect */
@@ -2918,13 +2917,13 @@ static int terminate(struct c4iw_dev *dev, struct sk_buff *skb)
        BUG_ON(!ep);
 
        if (ep && ep->com.qp) {
-               printk(KERN_WARNING MOD "TERM received tid %u qpid %u\n", tid,
-                      ep->com.qp->wq.sq.qid);
+               pr_warn("TERM received tid %u qpid %u\n",
+                       tid, ep->com.qp->wq.sq.qid);
                attrs.next_state = C4IW_QP_STATE_TERMINATE;
                c4iw_modify_qp(ep->com.qp->rhp, ep->com.qp,
                               C4IW_QP_ATTR_NEXT_STATE, &attrs, 1);
        } else
-               printk(KERN_WARNING MOD "TERM received tid %u no ep/qp\n", tid);
+               pr_warn("TERM received tid %u no ep/qp\n", tid);
        c4iw_put_ep(&ep->com);
 
        return 0;
@@ -2946,18 +2945,19 @@ static int fw4_ack(struct c4iw_dev *dev, struct sk_buff *skb)
        ep = get_ep_from_tid(dev, tid);
        if (!ep)
                return 0;
-       PDBG("%s ep %p tid %u credits %u\n", __func__, ep, ep->hwtid, credits);
+       pr_debug("%s ep %p tid %u credits %u\n",
+                __func__, ep, ep->hwtid, credits);
        if (credits == 0) {
-               PDBG("%s 0 credit ack ep %p tid %u state %u\n",
-                    __func__, ep, ep->hwtid, state_read(&ep->com));
+               pr_debug("%s 0 credit ack ep %p tid %u state %u\n",
+                        __func__, ep, ep->hwtid, state_read(&ep->com));
                goto out;
        }
 
        dst_confirm(ep->dst);
        if (ep->mpa_skb) {
-               PDBG("%s last streaming msg ack ep %p tid %u state %u "
-                    "initiator %u freeing skb\n", __func__, ep, ep->hwtid,
-                    state_read(&ep->com), ep->mpa_attr.initiator ? 1 : 0);
+               pr_debug("%s last streaming msg ack ep %p tid %u state %u initiator %u freeing skb\n",
+                        __func__, ep, ep->hwtid,
+                        state_read(&ep->com), ep->mpa_attr.initiator ? 1 : 0);
                mutex_lock(&ep->com.mutex);
                kfree_skb(ep->mpa_skb);
                ep->mpa_skb = NULL;
@@ -2975,7 +2975,7 @@ int c4iw_reject_cr(struct iw_cm_id *cm_id, const void *pdata, u8 pdata_len)
        int abort;
        struct c4iw_ep *ep = to_ep(cm_id);
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
 
        mutex_lock(&ep->com.mutex);
        if (ep->com.state != MPA_REQ_RCVD) {
@@ -3006,7 +3006,7 @@ int c4iw_accept_cr(struct iw_cm_id *cm_id, struct iw_cm_conn_param *conn_param)
        struct c4iw_qp *qp = get_qhp(h, conn_param->qpn);
        int abort = 0;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
 
        mutex_lock(&ep->com.mutex);
        if (ep->com.state != MPA_REQ_RCVD) {
@@ -3059,7 +3059,7 @@ int c4iw_accept_cr(struct iw_cm_id *cm_id, struct iw_cm_conn_param *conn_param)
                        ep->ird = 1;
        }
 
-       PDBG("%s %d ird %d ord %d\n", __func__, __LINE__, ep->ird, ep->ord);
+       pr_debug("%s %d ird %d ord %d\n", __func__, __LINE__, ep->ird, ep->ord);
 
        ep->com.cm_id = cm_id;
        ref_cm_id(&ep->com);
@@ -3188,7 +3188,7 @@ int c4iw_connect(struct iw_cm_id *cm_id, struct iw_cm_conn_param *conn_param)
        }
        ep = alloc_ep(sizeof(*ep), GFP_KERNEL);
        if (!ep) {
-               printk(KERN_ERR MOD "%s - cannot alloc ep.\n", __func__);
+               pr_err("%s - cannot alloc ep\n", __func__);
                err = -ENOMEM;
                goto out;
        }
@@ -3215,20 +3215,20 @@ int c4iw_connect(struct iw_cm_id *cm_id, struct iw_cm_conn_param *conn_param)
        ep->com.dev = dev;
        ep->com.qp = get_qhp(dev, conn_param->qpn);
        if (!ep->com.qp) {
-               PDBG("%s qpn 0x%x not found!\n", __func__, conn_param->qpn);
+               pr_debug("%s qpn 0x%x not found!\n", __func__, conn_param->qpn);
                err = -EINVAL;
                goto fail2;
        }
        ref_qp(ep);
-       PDBG("%s qpn 0x%x qp %p cm_id %p\n", __func__, conn_param->qpn,
-            ep->com.qp, cm_id);
+       pr_debug("%s qpn 0x%x qp %p cm_id %p\n", __func__, conn_param->qpn,
+                ep->com.qp, cm_id);
 
        /*
         * Allocate an active TID to initiate a TCP connection.
         */
        ep->atid = cxgb4_alloc_atid(dev->rdev.lldi.tids, ep);
        if (ep->atid == -1) {
-               printk(KERN_ERR MOD "%s - cannot alloc atid.\n", __func__);
+               pr_err("%s - cannot alloc atid\n", __func__);
                err = -ENOMEM;
                goto fail2;
        }
@@ -3258,9 +3258,9 @@ int c4iw_connect(struct iw_cm_id *cm_id, struct iw_cm_conn_param *conn_param)
                }
 
                /* find a route */
-               PDBG("%s saddr %pI4 sport 0x%x raddr %pI4 rport 0x%x\n",
-                    __func__, &laddr->sin_addr, ntohs(laddr->sin_port),
-                    ra, ntohs(raddr->sin_port));
+               pr_debug("%s saddr %pI4 sport 0x%x raddr %pI4 rport 0x%x\n",
+                        __func__, &laddr->sin_addr, ntohs(laddr->sin_port),
+                        ra, ntohs(raddr->sin_port));
                ep->dst = cxgb_find_route(&dev->rdev.lldi, get_real_dev,
                                          laddr->sin_addr.s_addr,
                                          raddr->sin_addr.s_addr,
@@ -3280,10 +3280,10 @@ int c4iw_connect(struct iw_cm_id *cm_id, struct iw_cm_conn_param *conn_param)
                }
 
                /* find a route */
-               PDBG("%s saddr %pI6 sport 0x%x raddr %pI6 rport 0x%x\n",
-                    __func__, laddr6->sin6_addr.s6_addr,
-                    ntohs(laddr6->sin6_port),
-                    raddr6->sin6_addr.s6_addr, ntohs(raddr6->sin6_port));
+               pr_debug("%s saddr %pI6 sport 0x%x raddr %pI6 rport 0x%x\n",
+                        __func__, laddr6->sin6_addr.s6_addr,
+                        ntohs(laddr6->sin6_port),
+                        raddr6->sin6_addr.s6_addr, ntohs(raddr6->sin6_port));
                ep->dst = cxgb_find_route6(&dev->rdev.lldi, get_real_dev,
                                           laddr6->sin6_addr.s6_addr,
                                           raddr6->sin6_addr.s6_addr,
@@ -3292,7 +3292,7 @@ int c4iw_connect(struct iw_cm_id *cm_id, struct iw_cm_conn_param *conn_param)
                                           raddr6->sin6_scope_id);
        }
        if (!ep->dst) {
-               printk(KERN_ERR MOD "%s - cannot find route.\n", __func__);
+               pr_err("%s - cannot find route\n", __func__);
                err = -EHOSTUNREACH;
                goto fail3;
        }
@@ -3300,13 +3300,13 @@ int c4iw_connect(struct iw_cm_id *cm_id, struct iw_cm_conn_param *conn_param)
        err = import_ep(ep, iptype, ra, ep->dst, ep->com.dev, true,
                        ep->com.dev->rdev.lldi.adapter_type, cm_id->tos);
        if (err) {
-               printk(KERN_ERR MOD "%s - cannot alloc l2e.\n", __func__);
+               pr_err("%s - cannot alloc l2e\n", __func__);
                goto fail4;
        }
 
-       PDBG("%s txq_idx %u tx_chan %u smac_idx %u rss_qid %u l2t_idx %u\n",
-               __func__, ep->txq_idx, ep->tx_chan, ep->smac_idx, ep->rss_qid,
-               ep->l2t->idx);
+       pr_debug("%s txq_idx %u tx_chan %u smac_idx %u rss_qid %u l2t_idx %u\n",
+                __func__, ep->txq_idx, ep->tx_chan, ep->smac_idx, ep->rss_qid,
+                ep->l2t->idx);
 
        state_set(&ep->com, CONNECTING);
        ep->tos = cm_id->tos;
@@ -3414,12 +3414,12 @@ int c4iw_create_listen(struct iw_cm_id *cm_id, int backlog)
 
        ep = alloc_ep(sizeof(*ep), GFP_KERNEL);
        if (!ep) {
-               printk(KERN_ERR MOD "%s - cannot alloc ep.\n", __func__);
+               pr_err("%s - cannot alloc ep\n", __func__);
                err = -ENOMEM;
                goto fail1;
        }
        skb_queue_head_init(&ep->com.ep_skb_list);
-       PDBG("%s ep %p\n", __func__, ep);
+       pr_debug("%s ep %p\n", __func__, ep);
        ep->com.cm_id = cm_id;
        ref_cm_id(&ep->com);
        ep->com.dev = dev;
@@ -3439,7 +3439,7 @@ int c4iw_create_listen(struct iw_cm_id *cm_id, int backlog)
                                            cm_id->m_local_addr.ss_family, ep);
 
        if (ep->stid == -1) {
-               printk(KERN_ERR MOD "%s - cannot alloc stid.\n", __func__);
+               pr_err("%s - cannot alloc stid\n", __func__);
                err = -ENOMEM;
                goto fail2;
        }
@@ -3473,7 +3473,7 @@ int c4iw_destroy_listen(struct iw_cm_id *cm_id)
        int err;
        struct c4iw_listen_ep *ep = to_listen_ep(cm_id);
 
-       PDBG("%s ep %p\n", __func__, ep);
+       pr_debug("%s ep %p\n", __func__, ep);
 
        might_sleep();
        state_set(&ep->com, DEAD);
@@ -3514,8 +3514,8 @@ int c4iw_ep_disconnect(struct c4iw_ep *ep, int abrupt, gfp_t gfp)
 
        mutex_lock(&ep->com.mutex);
 
-       PDBG("%s ep %p state %s, abrupt %d\n", __func__, ep,
-            states[ep->com.state], abrupt);
+       pr_debug("%s ep %p state %s, abrupt %d\n", __func__, ep,
+                states[ep->com.state], abrupt);
 
        /*
         * Ref the ep here in case we have fatal errors causing the
@@ -3568,8 +3568,8 @@ int c4iw_ep_disconnect(struct c4iw_ep *ep, int abrupt, gfp_t gfp)
        case MORIBUND:
        case ABORTING:
        case DEAD:
-               PDBG("%s ignoring disconnect ep %p state %u\n",
-                    __func__, ep, ep->com.state);
+               pr_debug("%s ignoring disconnect ep %p state %u\n",
+                        __func__, ep, ep->com.state);
                break;
        default:
                BUG();
@@ -3600,8 +3600,7 @@ int c4iw_ep_disconnect(struct c4iw_ep *ep, int abrupt, gfp_t gfp)
                                                     C4IW_QP_ATTR_NEXT_STATE,
                                                     &attrs, 1);
                                if (ret)
-                                       pr_err(MOD
-                                              "%s - qp <- error failed!\n",
+                                       pr_err("%s - qp <- error failed!\n",
                                               __func__);
                        }
                        fatal = 1;
@@ -3674,7 +3673,7 @@ static void passive_ofld_conn_reply(struct c4iw_dev *dev, struct sk_buff *skb,
        rpl_skb = (struct sk_buff *)(unsigned long)req->cookie;
        BUG_ON(!rpl_skb);
        if (req->retval) {
-               PDBG("%s passive open failure %d\n", __func__, req->retval);
+               pr_debug("%s passive open failure %d\n", __func__, req->retval);
                mutex_lock(&dev->rdev.stats.lock);
                dev->rdev.stats.pas_ofld_conn_fails++;
                mutex_unlock(&dev->rdev.stats.lock);
@@ -3800,6 +3799,8 @@ static void send_fw_pass_open_req(struct c4iw_dev *dev, struct sk_buff *skb,
        int ret;
 
        req_skb = alloc_skb(sizeof(struct fw_ofld_connection_wr), GFP_KERNEL);
+       if (!req_skb)
+               return;
        req = (struct fw_ofld_connection_wr *)__skb_put(req_skb, sizeof(*req));
        memset(req, 0, sizeof(*req));
        req->op_compl = htonl(WR_OP_V(FW_OFLD_CONNECTION_WR) | FW_WR_COMPL_F);
@@ -3890,7 +3891,8 @@ static int rx_pkt(struct c4iw_dev *dev, struct sk_buff *skb)
 
        lep = (struct c4iw_ep *)get_ep_from_stid(dev, stid);
        if (!lep) {
-               PDBG("%s connect request on invalid stid %d\n", __func__, stid);
+               pr_debug("%s connect request on invalid stid %d\n",
+                        __func__, stid);
                goto reject;
        }
 
@@ -3927,9 +3929,9 @@ static int rx_pkt(struct c4iw_dev *dev, struct sk_buff *skb)
        skb_set_transport_header(skb, (void *)tcph - (void *)rss);
        skb_get(skb);
 
-       PDBG("%s lip 0x%x lport %u pip 0x%x pport %u tos %d\n", __func__,
-            ntohl(iph->daddr), ntohs(tcph->dest), ntohl(iph->saddr),
-            ntohs(tcph->source), iph->tos);
+       pr_debug("%s lip 0x%x lport %u pip 0x%x pport %u tos %d\n", __func__,
+                ntohl(iph->daddr), ntohs(tcph->dest), ntohl(iph->saddr),
+                ntohs(tcph->source), iph->tos);
 
        dst = cxgb_find_route(&dev->rdev.lldi, get_real_dev,
                              iph->daddr, iph->saddr, tcph->dest,
@@ -4026,8 +4028,8 @@ static void process_timeout(struct c4iw_ep *ep)
        int abort = 1;
 
        mutex_lock(&ep->com.mutex);
-       PDBG("%s ep %p tid %u state %d\n", __func__, ep, ep->hwtid,
-            ep->com.state);
+       pr_debug("%s ep %p tid %u state %d\n", __func__, ep, ep->hwtid,
+                ep->com.state);
        set_bit(TIMEDOUT, &ep->com.history);
        switch (ep->com.state) {
        case MPA_REQ_SENT:
@@ -4157,8 +4159,8 @@ static int set_tcb_rpl(struct c4iw_dev *dev, struct sk_buff *skb)
        struct cpl_set_tcb_rpl *rpl = cplhdr(skb);
 
        if (rpl->status != CPL_ERR_NONE) {
-               printk(KERN_ERR MOD "Unexpected SET_TCB_RPL status %u "
-                      "for tid %u\n", rpl->status, GET_TID(rpl));
+               pr_err("Unexpected SET_TCB_RPL status %u for tid %u\n",
+                      rpl->status, GET_TID(rpl));
        }
        kfree_skb(skb);
        return 0;
@@ -4170,13 +4172,13 @@ static int fw6_msg(struct c4iw_dev *dev, struct sk_buff *skb)
        struct c4iw_wr_wait *wr_waitp;
        int ret;
 
-       PDBG("%s type %u\n", __func__, rpl->type);
+       pr_debug("%s type %u\n", __func__, rpl->type);
 
        switch (rpl->type) {
        case FW6_TYPE_WR_RPL:
                ret = (int)((be64_to_cpu(rpl->data[0]) >> 8) & 0xff);
                wr_waitp = (struct c4iw_wr_wait *)(__force unsigned long) rpl->data[1];
-               PDBG("%s wr_waitp %p ret %u\n", __func__, wr_waitp, ret);
+               pr_debug("%s wr_waitp %p ret %u\n", __func__, wr_waitp, ret);
                if (wr_waitp)
                        c4iw_wake_up(wr_waitp, ret ? -ret : 0);
                kfree_skb(skb);
@@ -4186,8 +4188,8 @@ static int fw6_msg(struct c4iw_dev *dev, struct sk_buff *skb)
                sched(dev, skb);
                break;
        default:
-               printk(KERN_ERR MOD "%s unexpected fw6 msg type %u\n", __func__,
-                      rpl->type);
+               pr_err("%s unexpected fw6 msg type %u\n",
+                      __func__, rpl->type);
                kfree_skb(skb);
                break;
        }
@@ -4203,19 +4205,18 @@ static int peer_abort_intr(struct c4iw_dev *dev, struct sk_buff *skb)
        ep = get_ep_from_tid(dev, tid);
        /* This EP will be dereferenced in peer_abort() */
        if (!ep) {
-               printk(KERN_WARNING MOD
-                      "Abort on non-existent endpoint, tid %d\n", tid);
+               pr_warn("Abort on non-existent endpoint, tid %d\n", tid);
                kfree_skb(skb);
                return 0;
        }
        if (cxgb_is_neg_adv(req->status)) {
-               PDBG("%s Negative advice on abort- tid %u status %d (%s)\n",
-                    __func__, ep->hwtid, req->status,
-                    neg_adv_str(req->status));
+               pr_debug("%s Negative advice on abort- tid %u status %d (%s)\n",
+                        __func__, ep->hwtid, req->status,
+                        neg_adv_str(req->status));
                goto out;
        }
-       PDBG("%s ep %p tid %u state %u\n", __func__, ep, ep->hwtid,
-            ep->com.state);
+       pr_debug("%s ep %p tid %u state %u\n", __func__, ep, ep->hwtid,
+                ep->com.state);
 
        c4iw_wake_up(&ep->com.wr_wait, -ECONNRESET);
 out: