From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrick McHardy Subject: Re: [BUG] 2.6.29-rc* QinQ vlan trunking regression Date: Thu, 05 Mar 2009 10:08:02 +0100 Message-ID: <49AF9672.50200@trash.net> References: <49AF87D6.5040104@linux.vnet.ibm.com> <49AF8CE0.6030301@trash.net> <20090305.005646.115456407.davem@davemloft.net> <20090305.005934.130220849.davem@davemloft.net> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="------------020003080003040006090208" Cc: blaschka@linux.vnet.ibm.com, bart@jukie.net, greearb@candelatech.com, shemminger@vyatta.com, dada1@cosmosbay.com, frank.blaschka@de.ibm.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org To: David Miller Return-path: Received: from stinky.trash.net ([213.144.137.162]:62015 "EHLO stinky.trash.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751405AbZCEJII (ORCPT ); Thu, 5 Mar 2009 04:08:08 -0500 In-Reply-To: <20090305.005934.130220849.davem@davemloft.net> Sender: netdev-owner@vger.kernel.org List-ID: This is a multi-part message in MIME format. --------------020003080003040006090208 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit David Miller wrote: > From: David Miller > Date: Thu, 05 Mar 2009 00:56:46 -0800 (PST) > > >> Then bond_neigh_setup() has the same bug, doesn't it? >> Indeed. But this seems to be the last one. > > Looking at the bond_main.c changes in: > > commit 008298231abbeb91bc7be9e8b078607b816d1a4a > Author: Stephen Hemminger > Date: Thu Nov 20 20:14:53 2008 -0800 > > netdev: add more functions to netdevice ops > > shows that it always behaved that way. > Yes, but that patch introduced the requirement to pass the correct device down since now the handlers need it to get to the ops of the underlying device. Previously they all relied on the handlers not using their private data. Signed-off-by: Patrick McHardy --------------020003080003040006090208 Content-Type: text/plain; name="x" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="x" diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c index 9fb3883..383ce48 100644 --- a/drivers/net/bonding/bond_main.c +++ b/drivers/net/bonding/bond_main.c @@ -4113,7 +4113,7 @@ static int bond_neigh_setup(struct net_device *dev, struct neigh_parms *parms) const struct net_device_ops *slave_ops = slave->dev->netdev_ops; if (slave_ops->ndo_neigh_setup) - return slave_ops->ndo_neigh_setup(dev, parms); + return slave_ops->ndo_neigh_setup(slave, parms); } return 0; } --------------020003080003040006090208--