Attention is currently required from: fixeria, jolly, laforge.

pespin has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/41531?usp=email )

Change subject: sctp_common: gen_sockopts/1: add heartbeat and RTO params ......................................................................

Patch Set 2:

(1 comment)

File config/sys.config:

https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/41531/comment/733d9bde_264b9... : PS2, Line 17: %% * rto_initial - RTO (retransmission time-out) initial, ms (default: 1000)

do you mind sharing to which sysctls/sockopts those values relate?

Linux sysctl defaults on 6.17.8-arch1-1: net.sctp.rto_initial = 3000 net.sctp.rto_max = 60000

net.sctp.rto_min = 1000

tbh, a rto_max of 1s for a satellite link looks a bit constrained to me.And iiuc by setting both rto_min and max to 1000 you are in practice establishing the RTO to 1000 in all cases, which may be suboptimal and problematic since you don't give the kernel space to accommodate to current connection status.