On 08/09/2013 02:39 PM, Karl Heiss wrote:
Hello,
I happened to be perusing the SCTP code in Linus' tree and noticed
that the following is happening in net/sctp/associola.c,
sctp_assoc_control_transport():
/* Don't inform ULP about transition from PF to
* active state and set cwnd to 1, see SCTP
* Quick failover draft section 5.1, point 5
*/
if (transport->state == SCTP_PF) {
ulp_notify = false;
transport->cwnd = 1;
}
From the quick failover draft, I see that the cwnd should be 1*MTU.
This doesn't seem correct as other places set the cwnd as:
transport->cwnd = asoc->pathmtu;
Am I missing something or is this just a simplification?
I think you are correct, that doesn't seem right.
Fix is on its way.
--
To unsubscribe from this list: send the line "unsubscribe linux-sctp" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
