diff options
-rw-r--r-- | net/packet/af_packet.c | 40 |
1 files changed, 20 insertions, 20 deletions
diff --git a/net/packet/af_packet.c b/net/packet/af_packet.c index 097a354ec8cd..01039d2b1695 100644 --- a/net/packet/af_packet.c +++ b/net/packet/af_packet.c | |||
@@ -243,40 +243,40 @@ static int packet_direct_xmit(struct sk_buff *skb) | |||
243 | const struct net_device_ops *ops = dev->netdev_ops; | 243 | const struct net_device_ops *ops = dev->netdev_ops; |
244 | netdev_features_t features; | 244 | netdev_features_t features; |
245 | struct netdev_queue *txq; | 245 | struct netdev_queue *txq; |
246 | int ret = NETDEV_TX_BUSY; | ||
246 | u16 queue_map; | 247 | u16 queue_map; |
247 | int ret; | ||
248 | 248 | ||
249 | if (unlikely(!netif_running(dev) || | 249 | if (unlikely(!netif_running(dev) || |
250 | !netif_carrier_ok(dev))) { | 250 | !netif_carrier_ok(dev))) |
251 | kfree_skb(skb); | 251 | goto drop; |
252 | return NET_XMIT_DROP; | ||
253 | } | ||
254 | 252 | ||
255 | features = netif_skb_features(skb); | 253 | features = netif_skb_features(skb); |
256 | if (skb_needs_linearize(skb, features) && | 254 | if (skb_needs_linearize(skb, features) && |
257 | __skb_linearize(skb)) { | 255 | __skb_linearize(skb)) |
258 | kfree_skb(skb); | 256 | goto drop; |
259 | return NET_XMIT_DROP; | ||
260 | } | ||
261 | 257 | ||
262 | queue_map = skb_get_queue_mapping(skb); | 258 | queue_map = skb_get_queue_mapping(skb); |
263 | txq = netdev_get_tx_queue(dev, queue_map); | 259 | txq = netdev_get_tx_queue(dev, queue_map); |
264 | 260 | ||
265 | __netif_tx_lock_bh(txq); | 261 | local_bh_disable(); |
266 | if (unlikely(netif_xmit_frozen_or_stopped(txq))) { | 262 | |
267 | ret = NETDEV_TX_BUSY; | 263 | HARD_TX_LOCK(dev, txq, smp_processor_id()); |
268 | kfree_skb(skb); | 264 | if (!netif_xmit_frozen_or_stopped(txq)) { |
269 | goto out; | 265 | ret = ops->ndo_start_xmit(skb, dev); |
266 | if (ret == NETDEV_TX_OK) | ||
267 | txq_trans_update(txq); | ||
270 | } | 268 | } |
269 | HARD_TX_UNLOCK(dev, txq); | ||
271 | 270 | ||
272 | ret = ops->ndo_start_xmit(skb, dev); | 271 | local_bh_enable(); |
273 | if (likely(dev_xmit_complete(ret))) | 272 | |
274 | txq_trans_update(txq); | 273 | if (!dev_xmit_complete(ret)) |
275 | else | ||
276 | kfree_skb(skb); | 274 | kfree_skb(skb); |
277 | out: | 275 | |
278 | __netif_tx_unlock_bh(txq); | ||
279 | return ret; | 276 | return ret; |
277 | drop: | ||
278 | kfree_skb(skb); | ||
279 | return NET_XMIT_DROP; | ||
280 | } | 280 | } |
281 | 281 | ||
282 | static struct net_device *packet_cached_dev_get(struct packet_sock *po) | 282 | static struct net_device *packet_cached_dev_get(struct packet_sock *po) |