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