dccp ccid-3: remove buggy RTT-sampling history lookup
authorGerrit Renker <gerrit@erg.abdn.ac.uk>
Tue, 14 Sep 2010 18:18:00 +0000 (20:18 +0200)
committerGerrit Renker <gerrit@erg.abdn.ac.uk>
Wed, 15 Sep 2010 10:36:02 +0000 (12:36 +0200)
commitd2c726309d88df3c5568486e4b5b9e4c3150903f
treef4aab69a2cd3f76a1cf483a20eb11295a2ea1395
parent20cbd3e120a0c20bebe420e1fed0e816730bb988
dccp ccid-3: remove buggy RTT-sampling history lookup

This removes the RTT-sampling function tfrc_tx_hist_rtt(), since

 1. it suffered from complex passing of return values (the return value both
    indicated successful lookup while the value doubled as RTT sample);

 2. when for some odd reason the sample value equalled 0, this triggered a bug
    warning about "bogus Ack", due to the ambiguity of the return value;

 3. on a passive host which has not sent anything the TX history is empty and
    thus will lead to unwanted "bogus Ack" warnings such as
    ccid3_hc_tx_packet_recv: server(e7b7d518): DATAACK with bogus ACK-28197148
    ccid3_hc_tx_packet_recv: server(e7b7d518): DATAACK with bogus ACK-26641606.

The fix is to replace the implicit encoding by performing the steps manually.

Furthermore, the "bogus Ack" warning has been removed, since it can actually be
triggered due to several reasons (network reordering, old packet, (3) above),
hence it is not very useful.

Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>
net/dccp/ccids/ccid3.c
net/dccp/ccids/lib/packet_history.c
net/dccp/ccids/lib/packet_history.h