| From foo@baz Sat Jul 26 10:08:04 PDT 2014 |
| From: Nikolay Aleksandrov <nikolay@redhat.com> |
| Date: Sun, 13 Jul 2014 09:47:47 +0200 |
| Subject: bonding: fix ad_select module param check |
| |
| From: Nikolay Aleksandrov <nikolay@redhat.com> |
| |
| [ Upstream commit 548d28bd0eac840d122b691279ce9f4ce6ecbfb6 ] |
| |
| Obvious copy/paste error when I converted the ad_select to the new |
| option API. "lacp_rate" there should be "ad_select" so we can get the |
| proper value. |
| |
| CC: Jay Vosburgh <j.vosburgh@gmail.com> |
| CC: Veaceslav Falico <vfalico@gmail.com> |
| CC: Andy Gospodarek <andy@greyhouse.net> |
| CC: David S. Miller <davem@davemloft.net> |
| |
| Fixes: 9e5f5eebe765 ("bonding: convert ad_select to use the new option |
| API") |
| Reported-by: Karim Scheik <karim.scheik@prisma-solutions.at> |
| Signed-off-by: Nikolay Aleksandrov <nikolay@redhat.com> |
| Signed-off-by: David S. Miller <davem@davemloft.net> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| --- |
| drivers/net/bonding/bond_main.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/drivers/net/bonding/bond_main.c |
| +++ b/drivers/net/bonding/bond_main.c |
| @@ -4068,7 +4068,7 @@ static int bond_check_params(struct bond |
| } |
| |
| if (ad_select) { |
| - bond_opt_initstr(&newval, lacp_rate); |
| + bond_opt_initstr(&newval, ad_select); |
| valptr = bond_opt_parse(bond_opt_get(BOND_OPT_AD_SELECT), |
| &newval); |
| if (!valptr) { |
