Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Thu, 13 Feb 2020 05:05:23 -0800
From:   "Paul E. McKenney" <paulmck@kernel.org>
To:     Hongbo Yao <yaohongbo@huawei.com>
Cc:     linux-kernel@vger.kernel.org, chenzhou10@huawei.com,
        dave@stgolabs.net, josh@joshtriplett.org
Subject: Re: [PATCH RESEND -next] torture: avoid build error without
 CONFIG_RCU_TORTURE_TEST
Message-ID: <20200213130523.GY2935@paulmck-ThinkPad-P72>
Reply-To: paulmck@kernel.org
References: <20200213115313.4794-1-yaohongbo@huawei.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20200213115313.4794-1-yaohongbo@huawei.com>
User-Agent: Mutt/1.9.4 (2018-02-28)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Thu, Feb 13, 2020 at 07:53:13PM +0800, Hongbo Yao wrote:
> If TORTURE_TEST=y(selected by TORTURE_LOCK_TEST) and RCU_TORTURE_TEST=n,
> the following error is seen while building kernel/torture.c
> 
> kernel/torture.c: In function torture_onoff:
> kernel/torture.c:239:3: error: implicit declaration of function
> rcutorture_sched_setaffinity; did you mean __NR_ia32_sched_setaffinity?
> [-Werror=implicit-function-declaration]
>    rcutorture_sched_setaffinity(current->pid, cpumask_of(0));
> 
> Using sched_setaffnity() instead of rcutorture_sched_setaffinity() to
> avoid the error.
> 
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Fixes: bc3db9afb849 ("EXP: rcutorture hack to force CPU hotplug onto CPU 0")
> Signed-off-by: Hongbo Yao <yaohongbo@huawei.com>

Thank you for finding this!

However, this was due to a debugging commit used to chase a particular
bug (hence the "EXP:" at the beginning of the subject line).  Now that
this bug has been fixes (woo-hoo!!!), I will be dropping this commit.
In case you are interested, the fix is here:

97be2ad9946f ("rcu: Make rcu_barrier() account for offline no-CBs CPUs")

Either way, thank you for your testing efforts!

							Thanx, Paul

> ---
>  kernel/torture.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/kernel/torture.c b/kernel/torture.c
> index b29adec50e01..834214cbd1cd 100644
> --- a/kernel/torture.c
> +++ b/kernel/torture.c
> @@ -236,7 +236,7 @@ torture_onoff(void *arg)
>  			schedule_timeout_interruptible(HZ / 10);
>  			continue;
>  		}
> -		rcutorture_sched_setaffinity(current->pid, cpumask_of(0));
> +		sched_setaffinity(current->pid, cpumask_of(0));
>  		cpu = (torture_random(&rand) >> 4) % (maxcpu + 1);
>  		if (!torture_offline(cpu,
>  				     &n_offline_attempts, &n_offline_successes,
> -- 
> 2.17.1
>