2002-11-27 22:20:59

by Kevin Brosius

[permalink] [raw]
Subject: hugetlbpage.c build failure?

I see the following build failure in bk current:

gcc -Wp,-MD,arch/i386/mm/.hugetlbpage.o.d -D__KERNEL__ -Iinclude -Wall
-Wstric
t-prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common
-fomit-frame-po
inter -pipe -g -mpreferred-stack-boundary=2 -march=athlon
-Iarch/i386/mach-gener
ic -nostdinc -iwithprefix include -DKBUILD_BASENAME=hugetlbpage
-DKBUILD_MODN
AME=hugetlbpage -c -o arch/i386/mm/hugetlbpage.o
arch/i386/mm/hugetlbpage.c


arch/i386/mm/hugetlbpage.c:610: parse error before '*' token
arch/i386/mm/hugetlbpage.c: In function `hugetlb_sysctl_handler':
arch/i386/mm/hugetlbpage.c:611: number of arguments doesn't match
prototype
include/linux/hugetlb.h:14: prototype declaration
arch/i386/mm/hugetlbpage.c:612: warning: implicit declaration of
function `proc_
dointvec'
arch/i386/mm/hugetlbpage.c:612: `table' undeclared (first use in this
function)
arch/i386/mm/hugetlbpage.c:612: (Each undeclared identifier is reported
only onc
e
arch/i386/mm/hugetlbpage.c:612: for each function it appears in.)
arch/i386/mm/hugetlbpage.c:612: `write' undeclared (first use in this
function)
arch/i386/mm/hugetlbpage.c:612: `file' undeclared (first use in this
function)
arch/i386/mm/hugetlbpage.c:612: `buffer' undeclared (first use in this
function)
arch/i386/mm/hugetlbpage.c:612: `length' undeclared (first use in this
function)
make[1]: *** [arch/i386/mm/hugetlbpage.o] Error 1
make: *** [arch/i386/mm] Error 2


--
Kevin


2002-11-27 22:45:04

by Greg KH

[permalink] [raw]
Subject: Re: hugetlbpage.c build failure?

On Wed, Nov 27, 2002 at 05:28:18PM -0500, Kevin Brosius wrote:
> I see the following build failure in bk current:

If you get those problems fixed, you're going to need this patch too :)

greg k-h

--- 1.4/fs/hugetlbfs/inode.c Wed Nov 20 03:19:02 2002
+++ edited/fs/hugetlbfs/inode.c Wed Nov 27 14:27:02 2002
@@ -209,7 +209,7 @@
if (inode->i_data.nrpages)
truncate_hugepages(&inode->i_data, 0);

- security_ops->inode_delete(inode);
+ security_inode_delete(inode);

clear_inode(inode);
destroy_inode(inode);
@@ -333,7 +333,7 @@
if (error)
goto out;

- error = security_ops->inode_setattr(dentry, attr);
+ error = security_inode_setattr(dentry, attr);
if (error)
goto out;


2002-11-27 22:54:27

by Arnd Bergmann

[permalink] [raw]
Subject: Re: hugetlbpage.c build failure?

Kevin Brosius wrote:
> arch/i386/mm/hugetlbpage.c:610: parse error before '*' token

The patch below fixed this for me

===== arch/i386/mm/hugetlbpage.c 1.17 vs edited =====
--- 1.17/arch/i386/mm/hugetlbpage.c Thu Nov 14 23:03:02 2002
+++ edited/arch/i386/mm/hugetlbpage.c Wed Nov 27 19:18:23 2002
@@ -14,6 +14,7 @@
#include <linux/slab.h>
#include <linux/module.h>
#include <linux/err.h>
+#include <linux/sysctl.h>
#include <asm/mman.h>
#include <asm/pgalloc.h>

2002-11-27 23:32:04

by Kevin Brosius

[permalink] [raw]
Subject: Re: hugetlbpage.c build failure?

Arnd Bergmann wrote:
>
>
> Kevin Brosius wrote:
> > arch/i386/mm/hugetlbpage.c:610: parse error before '*' token
>
> The patch below fixed this for me
>
> ===== arch/i386/mm/hugetlbpage.c 1.17 vs edited =====
> --- 1.17/arch/i386/mm/hugetlbpage.c Thu Nov 14 23:03:02 2002
> +++ edited/arch/i386/mm/hugetlbpage.c Wed Nov 27 19:18:23 2002
> @@ -14,6 +14,7 @@
> #include <linux/slab.h>
> #include <linux/module.h>
> #include <linux/err.h>
> +#include <linux/sysctl.h>
> #include <asm/mman.h>
> #include <asm/pgalloc.h>

Thanks guys, that does it here. Greg, was yours a run time fix? I don't
see any difference during build.

--
Kevin

2002-11-28 00:42:22

by Greg KH

[permalink] [raw]
Subject: Re: hugetlbpage.c build failure?

On Wed, Nov 27, 2002 at 06:39:24PM -0500, Kevin Brosius wrote:
> Arnd Bergmann wrote:
> >
> >
> > Kevin Brosius wrote:
> > > arch/i386/mm/hugetlbpage.c:610: parse error before '*' token
> >
> > The patch below fixed this for me
> >
> > ===== arch/i386/mm/hugetlbpage.c 1.17 vs edited =====
> > --- 1.17/arch/i386/mm/hugetlbpage.c Thu Nov 14 23:03:02 2002
> > +++ edited/arch/i386/mm/hugetlbpage.c Wed Nov 27 19:18:23 2002
> > @@ -14,6 +14,7 @@
> > #include <linux/slab.h>
> > #include <linux/module.h>
> > #include <linux/err.h>
> > +#include <linux/sysctl.h>
> > #include <asm/mman.h>
> > #include <asm/pgalloc.h>
>
> Thanks guys, that does it here. Greg, was yours a run time fix? I don't
> see any difference during build.

Should be needed during building, but I don't know the .config options
that are necessary for it to be needed, sorry.

greg k-h