Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1627182 > unrolled thread

[PATCH] rcuperf: refix rcuperf sched_ops definition

Started byArnd Bergmann <arnd@arndb.de>
First post2017-04-20 10:20 +0200
Last post2017-04-20 17:40 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] rcuperf: refix rcuperf sched_ops definition Arnd Bergmann <arnd@arndb.de> - 2017-04-20 10:20 +0200
    Re: [PATCH] rcuperf: refix rcuperf sched_ops definition "Paul E. McKenney" <paulmck@linux.vnet.ibm.com> - 2017-04-20 17:40 +0200

#1627182 — [PATCH] rcuperf: refix rcuperf sched_ops definition

FromArnd Bergmann <arnd@arndb.de>
Date2017-04-20 10:20 +0200
Subject[PATCH] rcuperf: refix rcuperf sched_ops definition
Message-ID<tyftL-6wC-1@gated-at.bofh.it>
It looks like my fixup was incorrectly folded into the original patch,
and still leaves a duplicated initializer but drops the .sync
callback pointer.

This should rectify it, please fold it as well.

Fixes: df29028c2a9c ("rcuperf: Add ability to performance-test call_rcu() and friends")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 kernel/rcu/rcuperf.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/rcu/rcuperf.c b/kernel/rcu/rcuperf.c
index 1e2a1424d1f5..5158ddba6716 100644
--- a/kernel/rcu/rcuperf.c
+++ b/kernel/rcu/rcuperf.c
@@ -308,7 +308,7 @@ static struct rcu_perf_ops sched_ops = {
 	.exp_completed	= rcu_exp_batches_completed_sched,
 	.async		= call_rcu_sched,
 	.gp_barrier	= rcu_barrier_sched,
-	.async		= srcu_call_rcu,
+	.sync		= synchronize_sched,
 	.exp_sync	= synchronize_sched_expedited,
 	.name		= "sched"
 };
-- 
2.9.0

[toc] | [next] | [standalone]


#1627595

From"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Date2017-04-20 17:40 +0200
Message-ID<tymlz-2fX-7@gated-at.bofh.it>
In reply to#1627182
On Thu, Apr 20, 2017 at 10:12:31AM +0200, Arnd Bergmann wrote:
> It looks like my fixup was incorrectly folded into the original patch,
> and still leaves a duplicated initializer but drops the .sync
> callback pointer.
> 
> This should rectify it, please fold it as well.
> 
> Fixes: df29028c2a9c ("rcuperf: Add ability to performance-test call_rcu() and friends")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

I really am not doing well on this commit, am I?  :-/

Thank you very much for your attention to detail and for the fixes!!!

							Thanx, Paul

> ---
>  kernel/rcu/rcuperf.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/kernel/rcu/rcuperf.c b/kernel/rcu/rcuperf.c
> index 1e2a1424d1f5..5158ddba6716 100644
> --- a/kernel/rcu/rcuperf.c
> +++ b/kernel/rcu/rcuperf.c
> @@ -308,7 +308,7 @@ static struct rcu_perf_ops sched_ops = {
>  	.exp_completed	= rcu_exp_batches_completed_sched,
>  	.async		= call_rcu_sched,
>  	.gp_barrier	= rcu_barrier_sched,
> -	.async		= srcu_call_rcu,
> +	.sync		= synchronize_sched,
>  	.exp_sync	= synchronize_sched_expedited,
>  	.name		= "sched"
>  };
> -- 
> 2.9.0
> 

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web