diff options
| author |   Jiri Pirko <jiri@resnulli.us> | 2014-11-19 14:04:57 +0100 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2014-11-21 14:20:17 -0500 |
| commit | b4bef1b57544b18899eb15569e3bafd8d2eeeff6 (patch) | |
| tree | 5f78030a09edac7df1e26c109d6aa2ef03927d5b /drivers/net/bonding/bond_alb.c | |
| parent | vlan: make __vlan_hwaccel_put_tag return void (diff) | |
| download | linux-dev-b4bef1b57544b18899eb15569e3bafd8d2eeeff6.tar.xz linux-dev-b4bef1b57544b18899eb15569e3bafd8d2eeeff6.zip | |
vlan: kill vlan_put_tag helper
Since both tx and rx paths work with skb->vlan_tci, there's no need for
this function anymore. Switch users directly to __vlan_hwaccel_put_tag.
Signed-off-by: Jiri Pirko <jiri@resnulli.us>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/bonding/bond_alb.c')
| -rw-r--r-- | drivers/net/bonding/bond_alb.c | 17 |
1 files changed, 4 insertions, 13 deletions
diff --git a/drivers/net/bonding/bond_alb.c b/drivers/net/bonding/bond_alb.c index e1f1a006af85..bb9e9fc45e1b 100644 --- a/drivers/net/bonding/bond_alb.c +++ b/drivers/net/bonding/bond_alb.c @@ -475,12 +475,8 @@ static void rlb_update_client(struct rlb_client_info *client_info) skb->dev = client_info->slave->dev; if (client_info->vlan_id) { - skb = vlan_put_tag(skb, htons(ETH_P_8021Q), client_info->vlan_id); - if (!skb) { - netdev_err(client_info->slave->bond->dev, - "failed to insert VLAN tag\n"); - continue; - } + __vlan_hwaccel_put_tag(skb, htons(ETH_P_8021Q), + client_info->vlan_id); } arp_xmit(skb); @@ -951,13 +947,8 @@ static void alb_send_lp_vid(struct slave *slave, u8 mac_addr[], skb->priority = TC_PRIO_CONTROL; skb->dev = slave->dev; - if (vid) { - skb = vlan_put_tag(skb, vlan_proto, vid); - if (!skb) { - netdev_err(slave->bond->dev, "failed to insert VLAN tag\n"); - return; - } - } + if (vid) + __vlan_hwaccel_put_tag(skb, vlan_proto, vid); dev_queue_xmit(skb); } |