[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Xen-devel] [PATCH v3] x86: Fixup IRQs when CPUs go down during shutdown
On 12/10/2015 09:28 AM, Jan Beulich wrote:
On 10.12.15 at 10:21, <ross.lagerwall@xxxxxxxxxx> wrote:
On 12/04/2015 02:42 PM, Jan Beulich wrote:
On 04.12.15 at 15:01, <ross.lagerwall@xxxxxxxxxx> wrote:
Commit fc0c3fa2ad5c ("x86/IO-APIC: fix setup of Xen internally used IRQs
(take 2)") introduced a regression on some hardware where Xen would hang
during shutdown, repeating the following message:
APIC error on CPU0: 08(08), Receive accept error
This appears to be because an interrupt (in this case from the serial
console) destined for a CPU other than the boot CPU is left unhandled so
an APIC error on CPU 0 is generated instead.
To fix this, before taking down the non-boot CPUs, call fixup_irqs()
with a CPU mask of only the boot CPU to reset the IRQ affinities
correctly.
Signed-off-by: Ross Lagerwall <ross.lagerwall@xxxxxxxxxx>
Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
Is this going to go onto staging?
I really was waiting for some kind of response to
... I really would have wanted the split of the functions to be
undone too (renaming the bool_t function parameter suitably).
in the mail that you just replied to (but stripped those parts off).
Oh, given the Reviewed-by tag, I thought you weren't expecting a
response. Anyway, I prefer the version split into two functions since it
is doing two logically separate tasks. I prefer having (arguably) better
code than having less code churn and a smaller patch size.
--
Ross Lagerwall
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel
|
