Author: np
Date: Wed Dec 19 23:29:01 2018
New Revision: 342266
URL: https://svnweb.freebsd.org/changeset/base/342266

Log:
  cxgbe/iw_cxgbe: Use DSGLs to write to card's memory when appropriate.
  
  Submitted by: Krishnamraju Eraparaju @ Chelsio
  Sponsored by: Chelsio Communications

Modified:
  head/sys/dev/cxgbe/iw_cxgbe/cm.c
  head/sys/dev/cxgbe/iw_cxgbe/iw_cxgbe.h
  head/sys/dev/cxgbe/iw_cxgbe/mem.c
  head/sys/dev/cxgbe/iw_cxgbe/provider.c
  head/sys/dev/cxgbe/iw_cxgbe/qp.c
  head/sys/dev/cxgbe/t4_main.c

Modified: head/sys/dev/cxgbe/iw_cxgbe/cm.c
==============================================================================
--- head/sys/dev/cxgbe/iw_cxgbe/cm.c    Wed Dec 19 23:28:56 2018        
(r342265)
+++ head/sys/dev/cxgbe/iw_cxgbe/cm.c    Wed Dec 19 23:29:01 2018        
(r342266)
@@ -1265,6 +1265,15 @@ static int snd_win = 128 * 1024;
 SYSCTL_INT(_hw_iw_cxgbe, OID_AUTO, snd_win, CTLFLAG_RWTUN, &snd_win, 0,
                "TCP send window in bytes (default = 128KB)");
 
