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:09:18 +0100 Message-ID: <49AF96BE.1020302@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> <49AF9672.50200@trash.net> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="------------000208020303060800090401" 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]:62039 "EHLO stinky.trash.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751431AbZCEJJW (ORCPT ); Thu, 5 Mar 2009 04:09:22 -0500 In-Reply-To: <49AF9672.50200@trash.net> Sender: netdev-owner@vger.kernel.org List-ID: This is a multi-part message in MIME format. --------------000208020303060800090401 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Patrick McHardy wrote: > 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 > Oops, the last patch was broken. --------------000208020303060800090401 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..e0578fe 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->dev, parms); } return 0; } --------------000208020303060800090401--