public inbox for netdev@vger.kernel.org
 help / color / mirror / Atom feed
From: Eric Dumazet <eric.dumazet@gmail.com>
To: David Miller <davem@davemloft.net>, Jay Vosburgh <fubar@us.ibm.com>
Cc: shemminger@vyatta.com, bphilips@suse.de,
	e1000-devel@lists.sourceforge.net, bruce.w.allan@intel.com,
	jesse.brandeburg@intel.com, john.ronciak@intel.com,
	jeffrey.t.kirsher@intel.com, netdev@vger.kernel.org,
	bhutchings@solarflare.com
Subject: [PATCH] bonding: enable gro by default
Date: Fri, 17 Sep 2010 21:25:07 +0200	[thread overview]
Message-ID: <1284751507.2235.17.camel@edumazet-laptop> (raw)
In-Reply-To: <20100917.115707.106807995.davem@davemloft.net>

Le vendredi 17 septembre 2010 à 11:57 -0700, David Miller a écrit :

> Applied, thanks Eric.
> 
> Can you do whatever change is needed to get it enabled by default?
> 

Sure ;)

[PATCH] bonding: enable gro by default

gro can be enabled by default on bonding devices.

Actual support depends on the lower devices.

One can still use ethtool to switch off GRO if needed.

Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
---
 drivers/net/bonding/bond_main.c |    4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c
index 3b16f62..80610a6 100644
--- a/drivers/net/bonding/bond_main.c
+++ b/drivers/net/bonding/bond_main.c
@@ -4678,6 +4678,10 @@ static void bond_setup(struct net_device *bond_dev)
 			       NETIF_F_HW_VLAN_RX |
 			       NETIF_F_HW_VLAN_FILTER);
 
+	/* By default, we enable GRO on bonding devices.
+	 * Actual support requires lowlevel drivers are GRO ready.
+	 */
+	bond_dev->features |= NETIF_F_GRO;
 }
 
 static void bond_work_cancel_all(struct bonding *bond)



  reply	other threads:[~2010-09-17 19:25 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-15  1:50 [PATCH] ixgbe: add GRO to vlan_features Brandon Philips
2010-09-15  4:35 ` David Miller
2010-09-15 16:37   ` [PATCH] vlan: enable GRO if real_dev supports it Brandon Philips
2010-09-15 17:08     ` Eric Dumazet
2010-09-15 17:14       ` David Miller
2010-09-15 17:24         ` Ben Hutchings
2010-09-15 17:29           ` David Miller
2010-09-15 19:24           ` [PATCH] net: enable GRO by default for vlan devices Brandon Philips
2010-09-15 22:03             ` Eric Dumazet
2010-09-15 22:28               ` Eric Dumazet
2010-09-16  1:16                 ` Stephen Hemminger
2010-09-16  5:51                   ` [E1000-devel] " David Miller
2010-09-16  6:21                     ` Stephen Hemminger
2010-09-16  6:38                       ` David Miller
2010-09-16  6:42                         ` Eric Dumazet
2010-09-16  8:11                           ` [PATCH] ethtool: change ethtool_set_gro() to use ethtool_op_get_rx_csum Eric Dumazet
2010-09-17 18:57                             ` David Miller
2010-09-17 19:25                               ` Eric Dumazet [this message]
2010-09-17 23:54                                 ` [PATCH] bonding: enable gro by default David Miller
2010-09-16  5:32               ` [PATCH] net: enable GRO by default for vlan devices David Miller
2010-09-15 18:59       ` [PATCH] vlan: enable GRO if real_dev supports it Brandon Philips
2010-09-15 16:38   ` [PATCH] ixgbe: add GRO to vlan_features Brandon Philips

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1284751507.2235.17.camel@edumazet-laptop \
    --to=eric.dumazet@gmail.com \
    --cc=bhutchings@solarflare.com \
    --cc=bphilips@suse.de \
    --cc=bruce.w.allan@intel.com \
    --cc=davem@davemloft.net \
    --cc=e1000-devel@lists.sourceforge.net \
    --cc=fubar@us.ibm.com \
    --cc=jeffrey.t.kirsher@intel.com \
    --cc=jesse.brandeburg@intel.com \
    --cc=john.ronciak@intel.com \
    --cc=netdev@vger.kernel.org \
    --cc=shemminger@vyatta.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox