Re: [PATCH -next] xfs: eliminate kconfig dependency warning

[Dave Chinner]

On Wed, Oct 13, 2010 at 09:01:42PM -0700, Randy Dunlap wrote:
> From: Randy Dunlap <randy.dunlap@xxxxxxxxxx>
> 
> Fix kconfig dependency warning to satisfy dependencies:
> 
> warning: (XFS_FS && BLOCK || NFSD && NETWORK_FILESYSTEMS && INET && FILE_LOCKING && BKL) selects EXPORTFS which has unmet direct dependencies (NETWORK_FILESYSTEMS)
> 
> Signed-off-by: Randy Dunlap <randy.dunlap@xxxxxxxxxx>
> ---
>  fs/xfs/Kconfig |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> --- linux-next-20101013.orig/fs/xfs/Kconfig
> +++ linux-next-20101013/fs/xfs/Kconfig
> @@ -1,6 +1,6 @@
>  config XFS_FS
>  	tristate "XFS filesystem support"
> -	depends on BLOCK
> +	depends on BLOCK && NETWORK_FILESYSTEMS
>  	select EXPORTFS
>  	help
>  	  XFS is a high performance journaling filesystem which originated

This looks wrong to me.  IIUC, this means if you turn of network
filesystem support (e.g no NFS, CIFS, etc), you can't select XFS
even though it is a local filesystem.

EXPORTFS functionality actually has use for local filesystems - the
open-by-handle interfaces in XFS are what uses it, and there are patches
about that make this generic for most filesystems. Hence it seems to
me that the correct thing to do is separate EXPORTFS from
NETWORK_FILESYSTEMS, not make local filesystems depend on a user
selecting network filesystem support....

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/