tun: drop useless debugging statements
authorMichal Kubecek <mkubecek@suse.cz>
Wed, 4 Mar 2020 16:24:09 +0000 (17:24 +0100)
committerDavid S. Miller <davem@davemloft.net>
Fri, 6 Mar 2020 05:38:02 +0000 (21:38 -0800)
Some of the tun_debug() statements only inform us about entering
a function which can be easily achieved with ftrace or kprobe. As
tun_debug() is no-op unless TUN_DEBUG is set which requires editing the
source and recompiling, setting up ftrace or kprobe is easier. Drop these
debug statements.

Also drop the tun_debug() statement informing about SIOCSIFHWADDR ioctl.
We can monitor these through rtnetlink and it makes little sense to log
address changes through ioctl but not changes through rtnetlink. Moreover,
this tun_debug() is called even if the actual address change fails which
makes it even less useful.

Signed-off-by: Michal Kubecek <mkubecek@suse.cz>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/tun.c

index 59290ef07497f264585135aec10beaab4e303e8f..15ae2050ab5bb0300a62d9b743d04b47df353fd1 100644 (file)
@@ -473,8 +473,6 @@ static void tun_flow_cleanup(struct timer_list *t)
        unsigned long count = 0;
        int i;
 
-       tun_debug(KERN_INFO, tun, "tun_flow_cleanup\n");
-
        spin_lock(&tun->lock);
        for (i = 0; i < TUN_NUM_FLOW_ENTRIES; i++) {
                struct tun_flow_entry *e;
@@ -1420,8 +1418,6 @@ static __poll_t tun_chr_poll(struct file *file, poll_table *wait)
 
        sk = tfile->socket.sk;
 
-       tun_debug(KERN_INFO, tun, "tun_chr_poll\n");
-
        poll_wait(file, sk_sleep(sk), wait);
 
        if (!ptr_ring_empty(&tfile->tx_ring))
@@ -2192,8 +2188,6 @@ static ssize_t tun_do_read(struct tun_struct *tun, struct tun_file *tfile,
        ssize_t ret;
        int err;
 
-       tun_debug(KERN_INFO, tun, "tun_do_read\n");
-
        if (!iov_iter_count(to)) {
                tun_ptr_free(ptr);
                return 0;
@@ -2838,8 +2832,6 @@ static int tun_set_iff(struct net *net, struct file *file, struct ifreq *ifr)
 
        netif_carrier_on(tun->dev);
 
-       tun_debug(KERN_INFO, tun, "tun_set_iff\n");
-
        /* Make sure persistent devices do not get stuck in
         * xoff state.
         */
@@ -2870,8 +2862,6 @@ err_free_dev:
 
 static void tun_get_iff(struct tun_struct *tun, struct ifreq *ifr)
 {
-       tun_debug(KERN_INFO, tun, "tun_get_iff\n");
-
        strcpy(ifr->ifr_name, tun->dev->name);
 
        ifr->ifr_flags = tun_flags(tun);
@@ -3206,9 +3196,6 @@ static long __tun_chr_ioctl(struct file *file, unsigned int cmd,
 
        case SIOCSIFHWADDR:
                /* Set hw address */
-               tun_debug(KERN_DEBUG, tun, "set hw address: %pM\n",
-                         ifr.ifr_hwaddr.sa_data);
-
                ret = dev_set_mac_address(tun->dev, &ifr.ifr_hwaddr, NULL);
                break;