* [PATCH] rcutorture: replace the function name with %s
@ 2021-01-23 9:54 Stephen Zhang
2021-01-23 16:11 ` Paul E. McKenney
0 siblings, 1 reply; 2+ messages in thread
From: Stephen Zhang @ 2021-01-23 9:54 UTC (permalink / raw)
To: dave, paulmck, josh, rostedt, mathieu.desnoyers, jiangshanlai, joel
Cc: rcu, linux-kernel, Stephen Zhang
Better to replace the function name with %s in case of changes.
Signed-off-by: Stephen Zhang <stephenzhangzsd@gmail.com>
---
kernel/rcu/rcutorture.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/kernel/rcu/rcutorture.c b/kernel/rcu/rcutorture.c
index 2440f89..e561641 100644
--- a/kernel/rcu/rcutorture.c
+++ b/kernel/rcu/rcutorture.c
@@ -2016,8 +2016,8 @@ static int rcu_torture_stall(void *args)
local_irq_disable();
else if (!stall_cpu_block)
preempt_disable();
- pr_alert("rcu_torture_stall start on CPU %d.\n",
- raw_smp_processor_id());
+ pr_alert("%s start on CPU %d.\n",
+ __func__, raw_smp_processor_id());
while (ULONG_CMP_LT((unsigned long)ktime_get_seconds(),
stop_at))
if (stall_cpu_block)
@@ -2028,7 +2028,7 @@ static int rcu_torture_stall(void *args)
preempt_enable();
cur_ops->readunlock(idx);
}
- pr_alert("rcu_torture_stall end.\n");
+ pr_alert("%s end.\n", __func__);
torture_shutdown_absorb("rcu_torture_stall");
while (!kthread_should_stop())
schedule_timeout_interruptible(10 * HZ);
--
1.8.3.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] rcutorture: replace the function name with %s
2021-01-23 9:54 [PATCH] rcutorture: replace the function name with %s Stephen Zhang
@ 2021-01-23 16:11 ` Paul E. McKenney
0 siblings, 0 replies; 2+ messages in thread
From: Paul E. McKenney @ 2021-01-23 16:11 UTC (permalink / raw)
To: Stephen Zhang
Cc: dave, josh, rostedt, mathieu.desnoyers, jiangshanlai, joel, rcu,
linux-kernel
On Sat, Jan 23, 2021 at 05:54:17PM +0800, Stephen Zhang wrote:
> Better to replace the function name with %s in case of changes.
>
> Signed-off-by: Stephen Zhang <stephenzhangzsd@gmail.com>
I queued both for the v5.13 merge window with the usual wordsmithing,
thank you!
Thanx, Paul
> ---
> kernel/rcu/rcutorture.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/kernel/rcu/rcutorture.c b/kernel/rcu/rcutorture.c
> index 2440f89..e561641 100644
> --- a/kernel/rcu/rcutorture.c
> +++ b/kernel/rcu/rcutorture.c
> @@ -2016,8 +2016,8 @@ static int rcu_torture_stall(void *args)
> local_irq_disable();
> else if (!stall_cpu_block)
> preempt_disable();
> - pr_alert("rcu_torture_stall start on CPU %d.\n",
> - raw_smp_processor_id());
> + pr_alert("%s start on CPU %d.\n",
> + __func__, raw_smp_processor_id());
> while (ULONG_CMP_LT((unsigned long)ktime_get_seconds(),
> stop_at))
> if (stall_cpu_block)
> @@ -2028,7 +2028,7 @@ static int rcu_torture_stall(void *args)
> preempt_enable();
> cur_ops->readunlock(idx);
> }
> - pr_alert("rcu_torture_stall end.\n");
> + pr_alert("%s end.\n", __func__);
> torture_shutdown_absorb("rcu_torture_stall");
> while (!kthread_should_stop())
> schedule_timeout_interruptible(10 * HZ);
> --
> 1.8.3.1
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-01-23 16:11 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-23 9:54 [PATCH] rcutorture: replace the function name with %s Stephen Zhang
2021-01-23 16:11 ` Paul E. McKenney
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).