RE: [PATCH rdma-next v1 11/13] IB/{core, cm, cma, ipoib}: Resolve route only while sending CM requests

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

 



Hi Jason, Doug,

> -----Original Message-----
> From: Leon Romanovsky [mailto:leon@xxxxxxxxxx]
> Sent: Tuesday, March 13, 2018 9:06 AM
> To: Doug Ledford <dledford@xxxxxxxxxx>; Jason Gunthorpe
> <jgg@xxxxxxxxxxxx>
> Cc: Leon Romanovsky <leonro@xxxxxxxxxxxx>; RDMA mailing list <linux-
> rdma@xxxxxxxxxxxxxxx>; Daniel Jurgens <danielj@xxxxxxxxxxxx>; Mark Bloch
> <markb@xxxxxxxxxxxx>; Parav Pandit <parav@xxxxxxxxxxxx>
> Subject: [PATCH rdma-next v1 11/13] IB/{core, cm, cma, ipoib}: Resolve route
> only while sending CM requests
> 

Subject should be: Resolve route only when receiving CM requests.
Can you please change the subject while applying this patch?



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