net ipv4: use preferred log methods
authorBastian Stender <bst@pengutronix.de>
Fri, 13 Nov 2015 10:40:34 +0000 (11:40 +0100)
committerDavid S. Miller <davem@davemloft.net>
Wed, 18 Nov 2015 18:37:20 +0000 (13:37 -0500)
Replace printk calls with preferred unconditional log method calls to keep
kernel messages clean.

Added newline to "too small MTU" message.

Signed-off-by: Bastian Stender <bst@pengutronix.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/ipconfig.c
net/ipv4/netfilter/arp_tables.c
net/ipv4/netfilter/nf_conntrack_l3proto_ipv4.c
net/ipv4/netfilter/nf_nat_snmp_basic.c

index 0bc7412d9e14a3395ab9c877ee6683f33748a3b1..e86e8a9738ea9b1e16526c2c06a4fb63319372a1 100644 (file)
 #include <net/checksum.h>
 #include <asm/processor.h>
 
-/* Define this to allow debugging output */
-#undef IPCONFIG_DEBUG
-
-#ifdef IPCONFIG_DEBUG
-#define DBG(x) printk x
-#else
-#define DBG(x) do { } while(0)
-#endif
-
 #if defined(CONFIG_IP_PNP_DHCP)
 #define IPCONFIG_DHCP
 #endif
@@ -227,7 +218,7 @@ static int __init ic_open_devs(void)
                        if (dev->mtu >= 364)
                                able |= IC_BOOTP;
                        else
-                               pr_warn("DHCP/BOOTP: Ignoring device %s, MTU %d too small",
+                               pr_warn("DHCP/BOOTP: Ignoring device %s, MTU %d too small\n",
                                        dev->name, dev->mtu);
                        if (!(dev->flags & IFF_NOARP))
                                able |= IC_RARP;
@@ -254,8 +245,8 @@ static int __init ic_open_devs(void)
                        else
                                d->xid = 0;
                        ic_proto_have_if |= able;
-                       DBG(("IP-Config: %s UP (able=%d, xid=%08x)\n",
-                               dev->name, able, d->xid));
+                       pr_debug("IP-Config: %s UP (able=%d, xid=%08x)\n",
+                                dev->name, able, d->xid);
                }
        }
 
@@ -311,7 +302,7 @@ static void __init ic_close_devs(void)
                next = d->next;
                dev = d->dev;
                if (dev != ic_dev && !netdev_uses_dsa(dev)) {
-                       DBG(("IP-Config: Downing %s\n", dev->name));
+                       pr_debug("IP-Config: Downing %s\n", dev->name);
                        dev_change_flags(dev, d->flags);
                }
                kfree(d);
@@ -464,7 +455,8 @@ static int __init ic_defaults(void)
                               &ic_myaddr);
                        return -1;
                }
-               printk("IP-Config: Guessing netmask %pI4\n", &ic_netmask);
+               pr_notice("IP-Config: Guessing netmask %pI4\n",
+                         &ic_netmask);
        }
 
        return 0;
@@ -675,9 +667,7 @@ ic_dhcp_init_options(u8 *options)
        u8 *e = options;
        int len;
 
-#ifdef IPCONFIG_DEBUG
-       printk("DHCP: Sending message type %d\n", mt);
-#endif
+       pr_debug("DHCP: Sending message type %d\n", mt);
 
        memcpy(e, ic_bootp_cookie, 4);  /* RFC1048 Magic Cookie */
        e += 4;
