From mboxrd@z Thu Jan 1 00:00:00 1970 From: Neal Cardwell Subject: Re: Linux ECN Handling Date: Tue, 21 Nov 2017 11:20:16 -0500 Message-ID: References: <20171019124312.GE16796@breakpoint.cc> <5A006CF6.1020608@iogearbox.net> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Cc: Steve Ibanez , Daniel Borkmann , Netdev , Florian Westphal , Mohammad Alizadeh , Lawrence Brakmo , Eric Dumazet To: Yuchung Cheng Return-path: Received: from mail-wr0-f182.google.com ([209.85.128.182]:40166 "EHLO mail-wr0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751242AbdKUQUi (ORCPT ); Tue, 21 Nov 2017 11:20:38 -0500 Received: by mail-wr0-f182.google.com with SMTP id s41so6378997wrc.7 for ; Tue, 21 Nov 2017 08:20:38 -0800 (PST) In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: On Tue, Nov 21, 2017 at 10:51 AM, Yuchung Cheng wrote: > On Tue, Nov 21, 2017 at 7:01 AM, Neal Cardwell wrote: >> >> The original motivation for only allowing TLP in the CA_Open state was >> to be conservative and avoid having the TLP impose extra load on the >> bottleneck when it may be congested. Plus if there are any SACKed >> packets in the SACK scoreboard then there are other existing >> mechanisms to do speedy loss recovery. > Neal I like your idea of covering more states in TLP. but shouldn't we > also fix the tso_deferral_logic to work better w/ PRR in CWR state, b/c > it's a general transmission issue. Yes, I agree it's also worthwhile to see if we can make PRR and TSO deferral play well together. Sorry, I should have been more clear about that. neal