+int use_dsgl = 1;
+SYSCTL_INT(_hw_iw_cxgbe, OID_AUTO, use_dsgl, CTLFLAG_RWTUN, &use_dsgl, 0,
+               "Use DSGL for PBL/FastReg (default=1)");
+
+int inline_threshold = 128;
+SYSCTL_INT(_hw_iw_cxgbe, OID_AUTO, inline_threshold, CTLFLAG_RWTUN, 
&inline_threshold, 0,
+               "inline vs dsgl threshold (default=128)");
+
+
 static void
 start_ep_timer(struct c4iw_ep *ep)
 {

Modified: head/sys/dev/cxgbe/iw_cxgbe/iw_cxgbe.h
==============================================================================
--- head/sys/dev/cxgbe/iw_cxgbe/iw_cxgbe.h      Wed Dec 19 23:28:56 2018        
(r342265)
+++ head/sys/dev/cxgbe/iw_cxgbe/iw_cxgbe.h      Wed Dec 19 23:29:01 2018        
(r342266)
@@ -70,6 +70,9 @@
 #define KTR_IW_CXGBE   KTR_SPARE3
 
 extern int c4iw_debug;
+extern int use_dsgl;
+extern int inline_threshold;
+
 #define PDBG(fmt, args...) \
 do { \
        if (c4iw_debug) \

Modified: head/sys/dev/cxgbe/iw_cxgbe/mem.c
==============================================================================
--- head/sys/dev/cxgbe/iw_cxgbe/mem.c   Wed Dec 19 23:28:56 2018        
(r342265)
+++ head/sys/dev/cxgbe/iw_cxgbe/mem.c   Wed Dec 19 23:29:01 2018        
(r342266)
@@ -45,9 +45,9 @@ __FBSDID("$FreeBSD$");
 #include <common/t4_msg.h>
 #include "iw_cxgbe.h"
 
-int use_dsgl = 1;
 #define T4_ULPTX_MIN_IO 32
 #define C4IW_MAX_INLINE_SIZE 96
+#define T4_ULPTX_MAX_DMA 1024
 
 static int
 mr_exceeds_hw_limits(struct c4iw_dev *dev, u64 length)
@@ -57,10 +57,60 @@ mr_exceeds_hw_limits(struct c4iw_dev *dev, u64 length)
 }
 
 static int
-write_adapter_mem(struct c4iw_rdev *rdev, u32 addr, u32 len, void *data)
+_c4iw_write_mem_dma_aligned(struct c4iw_rdev *rdev, u32 addr, u32 len,
+                               void *data, int wait)
 {
        struct adapter *sc = rdev->adap;
        struct ulp_mem_io *ulpmc;
+       struct ulptx_sgl *sgl;
+       u8 wr_len;
+       int ret = 0;
+       struct c4iw_wr_wait wr_wait;
+       struct wrqe *wr;
+
+       addr &= 0x7FFFFFF;
+
+       if (wait)
+               c4iw_init_wr_wait(&wr_wait);
+       wr_len = roundup(sizeof *ulpmc + sizeof *sgl, 16);
+
+       wr = alloc_wrqe(wr_len, &sc->sge.ctrlq[0]);
+       if (wr == NULL)
+               return -ENOMEM;
+       ulpmc = wrtod(wr);
+
+       memset(ulpmc, 0, wr_len);
+       INIT_ULPTX_WR(ulpmc, wr_len, 0, 0);
+       ulpmc->wr.wr_hi = cpu_to_be32(V_FW_WR_OP(FW_ULPTX_WR) |
+                                   (wait ? F_FW_WR_COMPL : 0));
+       ulpmc->wr.wr_lo = wait ? (u64)(unsigned long)&wr_wait : 0;
+       ulpmc->wr.wr_mid = cpu_to_be32(V_FW_WR_LEN16(DIV_ROUND_UP(wr_len, 16)));
+       ulpmc->cmd = cpu_to_be32(V_ULPTX_CMD(ULP_TX_MEM_WRITE) |
+                              V_T5_ULP_MEMIO_ORDER(1) |
+                       V_T5_ULP_MEMIO_FID(sc->sge.ofld_rxq[0].iq.abs_id));
+       ulpmc->dlen = cpu_to_be32(V_ULP_MEMIO_DATA_LEN(len>>5));
+       ulpmc->len16 = cpu_to_be32(DIV_ROUND_UP(wr_len-sizeof(ulpmc->wr), 16));
+       ulpmc->lock_addr = cpu_to_be32(V_ULP_MEMIO_ADDR(addr));
+
+       sgl = (struct ulptx_sgl *)(ulpmc + 1);
+       sgl->cmd_nsge = cpu_to_be32(V_ULPTX_CMD(ULP_TX_SC_DSGL) |
+                                   V_ULPTX_NSGE(1));
+       sgl->len0 = cpu_to_be32(len);
+       sgl->addr0 = cpu_to_be64((u64)data);
+
+       t4_wrq_tx(sc, wr);
+
+       if (wait)
+               ret = c4iw_wait_for_reply(rdev, &wr_wait, 0, 0, NULL, __func__);
+       return ret;
+}
+
+
+static int
+_c4iw_write_mem_inline(struct c4iw_rdev *rdev, u32 addr, u32 len, void *data)
+{
+       struct adapter *sc = rdev->adap;
+       struct ulp_mem_io *ulpmc;
        struct ulptx_idata *ulpsc;
        u8 wr_len, *to_dp, *from_dp;
        int copy_len, num_wqe, i, ret = 0;
@@ -84,7 +134,7 @@ write_adapter_mem(struct c4iw_rdev *rdev, u32 addr, u3
 
                wr = alloc_wrqe(wr_len, &sc->sge.ctrlq[0]);
                if (wr == NULL)
-                       return (0);
+                       return -ENOMEM;
                ulpmc = wrtod(wr);
 
                memset(ulpmc, 0, wr_len);
@@ -93,7 +143,8 @@ write_adapter_mem(struct c4iw_rdev *rdev, u32 addr, u3
                if (i == (num_wqe-1)) {
                        ulpmc->wr.wr_hi = cpu_to_be32(V_FW_WR_OP(FW_ULPTX_WR) |
                                                    F_FW_WR_COMPL);
-                       ulpmc->wr.wr_lo = (__force __be64)(unsigned long) 
&wr_wait;
+                       ulpmc->wr.wr_lo =
+                                      (__force __be64)(unsigned long) &wr_wait;
                } else
                        ulpmc->wr.wr_hi = cpu_to_be32(V_FW_WR_OP(FW_ULPTX_WR));
                ulpmc->wr.wr_mid = cpu_to_be32(
@@ -126,6 +177,69 @@ write_adapter_mem(struct c4iw_rdev *rdev, u32 addr, u3
        ret = c4iw_wait_for_reply(rdev, &wr_wait, 0, 0, NULL, __func__);
        return ret;
 }
+
+static int
+_c4iw_write_mem_dma(struct c4iw_rdev *rdev, u32 addr, u32 len, void *data)
+{
+       struct c4iw_dev *rhp = rdev_to_c4iw_dev(rdev);
+       u32 remain = len;
+       u32 dmalen;
+       int ret = 0;
+       dma_addr_t daddr;
+       dma_addr_t save;
+
+       daddr = dma_map_single(rhp->ibdev.dma_device, data, len, DMA_TO_DEVICE);
+       if (dma_mapping_error(rhp->ibdev.dma_device, daddr))
+               return -1;
+       save = daddr;
+
+       while (remain > inline_threshold) {
+               if (remain < T4_ULPTX_MAX_DMA) {
+                       if (remain & ~T4_ULPTX_MIN_IO)
+                               dmalen = remain & ~(T4_ULPTX_MIN_IO-1);
+                       else
+                               dmalen = remain;
+               } else
+                       dmalen = T4_ULPTX_MAX_DMA;
+               remain -= dmalen;
+               ret = _c4iw_write_mem_dma_aligned(rdev, addr, dmalen,
+                               (void *)daddr, !remain);
+               if (ret)
+                       goto out;
+               addr += dmalen >> 5;
+               data = (u64 *)data + dmalen;
+               daddr = daddr + dmalen;
+       }
+       if (remain)
+               ret = _c4iw_write_mem_inline(rdev, addr, remain, data);
+out:
+       dma_unmap_single(rhp->ibdev.dma_device, save, len, DMA_TO_DEVICE);
+       return ret;
+}
+
+/*
+ * write len bytes of data into addr (32B aligned address)
+ * If data is NULL, clear len byte of memory to zero.
+ */
+static int
+write_adapter_mem(struct c4iw_rdev *rdev, u32 addr, u32 len,
+                            void *data)
+{
+       if (rdev->adap->params.ulptx_memwrite_dsgl && use_dsgl) {
+               if (len > inline_threshold) {
+                       if (_c4iw_write_mem_dma(rdev, addr, len, data)) {
+                               log(LOG_ERR, "%s: dma map "
+                                      "failure (non fatal)\n", __func__);
+                               return _c4iw_write_mem_inline(rdev, addr, len,
+                                                             data);
+                       } else
+                               return 0;
+               } else
+                       return _c4iw_write_mem_inline(rdev, addr, len, data);
+       } else
+               return _c4iw_write_mem_inline(rdev, addr, len, data);
+}
+
 
 /*
  * Build and write a TPT entry.

Modified: head/sys/dev/cxgbe/iw_cxgbe/provider.c
==============================================================================
--- head/sys/dev/cxgbe/iw_cxgbe/provider.c      Wed Dec 19 23:28:56 2018        
(r342265)
+++ head/sys/dev/cxgbe/iw_cxgbe/provider.c      Wed Dec 19 23:29:01 2018        
(r342266)
@@ -46,7 +46,7 @@ __FBSDID("$FreeBSD$");
 
 #include "iw_cxgbe.h"
 #include "user.h"
-extern int use_dsgl;
+
 static int fastreg_support = 1;
 module_param(fastreg_support, int, 0644);
 MODULE_PARM_DESC(fastreg_support, "Advertise fastreg support (default = 1)");

Modified: head/sys/dev/cxgbe/iw_cxgbe/qp.c
==============================================================================
--- head/sys/dev/cxgbe/iw_cxgbe/qp.c    Wed Dec 19 23:28:56 2018        
(r342265)
+++ head/sys/dev/cxgbe/iw_cxgbe/qp.c    Wed Dec 19 23:29:01 2018        
(r342266)
@@ -65,7 +65,7 @@ struct cpl_set_tcb_rpl;
 
 #include "iw_cxgbe.h"
 #include "user.h"
-extern int use_dsgl;
+
 static int creds(struct toepcb *toep, struct inpcb *inp, size_t wrsize);
 static int max_fr_immd = T4_MAX_FR_IMMD;//SYSCTL parameter later...
 

Modified: head/sys/dev/cxgbe/t4_main.c
==============================================================================
--- head/sys/dev/cxgbe/t4_main.c        Wed Dec 19 23:28:56 2018        
(r342265)
+++ head/sys/dev/cxgbe/t4_main.c        Wed Dec 19 23:29:01 2018        
(r342266)
@@ -4092,6 +4092,18 @@ get_params__post_init(struct adapter *sc)
        else
                sc->params.filter2_wr_support = 0;
 
+       /*
+        * Find out whether we're allowed to use the ULPTX MEMWRITE DSGL.
+        * This is queried separately for the same reason as other params above.
+        */
+       param[0] = FW_PARAM_DEV(ULPTX_MEMWRITE_DSGL);
+       val[0] = 0;
+       rc = -t4_query_params(sc, sc->mbox, sc->pf, 0, 1, param, val);
+       if (rc == 0)
+               sc->params.ulptx_memwrite_dsgl = val[0] != 0;
+       else
+               sc->params.ulptx_memwrite_dsgl = false;
+
        /* get capabilites */
        bzero(&caps, sizeof(caps));
        caps.op_to_write = htobe32(V_FW_CMD_OP(FW_CAPS_CONFIG_CMD) |
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "[email protected]"

Reply via email to