diff options
author | Jianhua Xie <Jianhua.Xie@freescale.com> | 2014-09-02 04:30:27 (GMT) |
---|---|---|
committer | Jianhua Xie <jianhua.xie@freescale.com> | 2014-09-02 10:22:26 (GMT) |
commit | 3d55c3c0173c7ebfcc75c0fb7d26533b6b90bc71 (patch) | |
tree | dd73f611917e07143e05ec056df39c5aa7cb9346 | |
parent | 7eac4c0058f7400df0b4e2e2bcc6c05e8645ef88 (diff) | |
download | linux-fsl-qoriq-3d55c3c0173c7ebfcc75c0fb7d26533b6b90bc71.tar.xz |
bonding: fix missed readunlock() while hack LAG
LAG is hacking bond_3ad_xmit_xor() with Freescale PCD.
Original xmit policy is calculated by CPU and software,
after hacking, this part is replaced with FMan Keygen
hashing to instead.
This patch is fixing an errors on missing readunlock()
after called readlock() while hacking.
Signed-off-by: Jianhua Xie <jianhua.xie@freescale.com>
Change-Id: Ieffe4b340c83e6f2d6f20fd01124eaf1865c81a5
Reviewed-on: http://git.am.freescale.net:8181/17717
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
Reviewed-by: Chenhui Zhao <chenhui.zhao@freescale.com>
-rw-r--r-- | drivers/net/bonding/bond_3ad.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/bonding/bond_3ad.c b/drivers/net/bonding/bond_3ad.c index 51cb153..06aafbd 100644 --- a/drivers/net/bonding/bond_3ad.c +++ b/drivers/net/bonding/bond_3ad.c @@ -2446,7 +2446,8 @@ int bond_3ad_xmit_xor(struct sk_buff *skb, struct net_device *dev) if ((bond->slave_cnt == SLAVES_PER_BOND) && are_all_slaves_linkup(bond) && (bond->params.ohp) && (bond->params.ohp->oh_en == 1)) - return enqueue_pkt_to_oh(bond, skb, NULL); + res = enqueue_pkt_to_oh(bond, skb, NULL); + goto out; #endif slave_agg_no = bond->xmit_hash_policy(skb, slaves_in_agg); |