Re: [PATCH rdma-next 1/2] IB/cma: Resolve route only while receiving CM requests

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

 



On Wed, Mar 21, 2018 at 05:16:35PM +0200, Leon Romanovsky wrote:
> From: Parav Pandit <parav@xxxxxxxxxxxx>
> 
> Currently CM request for RoCE follows following flow.
> rdma_create_id()
> rdma_resolve_addr()
> rdma_resolve_route()
> For RC QPs:
> rdma_connect()
> ->cma_connect_ib()
>   ->ib_send_cm_req()
>     ->cm_init_av_by_path()
>       ->ib_init_ah_attr_from_path()
> For UD QPs:
> rdma_connect()
> ->cma_resolve_ib_udp()
>   ->ib_send_cm_sidr_req()
>     ->cm_init_av_by_path()
>       ->ib_init_ah_attr_from_path()
> 
> In both the flows, route is already resolved before sending CM requests.
> Therefore, code is refactored to avoid resolving route second time in
> ib_cm layer.
> ib_init_ah_attr_from_path() is extended to resolve route when it is not
> yet resolved for RoCE link layer. This is achieved by caller setting
> route_resolved field in path record whenever it has route already
> resolved.
> 
> Signed-off-by: Parav Pandit <parav@xxxxxxxxxxxx>
> Signed-off-by: Leon Romanovsky <leonro@xxxxxxxxxxxx>
> ---
>  drivers/infiniband/core/cm.c       |  7 ++++++-
>  drivers/infiniband/core/cma.c      |  1 +
>  drivers/infiniband/core/sa_query.c | 11 ++++++++---
>  include/rdma/ib_sa.h               |  8 ++++++++
>  4 files changed, 23 insertions(+), 4 deletions(-)

As discussed I applied the revised version from here to for-next:

https://patchwork.kernel.org/patch/10300505/

Thanks,
Jason
--
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