> The flag RDMA_CORE_CAP_IB_SA is set when link layer is InfiniBand but
> also when link layer is Ethernet. Therefore, link layer is not implied by

This flag should NOT be set for Ethernet.  What is setting this?

> this flag.
> 
> ---
>  drivers/infiniband/core/cma.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/infiniband/core/cma.c b/drivers/infiniband/core/cma.c
> index 3b943b7..f96d6fd 100644
> --- a/drivers/infiniband/core/cma.c
> +++ b/drivers/infiniband/core/cma.c
> @@ -1993,7 +1993,7 @@ int rdma_resolve_route(struct rdma_cm_id *id, int
> timeout_ms)
>               return -EINVAL;
> 
>       atomic_inc(&id_priv->refcount);
> -     if (rdma_cap_ib_sa(id->device, id->port_num))
> +     if (rdma_protocol_ib(id->device, id->port_num))

This change looks wrong.  The check here has nothing to do with the link layer. 
 It is stating that route resolution must be done using an SA.


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

Reply via email to