Re: [PATCH rdma-next 15/31] RDMA/cma: Avoid setting path record type twice

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

 



On Tue, 2017-11-14 at 14:52 +0200, Leon Romanovsky wrote:
> From: Parav Pandit <parav@xxxxxxxxxxxx>
> 
> Avoid setting path record type twice for RoCE.
> 
> Signed-off-by: Parav Pandit <parav@xxxxxxxxxxxx>
> Reviewed-by: Daniel Jurgens <danielj@xxxxxxxxxxxx>
> Signed-off-by: Leon Romanovsky <leon@xxxxxxxxxx>
> ---
>  drivers/infiniband/core/cma.c | 9 ++++-----
>  1 file changed, 4 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/infiniband/core/cma.c b/drivers/infiniband/core/cma.c
> index 168f1230756b..f986597201fa 100644
> --- a/drivers/infiniband/core/cma.c
> +++ b/drivers/infiniband/core/cma.c
> @@ -2567,7 +2567,11 @@ static int cma_resolve_iboe_route(struct rdma_id_private
> *id_priv)
>  	gid_type = cma_route_gid_type(addr->dev_addr.network,
>  				      supported_gids,
>  				      id_priv->gid_type);
> +	/* Use the hint from IP Stack to select GID Type */
> +	if (gid_type < ib_network_to_gid_type(addr->dev_addr.network))
> +		gid_type = ib_network_to_gid_type(addr->dev_addr.network);
>  	route->path_rec->rec_type = sa_conv_gid_to_pathrec_type(gid_type);
> +
>  	sa_path_set_ndev(route->path_rec, &init_net);
>  	sa_path_set_ifindex(route->path_rec, ndev->ifindex);
>  	sa_path_set_dmac(route->path_rec, addr->dev_addr.dst_dev_addr);
> @@ -2577,11 +2581,6 @@ static int cma_resolve_iboe_route(struct rdma_id_private
> *id_priv)
>  	rdma_ip2gid((struct sockaddr *)&id_priv->id.route.addr.dst_addr,
>  		    &route->path_rec->dgid);
>  
> -	/* Use the hint from IP Stack to select GID Type */
> -	if (gid_type < ib_network_to_gid_type(addr->dev_addr.network))
> -		gid_type = ib_network_to_gid_type(addr->dev_addr.network);
> -	route->path_rec->rec_type = sa_conv_gid_to_pathrec_type(gid_type);
> -
>  	if (((struct sockaddr *)&id_priv->id.route.addr.dst_addr)->sa_family != AF_IB)
>  		/* TODO: get the hoplimit from the inet/inet6 device */
>  		route->path_rec->hop_limit = addr->dev_addr.hoplimit;

Reviewed-by: Knut Omang <knut.omang@xxxxxxxxxx>

Knut
--
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