[Linux-cachefs] [Lists-linux-kernel-news] Re: [PATCH] NFS using CacheFS
Trond Myklebust
trond.myklebust at fys.uio.no
Mon Oct 4 21:53:57 UTC 2004
På må , 04/10/2004 klokka 22:45, skreiv Steve Dickson:
> 3) There is no user level support. I realize this is extremely cheesy
> but I noticed that the NFS posix mount option (in the 2.6 kernel)
> was no longer being used, so I high jacked it. Which means
> to make NFS to used CacheFS you need to use the posix option:
>
> mount -o posix server:/export/home /mnt/server/home
This is my one and only real gripe about it. The posix mount option is
clearly documented, so we really cannot play around with it. Why can't
you just add a separate cachefs flag?
Otherwise, I'm a bit dubious about the wisdom of putting
nfs_invalidatepage() and nfs_releasepage() into fs/nfs/file.c. These are
not file operations, but rather pure page cache operations. I would have
thought that either read.c or possibly nfs-cachefs.c would be more
appropriate.
Please note too that Chuck has made generic functions for copying and
comparing NFS filehandles. They should be used in nfs_cache_fh_match() &
co.
I'm a bit worried about the use of the raw IP address in
nfs_cache_server_match(). It seems to me that when we add the NFSv4.1
support for trunking over several different transport mechanisms (RDMA,
IPv4/v6 etc) on the same mountpoint, then we may end up with a problem.
We can probably leave it in for now, but later we may want to consider
switching to using server->hostname or something equivalent.
Otherwise, it looks good. Looking forward to try it out...
Cheers,
Trond
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
_______________________________________________
Lists-linux-kernel-news mailing list
Lists-linux-kernel-news at mailman.vmware.com
http://mailman.vmware.com/mailman/listinfo/lists-linux-kernel-news
More information about the Linux-cachefs
mailing list