mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 02:30:34 +02:00 
			
		
		
		
	net: Remove the now unused vrf_ptr
Signed-off-by: David Ahern <dsa@cumulusnetworks.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
		
							parent
							
								
									8e1ed7058b
								
							
						
					
					
						commit
						93a7e7e837
					
				
					 3 changed files with 2 additions and 38 deletions
				
			
		| 
						 | 
				
			
			@ -396,18 +396,15 @@ static void __vrf_insert_slave(struct slave_queue *queue, struct slave *slave)
 | 
			
		|||
 | 
			
		||||
static int do_vrf_add_slave(struct net_device *dev, struct net_device *port_dev)
 | 
			
		||||
{
 | 
			
		||||
	struct net_vrf_dev *vrf_ptr = kmalloc(sizeof(*vrf_ptr), GFP_KERNEL);
 | 
			
		||||
	struct slave *slave = kzalloc(sizeof(*slave), GFP_KERNEL);
 | 
			
		||||
	struct net_vrf *vrf = netdev_priv(dev);
 | 
			
		||||
	struct slave_queue *queue = &vrf->queue;
 | 
			
		||||
	int ret = -ENOMEM;
 | 
			
		||||
 | 
			
		||||
	if (!slave || !vrf_ptr)
 | 
			
		||||
	if (!slave)
 | 
			
		||||
		goto out_fail;
 | 
			
		||||
 | 
			
		||||
	slave->dev = port_dev;
 | 
			
		||||
	vrf_ptr->ifindex = dev->ifindex;
 | 
			
		||||
	vrf_ptr->tb_id = vrf->tb_id;
 | 
			
		||||
 | 
			
		||||
	/* register the packet handler for slave ports */
 | 
			
		||||
	ret = netdev_rx_handler_register(port_dev, vrf_handle_frame, dev);
 | 
			
		||||
| 
						 | 
				
			
			@ -424,7 +421,6 @@ static int do_vrf_add_slave(struct net_device *dev, struct net_device *port_dev)
 | 
			
		|||
 | 
			
		||||
	port_dev->flags |= IFF_SLAVE;
 | 
			
		||||
	__vrf_insert_slave(queue, slave);
 | 
			
		||||
	rcu_assign_pointer(port_dev->vrf_ptr, vrf_ptr);
 | 
			
		||||
	cycle_netdev(port_dev);
 | 
			
		||||
 | 
			
		||||
	return 0;
 | 
			
		||||
| 
						 | 
				
			
			@ -432,7 +428,6 @@ static int do_vrf_add_slave(struct net_device *dev, struct net_device *port_dev)
 | 
			
		|||
out_unregister:
 | 
			
		||||
	netdev_rx_handler_unregister(port_dev);
 | 
			
		||||
out_fail:
 | 
			
		||||
	kfree(vrf_ptr);
 | 
			
		||||
	kfree(slave);
 | 
			
		||||
	return ret;
 | 
			
		||||
}
 | 
			
		||||
| 
						 | 
				
			
			@ -448,21 +443,15 @@ static int vrf_add_slave(struct net_device *dev, struct net_device *port_dev)
 | 
			
		|||
/* inverse of do_vrf_add_slave */
 | 
			
		||||
static int do_vrf_del_slave(struct net_device *dev, struct net_device *port_dev)
 | 
			
		||||
{
 | 
			
		||||
	struct net_vrf_dev *vrf_ptr = rtnl_dereference(port_dev->vrf_ptr);
 | 
			
		||||
	struct net_vrf *vrf = netdev_priv(dev);
 | 
			
		||||
	struct slave_queue *queue = &vrf->queue;
 | 
			
		||||
	struct slave *slave;
 | 
			
		||||
 | 
			
		||||
	RCU_INIT_POINTER(port_dev->vrf_ptr, NULL);
 | 
			
		||||
 | 
			
		||||
	netdev_upper_dev_unlink(port_dev, dev);
 | 
			
		||||
	port_dev->flags &= ~IFF_SLAVE;
 | 
			
		||||
 | 
			
		||||
	netdev_rx_handler_unregister(port_dev);
 | 
			
		||||
 | 
			
		||||
	/* after netdev_rx_handler_unregister for synchronize_rcu */
 | 
			
		||||
	kfree(vrf_ptr);
 | 
			
		||||
 | 
			
		||||
	cycle_netdev(port_dev);
 | 
			
		||||
 | 
			
		||||
	slave = __vrf_find_slave_dev(queue, port_dev);
 | 
			
		||||
| 
						 | 
				
			
			@ -601,10 +590,6 @@ static int vrf_validate(struct nlattr *tb[], struct nlattr *data[])
 | 
			
		|||
 | 
			
		||||
static void vrf_dellink(struct net_device *dev, struct list_head *head)
 | 
			
		||||
{
 | 
			
		||||
	struct net_vrf_dev *vrf_ptr = rtnl_dereference(dev->vrf_ptr);
 | 
			
		||||
 | 
			
		||||
	RCU_INIT_POINTER(dev->vrf_ptr, NULL);
 | 
			
		||||
	kfree_rcu(vrf_ptr, rcu);
 | 
			
		||||
	unregister_netdevice_queue(dev, head);
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -612,7 +597,6 @@ static int vrf_newlink(struct net *src_net, struct net_device *dev,
 | 
			
		|||
		       struct nlattr *tb[], struct nlattr *data[])
 | 
			
		||||
{
 | 
			
		||||
	struct net_vrf *vrf = netdev_priv(dev);
 | 
			
		||||
	struct net_vrf_dev *vrf_ptr;
 | 
			
		||||
	int err;
 | 
			
		||||
 | 
			
		||||
	if (!data || !data[IFLA_VRF_TABLE])
 | 
			
		||||
| 
						 | 
				
			
			@ -622,24 +606,13 @@ static int vrf_newlink(struct net *src_net, struct net_device *dev,
 | 
			
		|||
 | 
			
		||||
	dev->priv_flags |= IFF_L3MDEV_MASTER;
 | 
			
		||||
 | 
			
		||||
	err = -ENOMEM;
 | 
			
		||||
	vrf_ptr = kmalloc(sizeof(*dev->vrf_ptr), GFP_KERNEL);
 | 
			
		||||
	if (!vrf_ptr)
 | 
			
		||||
		goto out_fail;
 | 
			
		||||
 | 
			
		||||
	vrf_ptr->ifindex = dev->ifindex;
 | 
			
		||||
	vrf_ptr->tb_id = vrf->tb_id;
 | 
			
		||||
 | 
			
		||||
	err = register_netdevice(dev);
 | 
			
		||||
	if (err < 0)
 | 
			
		||||
		goto out_fail;
 | 
			
		||||
 | 
			
		||||
	rcu_assign_pointer(dev->vrf_ptr, vrf_ptr);
 | 
			
		||||
 | 
			
		||||
	return 0;
 | 
			
		||||
 | 
			
		||||
out_fail:
 | 
			
		||||
	kfree(vrf_ptr);
 | 
			
		||||
	free_netdev(dev);
 | 
			
		||||
	return err;
 | 
			
		||||
}
 | 
			
		||||
| 
						 | 
				
			
			@ -683,10 +656,9 @@ static int vrf_device_event(struct notifier_block *unused,
 | 
			
		|||
 | 
			
		||||
	/* only care about unregister events to drop slave references */
 | 
			
		||||
	if (event == NETDEV_UNREGISTER) {
 | 
			
		||||
		struct net_vrf_dev *vrf_ptr = rtnl_dereference(dev->vrf_ptr);
 | 
			
		||||
		struct net_device *vrf_dev;
 | 
			
		||||
 | 
			
		||||
		if (!vrf_ptr || netif_is_l3_master(dev))
 | 
			
		||||
		if (netif_is_l3_master(dev))
 | 
			
		||||
			goto out;
 | 
			
		||||
 | 
			
		||||
		vrf_dev = netdev_master_upper_dev_get(dev);
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -1427,7 +1427,6 @@ enum netdev_priv_flags {
 | 
			
		|||
 *	@dn_ptr:	DECnet specific data
 | 
			
		||||
 *	@ip6_ptr:	IPv6 specific data
 | 
			
		||||
 *	@ax25_ptr:	AX.25 specific data
 | 
			
		||||
 *	@vrf_ptr:	VRF specific data
 | 
			
		||||
 *	@ieee80211_ptr:	IEEE 802.11 specific data, assign before registering
 | 
			
		||||
 *
 | 
			
		||||
 *	@last_rx:	Time of last Rx
 | 
			
		||||
| 
						 | 
				
			
			@ -1649,7 +1648,6 @@ struct net_device {
 | 
			
		|||
	struct dn_dev __rcu     *dn_ptr;
 | 
			
		||||
	struct inet6_dev __rcu	*ip6_ptr;
 | 
			
		||||
	void			*ax25_ptr;
 | 
			
		||||
	struct net_vrf_dev __rcu *vrf_ptr;
 | 
			
		||||
	struct wireless_dev	*ieee80211_ptr;
 | 
			
		||||
	struct wpan_dev		*ieee802154_ptr;
 | 
			
		||||
#if IS_ENABLED(CONFIG_MPLS_ROUTING)
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -11,12 +11,6 @@
 | 
			
		|||
#ifndef __LINUX_NET_VRF_H
 | 
			
		||||
#define __LINUX_NET_VRF_H
 | 
			
		||||
 | 
			
		||||
struct net_vrf_dev {
 | 
			
		||||
	struct rcu_head		rcu;
 | 
			
		||||
	int                     ifindex; /* ifindex of master dev */
 | 
			
		||||
	u32                     tb_id;   /* table id for VRF */
 | 
			
		||||
};
 | 
			
		||||
 | 
			
		||||
struct slave {
 | 
			
		||||
	struct list_head	list;
 | 
			
		||||
	struct net_device	*dev;
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in a new issue