Merge tag 'irqchip-6.3' of git://git.kernel.org/pub/scm/linux/kernel/git/maz/arm...
authorThomas Gleixner <tglx@linutronix.de>
Sat, 18 Feb 2023 23:07:56 +0000 (00:07 +0100)
committerThomas Gleixner <tglx@linutronix.de>
Sat, 18 Feb 2023 23:07:56 +0000 (00:07 +0100)
Pull irqchip updates from Marc Zyngier:

   - New and improved irqdomain locking, closing a number of races that
     became apparent now that we are able to probe drivers in parallel

   - A bunch of OF node refcounting bugs have been fixed

   - We now have a new IPI mux, lifted from the Apple AIC code and
     made common. It is expected that riscv will eventually benefit
     from it

   - Two small fixes for the Broadcom L2 drivers

   - Various cleanups and minor bug fixes

Link: https://lore.kernel.org/r/20230218143452.3817627-1-maz@kernel.org
14 files changed:
Documentation/kernel-hacking/locking.rst
Documentation/translations/it_IT/kernel-hacking/locking.rst
MAINTAINERS
block/blk-mq-cpumap.c
drivers/irqchip/irq-armada-370-xp.c
drivers/irqchip/irq-bcm2836.c
drivers/irqchip/irq-gic-v3.c
drivers/irqchip/irq-gic-v4.c
drivers/irqchip/irq-gic.c
include/linux/group_cpus.h [new file with mode: 0644]
kernel/irq/affinity.c
kernel/irq/manage.c
lib/Makefile
lib/group_cpus.c [new file with mode: 0644]

index c756786..dff0646 100644 (file)
@@ -1277,11 +1277,11 @@ Manfred Spraul points out that you can still do this, even if the data
 is very occasionally accessed in user context or softirqs/tasklets. The
 irq handler doesn't use a lock, and all other accesses are done as so::
 
-        spin_lock(&lock);
+        mutex_lock(&lock);
         disable_irq(irq);
         ...
         enable_irq(irq);
