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


Groups > linux.kernel > #1464548 > unrolled thread

[PATCH] cpu/hotplug: get rid of CPU_STARTING reference

Started bySebastian Andrzej Siewior <bigeasy@linutronix.de>
First post2016-08-17 14:30 +0200
Last post2016-08-17 16:30 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] cpu/hotplug: get rid of CPU_STARTING reference Sebastian Andrzej Siewior <bigeasy@linutronix.de> - 2016-08-17 14:30 +0200
    Re: [PATCH] cpu/hotplug: get rid of CPU_STARTING reference "Paul E. McKenney" <paulmck@linux.vnet.ibm.com> - 2016-08-17 16:30 +0200

#1464548 — [PATCH] cpu/hotplug: get rid of CPU_STARTING reference

FromSebastian Andrzej Siewior <bigeasy@linutronix.de>
Date2016-08-17 14:30 +0200
Subject[PATCH] cpu/hotplug: get rid of CPU_STARTING reference
Message-ID<s788N-XT-1@gated-at.bofh.it>
CPU_STARTING is scheduled for removal. There is no use of it in drivers
and just compatibility use core code. This patch removes it from an RCU
related comment.

Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
---

Paul, this "CPU_STARTING" refrences got in via "rcu: Exact CPU-online
tracking for RCU".

 kernel/cpu.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/cpu.c b/kernel/cpu.c
index 35b53a1f56a7..3788ad9d4073 100644
--- a/kernel/cpu.c
+++ b/kernel/cpu.c
@@ -897,7 +897,7 @@ void notify_cpu_starting(unsigned int cpu)
 	struct cpuhp_cpu_state *st = per_cpu_ptr(&cpuhp_state, cpu);
 	enum cpuhp_state target = min((int)st->target, CPUHP_AP_ONLINE);
 
-	rcu_cpu_starting(cpu);  /* All CPU_STARTING notifiers can use RCU. */
+	rcu_cpu_starting(cpu);	/* Enables RCU usage on this CPU. */
 	while (st->state < target) {
 		struct cpuhp_step *step;
 
-- 
2.9.3

[toc] | [next] | [standalone]


#1464606

From"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Date2016-08-17 16:30 +0200
Message-ID<s7a0W-2dr-19@gated-at.bofh.it>
In reply to#1464548
On Wed, Aug 17, 2016 at 02:21:04PM +0200, Sebastian Andrzej Siewior wrote:
> CPU_STARTING is scheduled for removal. There is no use of it in drivers
> and just compatibility use core code. This patch removes it from an RCU
> related comment.
> 
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>

Queued, thank you!

							Thanx, Paul

> ---
> 
> Paul, this "CPU_STARTING" refrences got in via "rcu: Exact CPU-online
> tracking for RCU".
> 
>  kernel/cpu.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/kernel/cpu.c b/kernel/cpu.c
> index 35b53a1f56a7..3788ad9d4073 100644
> --- a/kernel/cpu.c
> +++ b/kernel/cpu.c
> @@ -897,7 +897,7 @@ void notify_cpu_starting(unsigned int cpu)
>  	struct cpuhp_cpu_state *st = per_cpu_ptr(&cpuhp_state, cpu);
>  	enum cpuhp_state target = min((int)st->target, CPUHP_AP_ONLINE);
> 
> -	rcu_cpu_starting(cpu);  /* All CPU_STARTING notifiers can use RCU. */
> +	rcu_cpu_starting(cpu);	/* Enables RCU usage on this CPU. */
>  	while (st->state < target) {
>  		struct cpuhp_step *step;
> 
> -- 
> 2.9.3
> 

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web