On Thu, Oct 24, 2024 at 06:45:58PM +0200, Uladzislau Rezki (Sony) wrote: > There are two places where WARN_ON_ONCE() is called two times > in the error paths. One which is encapsulated into if() condition > and another one, which is unnecessary, is placed in the brackets. > > Remove an extra WARN_ON_ONCE() splat which is in brackets. > > Signed-off-by: Uladzislau Rezki (Sony) <urezki@xxxxxxxxx> For both: Reviewed-by: Paul E. McKenney <paulmck@xxxxxxxxxx> > --- > kernel/rcu/rcuscale.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/kernel/rcu/rcuscale.c b/kernel/rcu/rcuscale.c > index de7d511e6be4..1d8bb603c289 100644 > --- a/kernel/rcu/rcuscale.c > +++ b/kernel/rcu/rcuscale.c > @@ -889,13 +889,11 @@ kfree_scale_init(void) > > if (WARN_ON_ONCE(jiffies_at_lazy_cb - jif_start < 2 * HZ)) { > pr_alert("ERROR: call_rcu() CBs are not being lazy as expected!\n"); > - WARN_ON_ONCE(1); > goto unwind; > } > > if (WARN_ON_ONCE(jiffies_at_lazy_cb - jif_start > 3 * HZ)) { > pr_alert("ERROR: call_rcu() CBs are being too lazy!\n"); > - WARN_ON_ONCE(1); > goto unwind; > } > } > -- > 2.39.5 >