-        spin_unlock(&lock);
+        mutex_unlock(&lock);
 
 The disable_irq() prevents the irq handler from running
 (and waits for it to finish if it's currently running on other CPUs).
index b8ecf41..05d362b 100644 (file)
@@ -1307,11 +1307,11 @@ se i dati vengono occasionalmente utilizzati da un contesto utente o
 da un'interruzione software. Il gestore d'interruzione non utilizza alcun
 *lock*, e tutti gli altri accessi verranno fatti così::
 
-        spin_lock(&lock);
+        mutex_lock(&lock);
         disable_irq(irq);
         ...
         enable_irq(irq);
-        spin_unlock(&lock);
+        mutex_unlock(&lock);
 
 La funzione disable_irq() impedisce al gestore d'interruzioni
 d'essere eseguito (e aspetta che finisca nel caso fosse in esecuzione su
index 42fc47c..cb47488 100644 (file)
@@ -10935,6 +10935,8 @@ L:      linux-kernel@vger.kernel.org
 S:     Maintained
 T:     git git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git irq/core
 F:     kernel/irq/
+F:     include/linux/group_cpus.h
+F:     lib/group_cpus.c
 
 IRQCHIP DRIVERS
 M:     Thomas Gleixner <tglx@linutronix.de>
index 9c2fce1..0c612c1 100644 (file)
 #include <linux/mm.h>
 #include <linux/smp.h>
 #include <linux/cpu.h>
+#include <linux/group_cpus.h>
 
 #include <linux/blk-mq.h>
 #include "blk.h"
 #include "blk-mq.h"
 
-static int queue_index(struct blk_mq_queue_map *qmap,
-                      unsigned int nr_queues, const int q)
-{
-       return qmap->queue_offset + (q % nr_queues);
-}
-
-static int get_first_sibling(unsigned int cpu)
-{
-       unsigned int ret;
-
-       ret = cpumask_first(topology_sibling_cpumask(cpu));
-       if (ret < nr_cpu_ids)
-               return ret;
-
-       return cpu;
-}
-
 void blk_mq_map_queues(struct blk_mq_queue_map *qmap)
 {
-       unsigned int *map = qmap->mq_map;
-       unsigned int nr_queues = qmap->nr_queues;
-       unsigned int cpu, first_sibling, q = 0;
-
-       for_each_possible_cpu(cpu)
-               map[cpu] = -1;
-
-       /*
-        * Spread queues among present CPUs first for minimizing
-        * count of dead queues which are mapped by all un-present CPUs
-        */
-       for_each_present_cpu(cpu) {
-               if (q >= nr_queues)
-                       break;
-               map[cpu] = queue_index(qmap, nr_queues, q++);
+       const struct cpumask *masks;
+       unsigned int queue, cpu;
+
+       masks = group_cpus_evenly(qmap->nr_queues);
+       if (!masks) {
+               for_each_possible_cpu(cpu)
+                       qmap->mq_map[cpu] = qmap->queue_offset;
+               return;
        }
 
-       for_each_possible_cpu(cpu) {
-               if (map[cpu] != -1)
-                       continue;
-               /*
-                * First do sequential mapping between CPUs and queues.
-                * In case we still have CPUs to map, and we have some number of
-                * threads per cores then map sibling threads to the same queue
-                * for performance optimizations.
-                */
-               if (q < nr_queues) {
-                       map[cpu] = queue_index(qmap, nr_queues, q++);
-               } else {
-                       first_sibling = get_first_sibling(cpu);
-                       if (first_sibling == cpu)
-                               map[cpu] = queue_index(qmap, nr_queues, q++);
-                       else
-                               map[cpu] = map[first_sibling];
-               }
+       for (queue = 0; queue < qmap->nr_queues; queue++) {
+               for_each_cpu(cpu, &masks[queue])
+                       qmap->mq_map[cpu] = qmap->queue_offset + queue;
        }
+       kfree(masks);
 }
 EXPORT_SYMBOL_GPL(blk_mq_map_queues);
 
index ee18eb3..a555284 100644 (file)
@@ -454,8 +454,7 @@ static __init void armada_xp_ipi_init(struct device_node *node)
                return;
 
        irq_domain_update_bus_token(ipi_domain, DOMAIN_BUS_IPI);
-       base_ipi = __irq_domain_alloc_irqs(ipi_domain, -1, IPI_DOORBELL_END,
-                                          NUMA_NO_NODE, NULL, false, NULL);
+       base_ipi = irq_domain_alloc_irqs(ipi_domain, IPI_DOORBELL_END, NUMA_NO_NODE, NULL);
        if (WARN_ON(!base_ipi))
                return;
 
index 51491c3..e5f1059 100644 (file)
@@ -268,10 +268,7 @@ static void __init bcm2836_arm_irqchip_smp_init(void)
        ipi_domain->flags |= IRQ_DOMAIN_FLAG_IPI_SINGLE;
        irq_domain_update_bus_token(ipi_domain, DOMAIN_BUS_IPI);
 
-       base_ipi = __irq_domain_alloc_irqs(ipi_domain, -1, BITS_PER_MBOX,
-                                          NUMA_NO_NODE, NULL,
-                                          false, NULL);
-
+       base_ipi = irq_domain_alloc_irqs(ipi_domain, BITS_PER_MBOX, NUMA_NO_NODE, NULL);
        if (WARN_ON(!base_ipi))
                return;
 
index 997104d..bb57ab8 100644 (file)
@@ -1310,9 +1310,7 @@ static void __init gic_smp_init(void)
                                  gic_starting_cpu, NULL);
 
        /* Register all 8 non-secure SGIs */
-       base_sgi = __irq_domain_alloc_irqs(gic_data.domain, -1, 8,
-                                          NUMA_NO_NODE, &sgi_fwspec,
-                                          false, NULL);
+       base_sgi = irq_domain_alloc_irqs(gic_data.domain, 8, NUMA_NO_NODE, &sgi_fwspec);
        if (WARN_ON(base_sgi <= 0))
                return;
 
index a6277de..94d56a0 100644 (file)
@@ -139,9 +139,7 @@ static int its_alloc_vcpu_sgis(struct its_vpe *vpe, int idx)
        if (!vpe->sgi_domain)
                goto err;
 
-       sgi_base = __irq_domain_alloc_irqs(vpe->sgi_domain, -1, 16,
-                                              NUMA_NO_NODE, vpe,
-                                              false, NULL);
+       sgi_base = irq_domain_alloc_irqs(vpe->sgi_domain, 16, NUMA_NO_NODE, vpe);
        if (sgi_base <= 0)
                goto err;
 
@@ -176,9 +174,8 @@ int its_alloc_vcpu_irqs(struct its_vm *vm)
                vm->vpes[i]->idai = true;
        }
 
-       vpe_base_irq = __irq_domain_alloc_irqs(vm->domain, -1, vm->nr_vpes,
-                                              NUMA_NO_NODE, vm,
-                                              false, NULL);
+       vpe_base_irq = irq_domain_alloc_irqs(vm->domain, vm->nr_vpes,
+                                            NUMA_NO_NODE, vm);
        if (vpe_base_irq <= 0)
                goto err;
 
index 210bc2f..4fa4d8a 100644 (file)
@@ -868,9 +868,7 @@ static __init void gic_smp_init(void)
                                  "irqchip/arm/gic:starting",
                                  gic_starting_cpu, NULL);
 
-       base_sgi = __irq_domain_alloc_irqs(gic_data[0].domain, -1, 8,
-                                          NUMA_NO_NODE, &sgi_fwspec,
-                                          false, NULL);
+       base_sgi = irq_domain_alloc_irqs(gic_data[0].domain, 8, NUMA_NO_NODE, &sgi_fwspec);
        if (WARN_ON(base_sgi <= 0))
                return;
 
diff --git a/include/linux/group_cpus.h b/include/linux/group_cpus.h
new file mode 100644 (file)
index 0000000..e42807e
--- /dev/null
@@ -0,0 +1,14 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+/*
+ * Copyright (C) 2016 Thomas Gleixner.
+ * Copyright (C) 2016-2017 Christoph Hellwig.
+ */
+
+#ifndef __LINUX_GROUP_CPUS_H
+#define __LINUX_GROUP_CPUS_H
+#include <linux/kernel.h>
+#include <linux/cpu.h>
+
+struct cpumask *group_cpus_evenly(unsigned int numgrps);
+
+#endif
index d9a5c1d..44a4eba 100644 (file)
@@ -7,398 +7,7 @@
 #include <linux/kernel.h>
 #include <linux/slab.h>
 #include <linux/cpu.h>
