[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH 5/7] xen/arm: Fix assert in send_SGI_one
On Fri, 2013-08-30 at 14:30 +0100, Julien Grall wrote: > The GIC can handle maximum 8 cpus (0...7). The CPU id 7 is still valid. > > Signed-off-by: Julien Grall <julien.grall@xxxxxxxxxx> > --- > 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 4f3a8a5..6dcefd7 100644 > --- a/xen/arch/arm/gic.c > +++ b/xen/arch/arm/gic.c > @@ -481,7 +481,7 @@ void send_SGI_mask(const cpumask_t *cpumask, enum gic_sgi > sgi) > > void send_SGI_one(unsigned int cpu, enum gic_sgi sgi) > { > - ASSERT(cpu < 7); /* Targets bitmap only supports 8 CPUs */ > + ASSERT(cpu < NR_GIC_CPU_IF); /* Targets bitmap only supports 8 CPUs */ cpu here is a logical cpu id, not a gic id, I think? In which case although 8 happens to be correct: NR_GIC_CPU_IF is the wrong thing to be using and in any case this seems against the general principals of this series. > send_SGI_mask(cpumask_of(cpu), sgi); > } > _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |