[PATCH v3 5/5] rv: Remove struct rv_monitor::reacting

[Nam Cao]

The field 'reacting' in struct rv_monitor is set but never used. Delete it.

Reviewed-by: Gabriele Monaco <gmon...@redhat.com>
Signed-off-by: Nam Cao <nam...@linutronix.de>
---
 include/linux/rv.h            |  1 -
 kernel/trace/rv/rv_reactors.c | 19 +++++--------------
 2 files changed, 5 insertions(+), 15 deletions(-)

diff --git a/include/linux/rv.h b/include/linux/rv.h
index 2f867d6f72ba..80731242fe60 100644
--- a/include/linux/rv.h
+++ b/include/linux/rv.h
@@ -104,7 +104,6 @@ struct rv_monitor {
 #ifdef CONFIG_RV_REACTORS
        struct rv_reactor       *reactor;
        __printf(1, 2) void     (*react)(const char *msg, ...);
-       bool                    reacting;
 #endif
        struct list_head        list;
        struct rv_monitor       *parent;
diff --git a/kernel/trace/rv/rv_reactors.c b/kernel/trace/rv/rv_reactors.c
index a8e849e6cd85..106f2c4740f2 100644
--- a/kernel/trace/rv/rv_reactors.c
+++ b/kernel/trace/rv/rv_reactors.c
@@ -160,7 +160,7 @@ static const struct seq_operations monitor_reactors_seq_ops 
= {
 
 static void monitor_swap_reactors_single(struct rv_monitor *mon,
                                         struct rv_reactor *reactor,
-                                        bool reacting, bool nested)
+                                        bool nested)
 {
        bool monitor_enabled;
 
@@ -173,7 +173,6 @@ static void monitor_swap_reactors_single(struct rv_monitor 
*mon,
                rv_disable_monitor(mon);
 
        mon->reactor = reactor;
-       mon->reacting = reacting;
        mon->react = reactor->react;
 
        /* enable only once if iterating through a container */
@@ -181,8 +180,7 @@ static void monitor_swap_reactors_single(struct rv_monitor 
*mon,
                rv_enable_monitor(mon);
 }
 
-static void monitor_swap_reactors(struct rv_monitor *mon,
-                                 struct rv_reactor *reactor, bool reacting)
+static void monitor_swap_reactors(struct rv_monitor *mon, struct rv_reactor 
*reactor)
 {
        struct rv_monitor *p = mon;
 
@@ -190,7 +188,7 @@ static void monitor_swap_reactors(struct rv_monitor *mon,
                list_for_each_entry_continue(p, &rv_monitors_list, list) {
                        if (p->parent != mon)
                                break;
-                       monitor_swap_reactors_single(p, reactor, reacting, 
true);
+                       monitor_swap_reactors_single(p, reactor, true);
                }
        /*
         * This call enables and disables the monitor if they were active.
@@ -198,7 +196,7 @@ static void monitor_swap_reactors(struct rv_monitor *mon,
         * All nested monitors are enabled also if they were off, we may refine
         * this logic in the future.
         */
-       monitor_swap_reactors_single(mon, reactor, reacting, false);
+       monitor_swap_reactors_single(mon, reactor, false);
 }
 
 static ssize_t
@@ -210,7 +208,6 @@ monitor_reactors_write(struct file *file, const char __user 
*user_buf,
        struct rv_reactor *reactor;
        struct seq_file *seq_f;
        int retval = -EINVAL;
-       bool enable;
        char *ptr;
        int len;
 
@@ -243,12 +240,7 @@ monitor_reactors_write(struct file *file, const char 
__user *user_buf,
                if (strcmp(ptr, reactor->name) != 0)
                        continue;
 
-               if (strcmp(reactor->name, "nop"))
-                       enable = false;
-               else
-                       enable = true;
-
-               monitor_swap_reactors(mon, reactor, enable);
+               monitor_swap_reactors(mon, reactor);
 
                retval = count;
                break;
@@ -439,7 +431,6 @@ int reactor_populate_monitor(struct rv_monitor *mon)
         * Configure as the rv_nop reactor.
         */
        mon->reactor = get_reactor_rdef_by_name("nop");
-       mon->reacting = false;
 
        return 0;
 }
-- 
2.39.5