Commit 759af00e authored by Vlad Yasevich's avatar Vlad Yasevich Committed by David S. Miller

sctp: Properly timestamp outgoing data chunks for rtx purposes

Recent changes to the retransmit code exposed a long standing
bug where it was possible for a chunk to be time stamped
after the retransmit timer was reset.  This caused a rare
situation where the retrnamist timer has expired, but
nothing was marked for retrnasmission because all of
timesamps on data were less then 1 rto ago.  As result,
the timer was never restarted since nothing was retransmitted,
and this resulted in a hung association that did couldn't
complete the data transfer.  The solution is to timestamp
the chunk when it's added to the packet for transmission
purposes.  After the packet is trsnmitted the rtx timer
is restarted.  This guarantees that when the timer expires,
there will be data to retransmit.
Signed-off-by: default avatarVlad Yasevich <vladislav.yasevich@hp.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 6574df9a
...@@ -324,14 +324,16 @@ sctp_xmit_t sctp_packet_append_chunk(struct sctp_packet *packet, ...@@ -324,14 +324,16 @@ sctp_xmit_t sctp_packet_append_chunk(struct sctp_packet *packet,
switch (chunk->chunk_hdr->type) { switch (chunk->chunk_hdr->type) {
case SCTP_CID_DATA: case SCTP_CID_DATA:
retval = sctp_packet_append_data(packet, chunk); retval = sctp_packet_append_data(packet, chunk);
if (SCTP_XMIT_OK != retval)
goto finish;
/* Disallow SACK bundling after DATA. */ /* Disallow SACK bundling after DATA. */
packet->has_sack = 1; packet->has_sack = 1;
/* Disallow AUTH bundling after DATA */ /* Disallow AUTH bundling after DATA */
packet->has_auth = 1; packet->has_auth = 1;
/* Let it be knows that packet has DATA in it */ /* Let it be knows that packet has DATA in it */
packet->has_data = 1; packet->has_data = 1;
if (SCTP_XMIT_OK != retval) /* timestamp the chunk for rtx purposes */
goto finish; chunk->sent_at = jiffies;
break; break;
case SCTP_CID_COOKIE_ECHO: case SCTP_CID_COOKIE_ECHO:
packet->has_cookie_echo = 1; packet->has_cookie_echo = 1;
...@@ -470,7 +472,6 @@ int sctp_packet_transmit(struct sctp_packet *packet) ...@@ -470,7 +472,6 @@ int sctp_packet_transmit(struct sctp_packet *packet)
} else } else
chunk->resent = 1; chunk->resent = 1;
chunk->sent_at = jiffies;
has_data = 1; has_data = 1;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment