[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[xen master] x86/irq: fix reporting of spurious i8259 interrupts



commit 709f6c8ce6422475c372e67507606170a31ccb65
Author:     Roger Pau Monné <roger.pau@xxxxxxxxxx>
AuthorDate: Wed Aug 30 10:03:53 2023 +0200
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Wed Aug 30 10:03:53 2023 +0200

    x86/irq: fix reporting of spurious i8259 interrupts
    
    The return value of bogus_8259A_irq() is wrong: the function will
    return `true` when the IRQ is real and `false` when it's a spurious
    IRQ.  This causes the "No irq handler for vector ..." message in
    do_IRQ() to be printed for spurious i8259 interrupts which is not
    intended (and not helpful).
    
    Fix by inverting the return value of bogus_8259A_irq().
    
    Fixes: 132906348a14 ('x86/i8259: Handle bogus spurious interrupts more 
quietly')
    Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
    Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
---
 xen/arch/x86/i8259.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/arch/x86/i8259.c b/xen/arch/x86/i8259.c
index 6b35be10f0..ed9f55abe5 100644
--- a/xen/arch/x86/i8259.c
+++ b/xen/arch/x86/i8259.c
@@ -37,7 +37,7 @@ static bool _mask_and_ack_8259A_irq(unsigned int irq);
 
 bool bogus_8259A_irq(unsigned int irq)
 {
-    return _mask_and_ack_8259A_irq(irq);
+    return !_mask_and_ack_8259A_irq(irq);
 }
 
 static void cf_check mask_and_ack_8259A_irq(struct irq_desc *desc)
--
generated by git-patchbot for /home/xen/git/xen.git#master



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.