Message ID | 4A6960EF.5020204@gmail.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
On Fri, 2009-07-24 at 00:21 -0700, Eric Dumazet wrote: > Based on Patrick feedback, I believe we can add this followup too > > [PATCH net-next-2.6] bnx2x: Dont update vlan_features in bnx2x_set_tso() > > Patrick said : "vlan_features doesn't need to be updated, the resulting > dev->features of the VLAN device is computed as the intersection of > dev->features and dev->vlan_features." > Thanks Eric > Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com> Acked-by: Eilon Greenstein <eilong@broadcom.com> -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
From: "Eilon Greenstein" <eilong@broadcom.com> Date: Sun, 26 Jul 2009 13:53:43 +0300 > On Fri, 2009-07-24 at 00:21 -0700, Eric Dumazet wrote: >> Based on Patrick feedback, I believe we can add this followup too >> >> [PATCH net-next-2.6] bnx2x: Dont update vlan_features in bnx2x_set_tso() >> >> Patrick said : "vlan_features doesn't need to be updated, the resulting >> dev->features of the VLAN device is computed as the intersection of >> dev->features and dev->vlan_features." >> > Thanks Eric > >> Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com> > Acked-by: Eilon Greenstein <eilong@broadcom.com> Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/bnx2x_main.c b/drivers/net/bnx2x_main.c index c4c42b3..a2de0cd 100644 --- a/drivers/net/bnx2x_main.c +++ b/drivers/net/bnx2x_main.c @@ -9349,17 +9349,9 @@ static int bnx2x_set_tso(struct net_device *dev, u32 data) if (data) { dev->features |= (NETIF_F_TSO | NETIF_F_TSO_ECN); dev->features |= NETIF_F_TSO6; -#ifdef BCM_VLAN - dev->vlan_features |= (NETIF_F_TSO | NETIF_F_TSO_ECN); - dev->vlan_features |= NETIF_F_TSO6; -#endif } else { dev->features &= ~(NETIF_F_TSO | NETIF_F_TSO_ECN); dev->features &= ~NETIF_F_TSO6; -#ifdef BCM_VLAN - dev->vlan_features &= ~(NETIF_F_TSO | NETIF_F_TSO_ECN); - dev->vlan_features &= ~NETIF_F_TSO6; -#endif } return 0;