On 13/11/15 15:41, Jan Beulich wrote:
> ..., i.e. namely that of a PCI serial card with an IRQ above the
> legacy range. This had got broken by the switch to cpumask_any() in
> cpu_mask_to_apicid_phys(). Fix this by allowing all CPUs for that IRQ
> (via setup_vector_irq() properly updating a booting CPU's vector_irq[],
> thus avoiding "No irq handler for vector" messages and the interrupt
> not working).
>
> Cleanup coding style and types there at once.
>
> While doing this I also noticed that io_apic_set_pci_routing() can't
> be quite right: It sets up the destination _before_ getting a vector
> allocated (which on other than systems using the flat APIC mode
> affects the possible destinations), and also didn't restrict affinity
> to ->arch.cpu_mask (as established by assign_irq_vector()).
>
> Signed-off-by: Jan Beulich <jbeul...@suse.com>

Appears to cover the issues identified from v1.

Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com>

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to