Hi, On Fri, 2006-04-21 at 18:43 +0200, Adrian Bunk wrote: > On Fri, Apr 21, 2006 at 05:22:38PM +0100, Steven Whitehouse wrote: > > >... > > --- a/fs/Kconfig > > +++ b/fs/Kconfig > >... > > > > source "fs/nls/Kconfig" > > +source "fs/dlm/Kconfig" > >... > > File doesn't exist > > > --- a/fs/Makefile > > +++ b/fs/Makefile > > @@ -48,6 +48,7 @@ obj-$(CONFIG_SYSFS) += sysfs/ > > obj-y += devpts/ > > > > obj-$(CONFIG_PROFILING) += dcookies.o > > +obj-$(CONFIG_DLM) += dlm/ > >... > > Directory doesn't exist. > The above two anomalies are due to the git tree also containing the DLM (already in -mm) and I forgot to redo this patch which is the only one which overlaps with the DLM code. I expect that Andrew Morton will update the version of the DLM in -mm at the same time as taking GFS2 when it is deemed ready to go in. > > --- /dev/null > > +++ b/fs/gfs2/Kconfig > > @@ -0,0 +1,46 @@ > > +config GFS2_FS > > + tristate "GFS2 file system support" > > + default m > > + depends on EXPERIMENTAL > > + select FS_POSIX_ACL > > + select SYSFS > >... > > - tabs <-> spaces (tabs are correct) > - please remove the "default m" These two are done in: http://www.kernel.org/git/?p=linux/kernel/git/steve/gfs2-2.6.git;a=commitdiff;h=b5ea3e1ef307548bdd40fff6aba5fc96b002f284 > - "depends on SYSFS" instead of the select > This has been resolved by removing the dependency completely. > > +config GFS2_FS_LOCKING_DLM > > + tristate "GFS2 DLM locking module" > > + depends on GFS2_FS > > + select DLM > >... > > DLM in -mm depends on IPV6 || IPV6=n, so if you are select'ing it you > have to add this dependency to GFS2_FS_LOCKING_DLM. > > cu > Adrian > I will check this with the DLM developers and see what the best solution is. Thanks for the feedback, Steve. - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html