[PATCH] IB/mthca: Set RDMA/atomic capabilities correctly
authorRoland Dreier <roland@topspin.com>
Mon, 27 Jun 2005 21:36:41 +0000 (14:36 -0700)
committerLinus Torvalds <torvalds@ppc970.osdl.org>
Mon, 27 Jun 2005 22:11:45 +0000 (15:11 -0700)
mthca apparently had the meanings of the max_rd_atomic and max_dest_rd_atomic
QP attributes backwards.  max_rd_atomic limits the maximum number of
outstanding RDMA/atomic requests as an initiator (on a send queue), and
max_dest_rd_atomic specifies the resources allocated to handle RMDA/atomic
requests from the remote end of the connection.  We were programming our QP
context with these values swapped.

Signed-off-by: Roland Dreier <roland@topspin.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
drivers/infiniband/hw/mthca/mthca_qp.c

index 793c216b12996d073eeb1e343bb7148830f44fdb..6bed9a3e3f0f71beec0cfcea2b26b1aea48ddcbc 100644 (file)
@@ -724,9 +724,9 @@ int mthca_modify_qp(struct ib_qp *ibqp, struct ib_qp_attr *attr, int attr_mask)
                qp_param->opt_param_mask |= cpu_to_be32(MTHCA_QP_OPTPAR_RETRY_COUNT);
        }
 
-       if (attr_mask & IB_QP_MAX_DEST_RD_ATOMIC) {
-               qp_context->params1 |= cpu_to_be32(min(attr->max_dest_rd_atomic ?
-                                                      ffs(attr->max_dest_rd_atomic) - 1 : 0,
+       if (attr_mask & IB_QP_MAX_QP_RD_ATOMIC) {
+               qp_context->params1 |= cpu_to_be32(min(attr->max_rd_atomic ?
+                                                      ffs(attr->max_rd_atomic) - 1 : 0,
                                                       7) << 21);
                qp_param->opt_param_mask |= cpu_to_be32(MTHCA_QP_OPTPAR_SRA_MAX);
        }
@@ -764,10 +764,10 @@ int mthca_modify_qp(struct ib_qp *ibqp, struct ib_qp_attr *attr, int attr_mask)
                qp->atomic_rd_en = attr->qp_access_flags;
        }
 
-       if (attr_mask & IB_QP_MAX_QP_RD_ATOMIC) {
+       if (attr_mask & IB_QP_MAX_DEST_RD_ATOMIC) {
                u8 rra_max;
 
-               if (qp->resp_depth && !attr->max_rd_atomic) {
+               if (qp->resp_depth && !attr->max_dest_rd_atomic) {
                        /*
                         * Lowering our responder resources to zero.
                         * Turn off RDMA/atomics as responder.
@@ -778,7 +778,7 @@ int mthca_modify_qp(struct ib_qp *ibqp, struct ib_qp_attr *attr, int attr_mask)
                                                                MTHCA_QP_OPTPAR_RAE);
                }
 
-               if (!qp->resp_depth && attr->max_rd_atomic) {
+               if (!qp->resp_depth && attr->max_dest_rd_atomic) {
                        /*
                         * Increasing our responder resources from
                         * zero.  Turn on RDMA/atomics as appropriate.
@@ -799,7 +799,7 @@ int mthca_modify_qp(struct ib_qp *ibqp, struct ib_qp_attr *attr, int attr_mask)
                }
 
                for (rra_max = 0;
-                    1 << rra_max < attr->max_rd_atomic &&
+                    1 << rra_max < attr->max_dest_rd_atomic &&
                             rra_max < dev->qp_table.rdb_shift;
                     ++rra_max)
                        ; /* nothing */
@@ -807,7 +807,7 @@ int mthca_modify_qp(struct ib_qp *ibqp, struct ib_qp_attr *attr, int attr_mask)
                qp_context->params2      |= cpu_to_be32(rra_max << 21);
                qp_param->opt_param_mask |= cpu_to_be32(MTHCA_QP_OPTPAR_RRA_MAX);
 
-               qp->resp_depth = attr->max_rd_atomic;
+               qp->resp_depth = attr->max_dest_rd_atomic;
        }
 
        qp_context->params2 |= cpu_to_be32(MTHCA_QP_BIT_RSC);