On Wed, 30 May 2007 09:58:38 -0700 "Laurent Chavey" <[EMAIL PROTECTED]> wrote:
> should we then move/integrate that code as part of the bonding driver ? > > > On 5/30/07, Stephen Hemminger <[EMAIL PROTECTED]> wrote: > > On Tue, 29 May 2007 17:41:45 -0700 > > "Laurent Chavey" <[EMAIL PROTECTED]> wrote: > > > > > kernel version <= 2.6.20.1 > > > file drivers/net/bonding/bonding_main.c > > > function bond_compute_features() > > > > > > ----------- > > > Given a system with two different NIC. One driver sets > > > dev->features |= NETIF_F_HW_CSUM > > > the other driver sets > > > dev->features |= NETIF_F_IP_CSUM > > > > > > when enslaving the 2 device above, bond_compute_features() > > > does not set the intersection of the 2 CSUM > > > features (should be NETIF_F_IP_CSUM) > > > ----------- > > > > > > should the bond features in the case above include NETIF_F_IP_CSUM > > > - > > > To unsubscribe from this list: send the line "unsubscribe netdev" in > > > the body of a message to [EMAIL PROTECTED] > > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > > > The bridge code already has a more complete/complex recompute > > routine to handle this. > > > > -- > > Stephen Hemminger <[EMAIL PROTECTED]> > > Sure, that would be easier for maintenance. -- Stephen Hemminger <[EMAIL PROTECTED]> - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html