Skip to content

Commit

Permalink
bridge: Introduce br_send_bpdu_finish
Browse files Browse the repository at this point in the history
The function dev_queue_xmit_skb_sk is unncessary and very confusing.
Introduce br_send_bpdu_finish to remove the need for dev_queue_xmit_skb_sk,
and have br_send_bpdu_finish call dev_queue_xmit.

Signed-off-by: "Eric W. Biederman" <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
  • Loading branch information
ebiederm authored and davem330 committed Sep 18, 2015
1 parent f9e4306 commit 1f19c57
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion net/bridge/br_stp_bpdu.c
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,11 @@

#define LLC_RESERVE sizeof(struct llc_pdu_un)

static int br_send_bpdu_finish(struct sock *sk, struct sk_buff *skb)
{
return dev_queue_xmit(skb);
}

static void br_send_bpdu(struct net_bridge_port *p,
const unsigned char *data, int length)
{
Expand All @@ -56,7 +61,7 @@ static void br_send_bpdu(struct net_bridge_port *p,

NF_HOOK(NFPROTO_BRIDGE, NF_BR_LOCAL_OUT, NULL, skb,
NULL, skb->dev,
dev_queue_xmit_sk);
br_send_bpdu_finish);
}

static inline void br_set_ticks(unsigned char *dest, int j)
Expand Down

0 comments on commit 1f19c57

Please sign in to comment.