Re: [PATCH v5 04/10] IB/mlx5: Initialize mlx5_ib_qp signature related

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2/24/2014 8:46 AM, Nicholas A. Bellinger wrote:
On Sun, 2014-02-23 at 14:19 +0200, Sagi Grimberg wrote:
If user requested signature enable we Initialize
relevant mlx5_ib_qp members. we mark the qp as sig_enable
and we increase the effective SQ size, but still
limit the user max_send_wr to original size computed.
We also allow the create_qp routine to accept sig_enable
create flag.

Signed-off-by: Sagi Grimberg <sagig@xxxxxxxxxxxx>
---
  drivers/infiniband/hw/mlx5/mlx5_ib.h |    3 +++
  drivers/infiniband/hw/mlx5/qp.c      |   12 +++++++++---
  include/linux/mlx5/qp.h              |    1 +
  3 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/drivers/infiniband/hw/mlx5/mlx5_ib.h b/drivers/infiniband/hw/mlx5/mlx5_ib.h
index 79c4f14..e438f08 100644
--- a/drivers/infiniband/hw/mlx5/mlx5_ib.h
+++ b/drivers/infiniband/hw/mlx5/mlx5_ib.h
@@ -189,6 +189,9 @@ struct mlx5_ib_qp {
int create_type;
  	u32			pa_lkey;
+
+	/* Store signature errors */
+	bool			signature_en;
  };
struct mlx5_ib_cq_buf {
diff --git a/drivers/infiniband/hw/mlx5/qp.c b/drivers/infiniband/hw/mlx5/qp.c
index 7dfe8a1..01999f3 100644
--- a/drivers/infiniband/hw/mlx5/qp.c
+++ b/drivers/infiniband/hw/mlx5/qp.c
<SNIP>

@@ -665,7 +671,7 @@ static int create_kernel_qp(struct mlx5_ib_dev *dev,
  	int err;
uuari = &dev->mdev.priv.uuari;
-	if (init_attr->create_flags)
+	if (init_attr->create_flags & ~IB_QP_CREATE_SIGNATURE_EN)
  		return -EINVAL;
if (init_attr->qp_type == MLX5_IB_QPT_REG_UMR)

FYI, this particular block doesn't apply against >= v3.14-rc2 code.

Dropping it for now, and applying the rest as #5.

Please fix if necessary.

This block comes from Eli's commit that exists in Roland tree:

commit 1a4c3a3dc5fdeef2a7bdf4ac7d81df58c3c0a51e
Author: Eli Cohen <eli@xxxxxxxxxxxxxxxxxx>
Date:   Thu Feb 6 17:41:25 2014 +0200

    IB/mlx5: Don't set "block multicast loopback" capability

    Currently Connect-IB does not support blocking multicast loopback, so
    don't set IB_DEVICE_BLOCK_MULTICAST_LOOPBACK in the device caps.

    Reported by: Or Gerlitz <ogerlitz@xxxxxxxxxxxx>
    Signed-off-by: Eli Cohen <eli@xxxxxxxxxxxx>
    Signed-off-by: Roland Dreier <roland@xxxxxxxxxxxxxxx>

It's OK for you to drop this block since it reference this specific commit.

--nab

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux