[Devel] [PATCH rh7] net/dst: drop leftovers from diff-ve-net-devleak-debug-dump-dst in 2.6.32-x

[Kirill Tkhai]

On 17.12.2017 00:59, Konstantin Khorenko wrote:
> There was a debug patch in
> https://jira.sw.ru/browse/PCLIN-29838
> 
> intended to find out which dst entry holds a ref to netdevice forcing it
> to leak.
> 
> Don't see this problem in 3.10.0-x and the code is not used anyway now.
> 
> Original comment in diff-ve-net-devleak-debug-dump-dst:
> 
>   date: 2011/08/31 14:04:43;  author: xemul;  state: Exp;
>   Author: Pavel Emelyanov
>   Email: xemul at parallels.com
>   Subject: net: Dump dst cache contents when netdev leaks
> 
>   The experience shows that when netdev is leaked the holder is one
>   of dst entries. Add some debug for this.
> 
>   https://jira.sw.ru/browse/PCLIN-29838
> 
> Signed-off-by: Konstantin Khorenko <khorenko at virtuozzo.com>

Reviewed-by: Kirill Tkhai <ktkhai at virtuozzo.com>

> ---
>  include/net/dst.h |  6 ------
>  net/core/dev.c    |  1 -
>  net/core/dst.c    | 16 ----------------
>  net/ipv4/route.c  | 25 -------------------------
>  net/ipv6/route.c  | 24 ------------------------
>  5 files changed, 72 deletions(-)
> 
> diff --git a/include/net/dst.h b/include/net/dst.h
> index d2ea0fd..3af149e 100644
> --- a/include/net/dst.h
> +++ b/include/net/dst.h
> @@ -61,7 +61,6 @@ struct dst_entry {
>  #define DST_XFRM_TUNNEL		0x0080
>  #define DST_XFRM_QUEUE		0x0100
>  #define DST_METADATA		0x0200
> -#define DST_FREE		0x0400
>  
>  	RH_KABI_DEPRECATE(unsigned short, pending_confirm)
>  	short			error;
> @@ -194,11 +193,6 @@ dst_metric_raw(const struct dst_entry *dst, const int metric)
>  	return p[metric-1];
>  }
>  
> -void dst_dump_one(struct dst_entry *d);
> -void ip_rt_dump_dsts(void);
> -void dst_cache_dump(void);
> -extern void (*ip6_rt_dump_dsts)(void);
> -
>  static inline u32
>  dst_metric(const struct dst_entry *dst, const int metric)
>  {
> diff --git a/net/core/dev.c b/net/core/dev.c
> index 9d15a33..c2a73ae 100644
> --- a/net/core/dev.c
> +++ b/net/core/dev.c
> @@ -7372,7 +7372,6 @@ static void ve_netdev_leak(struct net_device *dev)
>  	pr_emerg("Device (%s:%d:%s:%p) marked as leaked\n",
>  			dev->name, netdev_refcnt_read(dev) - 1,
>  			ve_name(dev_net(dev)->owner_ve), dev);
> -	dst_cache_dump();
>  }
>  
>  /**
> diff --git a/net/core/dst.c b/net/core/dst.c
> index f5299a4..8f3026c 100644
> --- a/net/core/dst.c
> +++ b/net/core/dst.c
> @@ -265,7 +265,6 @@ struct dst_entry *dst_destroy(struct dst_entry * dst)
>  
>  	lwtstate_put(dst->lwtstate);
>  
> -	dst->flags |= DST_FREE;
>  	if (dst->flags & DST_METADATA)
>  		metadata_dst_free((struct metadata_dst *)dst);
>  	else
> @@ -299,21 +298,6 @@ static void dst_destroy_rcu(struct rcu_head *head)
>  		__dst_free(dst);
>  }
>  
> -void dst_dump_one(struct dst_entry *d)
> -{
> -	printk("\tdev %p err %d obs %d flags %x i/o %p/%p ref %d use %d\n",
> -			d->dev, (int)d->error, (int)d->obsolete, d->flags,
> -			d->input, d->output, atomic_read(&d->__refcnt), d->__use);
> -}
> -EXPORT_SYMBOL(dst_dump_one);
> -
> -void dst_cache_dump(void)
> -{
> -	ip_rt_dump_dsts();
> -	if (ip6_rt_dump_dsts)
> -		ip6_rt_dump_dsts();
> -}
> -
>  void dst_release(struct dst_entry *dst)
>  {
>  	if (dst) {
> diff --git a/net/ipv4/route.c b/net/ipv4/route.c
> index 9059a48..39d5df5 100644
> --- a/net/ipv4/route.c
> +++ b/net/ipv4/route.c
> @@ -2933,28 +2933,3 @@ void __init ip_static_sysctl_init(void)
>  	register_net_sysctl(&init_net, "net/ipv4/route", ipv4_route_table);
>  }
>  #endif
> -
> -#if 0
> -static void ip_rt_dump_dst(void *o)
> -{
> -	struct rtable *rt = (struct rtable *)o;
> -
> -	if (rt->dst.flags & DST_FREE)
> -		return;
> -
> -	printk("=== %p\n", o);
> -	dst_dump_one(&rt->dst);
> -	printk("\tgen %x flags %x type %d\n",
> -			rt->rt_genid, rt->rt_flags, (int)rt->rt_type);
> -}
> -#endif
> -
> -void ip_rt_dump_dsts(void)
> -{
> -	printk("IPv4 dst cache:\n");
> -	//FIXME
> -	//slab_obj_walk(ipv4_dst_ops.kmem_cachep, ip_rt_dump_dst);
> -}
> -
> -void (*ip6_rt_dump_dsts)(void);
> -EXPORT_SYMBOL_GPL(ip6_rt_dump_dsts);
> diff --git a/net/ipv6/route.c b/net/ipv6/route.c
> index 4a2a853..d0c303e 100644
> --- a/net/ipv6/route.c
> +++ b/net/ipv6/route.c
> @@ -3735,29 +3735,6 @@ void __init ip6_route_init_special_entries(void)
>    #endif
>  }
>  
> -#if 0
> -static void ip6_rt_dump_dst(void *o)
> -{
> -	struct rt6_info *r = (struct rt6_info *)o;
> -
> -	if (r->dst.flags & DST_FREE)
> -		return;
> -
> -	printk("=== %p\n", o);
> -	dst_dump_one(&r->dst);
> -	printk("\tflags %x ref %d prot %d\n",
> -			r->rt6i_flags, atomic_read(&r->rt6i_ref),
> -			(int)r->rt6i_protocol);
> -}
> -#endif
> -
> -static void _ip6_rt_dump_dsts(void)
> -{
> -	printk("IPv6 dst cache:\n");
> -	//FIXME
> -	//slab_obj_walk(ip6_dst_ops_template.kmem_cachep, ip6_rt_dump_dst);
> -}
> -
>  int __init ip6_route_init(void)
>  {
>  	int ret;
> @@ -3817,7 +3794,6 @@ int __init ip6_route_init(void)
>  		spin_lock_init(&ul->lock);
>  	}
>  
> -	ip6_rt_dump_dsts = _ip6_rt_dump_dsts;
>  out:
>  	return ret;
>  
>