]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
rcu: Delete unused rcu_gp_might_be_stalled() function
authorPaul E. McKenney <paulmck@kernel.org>
Wed, 16 Oct 2024 16:19:29 +0000 (09:19 -0700)
committerFrederic Weisbecker <frederic@kernel.org>
Wed, 23 Oct 2024 16:00:17 +0000 (18:00 +0200)
The rcu_gp_might_be_stalled() function is no longer used, so this commit
removes it.

Signed-off-by: Paul E. McKenney <paulmck@kernel.org>
Reviewed-by: Joel Fernandes (Google) <joel@joelfernandes.org>
Signed-off-by: Frederic Weisbecker <frederic@kernel.org>
include/linux/rcutiny.h
include/linux/rcutree.h
kernel/rcu/tree_stall.h

index 0ee270b3f5ed2fb8cb8aa5dc4d11e7a24c9675f3..fe42315f667fc5be7f2ed8eae6ea0c7193030846 100644 (file)
@@ -165,7 +165,6 @@ static inline bool rcu_inkernel_boot_has_ended(void) { return true; }
 static inline bool rcu_is_watching(void) { return true; }
 static inline void rcu_momentary_eqs(void) { }
 static inline void kfree_rcu_scheduler_running(void) { }
-static inline bool rcu_gp_might_be_stalled(void) { return false; }
 
 /* Avoid RCU read-side critical sections leaking across. */
 static inline void rcu_all_qs(void) { barrier(); }
index 90a684f94776ed8cccf53e0e35239c50729943d6..27d86d9127817e50f8d4dd79e1990d70a02435bb 100644 (file)
@@ -40,7 +40,6 @@ void kvfree_rcu_barrier(void);
 void rcu_barrier(void);
 void rcu_momentary_eqs(void);
 void kfree_rcu_scheduler_running(void);
-bool rcu_gp_might_be_stalled(void);
 
 struct rcu_gp_oldstate {
        unsigned long rgos_norm;
index 4432db6d0b99b3bfd79194abfe3be478e8929f59..d7cdd535e50b11536eb9d57ac18fa030a1ebfa8b 100644 (file)
@@ -76,36 +76,6 @@ int rcu_jiffies_till_stall_check(void)
 }
 EXPORT_SYMBOL_GPL(rcu_jiffies_till_stall_check);
 
-/**
- * rcu_gp_might_be_stalled - Is it likely that the grace period is stalled?
- *
- * Returns @true if the current grace period is sufficiently old that
- * it is reasonable to assume that it might be stalled.  This can be
- * useful when deciding whether to allocate memory to enable RCU-mediated
- * freeing on the one hand or just invoking synchronize_rcu() on the other.
- * The latter is preferable when the grace period is stalled.
- *
- * Note that sampling of the .gp_start and .gp_seq fields must be done
- * carefully to avoid false positives at the beginnings and ends of
- * grace periods.
- */
-bool rcu_gp_might_be_stalled(void)
-{
-       unsigned long d = rcu_jiffies_till_stall_check() / RCU_STALL_MIGHT_DIV;
-       unsigned long j = jiffies;
-
-       if (d < RCU_STALL_MIGHT_MIN)
-               d = RCU_STALL_MIGHT_MIN;
-       smp_mb(); // jiffies before .gp_seq to avoid false positives.
-       if (!rcu_gp_in_progress())
-               return false;
-       // Long delays at this point avoids false positive, but a delay
-       // of ULONG_MAX/4 jiffies voids your no-false-positive warranty.
-       smp_mb(); // .gp_seq before second .gp_start
-       // And ditto here.
-       return !time_before(j, READ_ONCE(rcu_state.gp_start) + d);
-}
-
 /* Don't do RCU CPU stall warnings during long sysrq printouts. */
 void rcu_sysrq_start(void)
 {