linux-kernel - Re: [patch] hugetlbfs: fix build failure on 64-bit x86

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20080726112212.GC11591@elte.hu>
Date:	Sat, 26 Jul 2008 13:22:12 +0200
From:	Ingo Molnar <mingo@...e.hu>
To:	Andrew Morton <akpm@...ux-foundation.org>
Cc:	Linus Torvalds <torvalds@...ux-foundation.org>,
	linux-kernel@...r.kernel.org
Subject: Re: [patch] hugetlbfs: fix build failure on 64-bit x86


* Andrew Morton <akpm@...ux-foundation.org> wrote:

> On Sat, 26 Jul 2008 11:16:54 +0200 Ingo Molnar <mingo@...e.hu> wrote:
> 
> > 
> > [resend, still relevant to -git]
> > 
> > >From 5215af24a20e5f448ee45cd1d97c4118b876c0f6 Mon Sep 17 00:00:00 2001
> > From: Ingo Molnar <mingo@...e.hu>
> > Date: Fri, 25 Jul 2008 20:45:52 +0200
> > Subject: [PATCH] hugetlbfs: fix build failure on 64-bit x86
> > 
> > fix:
> > 
> >  arch/x86/mm/built-in.o: In function `setup_hugepagesz':
> >  hugetlbpage.c:(.init.text+0xb49): undefined reference to `hugetlb_add_hstate'
> > 
> > Signed-off-by: Ingo Molnar <mingo@...e.hu>
> > ---
> >  arch/x86/mm/hugetlbpage.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/arch/x86/mm/hugetlbpage.c b/arch/x86/mm/hugetlbpage.c
> > index 8f307d9..977d185 100644
> > --- a/arch/x86/mm/hugetlbpage.c
> > +++ b/arch/x86/mm/hugetlbpage.c
> > @@ -425,7 +425,7 @@ hugetlb_get_unmapped_area(struct file *file, unsigned long addr,
> >  
> >  #endif /*HAVE_ARCH_HUGETLB_UNMAPPED_AREA*/
> >  
> > -#ifdef CONFIG_X86_64
> > +#if defined(CONFIG_X86_64) && defined(CONFIG_SYSCTL)
> >  static __init int setup_hugepagesz(char *opt)
> >  {
> >  	unsigned long ps = memparse(opt, &opt);
> 
> Seems wrong that a __setup handler would vanish if CONFIG_SYSCTL=n.
> 
> I think Nishanth's patch (sent to Linus a few hours ago) fixes this
> correctly, by making hugetlb_add_hstate available if CONFIG_SYSCTL=n:

agreed, that's cleaner/better.

> From: Nishanth Aravamudan <nacc@...ibm.com>
> 
> 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'

Acked-by: Ingo Molnar <mingo@...e.hu>

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/