Hi Bruce! This patchset adds a new open file cache for knfsd. As you well know, nfsd basically does an open() - read/write() - close() cycle for every nfsv3 READ or WRITE. It's also common for clients to "spray" several read and write requests in parallel or in quick succession, so we could skip a lot of that by simply caching these open filps. The idea here is to cache them in a hashtable for a little while (1s by default) in the expectation that clients may try to issue more reads or writes in quick succession. When there are any entries in the hashtable, there is a recurring workqueue job that will clean the cache. I've also added some hooks into sunrpc cache code that should allow us to purge the cache on an unexport event, so this shouldn't cause any problems with unmounting once you've unexported the fs. I did a little testing with it, but my test rig is pretty slow, and I couldn't measure much of a performance difference on a bog standard local fs. We do have some patches that allow the reexporting of NFSv4.1 via knfsd. Since NFS has a relatively slow open routine, this provides a rather large speedup. Without these patches: $ dd if=/dev/urandom of=/mnt/dp01/ddfile bs=4k count=256 oflag=direct 256+0 records in 256+0 records out 1048576 bytes (1.0 MB) copied, 54.3109 s, 19.3 kB/s With these patches: $ dd if=/dev/urandom of=/mnt/dp01/ddfile bs=4k count=256 oflag=direct 256+0 records in 256+0 records out 1048576 bytes (1.0 MB) copied, 1.05437 s, 995 kB/s It should also be possible to hook this code up to the nfs4_file too, but I haven't done that in this set. I'd like to get this in and settled before we start looking at that, since it'll mean a bit of reengineering of the NFSv4 code not to pass around struct file pointers. I'd like to have these considered for the v4.3 merge window if they look reasonable. Jeff Layton (9): nfsd: include linux/nfs4.h in export.h nfsd: move some file caching helpers to a common header nfsd: convert laundry_wq to something less nfsd4 specific nfsd: add a new struct file caching facility to nfsd nfsd: hook up nfsd_write to the new nfsd_file cache nfsd: hook up nfsd_read to the nfsd_file cache sunrpc: add a new cache_detail operation for when a cache is flushed nfsd: add a ->flush routine to svc_export_cache nfsd: allow the file cache expire time to be tunable fs/nfsd/Makefile | 3 +- fs/nfsd/export.c | 14 ++ fs/nfsd/export.h | 1 + fs/nfsd/filecache.c | 348 +++++++++++++++++++++++++++++++++++++++++++ fs/nfsd/filecache.h | 47 ++++++ fs/nfsd/nfs3proc.c | 2 +- fs/nfsd/nfs4state.c | 40 +---- fs/nfsd/nfsd.h | 1 + fs/nfsd/nfsproc.c | 2 +- fs/nfsd/nfssvc.c | 22 ++- fs/nfsd/vfs.c | 56 +++---- fs/nfsd/vfs.h | 2 +- include/linux/sunrpc/cache.h | 1 + net/sunrpc/cache.c | 3 + 14 files changed, 467 insertions(+), 75 deletions(-) create mode 100644 fs/nfsd/filecache.c create mode 100644 fs/nfsd/filecache.h -- 2.4.3 -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
