Skip to content

Commit 876e3d3

Browse files
Dag Moxnesjfvogel
authored andcommitted
rds:ib: Set RoCE ACK timeout before resolving route
The ACK timeout can now be used to estimate the PacketLifeTime for RoCE. Set RoCE ACK timeout before resolving route in RDS, so that the ACK timeout can be used for the PacketLifeTime for the route. Orabug: 30556591 Signed-off-by: Dag Moxnes <[email protected]> Reviewed-by: Håkon Bugge <[email protected]> Signed-off-by: Somasundaram Krishnasamy <[email protected]> Orabug: 30820081 UEK5 => UEK6 (cherry picked from commit 5aa8e27) cherry-pick-repo=UEK/production/linux-uek.git Signed-off-by: Gerd Rausch <[email protected]> Reviewed-by: Sharon Liu <[email protected]>
1 parent 5c53c2b commit 876e3d3

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

net/rds/ib_cm.c

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1424,9 +1424,6 @@ int rds_ib_cm_initiate_connect(struct rdma_cm_id *cm_id, bool isv6)
14241424
rds_ib_cm_fill_conn_param(conn, &conn_param, &dp,
14251425
conn->c_proposed_version, UINT_MAX, UINT_MAX,
14261426
frag, isv6, seq);
1427-
if (rds_ib_sysctl_local_ack_timeout &&
1428-
rdma_port_get_link_layer(cm_id->device, cm_id->port_num) == IB_LINK_LAYER_ETHERNET)
1429-
rdma_set_ack_timeout(cm_id, rds_ib_sysctl_local_ack_timeout);
14301427
ret = rdma_connect(cm_id, &conn_param);
14311428
if (ret) {
14321429
rds_rtd(RDS_RTD_CM, "RDS/IB: rdma_connect failed (%d)\n", ret);

net/rds/rdma_transport.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -165,6 +165,9 @@ static int rds_rdma_cm_event_handler_cmn(struct rdma_cm_id *cm_id,
165165
cm_id->route.addr.dev_addr.dst_dev_addr +
166166
rdma_addr_gid_offset(&cm_id->route.addr.dev_addr));
167167
rdma_set_service_type(cm_id, conn->c_tos);
168+
if (rds_ib_sysctl_local_ack_timeout &&
169+
rdma_port_get_link_layer(cm_id->device, cm_id->port_num) == IB_LINK_LAYER_ETHERNET)
170+
rdma_set_ack_timeout(cm_id, rds_ib_sysctl_local_ack_timeout);
168171

169172
/* XXX do we need to clean up if this fails? */
170173
ret = rdma_resolve_route(cm_id,

0 commit comments

Comments
 (0)