]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
rcuscale: Provide clear error when async specified without primitives
authorPaul E. McKenney <paulmck@kernel.org>
Fri, 2 Aug 2024 00:43:03 +0000 (17:43 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 17 Oct 2024 13:08:13 +0000 (15:08 +0200)
[ Upstream commit 11377947b5861fa59bf77c827e1dd7c081842cc9 ]

Currently, if the rcuscale module's async module parameter is specified
for RCU implementations that do not have async primitives such as RCU
Tasks Rude (which now lacks a call_rcu_tasks_rude() function), there
will be a series of splats due to calls to a NULL pointer.  This commit
therefore warns of this situation, but switches to non-async testing.

Signed-off-by: "Paul E. McKenney" <paulmck@kernel.org>
Signed-off-by: Neeraj Upadhyay <neeraj.upadhyay@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
kernel/rcu/rcuscale.c

index 3b9783eda67969544d773ba2162f93407b1b46db..cfe861bac1bd43728a881292ce4c42fb5894b045 100644 (file)
@@ -398,7 +398,7 @@ rcu_scale_writer(void *arg)
                        udelay(writer_holdoff);
                wdp = &wdpp[i];
                *wdp = ktime_get_mono_fast_ns();
-               if (gp_async) {
+               if (gp_async && !WARN_ON_ONCE(!cur_ops->async)) {
 retry:
                        if (!rhp)
                                rhp = kmalloc(sizeof(*rhp), GFP_KERNEL);
@@ -454,7 +454,7 @@ retry:
                        i++;
                rcu_scale_wait_shutdown();
        } while (!torture_must_stop());
-       if (gp_async) {
+       if (gp_async && cur_ops->async) {
                cur_ops->gp_barrier();
        }
        writer_n_durations[me] = i_max + 1;