netconsole: make netconsole_target->enabled a bool
authorTejun Heo <tj@kernel.org>
Thu, 25 Jun 2015 22:01:36 +0000 (15:01 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 26 Jun 2015 00:00:39 +0000 (17:00 -0700)
netconsole uses both bool and int for boolean values.  Let's convert
nt->enabled to bool for consistency.

Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: David Miller <davem@davemloft.net>
Cc: Kay Sievers <kay@vrfy.org>
Cc: Petr Mladek <pmladek@suse.cz>
Cc: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/net/netconsole.c

index 30c0524d701691f76c7fc8be409a4dbc19ade5bb..8dd1e554aa16daef995b85544ac9e40d72be5c5d 100644 (file)
@@ -104,7 +104,7 @@ struct netconsole_target {
 #ifdef CONFIG_NETCONSOLE_DYNAMIC
        struct config_item      item;
 #endif
-       int                     enabled;
+       bool                    enabled;
        struct mutex            mutex;
        struct netpoll          np;
 };
@@ -197,7 +197,7 @@ static struct netconsole_target *alloc_param_target(char *target_config)
        if (err)
                goto fail;
 
-       nt->enabled = 1;
+       nt->enabled = true;
 
        return nt;
 
@@ -322,13 +322,13 @@ static ssize_t store_enabled(struct netconsole_target *nt,
                return err;
        if (enabled < 0 || enabled > 1)
                return -EINVAL;
-       if (enabled == nt->enabled) {
+       if ((bool)enabled == nt->enabled) {
                pr_info("network logging has already %s\n",
                        nt->enabled ? "started" : "stopped");
                return -EINVAL;
        }
 
-       if (enabled) {  /* 1 */
+       if (enabled) {  /* true */
                /*
                 * Skip netpoll_parse_options() -- all the attributes are
                 * already configured via configfs. Just print them out.
@@ -340,13 +340,13 @@ static ssize_t store_enabled(struct netconsole_target *nt,
                        return err;
 
                pr_info("netconsole: network logging started\n");
-       } else {        /* 0 */
+       } else {        /* false */
                /* We need to disable the netconsole before cleaning it up
                 * otherwise we might end up in write_msg() with
-                * nt->np.dev == NULL and nt->enabled == 1
+                * nt->np.dev == NULL and nt->enabled == true
                 */
                spin_lock_irqsave(&target_list_lock, flags);
-               nt->enabled = 0;
+               nt->enabled = false;
                spin_unlock_irqrestore(&target_list_lock, flags);
                netpoll_cleanup(&nt->np);
        }
@@ -594,7 +594,7 @@ static struct config_item *make_netconsole_target(struct config_group *group,
 
        /*
         * Allocate and initialize with defaults.
-        * Target is disabled at creation (enabled == 0).
+        * Target is disabled at creation (!enabled).
         */
        nt = kzalloc(sizeof(*nt), GFP_KERNEL);
        if (!nt)
@@ -695,7 +695,7 @@ restart:
                                spin_lock_irqsave(&target_list_lock, flags);
                                dev_put(nt->np.dev);
                                nt->np.dev = NULL;
-                               nt->enabled = 0;
+                               nt->enabled = false;
                                stopped = true;
                                netconsole_target_put(nt);
                                goto restart;