On Thu, May 17, 2018 at 9:59 AM Yuchung Cheng <ych...@google.com> wrote:
> > Thanks for implementing this! Overall this patch seems nice to me.
> > Acked-by: Neal Cardwell <ncardw...@google.com>
> > BTW, I guess we should spread the word to maintainers of other major TCP
> > stacks that they need to be prepared for what may be a much higher
> > of compression/aggregation in the SACK stream. Linux stacks going back
> > years should be fine with this, but I'm not sure about the other major
> > (they may only allow sending one MSS per ACK-with-SACKs received).
> Patch looks really good but Neal's comment just reminds me a potential
> legacy issue.
> I recall at least Apple and Windows TCP stacks still need 3+ DUPACKs
> (!= a SACK covering 3+ packets) to trigger fast recovery. Will we have
> an issue there interacting w/ these stacks?
Then we should revert GRO :)
Really it is time for these stacks to catch up, or give up to QUIC :/