-#include <linux/sort.h>
-
-static void irq_spread_init_one(struct cpumask *irqmsk, struct cpumask *nmsk,
-                               unsigned int cpus_per_vec)
-{
-       const struct cpumask *siblmsk;
-       int cpu, sibl;
-
-       for ( ; cpus_per_vec > 0; ) {
-               cpu = cpumask_first(nmsk);
-
-               /* Should not happen, but I'm too lazy to think about it */
-               if (cpu >= nr_cpu_ids)
-                       return;
-
-               cpumask_clear_cpu(cpu, nmsk);
-               cpumask_set_cpu(cpu, irqmsk);
-               cpus_per_vec--;
-
-               /* If the cpu has siblings, use them first */
-               siblmsk = topology_sibling_cpumask(cpu);
-               for (sibl = -1; cpus_per_vec > 0; ) {
-                       sibl = cpumask_next(sibl, siblmsk);
-                       if (sibl >= nr_cpu_ids)
-                               break;
-                       if (!cpumask_test_and_clear_cpu(sibl, nmsk))
-                               continue;
-                       cpumask_set_cpu(sibl, irqmsk);
-                       cpus_per_vec--;
-               }
-       }
-}
-
-static cpumask_var_t *alloc_node_to_cpumask(void)
-{
-       cpumask_var_t *masks;
-       int node;
-
-       masks = kcalloc(nr_node_ids, sizeof(cpumask_var_t), GFP_KERNEL);
-       if (!masks)
-               return NULL;
-
-       for (node = 0; node < nr_node_ids; node++) {
-               if (!zalloc_cpumask_var(&masks[node], GFP_KERNEL))
-                       goto out_unwind;
-       }
-
-       return masks;
-
-out_unwind:
-       while (--node >= 0)
-               free_cpumask_var(masks[node]);
-       kfree(masks);
-       return NULL;
-}
-
-static void free_node_to_cpumask(cpumask_var_t *masks)
-{
-       int node;
-
-       for (node = 0; node < nr_node_ids; node++)
-               free_cpumask_var(masks[node]);
-       kfree(masks);
-}
-
-static void build_node_to_cpumask(cpumask_var_t *masks)
-{
-       int cpu;
-
-       for_each_possible_cpu(cpu)
-               cpumask_set_cpu(cpu, masks[cpu_to_node(cpu)]);
-}
-
-static int get_nodes_in_cpumask(cpumask_var_t *node_to_cpumask,
-                               const struct cpumask *mask, nodemask_t *nodemsk)
-{
-       int n, nodes = 0;
-
-       /* Calculate the number of nodes in the supplied affinity mask */
-       for_each_node(n) {
-               if (cpumask_intersects(mask, node_to_cpumask[n])) {
-                       node_set(n, *nodemsk);
-                       nodes++;
-               }
-       }
-       return nodes;
-}
-
-struct node_vectors {
-       unsigned id;
-
-       union {
-               unsigned nvectors;
-               unsigned ncpus;
-       };
-};
-
-static int ncpus_cmp_func(const void *l, const void *r)
-{
-       const struct node_vectors *ln = l;
-       const struct node_vectors *rn = r;
-
-       return ln->ncpus - rn->ncpus;
-}
-
-/*
- * Allocate vector number for each node, so that for each node:
- *
- * 1) the allocated number is >= 1
- *
- * 2) the allocated numbver is <= active CPU number of this node
- *
- * The actual allocated total vectors may be less than @numvecs when
- * active total CPU number is less than @numvecs.
- *
- * Active CPUs means the CPUs in '@cpu_mask AND @node_to_cpumask[]'
- * for each node.
- */
-static void alloc_nodes_vectors(unsigned int numvecs,
-                               cpumask_var_t *node_to_cpumask,
-                               const struct cpumask *cpu_mask,
-                               const nodemask_t nodemsk,
-                               struct cpumask *nmsk,
-                               struct node_vectors *node_vectors)
-{
-       unsigned n, remaining_ncpus = 0;
-
-       for (n = 0; n < nr_node_ids; n++) {
-               node_vectors[n].id = n;
-               node_vectors[n].ncpus = UINT_MAX;
-       }
-
-       for_each_node_mask(n, nodemsk) {
-               unsigned ncpus;
-
-               cpumask_and(nmsk, cpu_mask, node_to_cpumask[n]);
-               ncpus = cpumask_weight(nmsk);
-
-               if (!ncpus)
-                       continue;
-               remaining_ncpus += ncpus;
-               node_vectors[n].ncpus = ncpus;
-       }
-
-       numvecs = min_t(unsigned, remaining_ncpus, numvecs);
-
-       sort(node_vectors, nr_node_ids, sizeof(node_vectors[0]),
-            ncpus_cmp_func, NULL);
-
-       /*
-        * Allocate vectors for each node according to the ratio of this
-        * node's nr_cpus to remaining un-assigned ncpus. 'numvecs' is
-        * bigger than number of active numa nodes. Always start the
-        * allocation from the node with minimized nr_cpus.
-        *
-        * This way guarantees that each active node gets allocated at
-        * least one vector, and the theory is simple: over-allocation
-        * is only done when this node is assigned by one vector, so
-        * other nodes will be allocated >= 1 vector, since 'numvecs' is
-        * bigger than number of numa nodes.
-        *
-        * One perfect invariant is that number of allocated vectors for
-        * each node is <= CPU count of this node:
-        *
-        * 1) suppose there are two nodes: A and B
-        *      ncpu(X) is CPU count of node X
-        *      vecs(X) is the vector count allocated to node X via this
-        *      algorithm
-        *
-        *      ncpu(A) <= ncpu(B)
-        *      ncpu(A) + ncpu(B) = N
-        *      vecs(A) + vecs(B) = V
-        *
-        *      vecs(A) = max(1, round_down(V * ncpu(A) / N))
-        *      vecs(B) = V - vecs(A)
-        *
-        *      both N and V are integer, and 2 <= V <= N, suppose
-        *      V = N - delta, and 0 <= delta <= N - 2
-        *
-        * 2) obviously vecs(A) <= ncpu(A) because:
-        *
-        *      if vecs(A) is 1, then vecs(A) <= ncpu(A) given
-        *      ncpu(A) >= 1
-        *
-        *      otherwise,
-        *              vecs(A) <= V * ncpu(A) / N <= ncpu(A), given V <= N
-        *
-        * 3) prove how vecs(B) <= ncpu(B):
-        *
-        *      if round_down(V * ncpu(A) / N) == 0, vecs(B) won't be
-        *      over-allocated, so vecs(B) <= ncpu(B),
-        *
-        *      otherwise:
-        *
-        *      vecs(A) =
-        *              round_down(V * ncpu(A) / N) =
-        *              round_down((N - delta) * ncpu(A) / N) =
-        *              round_down((N * ncpu(A) - delta * ncpu(A)) / N)  >=
-        *              round_down((N * ncpu(A) - delta * N) / N)        =
-        *              cpu(A) - delta
-        *
-        *      then:
-        *
-        *      vecs(A) - V >= ncpu(A) - delta - V
-        *      =>
-        *      V - vecs(A) <= V + delta - ncpu(A)
-        *      =>
-        *      vecs(B) <= N - ncpu(A)
-        *      =>
-        *      vecs(B) <= cpu(B)
-        *
-        * For nodes >= 3, it can be thought as one node and another big
-        * node given that is exactly what this algorithm is implemented,
-        * and we always re-calculate 'remaining_ncpus' & 'numvecs', and
-        * finally for each node X: vecs(X) <= ncpu(X).
-        *
-        */
-       for (n = 0; n < nr_node_ids; n++) {
-               unsigned nvectors, ncpus;
-
-               if (node_vectors[n].ncpus == UINT_MAX)
-                       continue;
-
-               WARN_ON_ONCE(numvecs == 0);
-
-               ncpus = node_vectors[n].ncpus;
-               nvectors = max_t(unsigned, 1,
-                                numvecs * ncpus / remaining_ncpus);
-               WARN_ON_ONCE(nvectors > ncpus);
-
-               node_vectors[n].nvectors = nvectors;
-
-               remaining_ncpus -= ncpus;
-               numvecs -= nvectors;
-       }
-}
-
-static int __irq_build_affinity_masks(unsigned int startvec,
-                                     unsigned int numvecs,
-                                     unsigned int firstvec,
-                                     cpumask_var_t *node_to_cpumask,
-                                     const struct cpumask *cpu_mask,
-                                     struct cpumask *nmsk,
-                                     struct irq_affinity_desc *masks)
-{
-       unsigned int i, n, nodes, cpus_per_vec, extra_vecs, done = 0;
-       unsigned int last_affv = firstvec + numvecs;
-       unsigned int curvec = startvec;
-       nodemask_t nodemsk = NODE_MASK_NONE;
-       struct node_vectors *node_vectors;
-
-       if (cpumask_empty(cpu_mask))
-               return 0;
-
-       nodes = get_nodes_in_cpumask(node_to_cpumask, cpu_mask, &nodemsk);
-
-       /*
-        * If the number of nodes in the mask is greater than or equal the
-        * number of vectors we just spread the vectors across the nodes.
-        */
-       if (numvecs <= nodes) {
-               for_each_node_mask(n, nodemsk) {
-                       /* Ensure that only CPUs which are in both masks are set */
-                       cpumask_and(nmsk, cpu_mask, node_to_cpumask[n]);
-                       cpumask_or(&masks[curvec].mask, &masks[curvec].mask, nmsk);
-                       if (++curvec == last_affv)
-                               curvec = firstvec;
-               }
-               return numvecs;
-       }
-
-       node_vectors = kcalloc(nr_node_ids,
-                              sizeof(struct node_vectors),
-                              GFP_KERNEL);
-       if (!node_vectors)
-               return -ENOMEM;
-
-       /* allocate vector number for each node */
-       alloc_nodes_vectors(numvecs, node_to_cpumask, cpu_mask,
-                           nodemsk, nmsk, node_vectors);
-
-       for (i = 0; i < nr_node_ids; i++) {
-               unsigned int ncpus, v;
-               struct node_vectors *nv = &node_vectors[i];
-
-               if (nv->nvectors == UINT_MAX)
-                       continue;
-
-               /* Get the cpus on this node which are in the mask */
-               cpumask_and(nmsk, cpu_mask, node_to_cpumask[nv->id]);
-               ncpus = cpumask_weight(nmsk);
-               if (!ncpus)
-                       continue;
-
-               WARN_ON_ONCE(nv->nvectors > ncpus);
-
-               /* Account for rounding errors */
-               extra_vecs = ncpus - nv->nvectors * (ncpus / nv->nvectors);
-
-               /* Spread allocated vectors on CPUs of the current node */
-               for (v = 0; v < nv->nvectors; v++, curvec++) {
-                       cpus_per_vec = ncpus / nv->nvectors;
-
-                       /* Account for extra vectors to compensate rounding errors */
-                       if (extra_vecs) {
-                               cpus_per_vec++;
-                               --extra_vecs;
-                       }
-
-                       /*
-                        * wrapping has to be considered given 'startvec'
-                        * may start anywhere
-                        */
-                       if (curvec >= last_affv)
-                               curvec = firstvec;
-                       irq_spread_init_one(&masks[curvec].mask, nmsk,
-                                               cpus_per_vec);
-               }
-               done += nv->nvectors;
-       }
-       kfree(node_vectors);
-       return done;
-}
-
-/*
- * build affinity in two stages:
- *     1) spread present CPU on these vectors
- *     2) spread other possible CPUs on these vectors
- */
-static int irq_build_affinity_masks(unsigned int startvec, unsigned int numvecs,
-                                   unsigned int firstvec,
-                                   struct irq_affinity_desc *masks)
-{
-       unsigned int curvec = startvec, nr_present = 0, nr_others = 0;
-       cpumask_var_t *node_to_cpumask;
-       cpumask_var_t nmsk, npresmsk;
-       int ret = -ENOMEM;
-
-       if (!zalloc_cpumask_var(&nmsk, GFP_KERNEL))
-               return ret;
-
-       if (!zalloc_cpumask_var(&npresmsk, GFP_KERNEL))
-               goto fail_nmsk;
-
-       node_to_cpumask = alloc_node_to_cpumask();
-       if (!node_to_cpumask)
-               goto fail_npresmsk;
-
-       /* Stabilize the cpumasks */
-       cpus_read_lock();
-       build_node_to_cpumask(node_to_cpumask);
-
-       /* Spread on present CPUs starting from affd->pre_vectors */
-       ret = __irq_build_affinity_masks(curvec, numvecs, firstvec,
-                                        node_to_cpumask, cpu_present_mask,
-                                        nmsk, masks);
-       if (ret < 0)
-               goto fail_build_affinity;
-       nr_present = ret;
-
-       /*
-        * Spread on non present CPUs starting from the next vector to be
-        * handled. If the spreading of present CPUs already exhausted the
-        * vector space, assign the non present CPUs to the already spread
-        * out vectors.
-        */
-       if (nr_present >= numvecs)
-               curvec = firstvec;
-       else
-               curvec = firstvec + nr_present;
-       cpumask_andnot(npresmsk, cpu_possible_mask, cpu_present_mask);
-       ret = __irq_build_affinity_masks(curvec, numvecs, firstvec,
-                                        node_to_cpumask, npresmsk, nmsk,
-                                        masks);
-       if (ret >= 0)
-               nr_others = ret;
-
- fail_build_affinity:
-       cpus_read_unlock();
-
-       if (ret >= 0)
-               WARN_ON(nr_present + nr_others < numvecs);
-
-       free_node_to_cpumask(node_to_cpumask);
-
- fail_npresmsk:
-       free_cpumask_var(npresmsk);
-
- fail_nmsk:
-       free_cpumask_var(nmsk);
-       return ret < 0 ? ret : 0;
-}
+#include <linux/group_cpus.h>
 
 static void default_calc_sets(struct irq_affinity *affd, unsigned int affvecs)
 {
@@ -461,14 +70,18 @@ irq_create_affinity_masks(unsigned int nvecs, struct irq_affinity *affd)
         */
        for (i = 0, usedvecs = 0; i < affd->nr_sets; i++) {
                unsigned int this_vecs = affd->set_size[i];
-               int ret;
+               int j;
+               struct cpumask *result = group_cpus_evenly(this_vecs);
 
-               ret = irq_build_affinity_masks(curvec, this_vecs,
-                                              curvec, masks);
-               if (ret) {
+               if (!result) {
                        kfree(masks);
                        return NULL;
                }
+
+               for (j = 0; j < this_vecs; j++)
+                       cpumask_copy(&masks[curvec + j].mask, &result[j]);
+               kfree(result);
+
                curvec += this_vecs;
                usedvecs += this_vecs;
        }
index 5b7cf28..8ce7549 100644 (file)
@@ -723,10 +723,13 @@ EXPORT_SYMBOL(disable_irq_nosync);
  *     to complete before returning. If you use this function while
  *     holding a resource the IRQ handler may need you will deadlock.
  *
- *     This function may be called - with care - from IRQ context.
+ *     Can only be called from preemptible code as it might sleep when
+ *     an interrupt thread is associated to @irq.
+ *
  */
 void disable_irq(unsigned int irq)
 {
+       might_sleep();
        if (!__disable_irq_nosync(irq))
                synchronize_irq(irq);
 }
index 4d9461b..a4665a8 100644 (file)
@@ -353,6 +353,8 @@ obj-$(CONFIG_SBITMAP) += sbitmap.o
 
 obj-$(CONFIG_PARMAN) += parman.o
 
+obj-y += group_cpus.o
+
 # GCC library routines
 obj-$(CONFIG_GENERIC_LIB_ASHLDI3) += ashldi3.o
 obj-$(CONFIG_GENERIC_LIB_ASHRDI3) += ashrdi3.o
diff --git a/lib/group_cpus.c b/lib/group_cpus.c
new file mode 100644 (file)
index 0000000..9c837a3
--- /dev/null
@@ -0,0 +1,428 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Copyright (C) 2016 Thomas Gleixner.
+ * Copyright (C) 2016-2017 Christoph Hellwig.
+ */
+#include <linux/kernel.h>
+#include <linux/slab.h>
+#include <linux/cpu.h>
+#include <linux/sort.h>
+#include <linux/group_cpus.h>
+
+#ifdef CONFIG_SMP
+
+static void grp_spread_init_one(struct cpumask *irqmsk, struct cpumask *nmsk,
+                               unsigned int cpus_per_grp)
+{
+       const struct cpumask *siblmsk;
+       int cpu, sibl;
+
+       for ( ; cpus_per_grp > 0; ) {
+               cpu = cpumask_first(nmsk);
+
+               /* Should not happen, but I'm too lazy to think about it */
+               if (cpu >= nr_cpu_ids)
+                       return;
+
+               cpumask_clear_cpu(cpu, nmsk);
+               cpumask_set_cpu(cpu, irqmsk);
+               cpus_per_grp--;
+
+               /* If the cpu has siblings, use them first */
+               siblmsk = topology_sibling_cpumask(cpu);
+               for (sibl = -1; cpus_per_grp > 0; ) {
+                       sibl = cpumask_next(sibl, siblmsk);
+                       if (sibl >= nr_cpu_ids)
+                               break;
+                       if (!cpumask_test_and_clear_cpu(sibl, nmsk))
+                               continue;
+                       cpumask_set_cpu(sibl, irqmsk);
+                       cpus_per_grp--;
+               }
+       }
+}
+
+static cpumask_var_t *alloc_node_to_cpumask(void)
+{
+       cpumask_var_t *masks;
+       int node;
+
+       masks = kcalloc(nr_node_ids, sizeof(cpumask_var_t), GFP_KERNEL);
+       if (!masks)
+               return NULL;
+
+       for (node = 0; node < nr_node_ids; node++) {
+               if (!zalloc_cpumask_var(&masks[node], GFP_KERNEL))
+                       goto out_unwind;
+       }
+
+       return masks;
+
+out_unwind:
+       while (--node >= 0)
+               free_cpumask_var(masks[node]);
+       kfree(masks);
+       return NULL;
+}
+
+static void free_node_to_cpumask(cpumask_var_t *masks)
+{
+       int node;
+
+       for (node = 0; node < nr_node_ids; node++)
+               free_cpumask_var(masks[node]);
+       kfree(masks);
+}
+
+static void build_node_to_cpumask(cpumask_var_t *masks)
+{
+       int cpu;
+
+       for_each_possible_cpu(cpu)
+               cpumask_set_cpu(cpu, masks[cpu_to_node(cpu)]);
+}
+
+static int get_nodes_in_cpumask(cpumask_var_t *node_to_cpumask,
+                               const struct cpumask *mask, nodemask_t *nodemsk)
+{
+       int n, nodes = 0;
+
+       /* Calculate the number of nodes in the supplied affinity mask */
+       for_each_node(n) {
+               if (cpumask_intersects(mask, node_to_cpumask[n])) {
+                       node_set(n, *nodemsk);
+                       nodes++;
+               }
+       }
+       return nodes;
+}
+
+struct node_groups {
+       unsigned id;
+
+       union {
+               unsigned ngroups;
+               unsigned ncpus;
+       };
+};
+
+static int ncpus_cmp_func(const void *l, const void *r)
+{
+       const struct node_groups *ln = l;
+       const struct node_groups *rn = r;
+
+       return ln->ncpus - rn->ncpus;
+}
+
+/*
+ * Allocate group number for each node, so that for each node:
+ *
+ * 1) the allocated number is >= 1
+ *
+ * 2) the allocated number is <= active CPU number of this node
+ *
+ * The actual allocated total groups may be less than @numgrps when
+ * active total CPU number is less than @numgrps.
+ *
+ * Active CPUs means the CPUs in '@cpu_mask AND @node_to_cpumask[]'
+ * for each node.
+ */
+static void alloc_nodes_groups(unsigned int numgrps,
+                              cpumask_var_t *node_to_cpumask,
+                              const struct cpumask *cpu_mask,
+                              const nodemask_t nodemsk,
+                              struct cpumask *nmsk,
+                              struct node_groups *node_groups)
+{
+       unsigned n, remaining_ncpus = 0;
+
+       for (n = 0; n < nr_node_ids; n++) {
+               node_groups[n].id = n;
+               node_groups[n].ncpus = UINT_MAX;
+       }
+
+       for_each_node_mask(n, nodemsk) {
+               unsigned ncpus;
+
+               cpumask_and(nmsk, cpu_mask, node_to_cpumask[n]);
+               ncpus = cpumask_weight(nmsk);
+
+               if (!ncpus)
+                       continue;
+               remaining_ncpus += ncpus;
+               node_groups[n].ncpus = ncpus;
+       }
+
+       numgrps = min_t(unsigned, remaining_ncpus, numgrps);
+
+       sort(node_groups, nr_node_ids, sizeof(node_groups[0]),
+            ncpus_cmp_func, NULL);
+
+       /*
+        * Allocate groups for each node according to the ratio of this
+        * node's nr_cpus to remaining un-assigned ncpus. 'numgrps' is
+        * bigger than number of active numa nodes. Always start the
+        * allocation from the node with minimized nr_cpus.
+        *
+        * This way guarantees that each active node gets allocated at
+        * least one group, and the theory is simple: over-allocation
+        * is only done when this node is assigned by one group, so
+        * other nodes will be allocated >= 1 groups, since 'numgrps' is
+        * bigger than number of numa nodes.
+        *
+        * One perfect invariant is that number of allocated groups for
+        * each node is <= CPU count of this node:
+        *
+        * 1) suppose there are two nodes: A and B
+        *      ncpu(X) is CPU count of node X
+        *      grps(X) is the group count allocated to node X via this
+        *      algorithm
+        *
+        *      ncpu(A) <= ncpu(B)
+        *      ncpu(A) + ncpu(B) = N
+        *      grps(A) + grps(B) = G
+        *
+        *      grps(A) = max(1, round_down(G * ncpu(A) / N))
+        *      grps(B) = G - grps(A)
+        *
+        *      both N and G are integer, and 2 <= G <= N, suppose
+        *      G = N - delta, and 0 <= delta <= N - 2
+        *
+        * 2) obviously grps(A) <= ncpu(A) because:
+        *
+        *      if grps(A) is 1, then grps(A) <= ncpu(A) given
+        *      ncpu(A) >= 1
+        *
+        *      otherwise,
+        *              grps(A) <= G * ncpu(A) / N <= ncpu(A), given G <= N
+        *
+        * 3) prove how grps(B) <= ncpu(B):
+        *
+        *      if round_down(G * ncpu(A) / N) == 0, vecs(B) won't be
+        *      over-allocated, so grps(B) <= ncpu(B),
+        *
+        *      otherwise:
+        *
+        *      grps(A) =
+        *              round_down(G * ncpu(A) / N) =
+        *              round_down((N - delta) * ncpu(A) / N) =
+        *              round_down((N * ncpu(A) - delta * ncpu(A)) / N)  >=
+        *              round_down((N * ncpu(A) - delta * N) / N)        =
+        *              cpu(A) - delta
+        *
+        *      then:
+        *
+        *      grps(A) - G >= ncpu(A) - delta - G
+        *      =>
+        *      G - grps(A) <= G + delta - ncpu(A)
+        *      =>
+        *      grps(B) <= N - ncpu(A)
+        *      =>
+        *      grps(B) <= cpu(B)
+        *
+        * For nodes >= 3, it can be thought as one node and another big
+        * node given that is exactly what this algorithm is implemented,
+        * and we always re-calculate 'remaining_ncpus' & 'numgrps', and
+        * finally for each node X: grps(X) <= ncpu(X).
+        *
+        */
+       for (n = 0; n < nr_node_ids; n++) {
+               unsigned ngroups, ncpus;
+
+               if (node_groups[n].ncpus == UINT_MAX)
+                       continue;
+
+               WARN_ON_ONCE(numgrps == 0);
+
+               ncpus = node_groups[n].ncpus;
+               ngroups = max_t(unsigned, 1,
+                                numgrps * ncpus / remaining_ncpus);
+               WARN_ON_ONCE(ngroups > ncpus);
+
+               node_groups[n].ngroups = ngroups;
+
+               remaining_ncpus -= ncpus;
+               numgrps -= ngroups;
+       }
+}
+
+static int __group_cpus_evenly(unsigned int startgrp, unsigned int numgrps,
+                              cpumask_var_t *node_to_cpumask,
+                              const struct cpumask *cpu_mask,
+                              struct cpumask *nmsk, struct cpumask *masks)
+{
+       unsigned int i, n, nodes, cpus_per_grp, extra_grps, done = 0;
+       unsigned int last_grp = numgrps;
+       unsigned int curgrp = startgrp;
+       nodemask_t nodemsk = NODE_MASK_NONE;
+       struct node_groups *node_groups;
+
+       if (cpumask_empty(cpu_mask))
+               return 0;
+
+       nodes = get_nodes_in_cpumask(node_to_cpumask, cpu_mask, &nodemsk);
+
+       /*
+        * If the number of nodes in the mask is greater than or equal the
+        * number of groups we just spread the groups across the nodes.
+        */
+       if (numgrps <= nodes) {
+               for_each_node_mask(n, nodemsk) {
+                       /* Ensure that only CPUs which are in both masks are set */
+                       cpumask_and(nmsk, cpu_mask, node_to_cpumask[n]);
+                       cpumask_or(&masks[curgrp], &masks[curgrp], nmsk);
+                       if (++curgrp == last_grp)
+                               curgrp = 0;
+               }
+               return numgrps;
+       }
+
+       node_groups = kcalloc(nr_node_ids,
+                              sizeof(struct node_groups),
+                              GFP_KERNEL);
+       if (!node_groups)
+               return -ENOMEM;
+
+       /* allocate group number for each node */
+       alloc_nodes_groups(numgrps, node_to_cpumask, cpu_mask,
+                          nodemsk, nmsk, node_groups);
+       for (i = 0; i < nr_node_ids; i++) {
+               unsigned int ncpus, v;
+               struct node_groups *nv = &node_groups[i];
+
+               if (nv->ngroups == UINT_MAX)
+                       continue;
+
+               /* Get the cpus on this node which are in the mask */
+               cpumask_and(nmsk, cpu_mask, node_to_cpumask[nv->id]);
+               ncpus = cpumask_weight(nmsk);
+               if (!ncpus)
+                       continue;
+
+               WARN_ON_ONCE(nv->ngroups > ncpus);
+
+               /* Account for rounding errors */
+               extra_grps = ncpus - nv->ngroups * (ncpus / nv->ngroups);
+
+               /* Spread allocated groups on CPUs of the current node */
+               for (v = 0; v < nv->ngroups; v++, curgrp++) {
+                       cpus_per_grp = ncpus / nv->ngroups;
+
+                       /* Account for extra groups to compensate rounding errors */
+                       if (extra_grps) {
+                               cpus_per_grp++;
+                               --extra_grps;
+                       }
+
+                       /*
+                        * wrapping has to be considered given 'startgrp'
+                        * may start anywhere
+                        */
+                       if (curgrp >= last_grp)
+                               curgrp = 0;
+                       grp_spread_init_one(&masks[curgrp], nmsk,
+                                               cpus_per_grp);
+               }
+               done += nv->ngroups;
+       }
+       kfree(node_groups);
+       return done;
+}
+
+/**
+ * group_cpus_evenly - Group all CPUs evenly per NUMA/CPU locality
+ * @numgrps: number of groups
+ *
+ * Return: cpumask array if successful, NULL otherwise. And each element
+ * includes CPUs assigned to this group
+ *
+ * Try to put close CPUs from viewpoint of CPU and NUMA locality into
+ * same group, and run two-stage grouping:
+ *     1) allocate present CPUs on these groups evenly first
+ *     2) allocate other possible CPUs on these groups evenly
+ *
+ * We guarantee in the resulted grouping that all CPUs are covered, and
+ * no same CPU is assigned to multiple groups
+ */
+struct cpumask *group_cpus_evenly(unsigned int numgrps)
+{
+       unsigned int curgrp = 0, nr_present = 0, nr_others = 0;
+       cpumask_var_t *node_to_cpumask;
+       cpumask_var_t nmsk, npresmsk;
+       int ret = -ENOMEM;
+       struct cpumask *masks = NULL;
+
+       if (!zalloc_cpumask_var(&nmsk, GFP_KERNEL))
+               return NULL;
+
+       if (!zalloc_cpumask_var(&npresmsk, GFP_KERNEL))
+               goto fail_nmsk;
+
+       node_to_cpumask = alloc_node_to_cpumask();
+       if (!node_to_cpumask)
+               goto fail_npresmsk;
+
+       masks = kcalloc(numgrps, sizeof(*masks), GFP_KERNEL);
+       if (!masks)
+               goto fail_node_to_cpumask;
+
+       /* Stabilize the cpumasks */
+       cpus_read_lock();
+       build_node_to_cpumask(node_to_cpumask);
+
+       /* grouping present CPUs first */
+       ret = __group_cpus_evenly(curgrp, numgrps, node_to_cpumask,
+                                 cpu_present_mask, nmsk, masks);
+       if (ret < 0)
+               goto fail_build_affinity;
+       nr_present = ret;
+
+       /*
+        * Allocate non present CPUs starting from the next group to be
+        * handled. If the grouping of present CPUs already exhausted the
+        * group space, assign the non present CPUs to the already
+        * allocated out groups.
+        */
+       if (nr_present >= numgrps)
+               curgrp = 0;
+       else
+               curgrp = nr_present;
+       cpumask_andnot(npresmsk, cpu_possible_mask, cpu_present_mask);
+       ret = __group_cpus_evenly(curgrp, numgrps, node_to_cpumask,
+                                 npresmsk, nmsk, masks);
+       if (ret >= 0)
+               nr_others = ret;
+
+ fail_build_affinity:
+       cpus_read_unlock();
+
+       if (ret >= 0)
+               WARN_ON(nr_present + nr_others < numgrps);
+
+ fail_node_to_cpumask:
+       free_node_to_cpumask(node_to_cpumask);
+
+ fail_npresmsk:
+       free_cpumask_var(npresmsk);
+
+ fail_nmsk:
+       free_cpumask_var(nmsk);
+       if (ret < 0) {
+               kfree(masks);
+               return NULL;
+       }
+       return masks;
+}
+#else /* CONFIG_SMP */
+struct cpumask *group_cpus_evenly(unsigned int numgrps)
+{
+       struct cpumask *masks = kcalloc(numgrps, sizeof(*masks), GFP_KERNEL);
+
+       if (!masks)
+               return NULL;
+
+       /* assign all CPUs(cpu 0) to the 1st group only */
+       cpumask_copy(&masks[0], cpu_possible_mask);
+       return masks;
+}
+#endif /* CONFIG_SMP */