From: Ildar Ismagilov <[email protected]>

The srcu_funnel_exp_start() function checks to see if the srcu_struct
structure's expedited grace period counter needs updating to reflect a
newly arrived request for an expedited SRCU grace period.  Unfortunately,
the check is backwards, so this commit reverses the sense of the test.

Signed-off-by: Ildar Ismagilov <[email protected]>
Signed-off-by: Paul E. McKenney <[email protected]>
---
 kernel/rcu/srcutree.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/rcu/srcutree.c b/kernel/rcu/srcutree.c
index 9c6e0eaab4d5..045b559b9f22 100644
--- a/kernel/rcu/srcutree.c
+++ b/kernel/rcu/srcutree.c
@@ -626,7 +626,7 @@ static void srcu_funnel_exp_start(struct srcu_struct *sp, 
struct srcu_node *snp,
                spin_unlock_irqrestore_rcu_node(snp, flags);
        }
        spin_lock_irqsave_rcu_node(sp, flags);
-       if (!ULONG_CMP_LT(sp->srcu_gp_seq_needed_exp, s))
+       if (ULONG_CMP_LT(sp->srcu_gp_seq_needed_exp, s))
                sp->srcu_gp_seq_needed_exp = s;
        spin_unlock_irqrestore_rcu_node(sp, flags);
 }
-- 
2.5.2

Reply via email to