[SCSI] cxgb3i: fixed connection over vlan
authorkxie@chelsio.com <kxie@chelsio.com>
Thu, 23 Sep 2010 23:43:23 +0000 (16:43 -0700)
committerJames Bottomley <James.Bottomley@suse.de>
Thu, 7 Oct 2010 22:21:11 +0000 (17:21 -0500)
Signed-off-by: Karen Xie <kxie@chelsio.com>
Reviewed-by: Mike Christie <michaelc@cs.wisc.edu>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
drivers/scsi/cxgbi/cxgb3i/cxgb3i.c
drivers/scsi/cxgbi/libcxgbi.c
drivers/scsi/cxgbi/libcxgbi.h

index a2c207f57d61db9a4fe6663a20359450e7d5f86d..a129a170b47bb5a25628a94c301126dbb05c7c24 100644 (file)
@@ -589,9 +589,10 @@ static int do_act_open_rpl(struct t3cdev *tdev, struct sk_buff *skb, void *ctx)
        struct cxgbi_sock *csk = ctx;
        struct cpl_act_open_rpl *rpl = cplhdr(skb);
 
-       log_debug(1 << CXGBI_DBG_TOE | 1 << CXGBI_DBG_SOCK,
-               "csk 0x%p,%u,0x%lx,%u, status %u.\n",
-               csk, csk->state, csk->flags, csk->atid, rpl->status);
+       pr_info("csk 0x%p,%u,0x%lx,%u, status %u, %pI4:%u-%pI4:%u.\n",
+               csk, csk->state, csk->flags, csk->atid, rpl->status,
+               &csk->saddr.sin_addr.s_addr, ntohs(csk->saddr.sin_port),
+               &csk->daddr.sin_addr.s_addr, ntohs(csk->daddr.sin_port));
 
        if (rpl->status != CPL_ERR_TCAM_FULL &&
            rpl->status != CPL_ERR_CONN_EXIST &&
@@ -662,8 +663,7 @@ static int abort_status_to_errno(struct cxgbi_sock *csk, int abort_reason,
        switch (abort_reason) {
        case CPL_ERR_BAD_SYN: /* fall through */
        case CPL_ERR_CONN_RESET:
-               return csk->state > CTP_ESTABLISHED ?
-                       -EPIPE : -ECONNRESET;
+               return csk->state > CTP_ESTABLISHED ? -EPIPE : -ECONNRESET;
        case CPL_ERR_XMIT_TIMEDOUT:
        case CPL_ERR_PERSIST_TIMEDOUT:
        case CPL_ERR_FINWAIT2_TIMEDOUT:
@@ -945,17 +945,44 @@ static void release_offload_resources(struct cxgbi_sock *csk)
        csk->cdev = NULL;
 }
 
+static void update_address(struct cxgbi_hba *chba)
+{
+       if (chba->ipv4addr) {
+               if (chba->vdev &&
+                   chba->ipv4addr != cxgb3i_get_private_ipv4addr(chba->vdev)) {
+                       cxgb3i_set_private_ipv4addr(chba->vdev, chba->ipv4addr);
+                       cxgb3i_set_private_ipv4addr(chba->ndev, 0);
+                       pr_info("%s set %pI4.\n",
+                               chba->vdev->name, &chba->ipv4addr);
+               } else if (chba->ipv4addr !=
+                               cxgb3i_get_private_ipv4addr(chba->ndev)) {
+                       cxgb3i_set_private_ipv4addr(chba->ndev, chba->ipv4addr);
+                       pr_info("%s set %pI4.\n",
+                               chba->ndev->name, &chba->ipv4addr);
+               }
+       } else if (cxgb3i_get_private_ipv4addr(chba->ndev)) {
+               if (chba->vdev)
+                       cxgb3i_set_private_ipv4addr(chba->vdev, 0);
+               cxgb3i_set_private_ipv4addr(chba->ndev, 0);
+       }
+}
+
 static int init_act_open(struct cxgbi_sock *csk)
 {
        struct dst_entry *dst = csk->dst;
        struct cxgbi_device *cdev = csk->cdev;
        struct t3cdev *t3dev = (struct t3cdev *)cdev->lldev;
        struct net_device *ndev = cdev->ports[csk->port_id];
+       struct cxgbi_hba *chba = cdev->hbas[csk->port_id];
        struct sk_buff *skb = NULL;
 
        log_debug(1 << CXGBI_DBG_TOE | 1 << CXGBI_DBG_SOCK,
                "csk 0x%p,%u,0x%lx.\n", csk, csk->state, csk->flags);
 
+       update_address(chba);
+       if (chba->ipv4addr)
+               csk->saddr.sin_addr.s_addr = chba->ipv4addr;
+
        csk->rss_qid = 0;
        csk->l2t = t3_l2t_get(t3dev, dst->neighbour, ndev);
        if (!csk->l2t) {
@@ -984,6 +1011,12 @@ static int init_act_open(struct cxgbi_sock *csk)
        cxgbi_sock_reset_wr_list(csk);
        csk->err = 0;
 
+       log_debug(1 << CXGBI_DBG_TOE | 1 << CXGBI_DBG_SOCK,
+               "csk 0x%p,%u,0x%lx, %pI4:%u-%pI4:%u.\n",
+               csk, csk->state, csk->flags,
+               &csk->saddr.sin_addr.s_addr, ntohs(csk->saddr.sin_port),
+               &csk->daddr.sin_addr.s_addr, ntohs(csk->daddr.sin_port));
+
        cxgbi_sock_set_state(csk, CTP_ACTIVE_OPEN);
        send_act_open_req(csk, skb, csk->l2t);
        return 0;
@@ -1143,9 +1176,9 @@ static int ddp_alloc_gl_skb(struct cxgbi_ddp_info *ddp, int idx,
        for (i = 0; i < cnt; i++) {
                struct sk_buff *skb = alloc_wr(sizeof(struct ulp_mem_io) +
                                                PPOD_SIZE, 0, gfp);
-               if (skb) {
+               if (skb)
                        ddp->gl_skb[idx + i] = skb;
-               else {
+               else {
                        ddp_free_gl_skb(ddp, idx, i);
                        return -ENOMEM;
                }
index 6cfce2726ea3a6c51d8d3ae905797c8513d92b6d..be5661707dfaadd5a61d0bbf765fffa4fe115124 100644 (file)
@@ -195,16 +195,22 @@ EXPORT_SYMBOL_GPL(cxgbi_device_find_by_lldev);
 static struct cxgbi_device *cxgbi_device_find_by_netdev(struct net_device *ndev,
                                                        int *port)
 {
+       struct net_device *vdev = NULL;
        struct cxgbi_device *cdev, *tmp;
        int i;
 
-       if (ndev->priv_flags & IFF_802_1Q_VLAN)
+       if (ndev->priv_flags & IFF_802_1Q_VLAN) {
+               vdev = ndev;
                ndev = vlan_dev_real_dev(ndev);
+               log_debug(1 << CXGBI_DBG_DEV,
+                       "vlan dev %s -> %s.\n", vdev->name, ndev->name);
+       }
 
        mutex_lock(&cdev_mutex);
        list_for_each_entry_safe(cdev, tmp, &cdev_list, list_head) {
                for (i = 0; i < cdev->nports; i++) {
                        if (ndev == cdev->ports[i]) {
+                               cdev->hbas[i]->vdev = vdev;
                                mutex_unlock(&cdev_mutex);
                                if (port)
                                        *port = i;
@@ -218,24 +224,6 @@ static struct cxgbi_device *cxgbi_device_find_by_netdev(struct net_device *ndev,
        return NULL;
 }
 
-struct cxgbi_hba *cxgbi_hba_find_by_netdev(struct net_device *dev,
-                                       struct cxgbi_device *cdev)
-{
-       int i;
-
-       if (dev->priv_flags & IFF_802_1Q_VLAN)
-               dev = vlan_dev_real_dev(dev);
-
-       for (i = 0; i < cdev->nports; i++) {
-               if (cdev->hbas[i]->ndev == dev)
-                       return cdev->hbas[i];
-       }
-       log_debug(1 << CXGBI_DBG_DEV,
-               "ndev 0x%p, %s, cdev 0x%p, NO match found.\n",
-               dev, dev->name, cdev);
-       return NULL;
-}
-
 void cxgbi_hbas_remove(struct cxgbi_device *cdev)
 {
        int i;
@@ -532,12 +520,6 @@ static struct cxgbi_sock *cxgbi_check_route(struct sockaddr *dst_addr)
                        dst->neighbour->dev->name, ndev->name, mtu);
        }
 
-       if (ndev->priv_flags & IFF_802_1Q_VLAN) {
-               ndev = vlan_dev_real_dev(ndev);
-               pr_info("rt dev %s, vlan -> %s.\n",
-                       dst->neighbour->dev->name, ndev->name);
-       }
-
        cdev = cxgbi_device_find_by_netdev(ndev, &port);
        if (!cdev) {
                pr_info("dst %pI4, %s, NOT cxgbi device.\n",
@@ -561,10 +543,7 @@ static struct cxgbi_sock *cxgbi_check_route(struct sockaddr *dst_addr)
        csk->dst = dst;
        csk->daddr.sin_addr.s_addr = daddr->sin_addr.s_addr;
        csk->daddr.sin_port = daddr->sin_port;
-       if (cdev->hbas[port]->ipv4addr)
-               csk->saddr.sin_addr.s_addr = cdev->hbas[port]->ipv4addr;
-       else
-               csk->saddr.sin_addr.s_addr = rt->rt_src;
+       csk->saddr.sin_addr.s_addr = rt->rt_src;
 
        return csk;
 
index 48e6d623f76d340b40dcb0c70b1d679b106b3e4a..43025b71e295a6b672014efc617d82883ab5372c 100644 (file)
@@ -500,6 +500,7 @@ void cxgbi_sock_free_cpl_skbs(struct cxgbi_sock *);
 
 struct cxgbi_hba {
        struct net_device *ndev;
+       struct net_device *vdev;        /* vlan dev */
        struct Scsi_Host *shost;
        struct cxgbi_device *cdev;
        __be32 ipv4addr;