diff options
author | Boris Ostrovsky <boris.ostrovsky@oracle.com> | 2016-08-03 19:22:28 +0200 |
---|---|---|
committer | David Vrabel <david.vrabel@citrix.com> | 2016-08-24 19:44:46 +0200 |
commit | e1c105a9d75a1e9a44eb2f0b10204a6492057e4d (patch) | |
tree | 53b677f3c724f6c770c62fbb1ff9cb44e2eee837 /kernel/cpu.c | |
parent | xen/x86: Move irq allocation from Xen smp_op.cpu_up() (diff) | |
download | linux-e1c105a9d75a1e9a44eb2f0b10204a6492057e4d.tar.xz linux-e1c105a9d75a1e9a44eb2f0b10204a6492057e4d.zip |
hotplug: Prevent alloc/free of irq descriptors during cpu up/down (again)
Now that Xen no longer allocates irqs in _cpu_up() we can restore
commit a89941816726 ("hotplug: Prevent alloc/free of irq descriptors
during cpu up/down")
Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Acked-by: Thomas Gleixner <tglx@linutronix.de>
CC: x86@kernel.org
CC: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: David Vrabel <david.vrabel@citrix.com>
Diffstat (limited to '')
-rw-r--r-- | kernel/cpu.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/kernel/cpu.c b/kernel/cpu.c index 341bf80f80bd..ec12b726fa6f 100644 --- a/kernel/cpu.c +++ b/kernel/cpu.c @@ -349,8 +349,16 @@ static int bringup_cpu(unsigned int cpu) struct task_struct *idle = idle_thread_get(cpu); int ret; + /* + * Some architectures have to walk the irq descriptors to + * setup the vector space for the cpu which comes online. + * Prevent irq alloc/free across the bringup. + */ + irq_lock_sparse(); + /* Arch-specific enabling code. */ ret = __cpu_up(cpu, idle); + irq_unlock_sparse(); if (ret) { cpu_notify(CPU_UP_CANCELED, cpu); return ret; |