linux-kernel - Re: [PATCH] rcutorture: Traverse possible cpu to set maxcpu in rcu_nocb

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CALm+0cU9HbhCj0rc=-Cf9j3KUFjHRw2giMLV3+t_JrBTMCAaDA@mail.gmail.com>
Date:   Thu, 24 Aug 2023 16:59:58 +0800
From:   Z qiang <qiang.zhang1211@...il.com>
To:     paulmck@...nel.org, joel@...lfernandes.org
Cc:     linux-kernel@...r.kernel.org, rcu@...r.kernel.org
Subject: Re: [PATCH] rcutorture: Traverse possible cpu to set maxcpu in rcu_nocb_toggle()

Sorry for the repeat sending.


>
> Currently, the maxcpu is set by traversing online CPUs, however, if
> the rcutorture.onoff_holdoff is set zero and onoff_interval is set
> non-zero, and the some CPUs with larger cpuid has been offline before
> setting maxcpu, for these CPUs, even if they are online again, also
> cannot be offload or deoffload.
>
> This commit therefore use for_each_possible_cpu() instead of
> for_each_online_cpu() in rcu_nocb_toggle().
>
> Signed-off-by: Zqiang <qiang.zhang1211@...il.com>
> ---
>  kernel/rcu/rcutorture.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/kernel/rcu/rcutorture.c b/kernel/rcu/rcutorture.c
> index a58372bdf0c1..b75d0fe558ce 100644
> --- a/kernel/rcu/rcutorture.c
> +++ b/kernel/rcu/rcutorture.c
> @@ -2131,7 +2131,7 @@ static int rcu_nocb_toggle(void *arg)
>         VERBOSE_TOROUT_STRING("rcu_nocb_toggle task started");
>         while (!rcu_inkernel_boot_has_ended())
>                 schedule_timeout_interruptible(HZ / 10);
> -       for_each_online_cpu(cpu)
> +       for_each_possible_cpu(cpu)
>                 maxcpu = cpu;
>         WARN_ON(maxcpu < 0);
>         if (toggle_interval > ULONG_MAX)
> --
> 2.17.1
>