@@ -847,7 +837,8 @@ static void __init ic_bootp_send_if(struct ic_device *d, unsigned long jiffies_d
        else if (dev->type == ARPHRD_FDDI)
                b->htype = ARPHRD_ETHER;
        else {
-               printk("Unknown ARP type 0x%04x for device %s\n", dev->type, dev->name);
+               pr_warn("Unknown ARP type 0x%04x for device %s\n", dev->type,
+                       dev->name);
                b->htype = dev->type; /* can cause undefined behavior */
        }
 
@@ -904,14 +895,12 @@ static void __init ic_do_bootp_ext(u8 *ext)
        int i;
        __be16 mtu;
 
-#ifdef IPCONFIG_DEBUG
        u8 *c;
 
-       printk("DHCP/BOOTP: Got extension %d:",*ext);
+       pr_debug("DHCP/BOOTP: Got extension %d:", *ext);
        for (c=ext+2; c<ext+2+ext[1]; c++)
-               printk(" %02x", *c);
-       printk("\n");
-#endif
+               pr_debug(" %02x", *c);
+       pr_debug("\n");
 
        switch (*ext++) {
        case 1:         /* Subnet mask */
@@ -1080,9 +1069,7 @@ static int __init ic_bootp_recv(struct sk_buff *skb, struct net_device *dev, str
                                }
                        }
 
-#ifdef IPCONFIG_DEBUG
-                       printk("DHCP: Got message type %d\n", mt);
-#endif
+                       pr_debug("DHCP: Got message type %d\n", mt);
 
                        switch (mt) {
                        case DHCPOFFER:
@@ -1095,10 +1082,8 @@ static int __init ic_bootp_recv(struct sk_buff *skb, struct net_device *dev, str
                                /* Let's accept that offer. */
                                ic_myaddr = b->your_ip;
                                ic_servaddr = server_id;
-#ifdef IPCONFIG_DEBUG
-                               printk("DHCP: Offered address %pI4 by server %pI4\n",
-                                      &ic_myaddr, &b->iph.saddr);
-#endif
+                               pr_debug("DHCP: Offered address %pI4 by server %pI4\n",
+                                        &ic_myaddr, &b->iph.saddr);
                                /* The DHCP indicated server address takes
                                 * precedence over the bootp header one if
                                 * they are different.
@@ -1254,13 +1239,13 @@ static int __init ic_dynamic(void)
                    (ic_proto_enabled & IC_USE_DHCP) &&
                    ic_dhcp_msgtype != DHCPACK) {
                        ic_got_reply = 0;
-                       pr_cont(",");
+                       pr_notice(",");
                        continue;
                }
 #endif /* IPCONFIG_DHCP */
 
                if (ic_got_reply) {
-                       pr_cont(" OK\n");
+                       pr_notice(" OK\n");
                        break;
                }
 
@@ -1268,7 +1253,7 @@ static int __init ic_dynamic(void)
                        continue;
 
                if (! --retries) {
-                       pr_cont(" timed out!\n");
+                       pr_notice(" timed out!\n");
                        break;
                }
 
@@ -1278,7 +1263,7 @@ static int __init ic_dynamic(void)
                if (timeout > CONF_TIMEOUT_MAX)
                        timeout = CONF_TIMEOUT_MAX;
 
-               pr_cont(".");
+               pr_notice(".");
        }
 
 #ifdef IPCONFIG_BOOTP
@@ -1295,11 +1280,11 @@ static int __init ic_dynamic(void)
                return -1;
        }
 
-       printk("IP-Config: Got %s answer from %pI4, ",
+       pr_info("IP-Config: Got %s answer from %pI4, ",
                ((ic_got_reply & IC_RARP) ? "RARP"
-                : (ic_proto_enabled & IC_USE_DHCP) ? "DHCP" : "BOOTP"),
-              &ic_addrservaddr);
-       pr_cont("my address is %pI4\n", &ic_myaddr);
+               : (ic_proto_enabled & IC_USE_DHCP) ? "DHCP" : "BOOTP"),
+               &ic_addrservaddr);
+       pr_info("my address is %pI4\n", &ic_myaddr);
 
        return 0;
 }
@@ -1426,7 +1411,7 @@ static int __init ip_auto_config(void)
        if (!ic_enable)
                return 0;
 
-       DBG(("IP-Config: Entered.\n"));
+       pr_debug("IP-Config: Entered.\n");
 #ifdef IPCONFIG_DYNAMIC
  try_try_again:
 #endif
@@ -1548,8 +1533,8 @@ static int __init ip_auto_config(void)
                }
        for (i++; i < CONF_NAMESERVERS_MAX; i++)
                if (ic_nameservers[i] != NONE)
-                       pr_cont(", nameserver%u=%pI4", i, &ic_nameservers[i]);
-       pr_cont("\n");
+                       pr_info(", nameserver%u=%pI4", i, &ic_nameservers[i]);
+       pr_info("\n");
 #endif /* !SILENT */
 
        return 0;
@@ -1585,7 +1570,7 @@ static int __init ic_proto_name(char *name)
                                return 1;
                        *v = 0;
                        if (kstrtou8(client_id, 0, dhcp_client_identifier))
-                               DBG("DHCP: Invalid client identifier type\n");
+                               pr_debug("DHCP: Invalid client identifier type\n");
                        strncpy(dhcp_client_identifier + 1, v + 1, 251);
                        *v = ',';
                }
