* [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code @ 2017-01-19 14:37 Juri Lelli 2017-01-19 14:37 ` [PATCH 1/7] Documentation: arm: fix wrong reference number in DT definition Juri Lelli ` (7 more replies) 0 siblings, 8 replies; 16+ messages in thread From: Juri Lelli @ 2017-01-19 14:37 UTC (permalink / raw) To: linux-kernel Cc: linux-pm, linux-arm-kernel, devicetree, peterz, vincent.guittot, robh+dt, mark.rutland, linux, sudeep.holla, lorenzo.pieralisi, catalin.marinas, will.deacon, morten.rasmussen, dietmar.eggemann, juri.lelli, broonie, gregkh Hi, arm and arm64 topology.c share a lot of code related to parsing of capacity information. This set of patches proposes a solution (based on Will's, Catalin's and Mark's off-line suggestions) to move such common code in a single place: drivers/base/arch_topology.c (by creating such file and conditionally compiling it for arm and arm64 only). First 5 patches are actually fixes for the current code. Patch 6 is the actual refactoring. Last patch removes one of the extern symbols by changing a bit the now common code. We still remain with some other externs, which are not nice. Moving them in some header file solves the issue, should I just create a new include/ linux/arch_topology.h file and move them there? The set is based on top of linux/master (4.10-rc4 fb1d8e0e2c50) and it is also available from: git://linux-arm.org/linux-jl.git upstream/default_caps_factorize Best, - Juri Juri Lelli (7): Documentation: arm: fix wrong reference number in DT definition Documentation/ABI: add information about cpu_capacity arm: fix return value of parse_cpu_capacity arm: remove wrong CONFIG_PROC_SYSCTL ifdef arm64: remove wrong CONFIG_PROC_SYSCTL ifdef arm, arm64: factorize common cpu capacity default code arm,arm64,drivers: reduce scope of cap_parsing_failed Documentation/ABI/testing/sysfs-devices-system-cpu | 7 + Documentation/devicetree/bindings/arm/cpus.txt | 4 +- arch/arm/Kconfig | 1 + arch/arm/kernel/topology.c | 216 +------------------ arch/arm64/Kconfig | 1 + arch/arm64/kernel/topology.c | 218 +------------------ drivers/base/Kconfig | 8 + drivers/base/Makefile | 1 + drivers/base/arch_topology.c | 240 +++++++++++++++++++++ 9 files changed, 269 insertions(+), 427 deletions(-) create mode 100644 drivers/base/arch_topology.c -- 2.10.0 ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH 1/7] Documentation: arm: fix wrong reference number in DT definition 2017-01-19 14:37 [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code Juri Lelli @ 2017-01-19 14:37 ` Juri Lelli 2017-01-21 20:55 ` Rob Herring 2017-01-19 14:37 ` [PATCH 2/7] Documentation/ABI: add information about cpu_capacity Juri Lelli ` (6 subsequent siblings) 7 siblings, 1 reply; 16+ messages in thread From: Juri Lelli @ 2017-01-19 14:37 UTC (permalink / raw) To: linux-kernel Cc: linux-pm, linux-arm-kernel, devicetree, peterz, vincent.guittot, robh+dt, mark.rutland, linux, sudeep.holla, lorenzo.pieralisi, catalin.marinas, will.deacon, morten.rasmussen, dietmar.eggemann, juri.lelli, broonie, gregkh Reference to cpu capacity binding has a wrong number. Fix it. Reported-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> Signed-off-by: Juri Lelli <juri.lelli@arm.com> --- Documentation/devicetree/bindings/arm/cpus.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/arm/cpus.txt b/Documentation/devicetree/bindings/arm/cpus.txt index a1bcfeed5f24..c27376a27a92 100644 --- a/Documentation/devicetree/bindings/arm/cpus.txt +++ b/Documentation/devicetree/bindings/arm/cpus.txt @@ -246,7 +246,7 @@ nodes to be present and contain the properties described below. Usage: Optional Value type: <u32> Definition: - # u32 value representing CPU capacity [3] in + # u32 value representing CPU capacity [4] in DMIPS/MHz, relative to highest capacity-dmips-mhz in the system. @@ -473,5 +473,5 @@ cpus { [2] arm/msm/qcom,kpss-acc.txt [3] ARM Linux kernel documentation - idle states bindings Documentation/devicetree/bindings/arm/idle-states.txt -[3] ARM Linux kernel documentation - cpu capacity bindings +[4] ARM Linux kernel documentation - cpu capacity bindings Documentation/devicetree/bindings/arm/cpu-capacity.txt -- 2.10.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH 1/7] Documentation: arm: fix wrong reference number in DT definition 2017-01-19 14:37 ` [PATCH 1/7] Documentation: arm: fix wrong reference number in DT definition Juri Lelli @ 2017-01-21 20:55 ` Rob Herring 0 siblings, 0 replies; 16+ messages in thread From: Rob Herring @ 2017-01-21 20:55 UTC (permalink / raw) To: Juri Lelli Cc: linux-kernel, linux-pm, linux-arm-kernel, devicetree, peterz, vincent.guittot, mark.rutland, linux, sudeep.holla, lorenzo.pieralisi, catalin.marinas, will.deacon, morten.rasmussen, dietmar.eggemann, broonie, gregkh On Thu, Jan 19, 2017 at 02:37:51PM +0000, Juri Lelli wrote: > Reference to cpu capacity binding has a wrong number. Fix it. > > Reported-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> > Signed-off-by: Juri Lelli <juri.lelli@arm.com> > --- > Documentation/devicetree/bindings/arm/cpus.txt | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Acked-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH 2/7] Documentation/ABI: add information about cpu_capacity 2017-01-19 14:37 [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code Juri Lelli 2017-01-19 14:37 ` [PATCH 1/7] Documentation: arm: fix wrong reference number in DT definition Juri Lelli @ 2017-01-19 14:37 ` Juri Lelli 2017-01-19 14:37 ` [PATCH 3/7] arm: fix return value of parse_cpu_capacity Juri Lelli ` (5 subsequent siblings) 7 siblings, 0 replies; 16+ messages in thread From: Juri Lelli @ 2017-01-19 14:37 UTC (permalink / raw) To: linux-kernel Cc: linux-pm, linux-arm-kernel, devicetree, peterz, vincent.guittot, robh+dt, mark.rutland, linux, sudeep.holla, lorenzo.pieralisi, catalin.marinas, will.deacon, morten.rasmussen, dietmar.eggemann, juri.lelli, broonie, gregkh /sys/devices/system/cpu/cpu#/cpu_capacity describe information about CPUs heterogeneity (ref. to Documentation/devicetree/bindings/arm/ cpu-capacity.txt). Add such description. Signed-off-by: Juri Lelli <juri.lelli@arm.com> --- Documentation/ABI/testing/sysfs-devices-system-cpu | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/Documentation/ABI/testing/sysfs-devices-system-cpu b/Documentation/ABI/testing/sysfs-devices-system-cpu index 2a4a423d08e0..f3d5817c4ef0 100644 --- a/Documentation/ABI/testing/sysfs-devices-system-cpu +++ b/Documentation/ABI/testing/sysfs-devices-system-cpu @@ -366,3 +366,10 @@ Contact: Linux ARM Kernel Mailing list <linux-arm-kernel@lists.infradead.org> Description: AArch64 CPU registers 'identification' directory exposes the CPU ID registers for identifying model and revision of the CPU. + +What: /sys/devices/system/cpu/cpu#/cpu_capacity +Date: December 2016 +Contact: Linux kernel mailing list <linux-kernel@vger.kernel.org> +Description: information about CPUs heterogeneity. + + cpu_capacity: capacity of cpu#. -- 2.10.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 3/7] arm: fix return value of parse_cpu_capacity 2017-01-19 14:37 [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code Juri Lelli 2017-01-19 14:37 ` [PATCH 1/7] Documentation: arm: fix wrong reference number in DT definition Juri Lelli 2017-01-19 14:37 ` [PATCH 2/7] Documentation/ABI: add information about cpu_capacity Juri Lelli @ 2017-01-19 14:37 ` Juri Lelli 2017-01-19 14:37 ` [PATCH 4/7] arm: remove wrong CONFIG_PROC_SYSCTL ifdef Juri Lelli ` (4 subsequent siblings) 7 siblings, 0 replies; 16+ messages in thread From: Juri Lelli @ 2017-01-19 14:37 UTC (permalink / raw) To: linux-kernel Cc: linux-pm, linux-arm-kernel, devicetree, peterz, vincent.guittot, robh+dt, mark.rutland, linux, sudeep.holla, lorenzo.pieralisi, catalin.marinas, will.deacon, morten.rasmussen, dietmar.eggemann, juri.lelli, broonie, gregkh parse_cpu_capacity() has to return 0 on failure, but it currently returns 1 instead if raw_capacity kcalloc failed. Fix it by removing the negation of the return value. Cc: Russell King <linux@arm.linux.org.uk> Reported-by: Morten Rasmussen <morten.rasmussen@arm.com> Fixes: 06073ee26775 ('ARM: 8621/3: parse cpu capacity-dmips-mhz from DT') Signed-off-by: Juri Lelli <juri.lelli@arm.com> --- arch/arm/kernel/topology.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/kernel/topology.c b/arch/arm/kernel/topology.c index ebf47d91b804..b439f7fff86b 100644 --- a/arch/arm/kernel/topology.c +++ b/arch/arm/kernel/topology.c @@ -165,7 +165,7 @@ static int __init parse_cpu_capacity(struct device_node *cpu_node, int cpu) if (!raw_capacity) { pr_err("cpu_capacity: failed to allocate memory for raw capacities\n"); cap_parsing_failed = true; - return !ret; + return ret; } } capacity_scale = max(cpu_capacity, capacity_scale); -- 2.10.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 4/7] arm: remove wrong CONFIG_PROC_SYSCTL ifdef 2017-01-19 14:37 [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code Juri Lelli ` (2 preceding siblings ...) 2017-01-19 14:37 ` [PATCH 3/7] arm: fix return value of parse_cpu_capacity Juri Lelli @ 2017-01-19 14:37 ` Juri Lelli 2017-01-19 14:37 ` [PATCH 5/7] arm64: " Juri Lelli ` (3 subsequent siblings) 7 siblings, 0 replies; 16+ messages in thread From: Juri Lelli @ 2017-01-19 14:37 UTC (permalink / raw) To: linux-kernel Cc: linux-pm, linux-arm-kernel, devicetree, peterz, vincent.guittot, robh+dt, mark.rutland, linux, sudeep.holla, lorenzo.pieralisi, catalin.marinas, will.deacon, morten.rasmussen, dietmar.eggemann, juri.lelli, broonie, gregkh The sysfs cpu_capacity entry for each CPU has nothing to do with PROC_FS, nor it's in /proc/sys path. Remove such ifdef. Cc: Russell King <linux@arm.linux.org.uk> Reported-and-suggested-by: Sudeep Holla <sudeep.holla@arm.com> Fixes: 7e5930aaef5d ('ARM: 8622/3: add sysfs cpu_capacity attribute') Signed-off-by: Juri Lelli <juri.lelli@arm.com> --- arch/arm/kernel/topology.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/arm/kernel/topology.c b/arch/arm/kernel/topology.c index b439f7fff86b..c760a321935b 100644 --- a/arch/arm/kernel/topology.c +++ b/arch/arm/kernel/topology.c @@ -56,7 +56,6 @@ static void set_capacity_scale(unsigned int cpu, unsigned long capacity) per_cpu(cpu_scale, cpu) = capacity; } -#ifdef CONFIG_PROC_SYSCTL static ssize_t cpu_capacity_show(struct device *dev, struct device_attribute *attr, char *buf) @@ -113,7 +112,6 @@ static int register_cpu_capacity_sysctl(void) return 0; } subsys_initcall(register_cpu_capacity_sysctl); -#endif #ifdef CONFIG_OF struct cpu_efficiency { -- 2.10.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 5/7] arm64: remove wrong CONFIG_PROC_SYSCTL ifdef 2017-01-19 14:37 [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code Juri Lelli ` (3 preceding siblings ...) 2017-01-19 14:37 ` [PATCH 4/7] arm: remove wrong CONFIG_PROC_SYSCTL ifdef Juri Lelli @ 2017-01-19 14:37 ` Juri Lelli 2017-01-19 14:37 ` [PATCH 6/7] arm, arm64: factorize common cpu capacity default code Juri Lelli ` (2 subsequent siblings) 7 siblings, 0 replies; 16+ messages in thread From: Juri Lelli @ 2017-01-19 14:37 UTC (permalink / raw) To: linux-kernel Cc: linux-pm, linux-arm-kernel, devicetree, peterz, vincent.guittot, robh+dt, mark.rutland, linux, sudeep.holla, lorenzo.pieralisi, catalin.marinas, will.deacon, morten.rasmussen, dietmar.eggemann, juri.lelli, broonie, gregkh The sysfs cpu_capacity entry for each CPU has nothing to do with PROC_FS, nor it's in /proc/sys path. Remove such ifdef. Cc: Will Deacon <will.deacon@arm.com> Cc: Catalin Marinas <catalin.marinas@arm.com> Reported-and-suggested-by: Sudeep Holla <sudeep.holla@arm.com> Fixes: be8f185d8af4 ('arm64: add sysfs cpu_capacity attribute') Signed-off-by: Juri Lelli <juri.lelli@arm.com> --- arch/arm64/kernel/topology.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/arm64/kernel/topology.c b/arch/arm64/kernel/topology.c index 23e9e13bd2aa..62b370388d72 100644 --- a/arch/arm64/kernel/topology.c +++ b/arch/arm64/kernel/topology.c @@ -40,7 +40,6 @@ static void set_capacity_scale(unsigned int cpu, unsigned long capacity) per_cpu(cpu_scale, cpu) = capacity; } -#ifdef CONFIG_PROC_SYSCTL static ssize_t cpu_capacity_show(struct device *dev, struct device_attribute *attr, char *buf) @@ -97,7 +96,6 @@ static int register_cpu_capacity_sysctl(void) return 0; } subsys_initcall(register_cpu_capacity_sysctl); -#endif static u32 capacity_scale; static u32 *raw_capacity; -- 2.10.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 6/7] arm, arm64: factorize common cpu capacity default code 2017-01-19 14:37 [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code Juri Lelli ` (4 preceding siblings ...) 2017-01-19 14:37 ` [PATCH 5/7] arm64: " Juri Lelli @ 2017-01-19 14:37 ` Juri Lelli 2017-01-19 14:53 ` Russell King - ARM Linux 2017-01-19 16:00 ` Dietmar Eggemann 2017-01-19 14:37 ` [PATCH 7/7] arm,arm64,drivers: reduce scope of cap_parsing_failed Juri Lelli 2017-01-30 12:29 ` [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code Juri Lelli 7 siblings, 2 replies; 16+ messages in thread From: Juri Lelli @ 2017-01-19 14:37 UTC (permalink / raw) To: linux-kernel Cc: linux-pm, linux-arm-kernel, devicetree, peterz, vincent.guittot, robh+dt, mark.rutland, linux, sudeep.holla, lorenzo.pieralisi, catalin.marinas, will.deacon, morten.rasmussen, dietmar.eggemann, juri.lelli, broonie, gregkh, Russell King arm and arm64 share lot of code relative to parsing CPU capacity information from DT, using that information for appropriate scaling and exposing a sysfs interface for chaging such values at runtime. Factorize such code in a common place (driver/base/arch_topology.c) in preparation for further additions. Suggested-by: Will Deacon <will.deacon@arm.com> Suggested-by: Mark Rutland <mark.rutland@arm.com> Suggested-by: Catalin Marinas <catalin.marinas@arm.com> Cc: Russell King <linux@armlinux.org.uk> Cc: Catalin Marinas <catalin.marinas@arm.com> Cc: Will Deacon <will.deacon@arm.com> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Juri Lelli <juri.lelli@arm.com> --- arch/arm/Kconfig | 1 + arch/arm/kernel/topology.c | 213 ++------------------------------------ arch/arm64/Kconfig | 1 + arch/arm64/kernel/topology.c | 213 +------------------------------------- drivers/base/Kconfig | 8 ++ drivers/base/Makefile | 1 + drivers/base/arch_topology.c | 240 +++++++++++++++++++++++++++++++++++++++++++ 7 files changed, 260 insertions(+), 417 deletions(-) create mode 100644 drivers/base/arch_topology.c diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index 186c4c214e0a..f7059b3a1265 100644 --- a/arch/arm/Kconfig +++ b/arch/arm/Kconfig @@ -19,6 +19,7 @@ config ARM select EDAC_SUPPORT select EDAC_ATOMIC_SCRUB select GENERIC_ALLOCATOR + select GENERIC_ARCH_TOPOLOGY select GENERIC_ATOMIC64 if (CPU_V7M || CPU_V6 || !CPU_32v6K || !AEABI) select GENERIC_CLOCKEVENTS_BROADCAST if SMP select GENERIC_EARLY_IOREMAP diff --git a/arch/arm/kernel/topology.c b/arch/arm/kernel/topology.c index c760a321935b..51e9ed6439f1 100644 --- a/arch/arm/kernel/topology.c +++ b/arch/arm/kernel/topology.c @@ -43,75 +43,10 @@ * to run the rebalance_domains for all idle cores and the cpu_capacity can be * updated during this sequence. */ -static DEFINE_PER_CPU(unsigned long, cpu_scale) = SCHED_CAPACITY_SCALE; -static DEFINE_MUTEX(cpu_scale_mutex); -unsigned long arch_scale_cpu_capacity(struct sched_domain *sd, int cpu) -{ - return per_cpu(cpu_scale, cpu); -} - -static void set_capacity_scale(unsigned int cpu, unsigned long capacity) -{ - per_cpu(cpu_scale, cpu) = capacity; -} - -static ssize_t cpu_capacity_show(struct device *dev, - struct device_attribute *attr, - char *buf) -{ - struct cpu *cpu = container_of(dev, struct cpu, dev); - - return sprintf(buf, "%lu\n", - arch_scale_cpu_capacity(NULL, cpu->dev.id)); -} - -static ssize_t cpu_capacity_store(struct device *dev, - struct device_attribute *attr, - const char *buf, - size_t count) -{ - struct cpu *cpu = container_of(dev, struct cpu, dev); - int this_cpu = cpu->dev.id, i; - unsigned long new_capacity; - ssize_t ret; - - if (count) { - ret = kstrtoul(buf, 0, &new_capacity); - if (ret) - return ret; - if (new_capacity > SCHED_CAPACITY_SCALE) - return -EINVAL; - - mutex_lock(&cpu_scale_mutex); - for_each_cpu(i, &cpu_topology[this_cpu].core_sibling) - set_capacity_scale(i, new_capacity); - mutex_unlock(&cpu_scale_mutex); - } - - return count; -} - -static DEVICE_ATTR_RW(cpu_capacity); - -static int register_cpu_capacity_sysctl(void) -{ - int i; - struct device *cpu; - - for_each_possible_cpu(i) { - cpu = get_cpu_device(i); - if (!cpu) { - pr_err("%s: too early to get CPU%d device!\n", - __func__, i); - continue; - } - device_create_file(cpu, &dev_attr_cpu_capacity); - } - - return 0; -} -subsys_initcall(register_cpu_capacity_sysctl); +extern unsigned long +arch_scale_cpu_capacity(struct sched_domain *sd, int cpu); +extern void set_capacity_scale(unsigned int cpu, unsigned long capacity); #ifdef CONFIG_OF struct cpu_efficiency { @@ -140,145 +75,9 @@ static unsigned long *__cpu_capacity; static unsigned long middle_capacity = 1; static bool cap_from_dt = true; -static u32 *raw_capacity; -static bool cap_parsing_failed; -static u32 capacity_scale; - -static int __init parse_cpu_capacity(struct device_node *cpu_node, int cpu) -{ - int ret = 1; - u32 cpu_capacity; - - if (cap_parsing_failed) - return !ret; - - ret = of_property_read_u32(cpu_node, - "capacity-dmips-mhz", - &cpu_capacity); - if (!ret) { - if (!raw_capacity) { - raw_capacity = kcalloc(num_possible_cpus(), - sizeof(*raw_capacity), - GFP_KERNEL); - if (!raw_capacity) { - pr_err("cpu_capacity: failed to allocate memory for raw capacities\n"); - cap_parsing_failed = true; - return ret; - } - } - capacity_scale = max(cpu_capacity, capacity_scale); - raw_capacity[cpu] = cpu_capacity; - pr_debug("cpu_capacity: %s cpu_capacity=%u (raw)\n", - cpu_node->full_name, raw_capacity[cpu]); - } else { - if (raw_capacity) { - pr_err("cpu_capacity: missing %s raw capacity\n", - cpu_node->full_name); - pr_err("cpu_capacity: partial information: fallback to 1024 for all CPUs\n"); - } - cap_parsing_failed = true; - kfree(raw_capacity); - } - - return !ret; -} - -static void normalize_cpu_capacity(void) -{ - u64 capacity; - int cpu; - - if (!raw_capacity || cap_parsing_failed) - return; - - pr_debug("cpu_capacity: capacity_scale=%u\n", capacity_scale); - mutex_lock(&cpu_scale_mutex); - for_each_possible_cpu(cpu) { - capacity = (raw_capacity[cpu] << SCHED_CAPACITY_SHIFT) - / capacity_scale; - set_capacity_scale(cpu, capacity); - pr_debug("cpu_capacity: CPU%d cpu_capacity=%lu\n", - cpu, arch_scale_cpu_capacity(NULL, cpu)); - } - mutex_unlock(&cpu_scale_mutex); -} - -#ifdef CONFIG_CPU_FREQ -static cpumask_var_t cpus_to_visit; -static bool cap_parsing_done; -static void parsing_done_workfn(struct work_struct *work); -static DECLARE_WORK(parsing_done_work, parsing_done_workfn); - -static int -init_cpu_capacity_callback(struct notifier_block *nb, - unsigned long val, - void *data) -{ - struct cpufreq_policy *policy = data; - int cpu; - - if (cap_parsing_failed || cap_parsing_done) - return 0; - - switch (val) { - case CPUFREQ_NOTIFY: - pr_debug("cpu_capacity: init cpu capacity for CPUs [%*pbl] (to_visit=%*pbl)\n", - cpumask_pr_args(policy->related_cpus), - cpumask_pr_args(cpus_to_visit)); - cpumask_andnot(cpus_to_visit, - cpus_to_visit, - policy->related_cpus); - for_each_cpu(cpu, policy->related_cpus) { - raw_capacity[cpu] = arch_scale_cpu_capacity(NULL, cpu) * - policy->cpuinfo.max_freq / 1000UL; - capacity_scale = max(raw_capacity[cpu], capacity_scale); - } - if (cpumask_empty(cpus_to_visit)) { - normalize_cpu_capacity(); - kfree(raw_capacity); - pr_debug("cpu_capacity: parsing done\n"); - cap_parsing_done = true; - schedule_work(&parsing_done_work); - } - } - return 0; -} - -static struct notifier_block init_cpu_capacity_notifier = { - .notifier_call = init_cpu_capacity_callback, -}; - -static int __init register_cpufreq_notifier(void) -{ - if (cap_parsing_failed) - return -EINVAL; - - if (!alloc_cpumask_var(&cpus_to_visit, GFP_KERNEL)) { - pr_err("cpu_capacity: failed to allocate memory for cpus_to_visit\n"); - return -ENOMEM; - } - cpumask_copy(cpus_to_visit, cpu_possible_mask); - - return cpufreq_register_notifier(&init_cpu_capacity_notifier, - CPUFREQ_POLICY_NOTIFIER); -} -core_initcall(register_cpufreq_notifier); - -static void parsing_done_workfn(struct work_struct *work) -{ - cpufreq_unregister_notifier(&init_cpu_capacity_notifier, - CPUFREQ_POLICY_NOTIFIER); -} - -#else -static int __init free_raw_capacity(void) -{ - kfree(raw_capacity); - - return 0; -} -core_initcall(free_raw_capacity); -#endif +extern bool cap_parsing_failed; +extern void normalize_cpu_capacity(void); +extern int __init parse_cpu_capacity(struct device_node *cpu_node, int cpu); /* * Iterate all CPUs' descriptor in DT and compute the efficiency diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig index 111742126897..7534bb41ee09 100644 --- a/arch/arm64/Kconfig +++ b/arch/arm64/Kconfig @@ -36,6 +36,7 @@ config ARM64 select EDAC_SUPPORT select FRAME_POINTER select GENERIC_ALLOCATOR + select GENERIC_ARCH_TOPOLOGY select GENERIC_CLOCKEVENTS select GENERIC_CLOCKEVENTS_BROADCAST select GENERIC_CPU_AUTOPROBE diff --git a/arch/arm64/kernel/topology.c b/arch/arm64/kernel/topology.c index 62b370388d72..f629f7524d65 100644 --- a/arch/arm64/kernel/topology.c +++ b/arch/arm64/kernel/topology.c @@ -21,221 +21,14 @@ #include <linux/sched.h> #include <linux/slab.h> #include <linux/string.h> -#include <linux/cpufreq.h> #include <asm/cpu.h> #include <asm/cputype.h> #include <asm/topology.h> -static DEFINE_PER_CPU(unsigned long, cpu_scale) = SCHED_CAPACITY_SCALE; -static DEFINE_MUTEX(cpu_scale_mutex); - -unsigned long arch_scale_cpu_capacity(struct sched_domain *sd, int cpu) -{ - return per_cpu(cpu_scale, cpu); -} - -static void set_capacity_scale(unsigned int cpu, unsigned long capacity) -{ - per_cpu(cpu_scale, cpu) = capacity; -} - -static ssize_t cpu_capacity_show(struct device *dev, - struct device_attribute *attr, - char *buf) -{ - struct cpu *cpu = container_of(dev, struct cpu, dev); - - return sprintf(buf, "%lu\n", - arch_scale_cpu_capacity(NULL, cpu->dev.id)); -} - -static ssize_t cpu_capacity_store(struct device *dev, - struct device_attribute *attr, - const char *buf, - size_t count) -{ - struct cpu *cpu = container_of(dev, struct cpu, dev); - int this_cpu = cpu->dev.id, i; - unsigned long new_capacity; - ssize_t ret; - - if (count) { - ret = kstrtoul(buf, 0, &new_capacity); - if (ret) - return ret; - if (new_capacity > SCHED_CAPACITY_SCALE) - return -EINVAL; - - mutex_lock(&cpu_scale_mutex); - for_each_cpu(i, &cpu_topology[this_cpu].core_sibling) - set_capacity_scale(i, new_capacity); - mutex_unlock(&cpu_scale_mutex); - } - - return count; -} - -static DEVICE_ATTR_RW(cpu_capacity); - -static int register_cpu_capacity_sysctl(void) -{ - int i; - struct device *cpu; - - for_each_possible_cpu(i) { - cpu = get_cpu_device(i); - if (!cpu) { - pr_err("%s: too early to get CPU%d device!\n", - __func__, i); - continue; - } - device_create_file(cpu, &dev_attr_cpu_capacity); - } - - return 0; -} -subsys_initcall(register_cpu_capacity_sysctl); - -static u32 capacity_scale; -static u32 *raw_capacity; -static bool cap_parsing_failed; - -static void __init parse_cpu_capacity(struct device_node *cpu_node, int cpu) -{ - int ret; - u32 cpu_capacity; - - if (cap_parsing_failed) - return; - - ret = of_property_read_u32(cpu_node, - "capacity-dmips-mhz", - &cpu_capacity); - if (!ret) { - if (!raw_capacity) { - raw_capacity = kcalloc(num_possible_cpus(), - sizeof(*raw_capacity), - GFP_KERNEL); - if (!raw_capacity) { - pr_err("cpu_capacity: failed to allocate memory for raw capacities\n"); - cap_parsing_failed = true; - return; - } - } - capacity_scale = max(cpu_capacity, capacity_scale); - raw_capacity[cpu] = cpu_capacity; - pr_debug("cpu_capacity: %s cpu_capacity=%u (raw)\n", - cpu_node->full_name, raw_capacity[cpu]); - } else { - if (raw_capacity) { - pr_err("cpu_capacity: missing %s raw capacity\n", - cpu_node->full_name); - pr_err("cpu_capacity: partial information: fallback to 1024 for all CPUs\n"); - } - cap_parsing_failed = true; - kfree(raw_capacity); - } -} - -static void normalize_cpu_capacity(void) -{ - u64 capacity; - int cpu; - - if (!raw_capacity || cap_parsing_failed) - return; - - pr_debug("cpu_capacity: capacity_scale=%u\n", capacity_scale); - mutex_lock(&cpu_scale_mutex); - for_each_possible_cpu(cpu) { - pr_debug("cpu_capacity: cpu=%d raw_capacity=%u\n", - cpu, raw_capacity[cpu]); - capacity = (raw_capacity[cpu] << SCHED_CAPACITY_SHIFT) - / capacity_scale; - set_capacity_scale(cpu, capacity); - pr_debug("cpu_capacity: CPU%d cpu_capacity=%lu\n", - cpu, arch_scale_cpu_capacity(NULL, cpu)); - } - mutex_unlock(&cpu_scale_mutex); -} - -#ifdef CONFIG_CPU_FREQ -static cpumask_var_t cpus_to_visit; -static bool cap_parsing_done; -static void parsing_done_workfn(struct work_struct *work); -static DECLARE_WORK(parsing_done_work, parsing_done_workfn); - -static int -init_cpu_capacity_callback(struct notifier_block *nb, - unsigned long val, - void *data) -{ - struct cpufreq_policy *policy = data; - int cpu; - - if (cap_parsing_failed || cap_parsing_done) - return 0; - - switch (val) { - case CPUFREQ_NOTIFY: - pr_debug("cpu_capacity: init cpu capacity for CPUs [%*pbl] (to_visit=%*pbl)\n", - cpumask_pr_args(policy->related_cpus), - cpumask_pr_args(cpus_to_visit)); - cpumask_andnot(cpus_to_visit, - cpus_to_visit, - policy->related_cpus); - for_each_cpu(cpu, policy->related_cpus) { - raw_capacity[cpu] = arch_scale_cpu_capacity(NULL, cpu) * - policy->cpuinfo.max_freq / 1000UL; - capacity_scale = max(raw_capacity[cpu], capacity_scale); - } - if (cpumask_empty(cpus_to_visit)) { - normalize_cpu_capacity(); - kfree(raw_capacity); - pr_debug("cpu_capacity: parsing done\n"); - cap_parsing_done = true; - schedule_work(&parsing_done_work); - } - } - return 0; -} - -static struct notifier_block init_cpu_capacity_notifier = { - .notifier_call = init_cpu_capacity_callback, -}; - -static int __init register_cpufreq_notifier(void) -{ - if (cap_parsing_failed) - return -EINVAL; - - if (!alloc_cpumask_var(&cpus_to_visit, GFP_KERNEL)) { - pr_err("cpu_capacity: failed to allocate memory for cpus_to_visit\n"); - return -ENOMEM; - } - cpumask_copy(cpus_to_visit, cpu_possible_mask); - - return cpufreq_register_notifier(&init_cpu_capacity_notifier, - CPUFREQ_POLICY_NOTIFIER); -} -core_initcall(register_cpufreq_notifier); - -static void parsing_done_workfn(struct work_struct *work) -{ - cpufreq_unregister_notifier(&init_cpu_capacity_notifier, - CPUFREQ_POLICY_NOTIFIER); -} - -#else -static int __init free_raw_capacity(void) -{ - kfree(raw_capacity); - - return 0; -} -core_initcall(free_raw_capacity); -#endif +extern bool cap_parsing_failed; +extern void normalize_cpu_capacity(void); +extern int __init parse_cpu_capacity(struct device_node *cpu_node, int cpu); static int __init get_cpu_for_node(struct device_node *node) { diff --git a/drivers/base/Kconfig b/drivers/base/Kconfig index d718ae4b907a..307ea31187dd 100644 --- a/drivers/base/Kconfig +++ b/drivers/base/Kconfig @@ -339,4 +339,12 @@ config CMA_ALIGNMENT endif +config GENERIC_ARCH_TOPOLOGY + bool + help + Enable support for architectures common topology code: e.g., parsing + CPU capacity information from DT, usage of such information for + appropriate scaling, sysfs interface for changing capacity values at + runtime. + endmenu diff --git a/drivers/base/Makefile b/drivers/base/Makefile index f2816f6ff76a..397e5c344e6a 100644 --- a/drivers/base/Makefile +++ b/drivers/base/Makefile @@ -23,6 +23,7 @@ obj-$(CONFIG_SOC_BUS) += soc.o obj-$(CONFIG_PINCTRL) += pinctrl.o obj-$(CONFIG_DEV_COREDUMP) += devcoredump.o obj-$(CONFIG_GENERIC_MSI_IRQ_DOMAIN) += platform-msi.o +obj-$(CONFIG_GENERIC_ARCH_TOPOLOGY) += arch_topology.o obj-y += test/ diff --git a/drivers/base/arch_topology.c b/drivers/base/arch_topology.c new file mode 100644 index 000000000000..3faf89518892 --- /dev/null +++ b/drivers/base/arch_topology.c @@ -0,0 +1,240 @@ +/* + * driver/base/arch_topology.c - Arch specific cpu topology information + * + * Written by: Juri Lelli, ARM Ltd. + * + * Copyright (C) 2016, ARM Ltd. + * + * All rights reserved. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, but + * WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY OR FITNESS FOR A PARTICULAR PURPOSE, GOOD TITLE or + * NON INFRINGEMENT. See the GNU General Public License for more + * details. + * + */ +#include <linux/cpu.h> +#include <linux/cpufreq.h> +#include <linux/device.h> +#include <linux/of.h> +#include <linux/slab.h> +#include <linux/string.h> +#include <linux/topology.h> + +static DEFINE_MUTEX(cpu_scale_mutex); +static DEFINE_PER_CPU(unsigned long, cpu_scale) = SCHED_CAPACITY_SCALE; + +unsigned long arch_scale_cpu_capacity(struct sched_domain *sd, int cpu) +{ + return per_cpu(cpu_scale, cpu); +} + +void set_capacity_scale(unsigned int cpu, unsigned long capacity) +{ + per_cpu(cpu_scale, cpu) = capacity; +} + +static ssize_t cpu_capacity_show(struct device *dev, + struct device_attribute *attr, + char *buf) +{ + struct cpu *cpu = container_of(dev, struct cpu, dev); + + return sprintf(buf, "%lu\n", + arch_scale_cpu_capacity(NULL, cpu->dev.id)); +} + +static ssize_t cpu_capacity_store(struct device *dev, + struct device_attribute *attr, + const char *buf, + size_t count) +{ + struct cpu *cpu = container_of(dev, struct cpu, dev); + int this_cpu = cpu->dev.id, i; + unsigned long new_capacity; + ssize_t ret; + + if (count) { + ret = kstrtoul(buf, 0, &new_capacity); + if (ret) + return ret; + if (new_capacity > SCHED_CAPACITY_SCALE) + return -EINVAL; + + mutex_lock(&cpu_scale_mutex); + for_each_cpu(i, &cpu_topology[this_cpu].core_sibling) + set_capacity_scale(i, new_capacity); + mutex_unlock(&cpu_scale_mutex); + } + + return count; +} + +static DEVICE_ATTR_RW(cpu_capacity); + +static int register_cpu_capacity_sysctl(void) +{ + int i; + struct device *cpu; + + for_each_possible_cpu(i) { + cpu = get_cpu_device(i); + if (!cpu) { + pr_err("%s: too early to get CPU%d device!\n", + __func__, i); + continue; + } + device_create_file(cpu, &dev_attr_cpu_capacity); + } + + return 0; +} +subsys_initcall(register_cpu_capacity_sysctl); + +u32 capacity_scale; +u32 *raw_capacity; +bool cap_parsing_failed; + +void normalize_cpu_capacity(void) +{ + u64 capacity; + int cpu; + + if (!raw_capacity || cap_parsing_failed) + return; + + pr_debug("cpu_capacity: capacity_scale=%u\n", capacity_scale); + mutex_lock(&cpu_scale_mutex); + for_each_possible_cpu(cpu) { + pr_debug("cpu_capacity: cpu=%d raw_capacity=%u\n", + cpu, raw_capacity[cpu]); + capacity = (raw_capacity[cpu] << SCHED_CAPACITY_SHIFT) + / capacity_scale; + set_capacity_scale(cpu, capacity); + pr_debug("cpu_capacity: CPU%d cpu_capacity=%lu\n", + cpu, arch_scale_cpu_capacity(NULL, cpu)); + } + mutex_unlock(&cpu_scale_mutex); +} + +int __init parse_cpu_capacity(struct device_node *cpu_node, int cpu) +{ + int ret = 1; + u32 cpu_capacity; + + if (cap_parsing_failed) + return !ret; + + ret = of_property_read_u32(cpu_node, + "capacity-dmips-mhz", + &cpu_capacity); + if (!ret) { + if (!raw_capacity) { + raw_capacity = kcalloc(num_possible_cpus(), + sizeof(*raw_capacity), + GFP_KERNEL); + if (!raw_capacity) { + pr_err("cpu_capacity: failed to allocate memory for raw capacities\n"); + cap_parsing_failed = true; + return ret; + } + } + capacity_scale = max(cpu_capacity, capacity_scale); + raw_capacity[cpu] = cpu_capacity; + pr_debug("cpu_capacity: %s cpu_capacity=%u (raw)\n", + cpu_node->full_name, raw_capacity[cpu]); + } else { + if (raw_capacity) { + pr_err("cpu_capacity: missing %s raw capacity\n", + cpu_node->full_name); + pr_err("cpu_capacity: partial information: fallback to 1024 for all CPUs\n"); + } + cap_parsing_failed = true; + kfree(raw_capacity); + } + + return !ret; +} + +#ifdef CONFIG_CPU_FREQ +static cpumask_var_t cpus_to_visit; +static bool cap_parsing_done; +static void parsing_done_workfn(struct work_struct *work); +static DECLARE_WORK(parsing_done_work, parsing_done_workfn); + +static int +init_cpu_capacity_callback(struct notifier_block *nb, + unsigned long val, + void *data) +{ + struct cpufreq_policy *policy = data; + int cpu; + + if (cap_parsing_failed || cap_parsing_done) + return 0; + + switch (val) { + case CPUFREQ_NOTIFY: + pr_debug("cpu_capacity: init cpu capacity for CPUs [%*pbl] (to_visit=%*pbl)\n", + cpumask_pr_args(policy->related_cpus), + cpumask_pr_args(cpus_to_visit)); + cpumask_andnot(cpus_to_visit, + cpus_to_visit, + policy->related_cpus); + for_each_cpu(cpu, policy->related_cpus) { + raw_capacity[cpu] = arch_scale_cpu_capacity(NULL, cpu) * + policy->cpuinfo.max_freq / 1000UL; + capacity_scale = max(raw_capacity[cpu], capacity_scale); + } + if (cpumask_empty(cpus_to_visit)) { + normalize_cpu_capacity(); + kfree(raw_capacity); + pr_debug("cpu_capacity: parsing done\n"); + cap_parsing_done = true; + schedule_work(&parsing_done_work); + } + } + return 0; +} + +static struct notifier_block init_cpu_capacity_notifier = { + .notifier_call = init_cpu_capacity_callback, +}; + +static int __init register_cpufreq_notifier(void) +{ + if (cap_parsing_failed) + return -EINVAL; + + if (!alloc_cpumask_var(&cpus_to_visit, GFP_KERNEL)) { + pr_err("cpu_capacity: failed to allocate memory for cpus_to_visit\n"); + return -ENOMEM; + } + cpumask_copy(cpus_to_visit, cpu_possible_mask); + + return cpufreq_register_notifier(&init_cpu_capacity_notifier, + CPUFREQ_POLICY_NOTIFIER); +} +core_initcall(register_cpufreq_notifier); + +static void parsing_done_workfn(struct work_struct *work) +{ + cpufreq_unregister_notifier(&init_cpu_capacity_notifier, + CPUFREQ_POLICY_NOTIFIER); +} + +#else +static int __init free_raw_capacity(void) +{ + kfree(raw_capacity); + + return 0; +} +core_initcall(free_raw_capacity); +#endif -- 2.10.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH 6/7] arm, arm64: factorize common cpu capacity default code 2017-01-19 14:37 ` [PATCH 6/7] arm, arm64: factorize common cpu capacity default code Juri Lelli @ 2017-01-19 14:53 ` Russell King - ARM Linux 2017-01-19 15:51 ` Juri Lelli 2017-01-19 16:00 ` Dietmar Eggemann 1 sibling, 1 reply; 16+ messages in thread From: Russell King - ARM Linux @ 2017-01-19 14:53 UTC (permalink / raw) To: Juri Lelli Cc: linux-kernel, linux-pm, linux-arm-kernel, devicetree, peterz, vincent.guittot, robh+dt, mark.rutland, sudeep.holla, lorenzo.pieralisi, catalin.marinas, will.deacon, morten.rasmussen, dietmar.eggemann, broonie, gregkh On Thu, Jan 19, 2017 at 02:37:56PM +0000, Juri Lelli wrote: > +extern unsigned long > +arch_scale_cpu_capacity(struct sched_domain *sd, int cpu); > +extern void set_capacity_scale(unsigned int cpu, unsigned long capacity); These should be in a header file (please run your code through sparse). > +extern bool cap_parsing_failed; > +extern void normalize_cpu_capacity(void); > +extern int __init parse_cpu_capacity(struct device_node *cpu_node, int cpu); Same for these. > diff --git a/drivers/base/arch_topology.c b/drivers/base/arch_topology.c > new file mode 100644 > index 000000000000..3faf89518892 > --- /dev/null > +++ b/drivers/base/arch_topology.c > @@ -0,0 +1,240 @@ > +/* > + * driver/base/arch_topology.c - Arch specific cpu topology information > + * > + * Written by: Juri Lelli, ARM Ltd. > + * > + * Copyright (C) 2016, ARM Ltd. > + * > + * All rights reserved. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License as published by > + * the Free Software Foundation; either version 2 of the License, or > + * (at your option) any later version. The files that you've taken this code from are GPLv2, but you've thrown a GPLv2+ header on a file that's merely a copy of the original code. As some of the code you've moved to this new file is from Nicolas and Vincent, you need to seek their approval to make this change of license terms, or keep the original license terms. -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up according to speedtest.net. ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 6/7] arm, arm64: factorize common cpu capacity default code 2017-01-19 14:53 ` Russell King - ARM Linux @ 2017-01-19 15:51 ` Juri Lelli 0 siblings, 0 replies; 16+ messages in thread From: Juri Lelli @ 2017-01-19 15:51 UTC (permalink / raw) To: Russell King - ARM Linux Cc: linux-kernel, linux-pm, linux-arm-kernel, devicetree, peterz, vincent.guittot, robh+dt, mark.rutland, sudeep.holla, lorenzo.pieralisi, catalin.marinas, will.deacon, morten.rasmussen, dietmar.eggemann, broonie, gregkh Hi, On 19/01/17 14:53, Russell King - ARM Linux wrote: > On Thu, Jan 19, 2017 at 02:37:56PM +0000, Juri Lelli wrote: > > +extern unsigned long > > +arch_scale_cpu_capacity(struct sched_domain *sd, int cpu); > > +extern void set_capacity_scale(unsigned int cpu, unsigned long capacity); > > These should be in a header file (please run your code through sparse). > > > +extern bool cap_parsing_failed; > > +extern void normalize_cpu_capacity(void); > > +extern int __init parse_cpu_capacity(struct device_node *cpu_node, int cpu); > > Same for these. > Right, I was wondering where to put these in the cover letter. New header file (e.g., include/linux/arch_topology.h) or some existing one? > > diff --git a/drivers/base/arch_topology.c b/drivers/base/arch_topology.c > > new file mode 100644 > > index 000000000000..3faf89518892 > > --- /dev/null > > +++ b/drivers/base/arch_topology.c > > @@ -0,0 +1,240 @@ > > +/* > > + * driver/base/arch_topology.c - Arch specific cpu topology information > > + * > > + * Written by: Juri Lelli, ARM Ltd. > > + * > > + * Copyright (C) 2016, ARM Ltd. > > + * > > + * All rights reserved. > > + * > > + * This program is free software; you can redistribute it and/or modify > > + * it under the terms of the GNU General Public License as published by > > + * the Free Software Foundation; either version 2 of the License, or > > + * (at your option) any later version. > > The files that you've taken this code from are GPLv2, but you've thrown > a GPLv2+ header on a file that's merely a copy of the original code. > As some of the code you've moved to this new file is from Nicolas and > Vincent, you need to seek their approval to make this change of license > terms, or keep the original license terms. > Apologies. I'd say we keep the original one. I'll modify it in the next version. Thanks, - Juri ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 6/7] arm, arm64: factorize common cpu capacity default code 2017-01-19 14:37 ` [PATCH 6/7] arm, arm64: factorize common cpu capacity default code Juri Lelli 2017-01-19 14:53 ` Russell King - ARM Linux @ 2017-01-19 16:00 ` Dietmar Eggemann 2017-01-20 10:42 ` Juri Lelli 1 sibling, 1 reply; 16+ messages in thread From: Dietmar Eggemann @ 2017-01-19 16:00 UTC (permalink / raw) To: Juri Lelli, linux-kernel Cc: linux-pm, linux-arm-kernel, devicetree, peterz, vincent.guittot, robh+dt, mark.rutland, linux, sudeep.holla, lorenzo.pieralisi, catalin.marinas, will.deacon, morten.rasmussen, broonie, gregkh, Russell King On 19/01/17 14:37, Juri Lelli wrote: > arm and arm64 share lot of code relative to parsing CPU capacity > information from DT, using that information for appropriate scaling and > exposing a sysfs interface for chaging such values at runtime. > > Factorize such code in a common place (driver/base/arch_topology.c) in > preparation for further additions. > > Suggested-by: Will Deacon <will.deacon@arm.com> > Suggested-by: Mark Rutland <mark.rutland@arm.com> > Suggested-by: Catalin Marinas <catalin.marinas@arm.com> > Cc: Russell King <linux@armlinux.org.uk> > Cc: Catalin Marinas <catalin.marinas@arm.com> > Cc: Will Deacon <will.deacon@arm.com> > Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > Signed-off-by: Juri Lelli <juri.lelli@arm.com> > --- > arch/arm/Kconfig | 1 + > arch/arm/kernel/topology.c | 213 ++------------------------------------ > arch/arm64/Kconfig | 1 + > arch/arm64/kernel/topology.c | 213 +------------------------------------- > drivers/base/Kconfig | 8 ++ > drivers/base/Makefile | 1 + > drivers/base/arch_topology.c | 240 +++++++++++++++++++++++++++++++++++++++++++ > 7 files changed, 260 insertions(+), 417 deletions(-) > create mode 100644 drivers/base/arch_topology.c [...] > +extern unsigned long > +arch_scale_cpu_capacity(struct sched_domain *sd, int cpu); How about adding a driver specific prefix 'foo_' to all driver interfaces? I'm asking because I would rather like to do a #define arch_scale_cpu_capacity foo_scale_cpu_capacity then a #define arch_scale_cpu_capacity arch_scale_cpu_capacity in arch/arm64/include/asm/topology.h later to wire cpu-invariant load-tracking support up to the task scheduler for ARM64. That's probably true too for all the 'driver' interfaces which get used in arch/arm{,64}/kernel/topology.c. [...] ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 6/7] arm, arm64: factorize common cpu capacity default code 2017-01-19 16:00 ` Dietmar Eggemann @ 2017-01-20 10:42 ` Juri Lelli 0 siblings, 0 replies; 16+ messages in thread From: Juri Lelli @ 2017-01-20 10:42 UTC (permalink / raw) To: Dietmar Eggemann Cc: linux-kernel, linux-pm, linux-arm-kernel, devicetree, peterz, vincent.guittot, robh+dt, mark.rutland, linux, sudeep.holla, lorenzo.pieralisi, catalin.marinas, will.deacon, morten.rasmussen, broonie, gregkh, Russell King Hi Dietmar, On 19/01/17 16:00, Dietmar Eggemann wrote: > On 19/01/17 14:37, Juri Lelli wrote: > > arm and arm64 share lot of code relative to parsing CPU capacity > > information from DT, using that information for appropriate scaling and > > exposing a sysfs interface for chaging such values at runtime. > > > > Factorize such code in a common place (driver/base/arch_topology.c) in > > preparation for further additions. > > > > Suggested-by: Will Deacon <will.deacon@arm.com> > > Suggested-by: Mark Rutland <mark.rutland@arm.com> > > Suggested-by: Catalin Marinas <catalin.marinas@arm.com> > > Cc: Russell King <linux@armlinux.org.uk> > > Cc: Catalin Marinas <catalin.marinas@arm.com> > > Cc: Will Deacon <will.deacon@arm.com> > > Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > Signed-off-by: Juri Lelli <juri.lelli@arm.com> > > --- > > arch/arm/Kconfig | 1 + > > arch/arm/kernel/topology.c | 213 ++------------------------------------ > > arch/arm64/Kconfig | 1 + > > arch/arm64/kernel/topology.c | 213 +------------------------------------- > > drivers/base/Kconfig | 8 ++ > > drivers/base/Makefile | 1 + > > drivers/base/arch_topology.c | 240 +++++++++++++++++++++++++++++++++++++++++++ > > 7 files changed, 260 insertions(+), 417 deletions(-) > > create mode 100644 drivers/base/arch_topology.c > > [...] > > > +extern unsigned long > > +arch_scale_cpu_capacity(struct sched_domain *sd, int cpu); > > How about adding a driver specific prefix 'foo_' to all driver interfaces? > > I'm asking because I would rather like to do a > > #define arch_scale_cpu_capacity foo_scale_cpu_capacity > > then a > > #define arch_scale_cpu_capacity arch_scale_cpu_capacity > > in arch/arm64/include/asm/topology.h > > later to wire cpu-invariant load-tracking support up to the task > scheduler for ARM64. > > That's probably true too for all the 'driver' interfaces which get used > in arch/arm{,64}/kernel/topology.c. > Looks like a good way to improve clarity to me. I'll add a patch for the next version doing this and we see what people think. Best, - Juri ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH 7/7] arm,arm64,drivers: reduce scope of cap_parsing_failed 2017-01-19 14:37 [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code Juri Lelli ` (5 preceding siblings ...) 2017-01-19 14:37 ` [PATCH 6/7] arm, arm64: factorize common cpu capacity default code Juri Lelli @ 2017-01-19 14:37 ` Juri Lelli 2017-01-30 12:29 ` [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code Juri Lelli 7 siblings, 0 replies; 16+ messages in thread From: Juri Lelli @ 2017-01-19 14:37 UTC (permalink / raw) To: linux-kernel Cc: linux-pm, linux-arm-kernel, devicetree, peterz, vincent.guittot, robh+dt, mark.rutland, linux, sudeep.holla, lorenzo.pieralisi, catalin.marinas, will.deacon, morten.rasmussen, dietmar.eggemann, juri.lelli, broonie, gregkh Reduce the scope of cap_parsing_failed (making it static in drivers/base/arch_topology.c) by slightly changing {arm,arm64} DT parsing code. Suggested-by: Morten Rasmussen <morten.rasmussen@arm.com> Signed-off-by: Juri Lelli <juri.lelli@arm.com> --- arch/arm/kernel/topology.c | 3 +-- arch/arm64/kernel/topology.c | 5 +---- drivers/base/arch_topology.c | 4 ++-- 3 files changed, 4 insertions(+), 8 deletions(-) diff --git a/arch/arm/kernel/topology.c b/arch/arm/kernel/topology.c index 51e9ed6439f1..5d4679a5418b 100644 --- a/arch/arm/kernel/topology.c +++ b/arch/arm/kernel/topology.c @@ -75,7 +75,6 @@ static unsigned long *__cpu_capacity; static unsigned long middle_capacity = 1; static bool cap_from_dt = true; -extern bool cap_parsing_failed; extern void normalize_cpu_capacity(void); extern int __init parse_cpu_capacity(struct device_node *cpu_node, int cpu); @@ -164,7 +163,7 @@ static void __init parse_dt_topology(void) middle_capacity = ((max_capacity / 3) >> (SCHED_CAPACITY_SHIFT-1)) + 1; - if (cap_from_dt && !cap_parsing_failed) + if (cap_from_dt) normalize_cpu_capacity(); } diff --git a/arch/arm64/kernel/topology.c b/arch/arm64/kernel/topology.c index f629f7524d65..deb5ebc1bdfe 100644 --- a/arch/arm64/kernel/topology.c +++ b/arch/arm64/kernel/topology.c @@ -26,7 +26,6 @@ #include <asm/cputype.h> #include <asm/topology.h> -extern bool cap_parsing_failed; extern void normalize_cpu_capacity(void); extern int __init parse_cpu_capacity(struct device_node *cpu_node, int cpu); @@ -186,10 +185,8 @@ static int __init parse_dt_topology(void) * cluster with restricted subnodes. */ map = of_get_child_by_name(cn, "cpu-map"); - if (!map) { - cap_parsing_failed = true; + if (!map) goto out; - } ret = parse_cluster(map, 0); if (ret != 0) diff --git a/drivers/base/arch_topology.c b/drivers/base/arch_topology.c index 3faf89518892..cfe51f7e1a3e 100644 --- a/drivers/base/arch_topology.c +++ b/drivers/base/arch_topology.c @@ -99,7 +99,7 @@ subsys_initcall(register_cpu_capacity_sysctl); u32 capacity_scale; u32 *raw_capacity; -bool cap_parsing_failed; +static bool cap_parsing_failed; void normalize_cpu_capacity(void) { @@ -209,7 +209,7 @@ static struct notifier_block init_cpu_capacity_notifier = { static int __init register_cpufreq_notifier(void) { - if (cap_parsing_failed) + if (!raw_capacity) return -EINVAL; if (!alloc_cpumask_var(&cpus_to_visit, GFP_KERNEL)) { -- 2.10.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code 2017-01-19 14:37 [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code Juri Lelli ` (6 preceding siblings ...) 2017-01-19 14:37 ` [PATCH 7/7] arm,arm64,drivers: reduce scope of cap_parsing_failed Juri Lelli @ 2017-01-30 12:29 ` Juri Lelli 2017-01-30 17:51 ` Catalin Marinas 7 siblings, 1 reply; 16+ messages in thread From: Juri Lelli @ 2017-01-30 12:29 UTC (permalink / raw) To: linux-kernel Cc: linux-pm, linux-arm-kernel, devicetree, peterz, vincent.guittot, robh+dt, mark.rutland, linux, sudeep.holla, lorenzo.pieralisi, catalin.marinas, will.deacon, morten.rasmussen, dietmar.eggemann, broonie, gregkh Hi, ping. I'd need more advice on this set, especially on how and if patch 6 could fly. Thanks, - Juri On 19/01/17 14:37, Juri Lelli wrote: > Hi, > > arm and arm64 topology.c share a lot of code related to parsing of capacity > information. This set of patches proposes a solution (based on Will's, > Catalin's and Mark's off-line suggestions) to move such common code in a single > place: drivers/base/arch_topology.c (by creating such file and conditionally > compiling it for arm and arm64 only). > > First 5 patches are actually fixes for the current code. > > Patch 6 is the actual refactoring. > > Last patch removes one of the extern symbols by changing a bit the now common > code. We still remain with some other externs, which are not nice. Moving them > in some header file solves the issue, should I just create a new include/ > linux/arch_topology.h file and move them there? > > The set is based on top of linux/master (4.10-rc4 fb1d8e0e2c50) and it is also > available from: > > git://linux-arm.org/linux-jl.git upstream/default_caps_factorize > > Best, > > - Juri > > Juri Lelli (7): > Documentation: arm: fix wrong reference number in DT definition > Documentation/ABI: add information about cpu_capacity > arm: fix return value of parse_cpu_capacity > arm: remove wrong CONFIG_PROC_SYSCTL ifdef > arm64: remove wrong CONFIG_PROC_SYSCTL ifdef > arm, arm64: factorize common cpu capacity default code > arm,arm64,drivers: reduce scope of cap_parsing_failed > > Documentation/ABI/testing/sysfs-devices-system-cpu | 7 + > Documentation/devicetree/bindings/arm/cpus.txt | 4 +- > arch/arm/Kconfig | 1 + > arch/arm/kernel/topology.c | 216 +------------------ > arch/arm64/Kconfig | 1 + > arch/arm64/kernel/topology.c | 218 +------------------ > drivers/base/Kconfig | 8 + > drivers/base/Makefile | 1 + > drivers/base/arch_topology.c | 240 +++++++++++++++++++++ > 9 files changed, 269 insertions(+), 427 deletions(-) > create mode 100644 drivers/base/arch_topology.c > > -- > 2.10.0 > ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code 2017-01-30 12:29 ` [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code Juri Lelli @ 2017-01-30 17:51 ` Catalin Marinas 2017-01-30 18:21 ` Juri Lelli 0 siblings, 1 reply; 16+ messages in thread From: Catalin Marinas @ 2017-01-30 17:51 UTC (permalink / raw) To: Juri Lelli Cc: linux-kernel, mark.rutland, devicetree, lorenzo.pieralisi, vincent.guittot, linux-pm, peterz, broonie, will.deacon, gregkh, dietmar.eggemann, robh+dt, sudeep.holla, linux, morten.rasmussen, linux-arm-kernel On Mon, Jan 30, 2017 at 12:29:01PM +0000, Juri Lelli wrote: > I'd need more advice on this set, especially on how and if patch 6 could fly. Since you got some comments and said that you are going to fix them in the next version, I guess people are waiting for you to post a new series. -- Catalin ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code 2017-01-30 17:51 ` Catalin Marinas @ 2017-01-30 18:21 ` Juri Lelli 0 siblings, 0 replies; 16+ messages in thread From: Juri Lelli @ 2017-01-30 18:21 UTC (permalink / raw) To: Catalin Marinas Cc: linux-kernel, mark.rutland, devicetree, lorenzo.pieralisi, vincent.guittot, linux-pm, peterz, broonie, will.deacon, gregkh, dietmar.eggemann, robh+dt, sudeep.holla, linux, morten.rasmussen, linux-arm-kernel Hi Catalin, On 30/01/17 17:51, Catalin Marinas wrote: > On Mon, Jan 30, 2017 at 12:29:01PM +0000, Juri Lelli wrote: > > I'd need more advice on this set, especially on how and if patch 6 could fly. > > Since you got some comments and said that you are going to fix them in > the next version, I guess people are waiting for you to post a new > series. > While this is true for Dietmar's and part of Russell's comments, I was still waiting to understand where people think is better to move the externs (as Russell pointed out), though, and if the whole idea could fly. I could certainly come up with a proposal on this point, but I didn't simply want to spam people's mailboxes with a v2 (addressing relatively minor points, IMHO) if v1 was already completely off. Apologies if that wasn't clear from my replies. Maybe you are saying that no comments are a good sign after all. :) Best, - Juri ^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2017-01-30 18:20 UTC | newest] Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-01-19 14:37 [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code Juri Lelli 2017-01-19 14:37 ` [PATCH 1/7] Documentation: arm: fix wrong reference number in DT definition Juri Lelli 2017-01-21 20:55 ` Rob Herring 2017-01-19 14:37 ` [PATCH 2/7] Documentation/ABI: add information about cpu_capacity Juri Lelli 2017-01-19 14:37 ` [PATCH 3/7] arm: fix return value of parse_cpu_capacity Juri Lelli 2017-01-19 14:37 ` [PATCH 4/7] arm: remove wrong CONFIG_PROC_SYSCTL ifdef Juri Lelli 2017-01-19 14:37 ` [PATCH 5/7] arm64: " Juri Lelli 2017-01-19 14:37 ` [PATCH 6/7] arm, arm64: factorize common cpu capacity default code Juri Lelli 2017-01-19 14:53 ` Russell King - ARM Linux 2017-01-19 15:51 ` Juri Lelli 2017-01-19 16:00 ` Dietmar Eggemann 2017-01-20 10:42 ` Juri Lelli 2017-01-19 14:37 ` [PATCH 7/7] arm,arm64,drivers: reduce scope of cap_parsing_failed Juri Lelli 2017-01-30 12:29 ` [PATCH 0/7] Fix issues and factorize arm/arm64 capacity information code Juri Lelli 2017-01-30 17:51 ` Catalin Marinas 2017-01-30 18:21 ` Juri Lelli
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).