* [patch] hugetlbfs: fix build failure on 64-bit x86
@ 2008-07-26 9:16 Ingo Molnar
2008-07-26 9:58 ` Andrew Morton
0 siblings, 1 reply; 3+ messages in thread
From: Ingo Molnar @ 2008-07-26 9:16 UTC (permalink / raw)
To: Linus Torvalds, Andrew Morton; +Cc: linux-kernel
[resend, still relevant to -git]
>From 5215af24a20e5f448ee45cd1d97c4118b876c0f6 Mon Sep 17 00:00:00 2001
From: Ingo Molnar <mingo@elte.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@elte.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);
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [patch] hugetlbfs: fix build failure on 64-bit x86
2008-07-26 9:16 [patch] hugetlbfs: fix build failure on 64-bit x86 Ingo Molnar
@ 2008-07-26 9:58 ` Andrew Morton
2008-07-26 11:22 ` Ingo Molnar
0 siblings, 1 reply; 3+ messages in thread
From: Andrew Morton @ 2008-07-26 9:58 UTC (permalink / raw)
To: Ingo Molnar; +Cc: Linus Torvalds, linux-kernel
On Sat, 26 Jul 2008 11:16:54 +0200 Ingo Molnar <mingo@elte.hu> wrote:
>
> [resend, still relevant to -git]
>
> >From 5215af24a20e5f448ee45cd1d97c4118b876c0f6 Mon Sep 17 00:00:00 2001
> From: Ingo Molnar <mingo@elte.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@elte.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:
From: Nishanth Aravamudan <nacc@us.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'
Also reverts Ingo's
commit e44d1b2998d62a1f2f4d7eb17b56ba396535509f
Author: Ingo Molnar <mingo@elte.hu>
Date: Fri Jul 25 12:57:41 2008 +0200
mm/hugetlb.c: fix build failure with !CONFIG_SYSCTL
which fixed the build error but added some unused-static-function warnings.
Signed-off-by: Nishanth Aravamudan <nacc@us.ibm.com>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---
mm/hugetlb.c | 24 ++++++++++++------------
1 file changed, 12 insertions(+), 12 deletions(-)
diff -puN mm/hugetlb.c~hugetlb-fix-config_sysctl=n-build mm/hugetlb.c
--- a/mm/hugetlb.c~hugetlb-fix-config_sysctl=n-build
+++ a/mm/hugetlb.c
@@ -1026,18 +1026,6 @@ static void __init report_hugepages(void
}
}
-static unsigned int cpuset_mems_nr(unsigned int *array)
-{
- int node;
- unsigned int nr = 0;
-
- for_each_node_mask(node, cpuset_current_mems_allowed)
- nr += array[node];
-
- return nr;
-}
-
-#ifdef CONFIG_SYSCTL
#ifdef CONFIG_HIGHMEM
static void try_to_free_low(struct hstate *h, unsigned long count)
{
@@ -1386,6 +1374,18 @@ static int __init hugetlb_default_setup(
}
__setup("default_hugepagesz=", hugetlb_default_setup);
+static unsigned int cpuset_mems_nr(unsigned int *array)
+{
+ int node;
+ unsigned int nr = 0;
+
+ for_each_node_mask(node, cpuset_current_mems_allowed)
+ nr += array[node];
+
+ 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)
_
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [patch] hugetlbfs: fix build failure on 64-bit x86
2008-07-26 9:58 ` Andrew Morton
@ 2008-07-26 11:22 ` Ingo Molnar
0 siblings, 0 replies; 3+ messages in thread
From: Ingo Molnar @ 2008-07-26 11:22 UTC (permalink / raw)
To: Andrew Morton; +Cc: Linus Torvalds, linux-kernel
* Andrew Morton <akpm@linux-foundation.org> wrote:
> On Sat, 26 Jul 2008 11:16:54 +0200 Ingo Molnar <mingo@elte.hu> wrote:
>
> >
> > [resend, still relevant to -git]
> >
> > >From 5215af24a20e5f448ee45cd1d97c4118b876c0f6 Mon Sep 17 00:00:00 2001
> > From: Ingo Molnar <mingo@elte.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@elte.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@us.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@elte.hu>
Ingo
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2008-07-26 11:22 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-07-26 9:16 [patch] hugetlbfs: fix build failure on 64-bit x86 Ingo Molnar
2008-07-26 9:58 ` Andrew Morton
2008-07-26 11:22 ` Ingo Molnar
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).