From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756174Ab2DFLM6 (ORCPT ); Fri, 6 Apr 2012 07:12:58 -0400 Received: from www.linutronix.de ([62.245.132.108]:38249 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755820Ab2DFLM5 (ORCPT ); Fri, 6 Apr 2012 07:12:57 -0400 Date: Fri, 6 Apr 2012 13:12:52 +0200 (CEST) From: Thomas Gleixner To: Andreas Schwab cc: Grant Likely , linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, devicetree-discuss@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org, Benjamin Herrenschmidt , Milton Miller , Rob Herring Subject: Re: [PATCH v5 06/27] irq_domain/powerpc: eliminate irq_map; use irq_alloc_desc() instead In-Reply-To: Message-ID: References: <1329383368-12122-1-git-send-email-grant.likely@secretlab.ca> <1329383368-12122-7-git-send-email-grant.likely@secretlab.ca> <20120402202848.1EB833E02A7@localhost> <20120404154020.40C513E09D5@localhost> User-Agent: Alpine 2.02 (LFD 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-Linutronix-Spam-Score: -1.0 X-Linutronix-Spam-Level: - X-Linutronix-Spam-Status: No , -1.0 points, 5.0 required, ALL_TRUSTED=-1,SHORTCIRCUIT=-0.0001 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 5 Apr 2012, Andreas Schwab wrote: > Grant Likely writes: > > > I bet it is NR_IRQS related. You have SPARSE_IRQ enabled, which means > > the maximum number of irq_descs is IRQ_BITMAP_BITS (NR_IRQS + 8192). > > The actual definition uses NR_IRQS + 8196. Guess that's a typo. (Does > it really make sense to add NR_IRQS here?) > > > diff --git a/arch/powerpc/include/asm/irq.h b/arch/powerpc/include/asm/irq.h > > index cf417e51..9edf499 100644 > > --- a/arch/powerpc/include/asm/irq.h > > +++ b/arch/powerpc/include/asm/irq.h > > @@ -20,7 +20,7 @@ > > > > /* Define a way to iterate across irqs. */ > > #define for_each_irq(i) \ > > - for ((i) = 0; (i) < NR_IRQS; ++(i)) > > + for ((i) = 0; (i) < nr_irqs; ++(i)) > > There are exactly two uses of for_each_irq, one is related to cpu > hotplug, the other to kexec, so that cannot make any difference. Though that wants to be fixed nevertheless. Thanks, tglx