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

2014-02-24 Thread Sagi Grimberg

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 
---
  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 */
+   boolsignature_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




@@ -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 
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 
Signed-off-by: Eli Cohen 
Signed-off-by: Roland Dreier 

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 majord...@vger.kernel.org
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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


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

2014-02-23 Thread Nicholas A. Bellinger
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 
> ---
>  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 */
> + boolsignature_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



> @@ -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.

--nab

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