... > > /* > > diff --git a/xen/arch/arm/gic.c b/xen/arch/arm/gic.c index > > 390c8b0..e4512a8 100644 > > --- a/xen/arch/arm/gic.c > > +++ b/xen/arch/arm/gic.c > > @@ -565,12 +565,13 @@ static void do_sgi(struct cpu_user_regs *regs, > > enum gic_sgi sgi) void gic_interrupt(struct cpu_user_regs *regs, int > > is_fiq) { > > unsigned int irq; > > + unsigned int max_irq = gic_hw_ops->info->nr_lines; > > > > do { > > /* Reading IRQ will ACK it */ > > irq = gic_hw_ops->read_irq(); > > > > - if ( likely(irq >= 16 && irq < 1021) ) > > + if ( likely(irq >= 16 && irq < max_irq) ) > > { > > local_irq_enable(); > > do_IRQ(regs, irq, is_fiq); > > This change should belong to a separate patch. >
Looking at code paths and discussing with a colleague that partially wrote the patch I think this test is not necessary at all. I'll check it. Frediano _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel