From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Fri, 4 Nov 2016 15:22:19 +0100 From: Heiko Carstens Subject: Re: [PATCH 10/25] s390/smp: Make cpu notifier symetric References: <20161103145021.28528-1-bigeasy@linutronix.de> <20161103145021.28528-11-bigeasy@linutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161103145021.28528-11-bigeasy@linutronix.de> Message-Id: <20161104142219.GC4088@osiris> Sender: linux-kernel-owner@vger.kernel.org List-Archive: List-Post: To: Sebastian Andrzej Siewior Cc: linux-kernel@vger.kernel.org, rt@linutronix.de, Thomas Gleixner , Martin Schwidefsky , linux-s390@vger.kernel.org List-ID: On Thu, Nov 03, 2016 at 03:50:06PM +0100, Sebastian Andrzej Siewior wrote: > From: Thomas Gleixner > > There is no reason to remove the sysfs cpu files when the CPU is dead, they > can be removed when the cpu is prepared to go down. Doing it at > DOWN_PREPARE allows us to convert it to a symetric hotplug state in the > next step. > > Cc: Martin Schwidefsky > Cc: Heiko Carstens > Cc: linux-s390@vger.kernel.org > Signed-off-by: Thomas Gleixner > Signed-off-by: Sebastian Andrzej Siewior > --- > arch/s390/kernel/smp.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/arch/s390/kernel/smp.c b/arch/s390/kernel/smp.c > index 35531fe1c5ea..1a21e66c484a 100644 > --- a/arch/s390/kernel/smp.c > +++ b/arch/s390/kernel/smp.c > @@ -1056,9 +1056,10 @@ static int smp_cpu_notify(struct notifier_block *self, unsigned long action, > > switch (action & ~CPU_TASKS_FROZEN) { > case CPU_ONLINE: > + case CPU_DOWN_FAILED: > err = sysfs_create_group(&s->kobj, &cpu_online_attr_group); > break; > - case CPU_DEAD: > + case CPU_DOWN_PREPARE:: This won't compile... even though it will be removed with the following patch it would be good to fix this to keep this bisectable.