IB/mlx4: Fix check of max_qp_dest_rdma in modify QP
authorEli Cohen <eli@mellanox.co.il>
Thu, 17 May 2007 13:32:39 +0000 (16:32 +0300)
committerRoland Dreier <rolandd@cisco.com>
Sat, 19 May 2007 15:51:56 +0000 (08:51 -0700)
max_qp_dest_rdma is already in natural units - no need to shift.  This
was discovered by a test that deliberately requests more outstanding
atomic operation than the device supports.

Found by Sagi Rotem at Mellanox.

Signed-off-by: Eli Cohen <eli@mellanox.co.il>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
drivers/infiniband/hw/mlx4/qp.c

index 5cd70690845078ff298125f24c14380deca3598a..9c362fa7146597bd3df50a78ea733b7c36692710 100644 (file)
@@ -694,7 +694,7 @@ int mlx4_ib_modify_qp(struct ib_qp *ibqp, struct ib_qp_attr *attr,
        }
 
        if (attr_mask & IB_QP_MAX_DEST_RD_ATOMIC &&
-           attr->max_dest_rd_atomic > 1 << dev->dev->caps.max_qp_dest_rdma) {
+           attr->max_dest_rd_atomic > dev->dev->caps.max_qp_dest_rdma) {
                goto out;
        }