mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 10:40:15 +02:00 
			
		
		
		
	veth: move loopback logic to common location
The veth driver contains code to forward an skb from the start_xmit function of one network device into the receive path of another device. Moving that code into a common location lets us reuse the code for direct forwarding of data between macvlan ports, and possibly in other drivers. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Acked-by: Patrick McHardy <kaber@trash.net> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
		
							parent
							
								
									1bda8aa86b
								
							
						
					
					
						commit
						445409602c
					
				
					 3 changed files with 45 additions and 14 deletions
				
			
		| 
						 | 
					@ -155,8 +155,6 @@ static netdev_tx_t veth_xmit(struct sk_buff *skb, struct net_device *dev)
 | 
				
			||||||
	struct veth_net_stats *stats, *rcv_stats;
 | 
						struct veth_net_stats *stats, *rcv_stats;
 | 
				
			||||||
	int length, cpu;
 | 
						int length, cpu;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	skb_orphan(skb);
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
	priv = netdev_priv(dev);
 | 
						priv = netdev_priv(dev);
 | 
				
			||||||
	rcv = priv->peer;
 | 
						rcv = priv->peer;
 | 
				
			||||||
	rcv_priv = netdev_priv(rcv);
 | 
						rcv_priv = netdev_priv(rcv);
 | 
				
			||||||
| 
						 | 
					@ -168,20 +166,12 @@ static netdev_tx_t veth_xmit(struct sk_buff *skb, struct net_device *dev)
 | 
				
			||||||
	if (!(rcv->flags & IFF_UP))
 | 
						if (!(rcv->flags & IFF_UP))
 | 
				
			||||||
		goto tx_drop;
 | 
							goto tx_drop;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	if (skb->len > (rcv->mtu + MTU_PAD))
 | 
					 | 
				
			||||||
		goto rx_drop;
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
        skb->tstamp.tv64 = 0;
 | 
					 | 
				
			||||||
	skb->pkt_type = PACKET_HOST;
 | 
					 | 
				
			||||||
	skb->protocol = eth_type_trans(skb, rcv);
 | 
					 | 
				
			||||||
	if (dev->features & NETIF_F_NO_CSUM)
 | 
						if (dev->features & NETIF_F_NO_CSUM)
 | 
				
			||||||
		skb->ip_summed = rcv_priv->ip_summed;
 | 
							skb->ip_summed = rcv_priv->ip_summed;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	skb->mark = 0;
 | 
						length = skb->len + ETH_HLEN;
 | 
				
			||||||
	secpath_reset(skb);
 | 
						if (dev_forward_skb(rcv, skb) != NET_RX_SUCCESS)
 | 
				
			||||||
	nf_reset(skb);
 | 
							goto rx_drop;
 | 
				
			||||||
 | 
					 | 
				
			||||||
	length = skb->len;
 | 
					 | 
				
			||||||
 | 
					
 | 
				
			||||||
	stats->tx_bytes += length;
 | 
						stats->tx_bytes += length;
 | 
				
			||||||
	stats->tx_packets++;
 | 
						stats->tx_packets++;
 | 
				
			||||||
| 
						 | 
					@ -189,7 +179,6 @@ static netdev_tx_t veth_xmit(struct sk_buff *skb, struct net_device *dev)
 | 
				
			||||||
	rcv_stats->rx_bytes += length;
 | 
						rcv_stats->rx_bytes += length;
 | 
				
			||||||
	rcv_stats->rx_packets++;
 | 
						rcv_stats->rx_packets++;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	netif_rx(skb);
 | 
					 | 
				
			||||||
	return NETDEV_TX_OK;
 | 
						return NETDEV_TX_OK;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
