Re: Linux 2.4.20-pre2 compile error

[Stephane Wirtel <stephane.wirtel@belgacom.net>]

are you sure that the "int number" is employed only by devfs?

On mar, 13 aoû 2002, Adrian Bunk wrote:
> On Tue, 13 Aug 2002, Stephane Wirtel wrote:
> 
> > are you sure about the patch ?
> 
> It seems to be correct and I can verify both the compile error and that
> this patch fixes it. What do you consider to be wrong?
> 
> > best regards
> 
> cu
> Adrian
> 
> > On mar, 13 aoû 2002, Adrian Bunk wrote:
> > > On Tue, 13 Aug 2002, Chad Young wrote:
> > >
> > > > any idea what causes these errors?
> > > >
> > > > make[3]: Entering directory
> > > > `/home/skidley/kernel/linux-2.4.20-pre2/fs/partitions'
> > > > gcc -D__KERNEL__ -I/home/skidley/kernel/linux-2.4.20-pre2/include -Wall
> > > > -Wstrict-prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common
> > > > -fomit-frame-pointer -pipe -mpreferred-stack-boundary=2 -march=i686
> > > > -nostdinc -I /usr/lib/gcc-lib/i386-linux/2.95.4/include
> > > > -DKBUILD_BASENAME=check  -DEXPORT_SYMTAB -c check.c
> > > > check.c: In function `devfs_register_disc':
> > > > check.c:328: structure has no member named `number'
> > > > check.c:329: structure has no member named `number'
> > > > check.c: In function `devfs_register_partitions':
> > > > check.c:361: structure has no member named `number'
> > > >...
> > >
> > > The following patch made by Christoph Hellwig fixes it:
> > >
> > >
> > > --- linux-2.4.20-bk-20020810/include/linux/genhd.h	Sat Aug 10 14:37:16 2002
> > > +++ linux/include/linux/genhd.h	Mon Aug 12 23:40:37 2002
> > > @@ -62,7 +62,9 @@ struct hd_struct {
> > >  	unsigned long start_sect;
> > >  	unsigned long nr_sects;
> > >  	devfs_handle_t de;              /* primary (master) devfs entry  */
> > > -
> > > +#ifdef CONFIG_DEVFS_FS
> > > +	int number;
> > > +#endif /* CONFIG_DEVFS_FS */
> > >  #ifdef CONFIG_BLK_STATS
> > >  	/* Performance stats: */
> > >  	unsigned int ios_in_flight;
> > >
> 
> -
> 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/
> 

-- 
Stephane Wirtel <stephane.wirtel@belgacom.net>
Web : www.linux-mons.be	 "Linux Is Not UniX !!!"