On Thu, Oct 08, 2015 at 01:49:32PM +0100, Ben Hutchings wrote:
> On Sat, 2015-09-26 at 15:20 -0700, Guenter Roeck wrote:
> > drivers/net/bonding/bond_main.c: In function ???bond_get_old_active???:
> > drivers/net/bonding/bond_main.c:884:33: error: macro "bond_for_each_slave" requires 3 arguments, but only 2 given
> > drivers/net/bonding/bond_main.c:884:2: error: ???bond_for_each_slave??? undeclared (first use in this function)
> > drivers/net/bonding/bond_main.c:884:2: note: each undeclared identifier is reported only once for each function it appears in
> > drivers/net/bonding/bond_main.c:884:35: error: expected ???;??? before ???{??? token
> >
> > This affects a large number of builds.
> >
> > Culprit is 'bonding: correct the MAC address for "follow"
> > fail_over_mac policy', which may need a backport.
>
> bond_for_each_slave() needs an int parameter for a counter.
>
> Untested backport attached, based on what I did for 3.2.
I agree with the fix, it's similar to what I proposed (except that
you actually did the patch of course).
Willy
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html