ARM: sched: Avoid empty 'slow' HMP domain
authorJon Medhurst <tixy@linaro.org>
Fri, 12 Oct 2012 12:45:35 +0000 (13:45 +0100)
committerJon Medhurst <tixy@linaro.org>
Wed, 17 Jul 2013 10:12:26 +0000 (11:12 +0100)
On homogeneous (non-heterogeneous) systems all CPUs will be declared
'fast' and the slow cpu list will be empty. In this situation we need to
avoid adding an empty slow HMP domain otherwise the scheduler code will
blow up when it attempts to move a task to the slow domain.

Signed-off-by: Jon Medhurst <tixy@linaro.org>
arch/arm/kernel/topology.c

index 627e14e44af5cf314c52467eab8c27eaf3067d03..0d406ff2e5b66e056591c28c8a62b90e689d17a6 100644 (file)
@@ -368,10 +368,12 @@ void __init arch_get_hmp_domains(struct list_head *hmp_domains_list)
         * Must be ordered with respect to compute capacity.
         * Fastest domain at head of list.
         */
-       domain = (struct hmp_domain *)
-               kmalloc(sizeof(struct hmp_domain), GFP_KERNEL);
-       cpumask_copy(&domain->cpus, &hmp_slow_cpu_mask);
-       list_add(&domain->hmp_domains, hmp_domains_list);
+       if(!cpumask_empty(&hmp_slow_cpu_mask)) {
+               domain = (struct hmp_domain *)
+                       kmalloc(sizeof(struct hmp_domain), GFP_KERNEL);
+               cpumask_copy(&domain->cpus, &hmp_slow_cpu_mask);
+               list_add(&domain->hmp_domains, hmp_domains_list);
+       }
        domain = (struct hmp_domain *)
                kmalloc(sizeof(struct hmp_domain), GFP_KERNEL);
        cpumask_copy(&domain->cpus, &hmp_fast_cpu_mask);