Add checksum offload support to ipoib

Signed-off-by: Eli Cohen <[EMAIL PROTECTED]>
Signed-off-by: Ali Ayub <[EMAIL PROTECTED]>

---

Add checksum offload support to ipoib

Signed-off-by: Eli Cohen <[EMAIL PROTECTED]>
Signed-off-by: Ali Ayub <[EMAIL PROTECTED]>

---

Index: ofa_1_3_dev_kernel/drivers/infiniband/ulp/ipoib/ipoib.h
===================================================================
--- ofa_1_3_dev_kernel.orig/drivers/infiniband/ulp/ipoib/ipoib.h        
2007-09-24 12:09:21.000000000 +0200
+++ ofa_1_3_dev_kernel/drivers/infiniband/ulp/ipoib/ipoib.h     2007-09-24 
12:49:00.000000000 +0200
@@ -86,6 +86,7 @@ enum {
        IPOIB_MCAST_STARTED       = 8,
        IPOIB_FLAG_NETIF_STOPPED  = 9,
        IPOIB_FLAG_ADMIN_CM       = 10,
+       IPOIB_FLAG_RX_CSUM        = 11,
 
        IPOIB_MAX_BACKOFF_SECONDS = 16,
 
Index: ofa_1_3_dev_kernel/drivers/infiniband/ulp/ipoib/ipoib_cm.c
===================================================================
--- ofa_1_3_dev_kernel.orig/drivers/infiniband/ulp/ipoib/ipoib_cm.c     
2007-09-24 12:23:26.000000000 +0200
+++ ofa_1_3_dev_kernel/drivers/infiniband/ulp/ipoib/ipoib_cm.c  2007-09-24 
13:05:21.000000000 +0200
@@ -1258,6 +1258,13 @@ static ssize_t set_mode(struct device *d
                set_bit(IPOIB_FLAG_ADMIN_CM, &priv->flags);
                ipoib_warn(priv, "enabling connected mode "
                           "will cause multicast packet drops\n");
+
+               /* clear ipv6 flag too */
+               dev->features &= ~NETIF_F_IP_CSUM;
+
+               priv->tx_wr.send_flags &=
+                       ~(IB_SEND_UDP_TCP_CSUM | IB_SEND_IP_CSUM);
+
                ipoib_flush_paths(dev);
                return count;
        }
@@ -1266,6 +1273,10 @@ static ssize_t set_mode(struct device *d
                clear_bit(IPOIB_FLAG_ADMIN_CM, &priv->flags);
                dev->mtu = min(priv->mcast_mtu, dev->mtu);
                ipoib_flush_paths(dev);
+
+               if (priv->ca->flags & IB_DEVICE_IP_CSUM)
+                       dev->features |= NETIF_F_IP_CSUM; /* ipv6 too */
+
                return count;
        }
 
Index: ofa_1_3_dev_kernel/drivers/infiniband/ulp/ipoib/ipoib_ib.c
===================================================================
--- ofa_1_3_dev_kernel.orig/drivers/infiniband/ulp/ipoib/ipoib_ib.c     
2007-09-24 11:57:02.000000000 +0200
+++ ofa_1_3_dev_kernel/drivers/infiniband/ulp/ipoib/ipoib_ib.c  2007-09-24 
13:03:27.000000000 +0200
@@ -37,6 +37,7 @@
 
 #include <linux/delay.h>
 #include <linux/dma-mapping.h>
+#include <linux/ip.h>
 
 #include <rdma/ib_cache.h>
 
@@ -231,6 +232,16 @@ static void ipoib_ib_handle_rx_wc(struct
        skb->dev = dev;
        /* XXX get correct PACKET_ type here */
        skb->pkt_type = PACKET_HOST;
+
+       /* check rx csum */
+       if (test_bit(IPOIB_FLAG_RX_CSUM, &priv->flags) && likely(wc->csum_ok)) {
+               /* Note: this is a specific requirement for Mellanox
+                  HW but since it is the only HW currently supporting
+                  checksum offload I put it here */
+               if ((((struct iphdr *)(skb->data))->ihl) == 5)
+                       skb->ip_summed = CHECKSUM_UNNECESSARY;
+       }
+
        netif_receive_skb(skb);
 
 repost:
@@ -396,6 +407,15 @@ void ipoib_send(struct net_device *dev, 
                return;
        }
 
+       if (priv->ca->flags & IB_DEVICE_IP_CSUM &&
+           skb->ip_summed == CHECKSUM_PARTIAL)
+               priv->tx_wr.send_flags |=
+                       IB_SEND_UDP_TCP_CSUM | IB_SEND_IP_CSUM;
+       else
+               priv->tx_wr.send_flags &=
+                       ~(IB_SEND_UDP_TCP_CSUM | IB_SEND_IP_CSUM);
+
+
        if (unlikely(post_send(priv, priv->tx_head & (ipoib_sendq_size - 1),
                               address->ah, qpn,
                               tx_req->mapping, skb_headlen(skb),
Index: ofa_1_3_dev_kernel/drivers/infiniband/ulp/ipoib/ipoib_main.c
===================================================================
--- ofa_1_3_dev_kernel.orig/drivers/infiniband/ulp/ipoib/ipoib_main.c   
2007-09-24 12:23:00.000000000 +0200
+++ ofa_1_3_dev_kernel/drivers/infiniband/ulp/ipoib/ipoib_main.c        
2007-09-24 13:04:52.000000000 +0200
@@ -1109,6 +1109,29 @@ int ipoib_add_pkey_attr(struct net_devic
        return device_create_file(&dev->dev, &dev_attr_pkey);
 }
 
+static void set_tx_csum(struct net_device *dev)
+{
+       struct ipoib_dev_priv *priv = netdev_priv(dev);
+
+       if (test_bit(IPOIB_FLAG_ADMIN_CM, &priv->flags))
+               return;
+
+       if (!(priv->ca->flags & IB_DEVICE_IP_CSUM))
+               return;
+
+       dev->features |= NETIF_F_SG | NETIF_F_IP_CSUM; /* turn on ipv6 too */
+}
+
+static void set_rx_csum(struct net_device *dev)
+{
+       struct ipoib_dev_priv *priv = netdev_priv(dev);
+
+       if (!(priv->ca->flags & IB_DEVICE_IP_CSUM))
+               return;
+
+       set_bit(IPOIB_FLAG_RX_CSUM, &priv->flags);
+}
+
 static struct net_device *ipoib_add_port(const char *format,
                                         struct ib_device *hca, u8 port)
 {
@@ -1165,6 +1188,9 @@ static struct net_device *ipoib_add_port
                goto event_failed;
        }
 
+       set_tx_csum(priv->dev);
+       set_rx_csum(priv->dev);
+
        result = register_netdev(priv->dev);
        if (result) {
                printk(KERN_WARNING "%s: couldn't register ipoib port %d; error 
%d\n",

_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to