From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Dumazet Subject: Re: [PATCH v4 net-next] net: introduce dev_set_forwarding() Date: Thu, 07 Nov 2013 20:06:34 -0800 Message-ID: <1383883594.9412.158.camel@edumazet-glaptop2.roam.corp.google.com> References: <1383584130.1553.2.camel@bwh-desktop.uk.level5networks.com> <20131107.161755.355597965034457889.davem@davemloft.net> <20131107213111.GA6885@gondor.apana.org.au> <20131107.170651.1819930710384730745.davem@davemloft.net> <20131108021758.GA13467@gondor.apana.org.au> <1383878554.9412.153.camel@edumazet-glaptop2.roam.corp.google.com> <20131108032208.GA14056@gondor.apana.org.au> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: David Miller , bhutchings@solarflare.com, christoph.paasch@uclouvain.be, netdev@vger.kernel.org, hkchu@google.com, mwdalton@google.com To: Herbert Xu Return-path: Received: from mail-pa0-f46.google.com ([209.85.220.46]:47918 "EHLO mail-pa0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750923Ab3KHEGg (ORCPT ); Thu, 7 Nov 2013 23:06:36 -0500 Received: by mail-pa0-f46.google.com with SMTP id kl14so304764pab.19 for ; Thu, 07 Nov 2013 20:06:36 -0800 (PST) In-Reply-To: <20131108032208.GA14056@gondor.apana.org.au> Sender: netdev-owner@vger.kernel.org List-ID: On Fri, 2013-11-08 at 11:22 +0800, Herbert Xu wrote: > If it really hurt then they can always turn it off using ethtool. > With your patch that automatically turns it off virt folks have > no way of even turning it on. Let me remind you that before the GRO frag_list patch, virt folks had no choice anyway. GRO packets were limited to 16 MSS. Its not because we can build large packets that we must do so. This was an error of our TSO implementation.