Re: [Xen-devel] [PATCH] xen/arm: gic: Correctly check if an IRQ is valid in gic_interrupt

2015-03-05 Thread Ian Campbell 
On Tue, 2015-03-03 at 16:36 +, Julien Grall wrote:
 The IRQ 1020 is reserved for special purpose, so the check should be
 irq  1020 rather than irq  1021.
 
 Signed-off-by: Julien Grall julien.gr...@linaro.org

Acked + applied, thanks.



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

[Xen-devel] [PATCH] xen/arm: gic: Correctly check if an IRQ is valid in gic_interrupt

2015-03-03 Thread Julien Grall 
The IRQ 1020 is reserved for special purpose, so the check should be
irq  1020 rather than irq  1021.

Signed-off-by: Julien Grall julien.gr...@linaro.org
---
 xen/arch/arm/gic.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/arch/arm/gic.c b/xen/arch/arm/gic.c
index 390c8b0..940fb8e 100644
--- a/xen/arch/arm/gic.c
+++ b/xen/arch/arm/gic.c
@@ -570,7 +570,7 @@ void gic_interrupt(struct cpu_user_regs *regs, int is_fiq)
 /* Reading IRQ will ACK it */
 irq = gic_hw_ops-read_irq();
 
-if ( likely(irq = 16  irq  1021) )
+if ( likely(irq = 16  irq  1020) )
 {
 local_irq_enable();
 do_IRQ(regs, irq, is_fiq);
-- 
2.1.4


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