Re: [PATCH 2/7] genirq/affinity: assign vectors to all present CPUs

2017-05-23 Thread Christoph Hellwig
On Sun, May 21, 2017 at 08:31:47PM +0200, Thomas Gleixner wrote: > On Fri, 19 May 2017, Christoph Hellwig wrote: > > - /* Stabilize the cpumasks */ > > - get_online_cpus(); > > How is that protected against physical CPU hotplug? Physical CPU hotplug > manipulates the present mask. It does

Re: [PATCH 2/7] genirq/affinity: assign vectors to all present CPUs

2017-05-21 Thread Thomas Gleixner
On Fri, 19 May 2017, Christoph Hellwig wrote: > - /* Stabilize the cpumasks */ > - get_online_cpus(); How is that protected against physical CPU hotplug? Physical CPU hotplug manipulates the present mask. > - nodes = get_nodes_in_cpumask(cpu_online_mask, ); > + nodes =

[PATCH 2/7] genirq/affinity: assign vectors to all present CPUs

2017-05-19 Thread Christoph Hellwig
Currently we only assign spread vectors to online CPUs, which ties the IRQ mapping to the currently online devices and doesn't deal nicely with the fact that CPUs could come and go rapidly due to e.g. power management. Instead assign vectors to all present CPUs to avoid this churn.