1 files changed, 27 insertions, 30 deletions

diff --git a/drivers/net/netconsole.c b/drivers/net/netconsole.c
index c9a15925a1f7..ba2f5e710af1 100644
--- a/drivers/net/netconsole.c
+++ b/drivers/net/netconsole.c
@@ -34,6 +34,8 @@
  *
  ****************************************************************/
 
+#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
+
 #include <linux/mm.h>
 #include <linux/init.h>
 #include <linux/module.h>
@@ -320,8 +322,8 @@ static ssize_t store_enabled(struct netconsole_target *nt,
         if (enabled < 0 || enabled > 1)
                 return -EINVAL;
         if (enabled == nt->enabled) {
-                printk(KERN_INFO "netconsole: network logging has already %s\n",
-                                nt->enabled ? "started" : "stopped");
+                pr_info("network logging has already %s\n",
+                        nt->enabled ? "started" : "stopped");
                 return -EINVAL;
         }
 
@@ -336,7 +338,7 @@ static ssize_t store_enabled(struct netconsole_target *nt,
                 if (err)
                         return err;
 
-                printk(KERN_INFO "netconsole: network logging started\n");
+                pr_info("netconsole: network logging started\n");
         } else {        /* 0 */
                 /* We need to disable the netconsole before cleaning it up
                  * otherwise we might end up in write_msg() with
@@ -360,9 +362,8 @@ static ssize_t store_dev_name(struct netconsole_target *nt,
         size_t len;
 
         if (nt->enabled) {
-                printk(KERN_ERR "netconsole: target (%s) is enabled, "
-                                "disable to update parameters\n",
-                                config_item_name(&nt->item));
+                pr_err("target (%s) is enabled, disable to update parameters\n",
+                       config_item_name(&nt->item));
                 return -EINVAL;
         }
 
@@ -383,9 +384,8 @@ static ssize_t store_local_port(struct netconsole_target *nt,
         int rv;
 
         if (nt->enabled) {
-                printk(KERN_ERR "netconsole: target (%s) is enabled, "
-                                "disable to update parameters\n",
-                                config_item_name(&nt->item));
+                pr_err("target (%s) is enabled, disable to update parameters\n",
+                       config_item_name(&nt->item));
                 return -EINVAL;
         }
 
@@ -402,9 +402,8 @@ static ssize_t store_remote_port(struct netconsole_target *nt,
         int rv;
 
         if (nt->enabled) {
-                printk(KERN_ERR "netconsole: target (%s) is enabled, "
-                                "disable to update parameters\n",
-                                config_item_name(&nt->item));
+                pr_err("target (%s) is enabled, disable to update parameters\n",
+                       config_item_name(&nt->item));
                 return -EINVAL;
         }
 
@@ -419,9 +418,8 @@ static ssize_t store_local_ip(struct netconsole_target *nt,
                               size_t count)
 {
         if (nt->enabled) {
-                printk(KERN_ERR "netconsole: target (%s) is enabled, "
-                                "disable to update parameters\n",
-                                config_item_name(&nt->item));
+                pr_err("target (%s) is enabled, disable to update parameters\n",
+                       config_item_name(&nt->item));
                 return -EINVAL;
         }
 
@@ -429,7 +427,7 @@ static ssize_t store_local_ip(struct netconsole_target *nt,
                 const char *end;
                 if (in6_pton(buf, count, nt->np.local_ip.in6.s6_addr, -1, &end) > 0) {
                         if (*end && *end != '\n') {
-                                printk(KERN_ERR "netconsole: invalid IPv6 address at: <%c>\n", *end);
+                                pr_err("invalid IPv6 address at: <%c>\n", *end);
                                 return -EINVAL;
                         }
                         nt->np.ipv6 = true;
@@ -450,9 +448,8 @@ static ssize_t store_remote_ip(struct netconsole_target *nt,
                                size_t count)
 {
         if (nt->enabled) {
-                printk(KERN_ERR "netconsole: target (%s) is enabled, "
-                                "disable to update parameters\n",
-                                config_item_name(&nt->item));
+                pr_err("target (%s) is enabled, disable to update parameters\n",
+                       config_item_name(&nt->item));
                 return -EINVAL;
         }
 
@@ -460,7 +457,7 @@ static ssize_t store_remote_ip(struct netconsole_target *nt,
                 const char *end;
                 if (in6_pton(buf, count, nt->np.remote_ip.in6.s6_addr, -1, &end) > 0) {
                         if (*end && *end != '\n') {
-                                printk(KERN_ERR "netconsole: invalid IPv6 address at: <%c>\n", *end);
+                                pr_err("invalid IPv6 address at: <%c>\n", *end);
                                 return -EINVAL;
                         }
                         nt->np.ipv6 = true;
@@ -483,9 +480,8 @@ static ssize_t store_remote_mac(struct netconsole_target *nt,
         u8 remote_mac[ETH_ALEN];
 
         if (nt->enabled) {
-                printk(KERN_ERR "netconsole: target (%s) is enabled, "
-                                "disable to update parameters\n",
-                                config_item_name(&nt->item));
+                pr_err("target (%s) is enabled, disable to update parameters\n",
+                       config_item_name(&nt->item));
                 return -EINVAL;
         }
 
@@ -708,19 +704,20 @@ restart:
         }
         spin_unlock_irqrestore(&target_list_lock, flags);
         if (stopped) {
-                printk(KERN_INFO "netconsole: network logging stopped on "
-                       "interface %s as it ", dev->name);
+                const char *msg = "had an event";
                 switch (event) {
                 case NETDEV_UNREGISTER:
-                        printk(KERN_CONT "unregistered\n");
+                        msg = "unregistered";
                         break;
                 case NETDEV_RELEASE:
-                        printk(KERN_CONT "released slaves\n");
+                        msg = "released slaves";
                         break;
                 case NETDEV_JOIN:
-                        printk(KERN_CONT "is joining a master device\n");
+                        msg = "is joining a master device";
                         break;
                 }
+                pr_info("network logging stopped on interface %s as it %s\n",
+                        dev->name, msg);
         }
 
 done:
@@ -806,7 +803,7 @@ static int __init init_netconsole(void)
                 goto undonotifier;
 
         register_console(&netconsole);
-        printk(KERN_INFO "netconsole: network logging started\n");
+        pr_info("network logging started\n");
 
         return err;
 
@@ -814,7 +811,7 @@ undonotifier:
         unregister_netdevice_notifier(&netconsole_netdev_notifier);
 
 fail:
-        printk(KERN_ERR "netconsole: cleaning up\n");
+        pr_err("cleaning up\n");
 
         /*
          * Remove all targets and destroy them (only targets created