Whamcloud - gitweb
LU-9420 lnd: Remove a bad check which slipped in 91/26891/4
authorDoug Oucharek <doug.s.oucharek@intel.com>
Fri, 28 Apr 2017 21:44:26 +0000 (14:44 -0700)
committerOleg Drokin <oleg.drokin@intel.com>
Mon, 1 May 2017 17:50:22 +0000 (17:50 +0000)
When the patch for LU-5710 landed, a check for message size was landed
that should not have been.  This check was part of a patch in LU-7650
which was later pulled because it broke things. LU-5718 picked up this
code via its many rebases (it too forever to land LU-5718 which is the
core problem here).

This patch removes that messaage size check.

Test-Parameters: trivial
Signed-off-by: Doug Oucharek <doug.s.oucharek@intel.com>
Change-Id: I3d114ec16cfbfd994efd9aee55e28a09159597be
Reviewed-on: https://review.whamcloud.com/26891
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Sonia Sharma <sonia.sharma@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Amir Shehata <amir.shehata@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lnet/klnds/o2iblnd/o2iblnd_cb.c

index aa14b1f..908d683 100644 (file)
@@ -1079,14 +1079,6 @@ kiblnd_init_rdma(kib_conn_t *conn, kib_tx_t *tx, int type,
        LASSERT(tx->tx_nwrq == 0 && tx->tx_nsge == 0);
        LASSERT(type == IBLND_MSG_GET_DONE || type == IBLND_MSG_PUT_DONE);
 
-       if (kiblnd_rd_size(srcrd) > conn->ibc_max_frags << PAGE_SHIFT) {
-               CERROR("RDMA is too large for peer %s (%d), src size: %d dst size: %d\n",
-                      libcfs_nid2str(conn->ibc_peer->ibp_nid),
-                      conn->ibc_max_frags << PAGE_SHIFT,
-                      kiblnd_rd_size(srcrd), kiblnd_rd_size(dstrd));
-               GOTO(too_big, rc = -EMSGSIZE);
-       }
-
        for (srcidx = dstidx = wrq_sge = sge_nob = 0;
             resid > 0; resid -= sge_nob) {
                int     prev = dstidx;
@@ -1156,7 +1148,6 @@ kiblnd_init_rdma(kib_conn_t *conn, kib_tx_t *tx, int type,
                tx->tx_nsge++;
        }
 
-too_big:
        if (rc < 0)     /* no RDMA if completing with failure */
                tx->tx_nwrq = tx->tx_nsge = 0;