From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755936AbZBIWX0 (ORCPT ); Mon, 9 Feb 2009 17:23:26 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755651AbZBIWXH (ORCPT ); Mon, 9 Feb 2009 17:23:07 -0500 Received: from fg-out-1718.google.com ([72.14.220.158]:24564 "EHLO fg-out-1718.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755580AbZBIWXF (ORCPT ); Mon, 9 Feb 2009 17:23:05 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; b=lnXfYKgWGfx/yIWWTgVhZrfuRwU3aXY5aOZguSD5VXf0DLfFnolUQZrCDmhcWbvDGt OJC85oMOi1EQsToOScVJLM2MUvNjjb/cD0mj3BNEDEoXfMMR873uf4CZcD+q6gSzeXpC kBDDXSIxITZycT0FYpKZ3nO24tPnWXP71Z8oQ= Date: Tue, 10 Feb 2009 01:29:10 +0300 From: Alexey Dobriyan To: Mike Travis Cc: linux-kernel@vger.kernel.org, Andrew Morton Subject: Re: include/linux/irq.h:452: error: implicit declaration of function 'cpu_to_node' Message-ID: <20090209222910.GA9768@x200.localdomain> References: <20090121215306.GA10123@x200.localdomain> <4977B1F1.1090602@sgi.com> <20090209203502.GA7701@x200.localdomain> <4990A899.8070302@sgi.com> <4990A96A.3060603@sgi.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4990A96A.3060603@sgi.com> User-Agent: Mutt/1.5.18 (2008-05-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Feb 09, 2009 at 02:08:42PM -0800, Mike Travis wrote: > Mike Travis wrote: > > Alexey Dobriyan wrote: > >> On Wed, Jan 21, 2009 at 03:38:25PM -0800, Mike Travis wrote: > >>> Alexey Dobriyan wrote: > >>>> On mips-malta: > >>>> > >>>> CC arch/mips/mti-malta/malta-int.o > >>>> In file included from arch/mips/mti-malta/malta-int.c:25: > >>>> include/linux/irq.h: In function 'init_alloc_desc_masks': > >>>> include/linux/irq.h:452: error: implicit declaration of function 'cpu_to_node' > >>>> include/linux/irq.h:454: error: 'GFP_ATOMIC' undeclared (first use in this function) > >>>> > >>>> I _think_ init_alloc_desc_masks() should be made out of line to fix this. > >>> Hi Alexey, > >>> > >>> Does adding #include fix the error? If so, I can add this to > >>> linux/irq.h. > >>> > >>> The primary reason it's inline is that unless CONFIG_CPUMASK_OFFSTACK=y, then the > >>> entire routine is a NOP which optimizes out more completely as an inline function. > >> linux/slab.h helps. > > > > Hi Alexey, > > > > Since you reported this problem a similar issue was discovered on s390. However > > the fix for that was to include interrupt.h instead of irq.h. > > > > Could you try this as well? > > > > Here's a tail end pointer to the thread and Andrew's commit msg including the > > patch is below. > > > > http://marc.info/?l=linux-mm-commits&m=123360349631362&w=4 > > Sorry, I should have made a patch specific to mips, and here that is... Removal helps independently. ;-) Oh and please fold all of this into relevant commit. > --- linux-2.6-linus.orig/arch/mips/mti-malta/malta-int.c > +++ linux-2.6-linus/arch/mips/mti-malta/malta-int.c > @@ -22,7 +22,7 @@ > * with two internal 82C95 interrupt controllers. > */ > #include > -#include > +#include <==== > #include > #include > #include <==== P.S.: irq related headers suck.