diff options
Diffstat (limited to 'net/sctp/sm_sideeffect.c')
-rw-r--r-- | net/sctp/sm_sideeffect.c | 17 |
1 files changed, 15 insertions, 2 deletions
diff --git a/net/sctp/sm_sideeffect.c b/net/sctp/sm_sideeffect.c index e1d6076b4f59..0146cfb1f182 100644 --- a/net/sctp/sm_sideeffect.c +++ b/net/sctp/sm_sideeffect.c | |||
@@ -461,9 +461,15 @@ static void sctp_do_8_2_transport_strike(struct sctp_association *asoc, | |||
461 | * expires, set RTO <- RTO * 2 ("back off the timer"). The | 461 | * expires, set RTO <- RTO * 2 ("back off the timer"). The |
462 | * maximum value discussed in rule C7 above (RTO.max) may be | 462 | * maximum value discussed in rule C7 above (RTO.max) may be |
463 | * used to provide an upper bound to this doubling operation. | 463 | * used to provide an upper bound to this doubling operation. |
464 | * | ||
465 | * Special Case: the first HB doesn't trigger exponential backoff. | ||
466 | * The first unacknowleged HB triggers it. We do this with a flag | ||
467 | * that indicates that we have an outstanding HB. | ||
464 | */ | 468 | */ |
465 | transport->last_rto = transport->rto; | 469 | if (transport->hb_sent) { |
466 | transport->rto = min((transport->rto * 2), transport->asoc->rto_max); | 470 | transport->last_rto = transport->rto; |
471 | transport->rto = min((transport->rto * 2), transport->asoc->rto_max); | ||
472 | } | ||
467 | } | 473 | } |
468 | 474 | ||
469 | /* Worker routine to handle INIT command failure. */ | 475 | /* Worker routine to handle INIT command failure. */ |
@@ -621,6 +627,11 @@ static void sctp_cmd_transport_on(sctp_cmd_seq_t *cmds, | |||
621 | t->error_count = 0; | 627 | t->error_count = 0; |
622 | t->asoc->overall_error_count = 0; | 628 | t->asoc->overall_error_count = 0; |
623 | 629 | ||
630 | /* Clear the hb_sent flag to signal that we had a good | ||
631 | * acknowledgement. | ||
632 | */ | ||
633 | t->hb_sent = 0; | ||
634 | |||
624 | /* Mark the destination transport address as active if it is not so | 635 | /* Mark the destination transport address as active if it is not so |
625 | * marked. | 636 | * marked. |
626 | */ | 637 | */ |
@@ -657,6 +668,8 @@ static void sctp_cmd_transport_reset(sctp_cmd_seq_t *cmds, | |||
657 | 668 | ||
658 | /* Mark one strike against a transport. */ | 669 | /* Mark one strike against a transport. */ |
659 | sctp_do_8_2_transport_strike(asoc, t); | 670 | sctp_do_8_2_transport_strike(asoc, t); |
671 | |||
672 | t->hb_sent = 1; | ||
660 | } | 673 | } |
661 | 674 | ||
662 | /* Helper function to process the process SACK command. */ | 675 | /* Helper function to process the process SACK command. */ |