On Mon, Oct 05 2020 at 16:28, David Woodhouse wrote: > > #else /* CONFIG_SMP */ > > +#define irq_default_affinity (NULL) ... > static int alloc_descs(unsigned int start, unsigned int cnt, int node, > const struct irq_affinity_desc *affinity, > + const struct cpumask *default_affinity, > struct module *owner) > { > struct irq_desc *desc; > int i; > > /* Validate affinity mask(s) */ > + if (!default_affinity || cpumask_empty(default_affinity)) > + return -EINVAL; How is that supposed to work on UP? Aside of that I really hate to have yet another argument just because. Thanks, tglx