Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754980AbYGYUeK (ORCPT ); Fri, 25 Jul 2008 16:34:10 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751436AbYGYUd5 (ORCPT ); Fri, 25 Jul 2008 16:33:57 -0400 Received: from e34.co.us.ibm.com ([32.97.110.152]:45457 "EHLO e34.co.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751223AbYGYUd4 (ORCPT ); Fri, 25 Jul 2008 16:33:56 -0400 Date: Fri, 25 Jul 2008 13:33:51 -0700 From: Nishanth Aravamudan To: Alan Cox Cc: akpm@linux-foundation.org, torvalds@linux-foundation.org, mingo@elte.hu, randy.dunlap@oracle.com, linux-kernel@vger.kernel.org Subject: [PATCH RESEND] hugetlb: fix CONFIG_SYSCTL=n build Message-ID: <20080725203351.GA18407@us.ibm.com> References: <20080721.233512.128618365.anemo@mba.ocn.ne.jp> <20080721191519.GA15843@elte.hu> <20080721214258.787a67ca@lxorguk.ukuu.org.uk> <20080722072334.GA25241@elte.hu> <48865899.3030209@garzik.org> <20080723104520.GA20672@elte.hu> <20080724052603.GA8744@elte.hu> <20080725180935.5bd02bf5@lxorguk.ukuu.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20080725180935.5bd02bf5@lxorguk.ukuu.org.uk> User-Agent: Mutt/1.5.17+20080114 (2008-01-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2075 Lines: 60 [resending with Linus' address fixed] On 25.07.2008 [18:09:35 +0100], Alan Cox wrote: > On Thu, 24 Jul 2008 07:26:03 +0200 > Ingo Molnar wrote: > > > > > not sure it's known but despite all the fixes i still get: > > > > drivers/built-in.o: In function `ne_drv_resume': > > ne.c:(.text+0x146ba4): undefined reference to `NS8390_init' > > drivers/built-in.o: In function `ne_block_output': > > ne.c:(.text+0x146e00): undefined reference to `NS8390_init' > > This should do the trick. I can't test it currently as that configuration > now fails to build with mm errors in current git head.. > > mm/hugetlb.c: In function ???hugetlb_acct_memory???: mm/hugetlb.c:1507: > error: implicit declaration of function ???cpuset_mems_nr??? When moving the intialization functions around, I did not remove the dependency of some helpers on CONFIG_SYSCTL. This led to a large chunk of code being elided even though it is needed to build hugetlb.o. Fixes a build failure reported by Alan Cox: mm/hugetlb.c: In function ‘hugetlb_acct_memory’: mm/hugetlb.c:1507: error: implicit declaration of function ‘cpuset_mems_nr’ Signed-off-by: Nishanth Aravamudan diff --git a/mm/hugetlb.c b/mm/hugetlb.c index 41341c4..3be79dc 100644 --- a/mm/hugetlb.c +++ b/mm/hugetlb.c @@ -1026,7 +1026,6 @@ static void __init report_hugepages(void) } } -#ifdef CONFIG_SYSCTL #ifdef CONFIG_HIGHMEM static void try_to_free_low(struct hstate *h, unsigned long count) { @@ -1386,6 +1385,7 @@ static unsigned int cpuset_mems_nr(unsigned int *array) return nr; } +#ifdef CONFIG_SYSCTL int hugetlb_sysctl_handler(struct ctl_table *table, int write, struct file *file, void __user *buffer, size_t *length, loff_t *ppos) -- Nishanth Aravamudan IBM Linux Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/