github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-11-07 06:17:02 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

genirq: Discard cpumask of irqaffinity= boot parameter if parsing fails

Browse Source 
irq_affinity_setup() ignores the return value of cpulist_parse(). This
can result in using a "half parsed" cpulist. E.g. on a system with 8
cores, the parameter "irqaffinity=0,1,6-10,4" would result in using only
CPU 0 and 1, since the parsing fails at 6-10. Other boot parameters
which take a cpulist (e.g. isolcpus=, workqueue.unbound_cpus=) already
handle this by falling back to a default.

Check the return value of cpulist_parse() and let
init_irq_default_affinity() set the default in early_irq_init()
later, if parsing fails.

Signed-off-by: Lukas Beckmann <lbckmnn@mailbox.org>
Signed-off-by: Thomas Gleixner <tglx@kernel.org>
Reviewed-by: Radu Rendec <radu@rendec.net>
Link: https://patch.msgid.link/20260822160502.4665-1-lbckmnn@mailbox.org
This commit is contained in:
Lukas Beckmann 2026-08-22 18:05:00 +02:00 committed by Thomas Gleixner
parent cee9395acd
commit 94235414a0
1 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
kernel/irq/irqdesc.c
View File

@ -28,7 +28,15 @@ static struct lock_class_key irq_desc_lock_class;
static int __init irq_affinity_setup(char *str)
{
alloc_bootmem_cpumask_var(&irq_default_affinity);
cpulist_parse(str, irq_default_affinity);
if (cpulist_parse(str, irq_default_affinity) < 0) {
/*
* Clear the mask so that init_irq_default_affinity()
* defaults it later.
*/
cpumask_clear(irq_default_affinity);
pr_warn("irqaffinity: incorrect CPU range, using default\n");
return 1;
}
/*
* Set at least the boot cpu. We don't want to end up with
* bugreports caused by random commandline masks