ipv4: Kill RT_CACHE_DEBUG
authorDavid S. Miller <davem@davemloft.net>
Wed, 18 May 2011 22:23:21 +0000 (18:23 -0400)
committerDavid S. Miller <davem@davemloft.net>
Wed, 18 May 2011 22:23:21 +0000 (18:23 -0400)
It's way past it's usefulness.  And this gets rid of a bunch
of stray ->rt_{dst,src} references.

Even the comment documenting the macro was inaccurate (stated
default was 1 when it's 0).

If reintroduced, it should be done properly, with dynamic debug
facilities.

Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/dst.h
net/core/dst.c
net/ipv4/route.c

index 2588a9a88cc6dde8c1ea77981461723b1651b521..07a0402c52e6b3cec976652d7aa0c3d4a101d955 100644 (file)
 #include <net/neighbour.h>
 #include <asm/processor.h>
 
-/*
- * 0 - no debugging messages
- * 1 - rare events and bugs (default)
- * 2 - trace mode.
- */
-#define RT_CACHE_DEBUG         0
-
 #define DST_GC_MIN     (HZ/10)
 #define DST_GC_INC     (HZ/2)
 #define DST_GC_MAX     (120*HZ)
index 30f009327b6294ad77f9eaeea01b73cc7b0eb627..da47a299618a18f392fa0012cbfd6b12b6367259 100644 (file)
@@ -33,9 +33,6 @@
  * 3) This list is guarded by a mutex,
  *    so that the gc_task and dst_dev_event() can be synchronized.
  */
-#if RT_CACHE_DEBUG >= 2
-static atomic_t                         dst_total = ATOMIC_INIT(0);
-#endif
 
 /*
  * We want to keep lock & list close together
@@ -69,10 +66,6 @@ static void dst_gc_task(struct work_struct *work)
        unsigned long expires = ~0L;
        struct dst_entry *dst, *next, head;
        struct dst_entry *last = &head;
-#if RT_CACHE_DEBUG >= 2
-       ktime_t time_start = ktime_get();
-       struct timespec elapsed;
-#endif
 
        mutex_lock(&dst_gc_mutex);
        next = dst_busy_list;
@@ -146,15 +139,6 @@ loop:
 
        spin_unlock_bh(&dst_garbage.lock);
        mutex_unlock(&dst_gc_mutex);
-#if RT_CACHE_DEBUG >= 2
-       elapsed = ktime_to_timespec(ktime_sub(ktime_get(), time_start));
-       printk(KERN_DEBUG "dst_total: %d delayed: %d work_perf: %d"
-               " expires: %lu elapsed: %lu us\n",
-               atomic_read(&dst_total), delayed, work_performed,
-               expires,
-               elapsed.tv_sec * USEC_PER_SEC +
-                 elapsed.tv_nsec / NSEC_PER_USEC);
-#endif
 }
 
 int dst_discard(struct sk_buff *skb)
@@ -205,9 +189,6 @@ void *dst_alloc(struct dst_ops *ops, struct net_device *dev,
        dst->lastuse = jiffies;
        dst->flags = flags;
        dst->next = NULL;
-#if RT_CACHE_DEBUG >= 2
-       atomic_inc(&dst_total);
-#endif
        dst_entries_add(ops, 1);
        return dst;
 }
@@ -267,9 +248,6 @@ again:
                dst->ops->destroy(dst);
        if (dst->dev)
                dev_put(dst->dev);
-#if RT_CACHE_DEBUG >= 2
-       atomic_dec(&dst_total);
-#endif
        kmem_cache_free(dst->ops->kmem_cachep, dst);
 
        dst = child;
index cb93c32027d73dfb454b37742871af57031f2394..9c5ad86bc7833f2587cdd5b5d5ad080f14e94ba0 100644 (file)
@@ -968,10 +968,6 @@ static int rt_garbage_collect(struct dst_ops *ops)
                        break;
 
                expire >>= 1;
-#if RT_CACHE_DEBUG >= 2
-               printk(KERN_DEBUG "expire>> %u %d %d %d\n", expire,
-                               dst_entries_get_fast(&ipv4_dst_ops), goal, i);
-#endif
 
                if (dst_entries_get_fast(&ipv4_dst_ops) < ip_rt_max_size)
                        goto out;
@@ -992,10 +988,6 @@ work_done:
            dst_entries_get_fast(&ipv4_dst_ops) < ipv4_dst_ops.gc_thresh ||
            dst_entries_get_slow(&ipv4_dst_ops) < ipv4_dst_ops.gc_thresh)
                expire = ip_rt_gc_timeout;
-#if RT_CACHE_DEBUG >= 2
-       printk(KERN_DEBUG "expire++ %u %d %d %d\n", expire,
-                       dst_entries_get_fast(&ipv4_dst_ops), goal, rover);
-#endif
 out:   return 0;
 }
 
@@ -1179,16 +1171,6 @@ restart:
 
        rt->dst.rt_next = rt_hash_table[hash].chain;
 
-#if RT_CACHE_DEBUG >= 2
-       if (rt->dst.rt_next) {
-               struct rtable *trt;
-               printk(KERN_DEBUG "rt_cache @%02x: %pI4",
-                      hash, &rt->rt_dst);
-               for (trt = rt->dst.rt_next; trt; trt = trt->dst.rt_next)
-                       printk(" . %pI4", &trt->rt_dst);
-               printk("\n");
-       }
-#endif
        /*
         * Since lookup is lockfree, we must make sure
         * previous writes to rt are committed to memory
@@ -1347,10 +1329,6 @@ static struct dst_entry *ipv4_negative_advice(struct dst_entry *dst)
                        unsigned hash = rt_hash(rt->rt_key_dst, rt->rt_key_src,
                                                rt->rt_oif,
                                                rt_genid(dev_net(dst->dev)));
-#if RT_CACHE_DEBUG >= 1
-                       printk(KERN_DEBUG "ipv4_negative_advice: redirect to %pI4/%02x dropped\n",
-                               &rt->rt_dst, rt->rt_key_tos);
-#endif
                        rt_del(hash, rt);
                        ret = NULL;
                } else if (rt->peer &&