tx_drop:
 | 
					tx_drop:
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -1562,6 +1562,8 @@ extern int		dev_set_mac_address(struct net_device *,
 | 
				
			||||||
extern int		dev_hard_start_xmit(struct sk_buff *skb,
 | 
					extern int		dev_hard_start_xmit(struct sk_buff *skb,
 | 
				
			||||||
					    struct net_device *dev,
 | 
										    struct net_device *dev,
 | 
				
			||||||
					    struct netdev_queue *txq);
 | 
										    struct netdev_queue *txq);
 | 
				
			||||||
 | 
					extern int		dev_forward_skb(struct net_device *dev,
 | 
				
			||||||
 | 
										struct sk_buff *skb);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
extern int		netdev_budget;
 | 
					extern int		netdev_budget;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -105,6 +105,7 @@
 | 
				
			||||||
#include <net/dst.h>
 | 
					#include <net/dst.h>
 | 
				
			||||||
#include <net/pkt_sched.h>
 | 
					#include <net/pkt_sched.h>
 | 
				
			||||||
#include <net/checksum.h>
 | 
					#include <net/checksum.h>
 | 
				
			||||||
 | 
					#include <net/xfrm.h>
 | 
				
			||||||
#include <linux/highmem.h>
 | 
					#include <linux/highmem.h>
 | 
				
			||||||
#include <linux/init.h>
 | 
					#include <linux/init.h>
 | 
				
			||||||
#include <linux/kmod.h>
 | 
					#include <linux/kmod.h>
 | 
				
			||||||
| 
						 | 
					@ -1419,6 +1420,45 @@ static inline void net_timestamp(struct sk_buff *skb)
 | 
				
			||||||
		skb->tstamp.tv64 = 0;
 | 
							skb->tstamp.tv64 = 0;
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					/**
 | 
				
			||||||
 | 
					 * dev_forward_skb - loopback an skb to another netif
 | 
				
			||||||
 | 
					 *
 | 
				
			||||||
 | 
					 * @dev: destination network device
 | 
				
			||||||
 | 
					 * @skb: buffer to forward
 | 
				
			||||||
 | 
					 *
 | 
				
			||||||
 | 
					 * return values:
 | 
				
			||||||
 | 
					 *	NET_RX_SUCCESS	(no congestion)
 | 
				
			||||||
 | 
					 *	NET_RX_DROP     (packet was dropped)
 | 
				
			||||||
 | 
					 *
 | 
				
			||||||
 | 
					 * dev_forward_skb can be used for injecting an skb from the
 | 
				
			||||||
 | 
					 * start_xmit function of one device into the receive queue
 | 
				
			||||||
 | 
					 * of another device.
 | 
				
			||||||
 | 
					 *
 | 
				
			||||||
 | 
					 * The receiving device may be in another namespace, so
 | 
				
			||||||
 | 
					 * we have to clear all information in the skb that could
 | 
				
			||||||
 | 
					 * impact namespace isolation.
 | 
				
			||||||
 | 
					 */
 | 
				
			||||||
 | 
					int dev_forward_skb(struct net_device *dev, struct sk_buff *skb)
 | 
				
			||||||
 | 
					{
 | 
				
			||||||
 | 
						skb_orphan(skb);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						if (!(dev->flags & IFF_UP))
 | 
				
			||||||
 | 
							return NET_RX_DROP;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						if (skb->len > (dev->mtu + dev->hard_header_len))
 | 
				
			||||||
 | 
							return NET_RX_DROP;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						skb_dst_drop(skb);
 | 
				
			||||||
 | 
						skb->tstamp.tv64 = 0;
 | 
				
			||||||
 | 
						skb->pkt_type = PACKET_HOST;
 | 
				
			||||||
 | 
						skb->protocol = eth_type_trans(skb, dev);
 | 
				
			||||||
 | 
						skb->mark = 0;
 | 
				
			||||||
 | 
						secpath_reset(skb);
 | 
				
			||||||
 | 
						nf_reset(skb);
 | 
				
			||||||
 | 
						return netif_rx(skb);
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					EXPORT_SYMBOL_GPL(dev_forward_skb);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
/*
 | 
					/*
 | 
				
			||||||
 *	Support routine. Sends outgoing frames to any network
 | 
					 *	Support routine. Sends outgoing frames to any network
 | 
				
			||||||
 *	taps currently in use.
 | 
					 *	taps currently in use.
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue