Re: test13-pre4

From: Andrea Arcangeli (andrea@suse.de)
Date: Fri Dec 22 2000 - 10:24:41 EST


On Fri, Dec 22, 2000 at 04:01:50PM +0100, Andrea Arcangeli wrote:
> On Fri, Dec 22, 2000 at 03:31:45PM +0100, Andrea Arcangeli wrote:
> > On Sat, Dec 23, 2000 at 12:00:27AM +1100, Eyal Lebedinsky wrote:
> > > Linus Torvalds wrote:
> > > > - pre4:
> > > > - Andrea Arkangeli: update to LVM-0.9
> > >
> > > gcc -D__KERNEL__ -I/usr/local/src/linux/include -Wall
> > > -Wstrict-prototypes -O2 -fomit-frame-pointer -fno-strict-aliasing -pipe
> > > -mpreferred-stack-boundary=2 -march=i686 -malign-functions=4 -DMODULE
> > > -DMODVERSIONS -include
> > > /usr/local/src/linux/include/linux/modversions.h -c -o lvm.o lvm.c
> > > lvm.c: In function `lvm_do_vg_extend':
> > > lvm.c:2024: warning: implicit declaration of function
> > > `lvm_do_create_proc_entry_of_pv'
> > > lvm.c: In function `lvm_do_create_proc_entry_of_lv':
> > > lvm.c:3016: `pde' undeclared (first use in this function)
> > > lvm.c:3016: (Each undeclared identifier is reported only once
> > > lvm.c:3016: for each function it appears in.)
> > > lvm.c: At top level:
> > > lvm.c:3044: warning: type mismatch with previous implicit declaration
> > > lvm.c:2024: warning: previous implicit declaration of
> > > `lvm_do_create_proc_entry_of_pv'
> > > lvm.c:3044: warning: `lvm_do_create_proc_entry_of_pv' was previously
> > > implicitly declared to return `int'
> > > lvm.c: In function `lvm_do_create_proc_entry_of_pv':
> > > lvm.c:3050: `pde' undeclared (first use in this function)
> > > lvm.c: At top level:
> > > lvm.c:147: warning: `lvm_short_version' defined but not used
> > > make[2]: *** [lvm.o] Error 1
> > > make[2]: Leaving directory `/data2/usr/local/src/linux-2.4/drivers/md'
> >
> > Strange, test13-pre3 plus the 0.9 lvm patch compiled and worked fine
> > for me. I'll try to compile test13-pre4 now and I'll let you know.
>
> Ok, found it, you can workaround it with:
>
> CONFIG_LVM_PROC_FS=y
>
> (I never tried to compile LVM without procfs support, you really
> want /proc support too, anyways of course it should compile also
> without /proc support so it's a minor bug...)

Ok this should fix the problem and it cleanups a little bit the code:

        ftp://ftp.us.kernel.org/pub/linux/kernel/people/andrea/patches/v2.4/2.4.0-test13-pre4/lvm-proc-fixes-1

Please confirm and then I'll sumbit it to Linus.

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



This archive was generated by hypermail 2b29 : Sat Dec 23 2000 - 21:00:31 EST