From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: bond_compute_features() does not handle devices with different CSUM Date: Wed, 30 May 2007 09:27:57 -0700 Message-ID: <20070530092757.3417fb52@freepuppy> References: <97949e3e0705291741t4c485d38ud48b06e6d07e31bc@mail.gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org To: "Laurent Chavey" Return-path: Received: from smtp.osdl.org ([207.189.120.12]:57377 "EHLO smtp.osdl.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751104AbXE3QjU (ORCPT ); Wed, 30 May 2007 12:39:20 -0400 In-Reply-To: <97949e3e0705291741t4c485d38ud48b06e6d07e31bc@mail.gmail.com> Sender: netdev-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Tue, 29 May 2007 17:41:45 -0700 "Laurent Chavey" 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 majordomo@vger.kernel.org > 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