Skip to content

Commit 5aa8e27

Browse files
Dag MoxnesSomasundaram Krishnasamy
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]>
1 parent dfcea06 commit 5aa8e27

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
@@ -166,6 +166,9 @@ static int rds_rdma_cm_event_handler_cmn(struct rdma_cm_id *cm_id,
166166
cm_id->route.addr.dev_addr.dst_dev_addr +
167167
rdma_addr_gid_offset(&cm_id->route.addr.dev_addr));
168168
rdma_set_service_type(cm_id, conn->c_tos);
169+
if (rds_ib_sysctl_local_ack_timeout &&
170+
rdma_port_get_link_layer(cm_id->device, cm_id->port_num) == IB_LINK_LAYER_ETHERNET)
171+
rdma_set_ack_timeout(cm_id, rds_ib_sysctl_local_ack_timeout);
169172

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

0 commit comments

Comments
 (0)