On Tue, Sep 27, 2016 at 8:18 PM, Yuchung Cheng <ych...@google.com> wrote:
> On Tue, Sep 27, 2016 at 4:31 PM, Lawrence Brakmo <bra...@fb.com> wrote:
>> @@ -213,6 +215,8 @@ static int tcp_write_timeout(struct sock *sk)
>> tcp_mtu_probing(icsk, sk);
>>
>> dst_negative_advice(sk);
>> + } else if (prandom_u32_max(100) <
>> sysctl_tcp_rto_txhash_prob) {
> The sysctl is not really needed but that's just my opinion.
I agree that it sounds nice to simplify by not having a sysctl for
this and always calling sk_rethink_txhash() here.
neal