@@ -1644,7 +1629,7 @@ static int __init ip_auto_config_setup(char *addrs)
                if ((cp = strchr(ip, ':')))
                        *cp++ = '\0';
                if (strlen(ip) > 0) {
-                       DBG(("IP-Config: Parameter #%d: `%s'\n", num, ip));
+                       pr_debug("IP-Config: Parameter #%d: `%s'\n", num, ip);
                        switch (num) {
                        case 0:
                                if ((ic_myaddr = in_aton(ip)) == ANY)
@@ -1716,7 +1701,7 @@ static int __init vendor_class_identifier_setup(char *addrs)
        if (strlcpy(vendor_class_identifier, addrs,
                    sizeof(vendor_class_identifier))
            >= sizeof(vendor_class_identifier))
-               pr_warn("DHCP: vendorclass too long, truncated to \"%s\"",
+               pr_warn("DHCP: vendorclass too long, truncated to \"%s\"\n",
                        vendor_class_identifier);
        return 1;
 }
index 11dccba474b7964fe7d9ee472c48d5eca435ae41..b488cac9c5ca5496238bcc31a02887362655212b 100644 (file)
@@ -38,13 +38,13 @@ MODULE_DESCRIPTION("arptables core");
 /*#define DEBUG_ARP_TABLES_USER*/
 
 #ifdef DEBUG_ARP_TABLES
-#define dprintf(format, args...)  printk(format , ## args)
+#define dprintf(format, args...)  pr_debug(format, ## args)
 #else
 #define dprintf(format, args...)
 #endif
 
 #ifdef DEBUG_ARP_TABLES_USER
-#define duprintf(format, args...) printk(format , ## args)
+#define duprintf(format, args...) pr_debug(format, ## args)
 #else
 #define duprintf(format, args...)
 #endif
@@ -1905,7 +1905,7 @@ static int __init arp_tables_init(void)
        if (ret < 0)
                goto err4;
 
-       printk(KERN_INFO "arp_tables: (C) 2002 David S. Miller\n");
+       pr_info("arp_tables: (C) 2002 David S. Miller\n");
        return 0;
 
 err4:
index 461ca926fd39408613f62a3de8bfc5ff4cbce7f6..e3c46e8e276267b74b0482e85a208a70fa7f3d85 100644 (file)
@@ -451,7 +451,7 @@ static int __init nf_conntrack_l3proto_ipv4_init(void)
 
        ret = nf_register_sockopt(&so_getorigdst);
        if (ret < 0) {
-               printk(KERN_ERR "Unable to register netfilter socket option\n");
+               pr_err("Unable to register netfilter socket option\n");
                return ret;
        }
 
index ddb894ac1458ca40bf30d213e8b3b2c8a18880af..c9b52c361da2e6acc746c2de86d8c7f3af0a9b39 100644 (file)
@@ -1048,7 +1048,7 @@ static int snmp_parse_mangle(unsigned char *msg,
        if (!asn1_uint_decode (&ctx, end, &vers))
                return 0;
        if (debug > 1)
-               printk(KERN_DEBUG "bsalg: snmp version: %u\n", vers + 1);
+               pr_debug("bsalg: snmp version: %u\n", vers + 1);
        if (vers > 1)
                return 1;
 
@@ -1064,10 +1064,10 @@ static int snmp_parse_mangle(unsigned char *msg,
        if (debug > 1) {
                unsigned int i;
 
-               printk(KERN_DEBUG "bsalg: community: ");
+               pr_debug("bsalg: community: ");
                for (i = 0; i < comm.len; i++)
-                       printk("%c", comm.data[i]);
-               printk("\n");
+                       pr_cont("%c", comm.data[i]);
+               pr_cont("\n");
        }
        kfree(comm.data);
 
@@ -1091,9 +1091,9 @@ static int snmp_parse_mangle(unsigned char *msg,
                };
 
                if (pdutype > SNMP_PDU_TRAP2)
-                       printk(KERN_DEBUG "bsalg: bad pdu type %u\n", pdutype);
+                       pr_debug("bsalg: bad pdu type %u\n", pdutype);
                else
-                       printk(KERN_DEBUG "bsalg: pdu: %s\n", pdus[pdutype]);
+                       pr_debug("bsalg: pdu: %s\n", pdus[pdutype]);
        }
        if (pdutype != SNMP_PDU_RESPONSE &&
            pdutype != SNMP_PDU_TRAP1 && pdutype != SNMP_PDU_TRAP2)
@@ -1119,7 +1119,7 @@ static int snmp_parse_mangle(unsigned char *msg,
                        return 0;
 
                if (debug > 1)
-                       printk(KERN_DEBUG "bsalg: request: id=0x%lx error_status=%u "
+                       pr_debug("bsalg: request: id=0x%lx error_status=%u "
                        "error_index=%u\n", req.id, req.error_status,
                        req.error_index);
        }
@@ -1145,13 +1145,13 @@ static int snmp_parse_mangle(unsigned char *msg,
                }
 
                if (debug > 1) {
-                       printk(KERN_DEBUG "bsalg: object: ");
+                       pr_debug("bsalg: object: ");
                        for (i = 0; i < obj->id_len; i++) {
                                if (i > 0)
-                                       printk(".");
-                               printk("%lu", obj->id[i]);
+                                       pr_cont(".");
+                               pr_cont("%lu", obj->id[i]);
                        }
-                       printk(": type=%u\n", obj->type);
+                       pr_cont(": type=%u\n", obj->type);
 
                }