netdev - Re: [PATCH 5/7] [CCID-3]: Remove ugly RTT-sampling history lookup

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20080527.063651.236273094.davem@davemloft.net>
Date:	Tue, 27 May 2008 06:36:51 -0700 (PDT)
From:	David Miller <davem@...emloft.net>
To:	gerrit@....abdn.ac.uk
Cc:	dccp@...r.kernel.org, netdev@...r.kernel.org
Subject: Re: [PATCH 5/7] [CCID-3]: Remove ugly RTT-sampling history lookup

From: Gerrit Renker <gerrit@....abdn.ac.uk>
Date: Tue, 27 May 2008 09:32:45 +0100

> 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@....abdn.ac.uk>

This mixes bug fixes with cleanups, and also conflicts because I'm
not applying the hctx prefix changes.

Look, if you want to make the member names shorter, shorten the post-
prefix part of the name, instead of getting rid of the useful prefix
part.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html