From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753173AbcGSHHh (ORCPT ); Tue, 19 Jul 2016 03:07:37 -0400 Received: from terminus.zytor.com ([198.137.202.10]:45194 "EHLO terminus.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753071AbcGSHHd (ORCPT ); Tue, 19 Jul 2016 03:07:33 -0400 Date: Tue, 19 Jul 2016 00:07:00 -0700 From: tip-bot for Thomas Gleixner Message-ID: Cc: schwidefsky@de.ibm.com, peterz@infradead.org, brueckner@linux.vnet.ibm.com, sukadev@linux.vnet.ibm.com, linux-kernel@vger.kernel.org, adam.buchbinder@gmail.com, torvalds@linux-foundation.org, tglx@linutronix.de, heiko.carstens@de.ibm.com, mingo@kernel.org, bigeasy@linutronix.de, anna-maria@linutronix.de, hpa@zytor.com Reply-To: brueckner@linux.vnet.ibm.com, sukadev@linux.vnet.ibm.com, schwidefsky@de.ibm.com, peterz@infradead.org, linux-kernel@vger.kernel.org, adam.buchbinder@gmail.com, tglx@linutronix.de, torvalds@linux-foundation.org, mingo@kernel.org, bigeasy@linutronix.de, anna-maria@linutronix.de, heiko.carstens@de.ibm.com, hpa@zytor.com In-Reply-To: <20160713153334.436370635@linutronix.de> References: <20160713153334.436370635@linutronix.de> To: linux-tip-commits@vger.kernel.org Subject: [tip:smp/hotplug] s390/perf: Convert the hotplug notifier to state machine callbacks (Counter) Git-Commit-ID: 4f0f8217e69a0f6ff727913aa49cb64632ff7b6a X-Mailer: tip-git-log-daemon Robot-ID: Robot-Unsubscribe: Contact to get blacklisted from these emails MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Commit-ID: 4f0f8217e69a0f6ff727913aa49cb64632ff7b6a Gitweb: http://git.kernel.org/tip/4f0f8217e69a0f6ff727913aa49cb64632ff7b6a Author: Thomas Gleixner AuthorDate: Wed, 13 Jul 2016 17:16:21 +0000 Committer: Ingo Molnar CommitDate: Thu, 14 Jul 2016 09:34:37 +0200 s390/perf: Convert the hotplug notifier to state machine callbacks (Counter) Install the callbacks via the state machine and let the core invoke the callbacks on the already online CPUs. Signed-off-by: Thomas Gleixner Signed-off-by: Anna-Maria Gleixner Reviewed-by: Sebastian Andrzej Siewior Cc: Adam Buchbinder Cc: Heiko Carstens Cc: Hendrik Brueckner Cc: Linus Torvalds Cc: Martin Schwidefsky Cc: Peter Zijlstra Cc: Sukadev Bhattiprolu Cc: linux-s390@vger.kernel.org Cc: rt@linutronix.de Link: http://lkml.kernel.org/r/20160713153334.436370635@linutronix.de Signed-off-by: Ingo Molnar --- arch/s390/kernel/perf_cpum_cf.c | 46 +++++++++++++++++------------------------ include/linux/cpuhotplug.h | 1 + 2 files changed, 20 insertions(+), 27 deletions(-) diff --git a/arch/s390/kernel/perf_cpum_cf.c b/arch/s390/kernel/perf_cpum_cf.c index 7ec63b1..037c2a2 100644 --- a/arch/s390/kernel/perf_cpum_cf.c +++ b/arch/s390/kernel/perf_cpum_cf.c @@ -664,30 +664,22 @@ static struct pmu cpumf_pmu = { .cancel_txn = cpumf_pmu_cancel_txn, }; -static int cpumf_pmu_notifier(struct notifier_block *self, unsigned long action, - void *hcpu) +static int cpumf_pmf_setup(unsigned int cpu, int flags) { - int flags; - - switch (action & ~CPU_TASKS_FROZEN) { - case CPU_ONLINE: - case CPU_DOWN_FAILED: - flags = PMC_INIT; - local_irq_disable(); - setup_pmc_cpu(&flags); - local_irq_enable(); - break; - case CPU_DOWN_PREPARE: - flags = PMC_RELEASE; - local_irq_disable(); - setup_pmc_cpu(&flags); - local_irq_enable(); - break; - default: - break; - } + local_irq_disable(); + setup_pmc_cpu(&flags); + local_irq_enable(); + return 0; +} + +static int s390_pmu_online_cpu(unsigned int cpu) +{ + return cpumf_pmf_setup(cpu, PMC_INIT); +} - return NOTIFY_OK; +static int s390_pmu_offline_cpu(unsigned int cpu) +{ + return cpumf_pmf_setup(cpu, PMC_RELEASE); } static int __init cpumf_pmu_init(void) @@ -707,7 +699,7 @@ static int __init cpumf_pmu_init(void) if (rc) { pr_err("Registering for CPU-measurement alerts " "failed with rc=%i\n", rc); - goto out; + return rc; } cpumf_pmu.attr_groups = cpumf_cf_event_group(); @@ -716,10 +708,10 @@ static int __init cpumf_pmu_init(void) pr_err("Registering the cpum_cf PMU failed with rc=%i\n", rc); unregister_external_irq(EXT_IRQ_MEASURE_ALERT, cpumf_measurement_alert); - goto out; + return rc; } - perf_cpu_notifier(cpumf_pmu_notifier); -out: - return rc; + return cpuhp_setup_state(CPUHP_AP_PERF_S390_CF_ONLINE, + "AP_PERF_S390_CF_ONLINE", + s390_pmu_online_cpu, s390_pmu_offline_cpu); } early_initcall(cpumf_pmu_init); diff --git a/include/linux/cpuhotplug.h b/include/linux/cpuhotplug.h index b265e99..d4e09cb 100644 --- a/include/linux/cpuhotplug.h +++ b/include/linux/cpuhotplug.h @@ -42,6 +42,7 @@ enum cpuhp_state { CPUHP_AP_PERF_X86_RAPL_ONLINE, CPUHP_AP_PERF_X86_CQM_ONLINE, CPUHP_AP_PERF_X86_CSTATE_ONLINE, + CPUHP_AP_PERF_S390_CF_ONLINE, CPUHP_AP_NOTIFY_ONLINE, CPUHP_AP_ONLINE_DYN, CPUHP_AP_ONLINE_DYN_END = CPUHP_AP_ONLINE_DYN + 30,