)]}' { "log": [ { "commit": "f43bf0bebed7c33b698a8a25f95812f9e87c3843", "tree": "20c527153cde564ca3517cf6f214fa32727542bb", "parents": [ "2a3f5fd45938bd86ce8faf4cb26be4f7e9ae2941" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 12:01:04 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:20:52 2007 -0400" }, "message": "NFS: Add a boot parameter to disable 64 bit inode numbers\n\nThis boot parameter will allow legacy 32-bit applications which call stat()\nto continue to function even if the NFSv3/v4 server uses 64-bit inode\nnumbers.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "c7c209730d635226b81e9aeae63b6dc8f445569f", "tree": "0817778cf1b0504839f8a3e83797826fbf9acccc", "parents": [ "4f48af45842c6e78ab958c90344d3c940db4da15" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 28 19:22:40 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:20:23 2007 -0400" }, "message": "NFS: Get rid of some obsolete macros\n\n- NFS_READTIME, NFS_CHANGE_ATTR are completely unused.\n- Inline the few remaining uses of NFS_ATTRTIMEO, and remove.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "4f48af45842c6e78ab958c90344d3c940db4da15", "tree": "40f41baaf1931b47618034c21377a35b8ad926fe", "parents": [ "9697d2342e1a480bc14921c52f185c2fe01016e7" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 02 23:13:32 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:20:20 2007 -0400" }, "message": "NFS: Simplify filehandle revalidation\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "9697d2342e1a480bc14921c52f185c2fe01016e7", "tree": "bd4fba02d28777fc7705d8937a19740940fd5317", "parents": [ "a12802cab8520f86c9a80bbf87d10ee6171687d1" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 02 21:58:05 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:20:18 2007 -0400" }, "message": "NFS: Ensure that nfs_link() returns a hashed dentry\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "a12802cab8520f86c9a80bbf87d10ee6171687d1", "tree": "95e65c0541f0488b9fa6d31441cd584b2fbaf480", "parents": [ "b050aa791fad6b060d6ff59305f01289e18b058c" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 02 19:13:04 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:20:16 2007 -0400" }, "message": "NFS: Be strict about dentry revalidation when doing exclusive create\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "b050aa791fad6b060d6ff59305f01289e18b058c", "tree": "74786215218dfa0bb6cd45997de889a4f8526028", "parents": [ "efbb06b7f98a154ef51ad41674548af5cc1fd005" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 02 19:02:07 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:20:13 2007 -0400" }, "message": "NFS: Don\u0027t zap the readdir caches upon error\n\nIf necessary, the caches will get zapped under normal revalidation.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "efbb06b7f98a154ef51ad41674548af5cc1fd005", "tree": "7324549bd5138c1a439ebe95d1ca5149b9e4cf79", "parents": [ "81c768808c78283e1b4ed4cd2cad2571294b2090" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 02 17:11:54 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:20:11 2007 -0400" }, "message": "NFS: Remove the redundant nfs_reval_fsid()\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "d75340cc4de5c187fbf0bba234309ca86cf0a2fb", "tree": "aa53e3248bf3d61283f94187ffc83b18fa06bb7a", "parents": [ "216d5d06883edfaf992ada0d72a2a22fdfdbd296" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon Oct 01 21:42:01 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:20:06 2007 -0400" }, "message": "NFSv4: Fix nfs_atomic_open() to set the verifier on negative dentries too\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "216d5d06883edfaf992ada0d72a2a22fdfdbd296", "tree": "4b3ff79736cb09a2bb565383232243971a9a8e2f", "parents": [ "0a5ebc148879be68acdb12fbe72b65cb88c410d9" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon Oct 01 20:10:12 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:20:03 2007 -0400" }, "message": "NFSv4: Use NFSv2/v3 rules for negative dentries in nfs_open_revalidate\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "0a5ebc148879be68acdb12fbe72b65cb88c410d9", "tree": "b9fb0b5a2689b33daa97255b8b302383a304785a", "parents": [ "f2c77f4e62a2290ae46b5b0449eb72d72afe691e" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 02 12:57:24 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:20:01 2007 -0400" }, "message": "NFSv4: Don\u0027t revalidate the directory in nfs_atomic_lookup()\n\nWhy bother, since the call to nfs4_atomic_open() will do it for us.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "f2c77f4e62a2290ae46b5b0449eb72d72afe691e", "tree": "f0b68a5be10fdebf477c3b5af1dbdce3a6150301", "parents": [ "6d2b2966869142660f46d1e06cf9d15c3debcf77" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 02 12:54:39 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:19:58 2007 -0400" }, "message": "NFS: Optimise nfs_lookup_revalidate()\n\nWe don\u0027t need to call nfs_revalidate_inode() on the directory if we already\nknow that the verifiers don\u0027t match.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "60ccd4ec4170c9487e3792322626acd160197bce", "tree": "b35353518bd7945a40e39edfa543ce80aaa3f8b4", "parents": [ "80eb209def76d375677840800eb838abce1e6639" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Sep 29 17:48:19 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:19:53 2007 -0400" }, "message": "NFS: Remove nfs_begin_data_update/nfs_end_data_update\n\nThe lower level routines in fs/nfs/proc.c, fs/nfs/nfs3proc.c and\nfs/nfs/nfs4proc.c should already be dealing with the revalidation issues.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "a1643a92f6de92074116922a2d2906dd33499ff4", "tree": "9047e929e13d48b53a8d3512ce31ac9ab6b1832d", "parents": [ "3258b4fa552c4f994b5e6490a8ad88f5d7e0e648" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Sep 29 17:25:43 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:19:48 2007 -0400" }, "message": "NFS: NFS_CACHEINV() should not test for nfs_caches_unstable()\n\nThe fact that we\u0027re in the process of modifying the inode does not mean\nthat we should not invalidate the attribute and data caches. The defensive\nthing is to always invalidate when we\u0027re confronted with inode\nmtime/ctime or change_attribute updates that we do not immediately\nrecognise.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "3258b4fa552c4f994b5e6490a8ad88f5d7e0e648", "tree": "1ec891ada36618119544ae6c870665a53b120fcd", "parents": [ "cf8ba45e0554f1c8838fcfe43a93114f177af839" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon Oct 01 13:54:51 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:19:45 2007 -0400" }, "message": "NFS: Remove bogus nfs_mark_for_revalidate() in nfs_lookup\n\nThe parent of the newly materialised dentry has just been revalidated...\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "cf8ba45e0554f1c8838fcfe43a93114f177af839", "tree": "e8799c4eac30b4c42c2be7a02e4848aac6113f44", "parents": [ "f38211100d4823be530577dc3452f838861222ec" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon Oct 01 13:46:53 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:19:42 2007 -0400" }, "message": "NFS: don\u0027t cache the verifer across -\u003elookup() calls\n\nIf the -\u003elookup() call causes the directory verifier to change, then there\nis still no need to use the old verifier, since our dentry has been\nverified.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "446e534985bada0ad7451c08cf213c06695f9b67", "tree": "6f364b49ad9f6bc2c82ce28ce4fbd75aee372286", "parents": [ "d4d9cdcb470784df76304f75d0ce88f20f15fa6a" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed Oct 03 15:58:38 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:19:30 2007 -0400" }, "message": "NFS: Fix a bug in nfs_open_revalidate()\n\nWe want to set the verifier when the call to nfs4_open_revalidate()\n_succeeds_.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "d4d9cdcb470784df76304f75d0ce88f20f15fa6a", "tree": "7066e4640947f078855b47501e69f76ff775a834", "parents": [ "5724ab37872042176916441930e78fd353be1e5e" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 02 18:38:53 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:19:27 2007 -0400" }, "message": "NFS: Don\u0027t hash the negative dentry when optimising for an O_EXCL open\n\nWe don\u0027t want to leave an unverified hashed negative dentry if the\nexclusive create fails to complete.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "5724ab37872042176916441930e78fd353be1e5e", "tree": "255ba54f98f20ef22584bfb7308bf80f0c4fa2ad", "parents": [ "fab728e156b3cbfe31f05d6e7cdebe3d5eaff878" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon Oct 01 21:51:38 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:19:25 2007 -0400" }, "message": "NFS: nfs_instantiate() should set the dentry verifier\n\nThat will also allow us to remove the calls in mknod and mkdir.\nIn addition it will ensure that symlinks set it correctly.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "fab728e156b3cbfe31f05d6e7cdebe3d5eaff878", "tree": "4a0a9ab57867e33342119e61d52c16d0c9eb7a32", "parents": [ "4b841736bc16b320bcdb1e8ece585b3ced9a8811" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Sep 29 17:41:33 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:19:22 2007 -0400" }, "message": "NFS: Ensure nfs_instantiate() invalidates the parent dir on error\n\nAlso ensure that it drops the dentry in this case.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "4b841736bc16b320bcdb1e8ece585b3ced9a8811", "tree": "c79e4c99393b8ebfd54617f0ffa791dacc443cb5", "parents": [ "8edb01828837302055a8f0afddb2256659480bc5" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Sep 29 17:15:01 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:19:20 2007 -0400" }, "message": "NFS: Fix nfs_verify_change_attribute()\n\nWe don\u0027t care about whether or not some other process on our client is\nchanging the directory while we\u0027re in nfs_lookup_revalidate(), because the\ndcache will take care of ensuring local atomicity.\nWe can therefore remove the test for nfs_caches_unstable().\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "c4812998398d9cbce8646494704c52297359ede0", "tree": "8c7b786779b03d1960c25800a7a27a10f8318e99", "parents": [ "57fa76f2da05d0fee597b26bbc1f05242252beab" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 28 17:11:45 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:19:03 2007 -0400" }, "message": "NFS: Fix atime revalidation in readdir()\n\nNFSv3 will correctly update atime on a readdir call, so there is no need to\nset the NFS_INO_INVALID_ATIME flag unless the call to nfs_refresh_inode()\nfails.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "57fa76f2da05d0fee597b26bbc1f05242252beab", "tree": "c3ee6a3b75315b10bf2d1c9396d7e07c1b8f4830", "parents": [ "47aabaa7e45385fee4a535a6f6e523ff944e1684" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sun Sep 30 18:01:13 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:19:00 2007 -0400" }, "message": "NFS: Don\u0027t use readdirplus data if the page cache is invalid\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "6ecc5e8fcad7ad64d68c098249359831331bd299", "tree": "a0880db55be695d2975eb2d993d4cc918d0d3d83", "parents": [ "7957c1418f4b6c66e28d4ac3c4d7a8c19d526c48" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 28 14:20:33 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:18:49 2007 -0400" }, "message": "NFS: Fix dcache revalidation bugs\n\nWe don\u0027t need to force a dentry lookup just because we\u0027re making changes to\nthe directory.\n\nDon\u0027t update nfsi-\u003ecache_change_attribute in nfs_end_data_update: that\noverrides the NFSv3/v4 weak consistency checking that tells us our update\nwas the only one, and that tells us the dcache is still valid.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "af22f94ae02ab9dd4fd7fe628c8434a59cc293be", "tree": "e3794a90fc0720c33266dfa682e264a4041e7a14", "parents": [ "cd3758e37ddea66fccca7d93c4b601e8a2e51926" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Aug 10 17:45:10 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:18:34 2007 -0400" }, "message": "NFSv4: Simplify _nfs4_do_access()\n\nCurrently, _nfs4_do_access() is just a copy of nfs_do_access() with added\nconversion of the open flags into an access mask. This patch merges the\nduplicate functionality.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "cd3758e37ddea66fccca7d93c4b601e8a2e51926", "tree": "07566d3ee457524a97b534a4f214b9178b1278c8", "parents": [ "c03025d55540bd648f2546659090140ecc835572" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Aug 10 17:44:32 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:18:31 2007 -0400" }, "message": "NFS: Replace file-\u003eprivate_data with calls to nfs_file_open_context()\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "8fb559f87fee7f71dbf9a595095ad7d8e84c55e7", "tree": "1d38a0d8b750f7515390c9741b12364f6f3426c7", "parents": [ "77a55a1fe8f26f7d022986a599b68002e21d968a" ], "author": { "name": "Chuck Lever", "email": "chuck.lever@oracle.com", "time": "Mon Sep 24 15:40:16 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:18:26 2007 -0400" }, "message": "NFS: Eliminate nfs_refresh_verifier()\n\nnfs_set_verifier() and nfs_refresh_verifier() do exactly the same thing, so\nreplace one with the other.\n\nSigned-off-by: Chuck Lever \u003cchuck.lever@oracle.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "77a55a1fe8f26f7d022986a599b68002e21d968a", "tree": "63e804b1105d535149e0fd558c96a8c3efcc8dce", "parents": [ "92f6c178250170222f6d80c8ae725400765aa7a4" ], "author": { "name": "Chuck Lever", "email": "chuck.lever@oracle.com", "time": "Mon Sep 24 15:40:11 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:18:24 2007 -0400" }, "message": "NFS: Eliminate nfs_renew_times()\n\nThe nfs_renew_times() function plants the current time in jiffies in\ndentry-\u003ed_time. But a call to nfs_renew_times() is always followed by\nanother call that overwrites dentry-\u003ed_time. Get rid of the\nnfs_renew_times() calls.\n\nSigned-off-by: Chuck Lever \u003cchuck.lever@oracle.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "92f6c178250170222f6d80c8ae725400765aa7a4", "tree": "f0da7a93122db0ed40cb5175294e8492082144ba", "parents": [ "1321d8d971028e796978f6a48d195c09158b3bcd" ], "author": { "name": "Chuck Lever", "email": "chuck.lever@oracle.com", "time": "Mon Sep 24 15:40:06 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:18:22 2007 -0400" }, "message": "NFS: Don\u0027t call nfs_renew_times() in nfs_dentry_iput()\n\nNegative dentries need to be reverified after an asynchronous unlink.\n\nQuoth Trond:\n\n\"Unfortunately I don\u0027t think that we can avoid revalidating the\nresulting negative dentry since the UNLINK call is asynchronous,\nand so the new verifier on the directory will only be known a\nposteriori.\"\n\nSigned-off-by: Chuck Lever \u003cchuck.lever@oracle.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "c7e15961115028b99f6142266b5fb08acca0e8dd", "tree": "21d6ca8d97234664f242e35430ba4f0dbf61df8e", "parents": [ "4e769b934e7638038e232c05b64f644e7269a90f" ], "author": { "name": "Fabio Olive Leite", "email": "fleite@redhat.com", "time": "Thu Jul 26 22:59:00 2007 -0300" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:15:33 2007 -0400" }, "message": "Re: [NFS] [PATCH] Attribute timeout handling and wrapping u32 jiffies\n\nI would like to discuss the idea that the current checks for attribute\ntimeout using time_after are inadequate for 32bit architectures, since\ntime_after works correctly only when the two timestamps being compared\nare within 2^31 jiffies of each other. The signed overflow caused by\ncomparing values more than 2^31 jiffies apart will flip the result,\ncausing incorrect assumptions of validity.\n\n2^31 jiffies is a fairly large period of time (~25 days) when compared\nto the lifetime of most kernel data structures, but for long lived NFS\nmounts that can sit idle for months (think that for some reason autofs\ncannot be used), it is easy to compare inode attribute timestamps with\nvery disparate or even bogus values (as in when jiffies have wrapped\nmany times, where the comparison doesn\u0027t even make sense).\n\nCurrently the code tests for attribute timeout by simply adding the\ndesired amount of jiffies to the stored timestamp and comparing that\nwith the current timestamp of obtained attribute data with time_after.\nThis is incorrect, as it returns true for the desired timeout period\nand another full 2^31 range of jiffies.\n\nIn testing with artificial jumps (several small jumps, not one big\ncrank) of the jiffies I was able to reproduce a problem found in a\nserver with very long lived NFS mounts, where attributes would not be\nrefreshed even after touching files and directories in the server:\n\nInitial uptime:\n03:42:01 up 6 min, 0 users, load average: 0.01, 0.12, 0.07\n\nNFS volume is mounted and time is advanced:\n03:38:09 up 25 days, 2 min, 0 users, load average: 1.22, 1.05, 1.08\n\n# ls -l /local/A/foo/bar /nfs/A/foo/bar\n-rw-r--r-- 1 root root 0 Dec 17 03:38 /local/A/foo/bar\n-rw-r--r-- 1 root root 0 Nov 22 00:36 /nfs/A/foo/bar\n\n# touch /local/A/foo/bar\n\n# ls -l /local/A/foo/bar /nfs/A/foo/bar\n-rw-r--r-- 1 root root 0 Dec 17 03:47 /local/A/foo/bar\n-rw-r--r-- 1 root root 0 Nov 22 00:36 /nfs/A/foo/bar\n\nWe can see the local mtime is updated, but the NFS mount still shows\nthe old value. The patch below makes it work:\n\nInitial setup...\n07:11:02 up 25 days, 1 min, 0 users, load average: 0.15, 0.03, 0.04\n\n# ls -l /local/A/foo/bar /nfs/A/foo/bar\n-rw-r--r-- 1 root root 0 Jan 11 07:11 /local/A/foo/bar\n-rw-r--r-- 1 root root 0 Jan 11 07:11 /nfs/A/foo/bar\n\n# touch /local/A/foo/bar\n\n# ls -l /local/A/foo/bar /nfs/A/foo/bar\n-rw-r--r-- 1 root root 0 Jan 11 07:14 /local/A/foo/bar\n-rw-r--r-- 1 root root 0 Jan 11 07:14 /nfs/A/foo/bar\n\nSigned-off-by: Fabio Olive Leite \u003cfleite@redhat.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "4e769b934e7638038e232c05b64f644e7269a90f", "tree": "6508c1d273f22d3704eb849fb90404bade7cb4a7", "parents": [ "50e437d522a6cc34a882b2f740297f1b6b4c3af3" ], "author": { "name": "Peter Staubach", "email": "staubach@redhat.com", "time": "Fri Aug 03 15:07:10 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 09 17:15:29 2007 -0400" }, "message": "64 bit ino support for NFS client\n\nHi.\n\nAttached is a patch to modify the NFS client code to support\n64 bit ino\u0027s, as appropriate for the system and the NFS\nprotocol version.\n\nThe code basically just expand the NFS interfaces for routines\nwhich handle ino\u0027s from using ino_t to u64 and then uses the\nfileid in the nfs_inode instead of i_ino in the inode. The\ncode paths that were updated are in the getattr method and\nthe readdir methods.\n\nThis should be no real change on 64 bit platforms. Since\nthe ino_t is an unsigned long, it would already be 64 bits\nwide.\n\n Thanx...\n\n ps\n\nSigned-off-by: Peter Staubach \u003cstaubach@redhat.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "54af3bb543c071769141387a42deaaab5074da55", "tree": "d24e5b23462a51a4e932b14feb5ae943b0dbba1e", "parents": [ "bd8936632475665bfd90c1180c9c1301ee838db8" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 28 12:27:41 2007 -0400" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Fri Sep 28 15:36:42 2007 -0700" }, "message": "NFS: Fix an Oops in encode_lookup()\n\nIt doesn\u0027t look as if the NFS file name limit is being initialised correctly\nin the struct nfs_server. Make sure that we limit whatever is being set in\nnfs_probe_fsinfo() and nfs_init_server().\n\nAlso ensure that readdirplus and nfs4_path_walk respect our file name\nlimits.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "e4eff1a622edd6ab7b73acd5d8763aa2fa3fee49", "tree": "257d6675733d4af122a77054281e1d7d5062d904", "parents": [ "4fdc17b2a7f4d9db5b08e0f963d0027f714e4104" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Jul 14 15:39:58 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Thu Jul 19 15:21:39 2007 -0400" }, "message": "SUNRPC: Clean up the sillyrename code\n\nFix a couple of bugs:\n - Don\u0027t rely on the parent dentry still being valid when the call completes.\n Fixes a race with shrink_dcache_for_umount_subtree()\n\n - Don\u0027t remove the file if the filehandle has been labelled as stale.\n\nFix a couple of inefficiencies\n - Remove the global list of sillyrenamed files. Instead we can cache the\n sillyrename information in the dentry-\u003ed_fsdata\n - Move common code from unlink_setup/unlink_done into fs/nfs/unlink.c\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "3062c532ad410fe0e8320566fe2879a396be6701", "tree": "a6187e4988378bd6cc9712f0881417e0f360917b", "parents": [ "be879c4e249a8875d7129f3b0c1bb62584dafbd8" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Jul 14 17:36:45 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Thu Jul 19 15:21:39 2007 -0400" }, "message": "NFS: Use dentry-\u003ed_time to store the parent directory verifier.\n\nThis will free up the d_fsdata field for other use.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "a50f7951a31d3b976e829250853f89c9d2da32c0", "tree": "de9e8a3f93ee724471a49d9edb1155adef8d41f5", "parents": [ "e2f032e9ef66e33089d09452892696ea97d1dca1" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Jun 05 19:23:43 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Jul 10 23:40:25 2007 -0400" }, "message": "NFS: Fix an Oops in the nfs_access_cache_shrinker()\n\nThe nfs_access_cache_shrinker may race with nfs_access_zap_cache().\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "83d93f2229348837bf988a1048f7f38789474471", "tree": "cd4c6fc06cea4f7f761ceac187671f9cb5348d20", "parents": [ "a0356862bcbeb20acf64bc1a82d28a4c5bb957a7" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jun 07 09:58:08 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Jul 10 23:40:25 2007 -0400" }, "message": "NFS: Use GFP_HIGHUSER for page allocation in nfs_symlink()\n\nnfs_symlink() allocates a GFP_KERNEL page for the pagecache. Most\npagecache pages are allocated using GFP_HIGHUSER, and there\u0027s no reason\nnot to do that in nfs_symlink() as well.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\n" }, { "commit": "a0356862bcbeb20acf64bc1a82d28a4c5bb957a7", "tree": "20dd38a076a17c35e63eadddc77ccf0f3633f377", "parents": [ "b39e625b6e75aa70e26c13f9378756bb5f2af032" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Jun 05 13:26:15 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Jul 10 23:40:24 2007 -0400" }, "message": "NFS: Fix nfs_reval_fsid()\n\nWe don\u0027t need to revalidate the fsid on the root directory. It suffices to\nrevalidate it on the current directory.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "ad389da79f7bf9dc12dbc79c9c2740f9ed2f13d1", "tree": "9fd51d8a43aaba790a87f721d2796e36ab9f66e7", "parents": [ "539cd03a5708c9861a3e738e6f363ad743c85ddf" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Jun 05 12:30:00 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Jul 10 23:40:24 2007 -0400" }, "message": "NFSv4: Ensure asynchronous open() calls always pin the mountpoint\n\nA number of race conditions may currently ensue if the user presses ^C\nand then unmounts the partition while an asynchronous open() is in\nprogress.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "e8edc6e03a5c8562dc70a6d969f732bdb355a7e7", "tree": "fc86c863655128a7041dfe613d14393d761fa7b9", "parents": [ "ff1be9ad61e3e17ba83702d8ed0b534e5b8ee15c" ], "author": { "name": "Alexey Dobriyan", "email": "adobriyan@gmail.com", "time": "Mon May 21 01:22:52 2007 +0400" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Mon May 21 09:18:19 2007 -0700" }, "message": "Detach sched.h from mm.h\n\nFirst thing mm.h does is including sched.h solely for can_do_mlock() inline\nfunction which has \"current\" dereference inside. By dealing with can_do_mlock()\nmm.h can be detached from sched.h which is good. See below, why.\n\nThis patch\na) removes unconditional inclusion of sched.h from mm.h\nb) makes can_do_mlock() normal function in mm/mlock.c\nc) exports can_do_mlock() to not break compilation\nd) adds sched.h inclusions back to files that were getting it indirectly.\ne) adds less bloated headers to some files (asm/signal.h, jiffies.h) that were\n getting them indirectly\n\nNet result is:\na) mm.h users would get less code to open, read, preprocess, parse, ... if\n they don\u0027t need sched.h\nb) sched.h stops being dependency for significant number of files:\n on x86_64 allmodconfig touching sched.h results in recompile of 4083 files,\n after patch it\u0027s only 3744 (-8.3%).\n\nCross-compile tested on\n\n\tall arm defconfigs, all mips defconfigs, all powerpc defconfigs,\n\talpha alpha-up\n\tarm\n\ti386 i386-up i386-defconfig i386-allnoconfig\n\tia64 ia64-up\n\tm68k\n\tmips\n\tparisc parisc-up\n\tpowerpc powerpc-up\n\ts390 s390-up\n\tsparc sparc-up\n\tsparc64 sparc64-up\n\tum-x86_64\n\tx86_64 x86_64-up x86_64-defconfig x86_64-allnoconfig\n\nas well as my two usual configs.\n\nSigned-off-by: Alexey Dobriyan \u003cadobriyan@gmail.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "10afec9081fee7e48995fa396fba22c7de4b99d4", "tree": "edf1e3abf62d95eeef178dc8060cf033cc7940df", "parents": [ "9c9cc93ad2a5d9972672e03685af20e8cea1e5a4" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon May 14 17:16:04 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon May 14 19:33:46 2007 -0400" }, "message": "NFS: Fix some \u0027sparse\u0027 warnings...\n\n - fs/nfs/dir.c:610:8: warning: symbol \u0027nfs_llseek_dir\u0027 was not declared.\n Should it be static?\n - fs/nfs/dir.c:636:5: warning: symbol \u0027nfs_fsync_dir\u0027 was not declared.\n Should it be static?\n - fs/nfs/write.c:925:19: warning: symbol \u0027req\u0027 shadows an earlier one\n - fs/nfs/write.c:61:6: warning: symbol \u0027nfs_commit_rcu_free\u0027 was not\n declared. Should it be static?\n - fs/nfs/nfs4proc.c:793:5: warning: symbol \u0027nfs4_recover_expired_lease\u0027\n was not declared. Should it be static?\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "7a13e932281e7042a592f4f14db0b348199e7aac", "tree": "474245a3ed19f26528fd8b1bdda946fa9cd751d7", "parents": [ "fee7f23feaf0845fdfd47d20cddc75652552fbb8" ], "author": { "name": "Jesper Juhl", "email": "jesper.juhl@gmail.com", "time": "Thu Apr 26 00:29:02 2007 -0700" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed May 09 17:58:01 2007 -0400" }, "message": "NFS: Kill the obsolete NFS_PARANOIA\n\nSigned-off-by: Jesper Juhl \u003cjesper.juhl@gmail.com\u003e\nAcked-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "e70c490810dc683fad39e57cf00e69d5f120c542", "tree": "24276f69c064376d3c69979f61e181b377560795", "parents": [ "e62c2bba1fb7cf068eb78d731da46e4447a9efb1" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed May 09 09:00:18 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed May 09 17:57:59 2007 -0400" }, "message": "NFS: Remove redundant check in nfs_check_verifier()\n\nThe check for nfs_attribute_timeout(dir) in nfs_check_verifier is\nredundant: nfs_lookup_revalidate() will already call nfs_revalidate_inode()\non the parent dir when necessary.\n\nThe only case where this is not done is the case of a negative dentry. Fix\nthis case by moving up the revalidation code.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "e62c2bba1fb7cf068eb78d731da46e4447a9efb1", "tree": "bce53fe724400f7cfa1ae4621aafc6aa70bd564d", "parents": [ "38cb162b7585d837083b8365da1eb32687c5164c" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed May 09 09:00:17 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed May 09 17:57:58 2007 -0400" }, "message": "NFS: Fix a jiffie wraparound issue\n\ndentry verifiers are always set to the parent directory\u0027s\ncache_change_attribute. There is no reason to be testing for anything other\nthan equality when we\u0027re trying to find out if the dentry has been checked\nsince the last time the directory was modified.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "6fe6900e1e5b6fa9e5c59aa5061f244fe3f467e2", "tree": "8bbfe5072279227cc50a941ad4813908082426a1", "parents": [ "714b8171af9c930a59a0da8f6fe50518e70ab035" ], "author": { "name": "Nick Piggin", "email": "npiggin@suse.de", "time": "Sun May 06 14:49:04 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Mon May 07 12:12:51 2007 -0700" }, "message": "mm: make read_cache_page synchronous\n\nEnsure pages are uptodate after returning from read_cache_page, which allows\nus to cut out most of the filesystem-internal PageUptodate calls.\n\nI didn\u0027t have a great look down the call chains, but this appears to fixes 7\npossible use-before uptodate in hfs, 2 in hfsplus, 1 in jfs, a few in\necryptfs, 1 in jffs2, and a possible cleared data overwritten with readpage in\nblock2mtd. All depending on whether the filler is async and/or can return\nwith a !uptodate page.\n\nSigned-off-by: Nick Piggin \u003cnpiggin@suse.de\u003e\nCc: Hugh Dickins \u003chugh@veritas.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "83672d392f7bcf556f7920d6715e4174d9373ee0", "tree": "43af2f3428bd5a8eec3b089faf5aaef6ff7a4189", "parents": [ "1f4eab7e7c1d90dcd8ca4d7c064ee78dfbb345eb" ], "author": { "name": "Neil Brown", "email": "neilb@suse.de", "time": "Mon Feb 26 12:48:25 2007 +1100" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon Apr 30 22:17:19 2007 -0700" }, "message": "NFS: Fix directory caching problem - with test case and patch.\n\nTry running this script in an NFS mounted directory (Client relatively\nrecent - 2.6.18 has the problem as does 2.6.20).\n\n------------------------------------------------------\n#!/bin/bash\n#\n# This script will produce the following errormessage from tar:\n#\n# tar: newdir/innerdir/innerfile: file changed as we read it\n\n# create dirs\nrm -rf nfstest\nmkdir -p nfstest/dir/innerdir\n\n# create files (should not be empty)\necho \"Hello World!\" \u003enfstest/dir/file\necho \"Hello World!\" \u003enfstest/dir/innerdir/innerfile\n\n# problem only happens if we sleep before chmod\nsleep 1\n\n# change file modes\nchmod -R a+r nfstest\n\n# rename dir\nmv nfstest/dir nfstest/newdir\n\n# tar it\ntar -cf nfstest/nfstest.tar -C nfstest newdir\n\n# restore old dir name\nmv nfstest/newdir nfstest/dir\n--------------------------------------------------------\n\nWhat happens:\n\nThe \u0027chmod -R\u0027 does a readdir_plus in each directory and the results\nget cached in the page cache. It then updates the ctime on each file\nby one second. When this happens, the post-op attributes are used to\nupdate the ctime stored on the client to match the value in the kernel.\n\nThe \u0027mv\u0027 calls shrink_dcache_parent on the directory tree which\nflushes all the dentries (so a new lookup will be required) but\ndoesn\u0027t flush the inodes or pagecache.\n\nThe \u0027tar\u0027 does a readdir on each directory, but (in the case of\n\u0027innerdir\u0027 at least) satisfies it from the pagecache and uses the\nREADDIRPLUS data to update all the inodes. In the case of\n\u0027innerdir/innerfile\u0027, the ctime is out of date.\n\n\u0027tar\u0027 then calls \u0027lstat\u0027 on innerdir/innerfile getting an old ctime.\nIt then opens the file (triggering a GETATTR), reads the content, and\nthen calls fstat to see if anything has changed. It finds that ctime\nhas changed and so complains.\n\nThe problem seems to be that the cache readdirplus info is kept around\nfor too long.\n\nMy patch below discards pagecache data for directories when\ndentry_iput is called on them. This effectively removes the symptom\nwhich convinces me that I correctly understand the problem. However\nI\u0027m not convinced that is a proper solution, as there could easily be\nother races that trigger the same problem without being affected by\nthis \u0027fix\u0027.\n\nOne possibility would be to require that readdirplus pagecache data be\nonly used *once* to instantiate an inode. Somehow it should then be\ninvalidated so that if the dentry subsequently disappears, it will\ncause a new request to the server to fill in the stat data.\n\nAnother possibility is to compare the cache_change_attribute on the\ninode with something similar for the readdirplus info and reject the\ninfo from readdirplus if it is too old.\n\nI haven\u0027t tried to implement these and would value other opinions\nbefore I do.\n\nThanks,\nNeilBrown\n\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "1f4eab7e7c1d90dcd8ca4d7c064ee78dfbb345eb", "tree": "06c431bedac14bd907e616c390743795abfb1622", "parents": [ "74dd34e6e8bb127ff4c182423154b294729b663b" ], "author": { "name": "Neil Brown", "email": "neilb@suse.de", "time": "Mon Apr 16 09:35:27 2007 +1000" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon Apr 30 22:17:18 2007 -0700" }, "message": "NFS: Set meaningful value for fattr-\u003etime_start in readdirplus results.\n\nDon\u0027t use uninitialsed value for fattr-\u003etime_start in readdirplus results.\n\nThe \u0027fattr\u0027 structure filled in by nfs3_decode_direct does not get a\nvalue for -\u003etime_start set.\nThus if an entry is for an inode that we already have in cache,\nwhen nfs_readdir_lookup calls nfs_fhget, it will call nfs_refresh_inode\nand may update the inode with out-of-date information.\n\nDirectories are read a page at a time, so each page could have a\ndifferent timestamp that \"should\" be used to set the time_start for\nthe fattr for info in that page. However storing the timestamp per\npage is awkward. (We could stick in the first 4 bytes and only read 4092\nbytes, but that is a bigger code change than I am interested it).\n\nThis patch ignores the readdir_plus attributes if a readdir finds the\ninformation already in cache, and otherwise sets -\u003etime_start to the time\nthe readdir request was sent to the server.\n\nIt might be nice to store - in the directory inode - the time stamp for\nthe earliest readdir request that is still in the page cache, so that we\ndon\u0027t ignore attribute data that we don\u0027t have to. This patch doesn\u0027t do\nthat.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "e1552e199857109d4b25b9163eff4646726eee3d", "tree": "344af8502076a414c7b579bc33e0c98f21a20b2c", "parents": [ "c9c57929d23e44f258d1b6e7f089e72c85f0bd1c" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Apr 14 19:07:28 2007 -0400" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Sat Apr 14 21:46:47 2007 -0700" }, "message": "NFS: Fix an Oops in nfs_setattr()\n\nIt looks like nfs_setattr() and nfs_rename() also need to test whether the\ntarget is a regular file before calling nfs_wb_all()...\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "d9bc125caf592b7d081021f32ce5b717efdf70c8", "tree": "263b7066ba22ddce21db610c0300f6eaac6f2064", "parents": [ "43d78ef2ba5bec26d0315859e8324bfc0be23766", "ec2f9d1331f658433411c58077871e1eef4ee1b4" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon Feb 12 22:43:25 2007 -0800" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon Feb 12 22:43:25 2007 -0800" }, "message": "Merge branch \u0027master\u0027 of /home/trondmy/kernel/linux-2.6/\n\nConflicts:\n\n\tnet/sunrpc/auth_gss/gss_krb5_crypto.c\n\tnet/sunrpc/auth_gss/gss_spkm3_token.c\n\tnet/sunrpc/clnt.c\n\nMerge with mainline and fix conflicts.\n" }, { "commit": "92e1d5be91a0e3ffa5c4697eeb09b2aa22792122", "tree": "4eb22a9f6c38e9f4cc2a5100cd6659b0af08b7ae", "parents": [ "754661f143e70d66eae6c48532ca245aa05dec0e" ], "author": { "name": "Arjan van de Ven", "email": "arjan@linux.intel.com", "time": "Mon Feb 12 00:55:39 2007 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Mon Feb 12 09:48:46 2007 -0800" }, "message": "[PATCH] mark struct inode_operations const 2\n\nMany struct inode_operations in the kernel can be \"const\". Marking them const\nmoves these to the .rodata section, which avoids false sharing with potential\ndirty data. In addition it\u0027ll catch accidental writes at compile time to\nthese shared resources.\n\nSigned-off-by: Arjan van de Ven \u003carjan@linux.intel.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "c79ba787c11e767ffaf8d723923afda99ba6c63c", "tree": "746cd4018ab4dd5b28368b8e9a4d4c23c8e5b7b0", "parents": [ "ef75c7974b383769ae5741cf930b8aa4dcaef395" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed Jan 31 08:16:24 2007 -0500" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Feb 03 15:35:05 2007 -0800" }, "message": "NFS: Dont clobber more uptodate values in nfs_set_verifier()\n\nnfs_lookup_revalidate and friends are not serialised, so it is currently\nquite possible for the dentry to be revalidated, and then have the\nupdated verifier replaced with an older value by another process.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "ef75c7974b383769ae5741cf930b8aa4dcaef395", "tree": "4ed78762d171e41badb94c1bca93ae0be3e5af16", "parents": [ "df1d5d23d3a1a713c69b0f9ec67c59aeca3ce6b3" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Jan 16 10:09:44 2007 -0500" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Feb 03 15:35:04 2007 -0800" }, "message": "NFS: Also use readdir info to revalidate positive dentries\n\nIf the fileid of the cached dentry fails to match that returned by\nthe readdir call, then we should also d_drop. Try to take into account the\nfact that on NFSv4, readdir may return the \"mounted_on_fileid\" by looking\nfor submounts.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "df1d5d23d3a1a713c69b0f9ec67c59aeca3ce6b3", "tree": "2d5c3d7ba82c3f7010089414db2f0563ff6bd3e0", "parents": [ "ccfeb506231348a3c60ab0fdb5753a574653e3c0" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon Jan 15 13:56:29 2007 -0500" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Feb 03 15:35:04 2007 -0800" }, "message": "NFS: Fix a readdir/lookup inefficiency.\n\nMake sure that nfs_readdir_lookup() handles negative dentries correctly.\nIf d_lookup() returns a negative dentry, then we need to d_drop() that\nsince readdir shows that it should be positive.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "ccfeb506231348a3c60ab0fdb5753a574653e3c0", "tree": "17e2261c396d927e389c7f018008e5537bc27ddb", "parents": [ "ab91f264cfbafd079dcb1bd02e9803c2dd65de19" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Jan 13 02:28:12 2007 -0500" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Feb 03 15:35:04 2007 -0800" }, "message": "NFS: Fix up \"rm -rf\"...\n\nWhen a file is being scheduled for deletion by means of the sillyrename\nmechanism, it makes sense to start out writeback of the dirty data as\nsoon as possible in order to ensure that the delete can occur. Examples of\ncases where this is an issue include \"rm -rf\", which will busy-wait until\nthe file is closed, and the sillyrename completes.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "717d44e849219781ced028a40fcc59d3e1f49e4c", "tree": "aa34a9b84377d18ff58901cc342b84c7e8b81dca", "parents": [ "bde8f00ce64d9824a4f227c8594e335a1a10d044" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed Jan 24 11:54:55 2007 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Wed Jan 24 12:31:06 2007 -0800" }, "message": "[PATCH] NFS: Fix races in nfs_revalidate_mapping()\n\nPrevent the call to invalidate_inode_pages2() from racing with file writes\nby taking the inode-\u003ei_mutex across the page cache flush and invalidate.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "01cce933d8b524d9312f5098c70fa1b6ac190572", "tree": "7601e02e874a6eb44faca3cdf06664c7377ac687", "parents": [ "2485822d51f8b338d289abe00eb7ce5249794a08" ], "author": { "name": "Josef \"Jeff\" Sipek", "email": "jsipek@cs.sunysb.edu", "time": "Fri Dec 08 02:36:40 2006 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.osdl.org", "time": "Fri Dec 08 08:28:41 2006 -0800" }, "message": "[PATCH] nfs: change uses of f_{dentry,vfsmnt} to use f_path\n\nChange all the uses of f_{dentry,vfsmnt} to f_path.{dentry,mnt} in the nfs\nclient code.\n\nSigned-off-by: Josef \"Jeff\" Sipek \u003cjsipek@cs.sunysb.edu\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "fc22617e451f23b466d4d63bb016f5f6111b69e4", "tree": "64a772622e9bcd49f23ffcde408eb3517082ce6f", "parents": [ "9eaef27b36a6b716384948da94b8fc5bfba7b712" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Oct 21 10:24:24 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Sat Oct 21 13:35:06 2006 -0700" }, "message": "[PATCH] NFS: Cache invalidation fixup\n\nIf someone has renamed a directory on the server, triggering the d_move\ncode in d_materialise_unique(), then we need to invalidate the cached\ndirectory information in the source parent directory.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\nCc: Miklos Szeredi \u003cmiklos@szeredi.hu\u003e\nCc: Maneesh Soni \u003cmaneesh@in.ibm.com\u003e\nCc: Dipankar Sarma \u003cdipankar@in.ibm.com\u003e\nCc: Neil Brown \u003cneilb@cse.unsw.edu.au\u003e\nCc: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nCc: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "9eaef27b36a6b716384948da94b8fc5bfba7b712", "tree": "ab69cf9c3863bb7ca564574c914e5b6bf8ad3162", "parents": [ "3f7705eab6722ad1a346d748c4aad55755d6c241" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Oct 21 10:24:20 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Sat Oct 21 13:35:06 2006 -0700" }, "message": "[PATCH] VFS: Make d_materialise_unique() enforce directory uniqueness\n\nIf the caller tries to instantiate a directory using an inode that already\nhas a dentry alias, then we attempt to rename the existing dentry instead\nof instantiating a new one. Fail with an ELOOP error if the rename would\naffect one of our parent directories.\n\nThis behaviour is needed in order to avoid issues such as\n\n http://bugzilla.kernel.org/show_bug.cgi?id\u003d7178\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\nCc: Miklos Szeredi \u003cmiklos@szeredi.hu\u003e\nCc: Maneesh Soni \u003cmaneesh@in.ibm.com\u003e\nCc: Dipankar Sarma \u003cdipankar@in.ibm.com\u003e\nCc: Neil Brown \u003cneilb@cse.unsw.edu.au\u003e\nCc: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nCc: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "0dbb4c6799cf8fa8c5ba1926153a30960117477d", "tree": "9d8f75f2f407a5758fa4ef3d8c03694f138c3304", "parents": [ "8687b63afbe42103730bff4d3f7bfff3463c303e" ], "author": { "name": "Al Viro", "email": "viro@ftp.linux.org.uk", "time": "Thu Oct 19 23:28:49 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Fri Oct 20 10:26:40 2006 -0700" }, "message": "[PATCH] xdr annotations: NFS readdir entries\n\non-the-wire data is big-endian\n\n[in large part pulled from Alexey\u0027s patch]\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nAcked-by: Trond Myklebust \u003ctrond.myklebust@fys.uio.no\u003e\nAcked-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "39cf8a1374dc51fea169190674d5e4996a7d7ea2", "tree": "ca6e201861ed911c2b78dd81b56505527308c9a4", "parents": [ "cd9ae2b6a75bb1fa0d370929c2d7a7da1ed719d9" ], "author": { "name": "Chuck Lever", "email": "chuck.lever@oracle.com", "time": "Thu Oct 19 23:28:41 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Fri Oct 20 10:26:39 2006 -0700" }, "message": "[PATCH] NFS: fix minor bug in new NFS symlink code\n\nThe original code confused a zero return code from pagevec_add() as success.\n\nTest plan:\nNone.\n\nSigned-off-by: Chuck Lever \u003cchuck.lever@oracle.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "cd9ae2b6a75bb1fa0d370929c2d7a7da1ed719d9", "tree": "1bf943b54f47bfbf5198b08b930bd09099ff99b7", "parents": [ "13bbc06af8a5f65df0f888b442e557c617cadba7" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Thu Oct 19 23:28:40 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Fri Oct 20 10:26:39 2006 -0700" }, "message": "[PATCH] NFS: Deal with failure of invalidate_inode_pages2()\n\nIf invalidate_inode_pages2() fails, then it should in principle just be\nbecause the current process was signalled. In that case, we just want to\nensure that the inode\u0027s page cache remains marked as invalid.\n\nAlso add a helper to allow the O_DIRECT code to simply mark the page cache as\ninvalid once it is finished writing, instead of calling\ninvalidate_inode_pages2() itself.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "ce71ec36840368b877fb63bd14c8e67ab62d08b1", "tree": "55ea3caaa339881dfd66d787b3dbbb964825d07a", "parents": [ "17ff785691503f63ec648df82a7fdaece7695561" ], "author": { "name": "Dave Hansen", "email": "haveblue@us.ibm.com", "time": "Sat Sep 30 23:29:06 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Sun Oct 01 00:39:30 2006 -0700" }, "message": "[PATCH] r/o bind mounts: monitor zeroing of i_nlink\n\nSome filesystems, instead of simply decrementing i_nlink, simply zero it\nduring an unlink operation. We need to catch these in addition to the\ndecrement operations.\n\nSigned-off-by: Dave Hansen \u003chaveblue@us.ibm.com\u003e\nAcked-by: Christoph Hellwig \u003chch@lst.de\u003e\nCc: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "9a53c3a783c2fa9b969628e65695c11c3e51e673", "tree": "5a6115e18ee105246d46e3db3d5b07749d232f5b", "parents": [ "aab520e2f6c80160cabd187a8d0292d1cec8ff68" ], "author": { "name": "Dave Hansen", "email": "haveblue@us.ibm.com", "time": "Sat Sep 30 23:29:03 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Sun Oct 01 00:39:30 2006 -0700" }, "message": "[PATCH] r/o bind mounts: unlink: monitor i_nlink\n\nWhen a filesystem decrements i_nlink to zero, it means that a write must be\nperformed in order to drop the inode from the filesystem.\n\nWe\u0027re shortly going to have keep filesystems from being remounted r/o between\nthe time that this i_nlink decrement and that write occurs.\n\nSo, add a little helper function to do the decrements. We\u0027ll tie into it in a\nbit to note when i_nlink hits zero.\n\nSigned-off-by: Dave Hansen \u003chaveblue@us.ibm.com\u003e\nAcked-by: Christoph Hellwig \u003chch@lst.de\u003e\nCc: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "349457ccf2592c14bdf13b6706170ae2e94931b1", "tree": "3670945b5a62617d38cf1f317487387032d3da4d", "parents": [ "1390334b4c697b7588d5661fcf6acaeec409cf4c" ], "author": { "name": "Mark Fasheh", "email": "mark.fasheh@oracle.com", "time": "Fri Sep 08 14:22:21 2006 -0700" }, "committer": { "name": "Mark Fasheh", "email": "mark.fasheh@oracle.com", "time": "Sun Sep 24 13:50:45 2006 -0700" }, "message": "[PATCH] Allow file systems to manually d_move() inside of -\u003erename()\n\nSome file systems want to manually d_move() the dentries involved in a\nrename. We can do this by making use of the FS_ODD_RENAME flag if we just\nhave nfs_rename() unconditionally do the d_move(). While there, we rename\nthe flag to be more descriptive.\n\nOCFS2 uses this to protect that part of the rename operation with a cluster\nlock.\n\nSigned-off-by: Mark Fasheh \u003cmark.fasheh@oracle.com\u003e\nCc: Trond Myklebust \u003ctrond.myklebust@fys.uio.no\u003e\nCc: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nCc: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\n" }, { "commit": "fd6840714d9cf6e93f1d42b904860a94df316b85", "tree": "e72a0ef8bff8f1b6183596dbce3de34570ee6180", "parents": [ "762d4527c2fc19d821a13d9a3455ccc2d4073731" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Sep 05 12:27:44 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 22 23:25:01 2006 -0400" }, "message": "NFS: nfs_lookup - don\u0027t hash dentry when optimising away the lookup\n\nIf the open intents tell us that a given lookup is going to result in a,\nexclusive create, we currently optimize away the lookup call itself. The\nreason is that the lookup would not be atomic with the create RPC call, so\nwhy do it in the first place?\n\nA problem occurs, however, if the VFS aborts the exclusive create operation\nafter the lookup, but before the call to create the file/directory: in this\ncase we will end up with a hashed negative dentry in the dcache that has\nnever been looked up.\nFix this by only actually hashing the dentry once the create operation has\nbeen successfully completed.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "94a6d75320b3681e6e728b70e18bd186cb55e682", "tree": "0957071549d76ceb3857e419998818b11bce7269", "parents": [ "873101b33776780d32610fc4c90c7358a5e98f51" ], "author": { "name": "Chuck Lever", "email": "chuck.lever@oracle.com", "time": "Tue Aug 22 20:06:23 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 22 23:24:53 2006 -0400" }, "message": "NFS: Use cached page as buffer for NFS symlink requests\n\nNow that we have a copy of the symlink path in the page cache, we can pass\na struct page down to the XDR routines instead of a string buffer.\n\nTest plan:\nConnectathon, all NFS versions.\n\nSigned-off-by: Chuck Lever \u003cchuck.lever@oracle.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "873101b33776780d32610fc4c90c7358a5e98f51", "tree": "39bff3860ecc522e8a716c8cf2eebf55588f081a", "parents": [ "4f390c152bc87165da4b1f5b7d870b46fb106d4e" ], "author": { "name": "Chuck Lever", "email": "chuck.lever@oracle.com", "time": "Tue Aug 22 20:06:23 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 22 23:24:53 2006 -0400" }, "message": "NFS: copy symlinks into page cache before sending NFS SYMLINK request\n\nCurrently the NFS client does not cache symlinks it creates. They get\ncached only when the NFS client reads them back from the server.\n\nCopy the symlink into the page cache before sending it.\n\nTest plan:\nConnectathon, all NFS versions.\n\nSigned-off-by: Chuck Lever \u003cchuck.lever@oracle.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "4f390c152bc87165da4b1f5b7d870b46fb106d4e", "tree": "643b5d12f76bd7d3688380fbaf69f607a34a06bf", "parents": [ "d3db90e270791b21cd00d3c094884bffa907cc9e" ], "author": { "name": "Chuck Lever", "email": "chuck.lever@oracle.com", "time": "Tue Aug 22 20:06:22 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 22 23:24:52 2006 -0400" }, "message": "NFS: Fix double d_drop in nfs_instantiate() error path\n\nIf the LOOKUP or GETATTR in nfs_instantiate fail, nfs_instantiate will do a\nd_drop before returning. But some callers already do a d_drop in the case\nof an error return. Make certain we do only one d_drop in all error paths.\n\nThis issue was introduced because over time, the symlink proc API diverged\nslightly from the create/mkdir/mknod proc API. To prevent other coding\nmistakes of this type, change the symlink proc API to be more like\ncreate/mkdir/mknod and move the nfs_instantiate call into the symlink proc\nroutines so it is used in exactly the same way for create, mkdir, mknod,\nand symlink.\n\nTest plan:\nConnectathon, all versions of NFS.\n\nSigned-off-by: Chuck Lever \u003cchuck.lever@oracle.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "d3db90e270791b21cd00d3c094884bffa907cc9e", "tree": "3d6aea8fabd34fba5db5454867811b087bb71e05", "parents": [ "b86acd501a34227e0ed2b2d54dc8002c1701ce17" ], "author": { "name": "Chuck Lever", "email": "chuck.lever@oracle.com", "time": "Tue Aug 22 20:06:22 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 22 23:24:52 2006 -0400" }, "message": "NFS: remove a no-longer-needed error check in nfs_symlink()\n\nIn the early days of NFS, there was no duplicate reply cache on the server.\nThus retransmitted non-idempotent requests often found that the request had\nalready completed on the server. To avoid passing an unanticipated return\ncode to unsuspecting applications, NFS clients would often shunt error\ncodes that implied the request had been retried but already completed.\n\nThanks to NFS over TCP, duplicate reply caches on the server, and network\nperformance and reliability improvements, it is safe to remove such checks.\n\nTest plan:\nNone.\n\nSigned-off-by: Chuck Lever \u003cchuck.lever@oracle.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "54ceac4515986030c2502960be620198dd8fe25b", "tree": "b4ae4305c5652c0fe883ef5ea3243da91dbd2b34", "parents": [ "cf6d7b5de8535a9f0088c5cc28ee2dae87371b4a" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Tue Aug 22 20:06:13 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 22 23:24:37 2006 -0400" }, "message": "NFS: Share NFS superblocks per-protocol per-server per-FSID\n\nThe attached patch makes NFS share superblocks between mounts from the same\nserver and FSID over the same protocol.\n\nIt does this by creating each superblock with a false root and returning the\nreal root dentry in the vfsmount presented by get_sb(). The root dentry set\nstarts off as an anonymous dentry if we don\u0027t already have the dentry for its\ninode, otherwise it simply returns the dentry we already have.\n\nWe may thus end up with several trees of dentries in the superblock, and if at\nsome later point one of anonymous tree roots is discovered by normal filesystem\nactivity to be located in another tree within the superblock, the anonymous\nroot is named and materialises attached to the second tree at the appropriate\npoint.\n\nWhy do it this way? Why not pass an extra argument to the mount() syscall to\nindicate the subpath and then pathwalk from the server root to the desired\ndirectory? You can\u0027t guarantee this will work for two reasons:\n\n (1) The root and intervening nodes may not be accessible to the client.\n\n With NFS2 and NFS3, for instance, mountd is called on the server to get\n the filehandle for the tip of a path. mountd won\u0027t give us handles for\n anything we don\u0027t have permission to access, and so we can\u0027t set up NFS\n inodes for such nodes, and so can\u0027t easily set up dentries (we\u0027d have to\n have ghost inodes or something).\n\n With this patch we don\u0027t actually create dentries until we get handles\n from the server that we can use to set up their inodes, and we don\u0027t\n actually bind them into the tree until we know for sure where they go.\n\n (2) Inaccessible symbolic links.\n\n If we\u0027re asked to mount two exports from the server, eg:\n\n\tmount warthog:/warthog/aaa/xxx /mmm\n\tmount warthog:/warthog/bbb/yyy /nnn\n\n We may not be able to access anything nearer the root than xxx and yyy,\n but we may find out later that /mmm/www/yyy, say, is actually the same\n directory as the one mounted on /nnn. What we might then find out, for\n example, is that /warthog/bbb was actually a symbolic link to\n /warthog/aaa/xxx/www, but we can\u0027t actually determine that by talking to\n the server until /warthog is made available by NFS.\n\n This would lead to having constructed an errneous dentry tree which we\n can\u0027t easily fix. We can end up with a dentry marked as a directory when\n it should actually be a symlink, or we could end up with an apparently\n hardlinked directory.\n\n With this patch we need not make assumptions about the type of a dentry\n for which we can\u0027t retrieve information, nor need we assume we know its\n place in the grand scheme of things until we actually see that place.\n\nThis patch reduces the possibility of aliasing in the inode and page caches for\ninodes that may be accessed by more than one NFS export. It also reduces the\nnumber of superblocks required for NFS where there are many NFS exports being\nused from a server (home directory server + autofs for example).\n\nThis in turn makes it simpler to do local caching of network filesystems, as it\ncan then be guaranteed that there won\u0027t be links from multiple inodes in\nseparate superblocks to the same cache file.\n\nObviously, cache aliasing between different levels of NFS protocol could still\nbe a problem, but at least that gives us another key to use when indexing the\ncache.\n\nThis patch makes the following changes:\n\n (1) The server record construction/destruction has been abstracted out into\n its own set of functions to make things easier to get right. These have\n been moved into fs/nfs/client.c.\n\n All the code in fs/nfs/client.c has to do with the management of\n connections to servers, and doesn\u0027t touch superblocks in any way; the\n remaining code in fs/nfs/super.c has to do with VFS superblock management.\n\n (2) The sequence of events undertaken by NFS mount is now reordered:\n\n (a) A volume representation (struct nfs_server) is allocated.\n\n (b) A server representation (struct nfs_client) is acquired. This may be\n \t allocated or shared, and is keyed on server address, port and NFS\n \t version.\n\n (c) If allocated, the client representation is initialised. The state\n \t member variable of nfs_client is used to prevent a race during\n \t initialisation from two mounts.\n\n (d) For NFS4 a simple pathwalk is performed, walking from FH to FH to find\n \t the root filehandle for the mount (fs/nfs/getroot.c). For NFS2/3 we\n \t are given the root FH in advance.\n\n (e) The volume FSID is probed for on the root FH.\n\n (f) The volume representation is initialised from the FSINFO record\n \t retrieved on the root FH.\n\n (g) sget() is called to acquire a superblock. This may be allocated or\n \t shared, keyed on client pointer and FSID.\n\n (h) If allocated, the superblock is initialised.\n\n (i) If the superblock is shared, then the new nfs_server record is\n \t discarded.\n\n (j) The root dentry for this mount is looked up from the root FH.\n\n (k) The root dentry for this mount is assigned to the vfsmount.\n\n (3) nfs_readdir_lookup() creates dentries for each of the entries readdir()\n returns; this function now attaches disconnected trees from alternate\n roots that happen to be discovered attached to a directory being read (in\n the same way nfs_lookup() is made to do for lookup ops).\n\n The new d_materialise_unique() function is now used to do this, thus\n permitting the whole thing to be done under one set of locks, and thus\n avoiding any race between mount and lookup operations on the same\n directory.\n\n (4) The client management code uses a new debug facility: NFSDBG_CLIENT which\n is set by echoing 1024 to /proc/net/sunrpc/nfs_debug.\n\n (5) Clone mounts are now called xdev mounts.\n\n (6) Use the dentry passed to the statfs() op as the handle for retrieving fs\n statistics rather than the root dentry of the superblock (which is now a\n dummy).\n\nSigned-Off-By: David Howells \u003cdhowells@redhat.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "8fa5c000d7f986ef9cdc6d95f9f7fcee20e0a7d6", "tree": "356b811803db9775fd9c870d189a9ff75885ae6c", "parents": [ "1f163415dc05983830bcc47b33c155b2528b1574" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Tue Aug 22 20:06:12 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 22 23:24:35 2006 -0400" }, "message": "NFS: Move rpc_ops from nfs_server to nfs_client\n\nMove the rpc_ops from the nfs_server struct to the nfs_client struct as they\u0027re\ncommon to all server records of a particular NFS protocol version.\n\nSigned-Off-By: David Howells \u003cdhowells@redhat.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "979df72e6f963b42ee484f2eca049c3344da0ba7", "tree": "f3ba48a16203c899dbe57482c1ff7f1c02e0c9ca", "parents": [ "cfcea3e8c66c2dcde98d5c2693d4bff50b5cac97" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Jul 25 11:28:19 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 22 23:24:29 2006 -0400" }, "message": "NFS: Add an ACCESS cache memory shrinker\n\nA pinned inode may in theory end up filling memory with cached ACCESS\ncalls. This patch ensures that the VM may shrink away the cache in these\nparticular cases.\nThe shrinker works by iterating through the list of inodes on the global\nnfs_access_lru_list, and removing the least recently used access\ncache entry until it is done (or until the entire cache is empty).\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "cfcea3e8c66c2dcde98d5c2693d4bff50b5cac97", "tree": "ee7b995a48c43de7355fa33079ee7aaad020d6f3", "parents": [ "1c3c07e9f6cc50dab2aeb8051325e317d4f6c70e" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Jul 25 11:28:18 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 22 23:24:29 2006 -0400" }, "message": "NFS: Add a global LRU list for the ACCESS cache\n\n...in order to allow the addition of a memory shrinker.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "1c3c07e9f6cc50dab2aeb8051325e317d4f6c70e", "tree": "96bbc2a304e9a0e831daf589d1662c6129303796", "parents": [ "3eeab61aa3ddd3c0bedb7449ada1599de22fdb5a" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Jul 25 11:28:18 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 22 23:24:28 2006 -0400" }, "message": "NFS: Add a new ACCESS rpc call cache to the linux nfs client\n\nThe current access cache only allows one entry at a time to be cached for each\ninode. Add a per-inode red-black tree in order to allow more than one to\nbe cached at a time.\n\nShould significantly cut down the time spent in path traversal for shared\ndirectories such as ${PATH}, /usr/share, etc.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "4e0641a7ad98fca5646a6be17605bc80f6c4ebde", "tree": "5951d35bb1b2ed5a0452334221689258b5e3165f", "parents": [ "bce3481c91801665e17f8daf59ede946129f3d3f" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed Jul 05 13:05:13 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed Jul 05 13:17:13 2006 -0400" }, "message": "NFS: Optimise away an excessive GETATTR call when a file is symlinked\n\nIn the case when compiling via a symlink tree, we want to ensure that the\nclose-to-open GETATTR call is applied only to the final file, and not to\nthe symlink.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "55a975937d40cac582e981ddc8ed783b3dcc043c", "tree": "13507eaebdab5e9c9dd615fe373ec6c0ddcbade4", "parents": [ "8b4bdcf8995dd92b23d2ec22b32aee8fbbb50e1c" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Jun 09 09:34:19 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Jun 09 09:34:19 2006 -0400" }, "message": "NFS: Ensure the client submounts, when it crosses a server mountpoint.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "44b11874ff583b6e766a05856b04f3c492c32b84", "tree": "b17effcaf70ed67f61b1ec2ed2dea1ea2f884d60", "parents": [ "38478b24e37587f1c4fedf8ac070ca54f052ed28" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Thu May 25 01:40:59 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Jun 09 09:34:09 2006 -0400" }, "message": "NFS: Separate metadata and page cache revalidation mechanisms\n\nSeparate out the function of revalidating the inode metadata, and\nrevalidating the mapping. The former may be called by lookup(),\nand only really needs to check that permissions, ctime, etc haven\u0027t changed\nwhereas the latter needs only done when we want to read data from the page\ncache, and may need to sync and then invalidate the mapping.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "7451c4f0ee53e36fd74168af8df75b28fd04a2aa", "tree": "d981de4189c559d8e0a9f291c9919dde396c9a04", "parents": [ "b9d9506d944865876e67281a4e4269d823ce5381" ], "author": { "name": "Carsten Otte", "email": "cotte@de.ibm.com", "time": "Wed Apr 19 13:06:37 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed Apr 19 13:06:37 2006 -0400" }, "message": "NFS: remove needless check in nfs_opendir()\n\nLocal variable res was initialized to 0 - no check needed here.\n\nSigned-off-by: Carsten Otte \u003ccotte@de.ibm.com\u003e\nCc: Trond Myklebust \u003ctrond.myklebust@fys.uio.no\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "4b6f5d20b04dcbc3d888555522b90ba6d36c4106", "tree": "420f271eaef7d3def7d4433b151c3cb6d7a54770", "parents": [ "99ac48f54a91d02140c497edc31dc57d4bc5c85d" ], "author": { "name": "Arjan van de Ven", "email": "arjan@infradead.org", "time": "Tue Mar 28 01:56:42 2006 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Tue Mar 28 09:16:06 2006 -0800" }, "message": "[PATCH] Make most file operations structs in fs/ const\n\nThis is a conversion to make the various file_operations structs in fs/\nconst. Basically a regexp job, with a few manual fixups\n\nThe goal is both to increase correctness (harder to accidentally write to\nshared datastructures) and reducing the false sharing of cachelines with\nthings that get dirty in .data (while .rodata is nicely read only and thus\ncache clean)\n\nSigned-off-by: Arjan van de Ven \u003carjan@infradead.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "03f28e3a2059fc466761d872122f30acb7be61ae", "tree": "d478e553b79520c34bb3d06e75b59609de2993f4", "parents": [ "01d0ae8beaee75d954900109619b700fe68707d9" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon Mar 20 13:44:48 2006 -0500" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon Mar 20 13:44:48 2006 -0500" }, "message": "NFS: Make nfs_fhget() return appropriate error values\n\nCurrently it returns NULL, which usually gets interpreted as ENOMEM. In\nfact it can mean a host of issues.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "1e7cb3dc12dbbac690d78c84f9c7cb11132ed121", "tree": "a6bc36250a6fe5a754b464099a50baf98ce36e78", "parents": [ "e95b85ec9d8c8ad4667f746aa4c9d22c281efc44" ], "author": { "name": "Chuck Lever", "email": "cel@netapp.com", "time": "Mon Mar 20 13:44:24 2006 -0500" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon Mar 20 13:44:24 2006 -0500" }, "message": "NFS: directory trace messages\n\nReuse NFSDBG_DIRCACHE and NFSDBG_LOOKUPCACHE to provide additional\ndiagnostic messages that trace the operation of the NFS client\u0027s\ndirectory behavior. A few new messages are now generated when NFSDBG_VFS\nis active, as well, to trace normal VFS activity. This compromise\nprovides better trace debugging for those who use pre-built kernels,\nwithout adding a lot of extra noise to the standard debug settings.\n\nTest-plan:\nEnable NFS trace debugging with flags 1, 2, or 4. You should be able to\nsee different types of trace messages with each flag setting.\n\nSigned-off-by: Chuck Lever \u003ccel@netapp.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "91d5b47023b608227d605d1e916b29dd0215bff7", "tree": "31dbb285639ea68db3abc0c4129988f02d50f11a", "parents": [ "d9ef5a8c26aab09762afce43df64736720b4860e" ], "author": { "name": "Chuck Lever", "email": "cel@netapp.com", "time": "Mon Mar 20 13:44:14 2006 -0500" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon Mar 20 13:44:14 2006 -0500" }, "message": "NFS: add I/O performance counters\n\nInvoke the byte and event counter macros where we want to count bytes and\nevents.\n\nClean-up: fix a possible NULL dereference in nfs_lock, and simplify\nnfs_file_open.\n\nTest-plan:\nfsx and iozone on UP and SMP systems, with and without pre-emption. Watch\nfor memory overwrite bugs, and performance loss (significantly more CPU\nrequired per op).\n\nSigned-off-by: Chuck Lever \u003ccel@netapp.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "1b1dcc1b57a49136f118a0f16367256ff9994a69", "tree": "b0b36d4f41d28c9d6514fb309d33c1a084d6309b", "parents": [ "794ee1baee1c26be40410233e6c20bceb2b03c08" ], "author": { "name": "Jes Sorensen", "email": "jes@sgi.com", "time": "Mon Jan 09 15:59:24 2006 -0800" }, "committer": { "name": "Ingo Molnar", "email": "mingo@hera.kernel.org", "time": "Mon Jan 09 15:59:24 2006 -0800" }, "message": "[PATCH] mutex subsystem, semaphore to mutex: VFS, -\u003ei_sem\n\nThis patch converts the inode semaphore to a mutex. I have tested it on\nXFS and compiled as much as one can consider on an ia64. Anyway your\nluck with it might be different.\n\nModified-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n\n(finished the conversion)\n\nSigned-off-by: Jes Sorensen \u003cjes@sgi.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "24174119c73983d5217da8f56a12c79a9b57e056", "tree": "38904c36afc94c6833cb9296c705e654ff286e43", "parents": [ "40859d7ee64ed6bfad8a4e93f9bb5c1074afadff" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Jan 03 09:55:33 2006 +0100" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Jan 06 14:58:50 2006 -0500" }, "message": "NFSv4: Ensure that we return the delegation on the target of a rename too.\n\n Signed-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "5ba7cc4801ae0fe74b6e0160f008521ae71d9f5d", "tree": "8e7c236d2adeee1c23d99a2e7d20203cdc37a4e6", "parents": [ "bb713d6d38f7be4f4e7d790cddb1b076e7da6699" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Dec 03 15:20:17 2005 -0500" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Dec 03 15:20:17 2005 -0500" }, "message": "NFS: Fix post-op attribute revalidation...\n\n - Missing nfs_mark_for_revalidate in nfs_proc_link()\n - Missing nfs_mark_for_revalidate in nfs_rename()\n\n Signed-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "34ea818846e6cccdd6e05354ab8c8a6d7e34657a", "tree": "d00d3d61c03454856c9047afaab3ba60a2e84f32", "parents": [ "2c56617d76fa7b2a709d58abd7ed68889cc88b90" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Nov 04 15:35:02 2005 -0500" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Nov 04 15:35:02 2005 -0500" }, "message": "NFSv4: Return any delegations before sillyrenaming the file\n\n I missed this one... Any form of rename will result in a delegation\n recall, so it is more efficient to return the one we hold before\n trying the rename.\n\n Signed-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "cf809556149f076b7a020c10e066b2b96e79b6a1", "tree": "714d3e58b4fe5af9b144c735601946949b217820", "parents": [ "516a6af641bb50c608329a5bd751acd0d65cc4ab" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Thu Oct 27 22:12:42 2005 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Thu Oct 27 22:12:42 2005 -0400" }, "message": "NFS: Ensure that nfs_link() instantiates the dentry correctly\n\n Signed-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "0c70b50150cfb0b43ff500a8a394a52b4d5f1350", "tree": "292d31ce5378edb856076d2c610c323b76af0b09", "parents": [ "decf491f3076190262d4c649bed877650623903a" ], "author": { "name": "Chuck Lever", "email": "cel@netapp.com", "time": "Tue Oct 25 11:48:36 2005 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Thu Oct 27 22:12:40 2005 -0400" }, "message": "NFS: nfs_lookup doesn\u0027t need to revalidate the parent directory\u0027s inode\n\n nfs_lookup() used to consult a lookup cache before trying an actual wire\n lookup operation. The lookup cache would be invalid, of course, if the\n parent directory\u0027s mtime had changed, so nfs_lookup performed an inode\n revalidation on the parent.\n\n Since nfs_lookup() doesn\u0027t use a cache anymore, the revalidation is no\n longer necessary. There are cases where it will generate a lot of\n unnecessary GETATTR traffic.\n\n See http://bugzilla.linux-nfs.org/show_bug.cgi?id\u003d9\n\n Test-plan:\n Use lndir and \"rm -rf\" and watch for excess GETATTR traffic or application\n level errors.\n\n Signed-off-by: Chuck Lever \u003ccel@netapp.com\u003e\n Signed-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "0e574af1be5f569a5d7f2800333b0bfb358a5e34", "tree": "bb5b62dece21b8906222dfee022020ed6f81e34c", "parents": [ "4c2cb58c552a34744979a99ccf01762d5eb7e288" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Thu Oct 27 22:12:38 2005 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Thu Oct 27 22:12:38 2005 -0400" }, "message": "NFS: Cleanup initialisation of struct nfs_fattr\n\n Signed-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "6fe43f9e3701f7a9f2be151a5e6cfe94b87e92f9", "tree": "edf67143a032c7b7867ee999f1deb01437a6bc99", "parents": [ "4c780a4688b421baa896b59778c05d7e068e479f" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 18 14:20:22 2005 -0700" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 18 14:20:22 2005 -0700" }, "message": "NFS: Fix rename of directory onto empty directory\n\n If someone tries to rename a directory onto an empty directory, we\n currently fail and return EBUSY.\n This patch ensures that we try the rename if both source and target\n are directories, and that we fail with a correct error of EISDIR if\n the source is not a directory.\n\n Signed-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "cae7a073a4c5484cc5713eab606bf54b46724ab3", "tree": "e5cdcf0376da5b04bca9bbbf8f226abe13f33275", "parents": [ "cdce5d6b94b6182f6d8a5b7b52923933e98cbc92" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 18 14:20:19 2005 -0700" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 18 14:20:19 2005 -0700" }, "message": "NFSv4: Return delegation upon rename or removal of file.\n\n Signed-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "6f926b5ba75c568296ec227e7d782db4ddbdca5c", "tree": "6c9aad3139ae4e54210ab5ff37446c357c54b521", "parents": [ "02a913a73b52071e93f4b76db3e86138d19efffd" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 18 14:20:18 2005 -0700" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 18 14:20:18 2005 -0700" }, "message": "[NFS]: Check that the server returns a valid regular file to our OPEN request\n\n Since it appears that some servers don\u0027t...\n\n Signed-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "02a913a73b52071e93f4b76db3e86138d19efffd", "tree": "1dc1abbd2d8f57a6fd593dd252d6d7ecc7c811c5", "parents": [ "834f2a4a1554dc5b2598038b3fe8703defcbe467" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 18 14:20:17 2005 -0700" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 18 14:20:17 2005 -0700" }, "message": "NFSv4: Eliminate nfsv4 open race...\n\n Make NFSv4 return the fully initialized file pointer with the\n stateid that it created in the lookup w/intent.\n\n Signed-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "9aa48b7e270d13c8781414dce081a42cae20a80d", "tree": "788749de63303b280155cd84c5a47434f029d6e0", "parents": [ "449231d6ddf50ca46b7fb2f76ecf790135222913" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Thu Aug 25 16:25:35 2005 -0700" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 23 12:38:01 2005 -0400" }, "message": "NFS: Don\u0027t expose internal READDIR errors to userspace\n\n Fixes a condition whereby the kernel is returning the non-POSIX error\n EBADCOOKIE to userspace.\n\n Signed-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "20509f1bc553ed7fafa88fa8d01c6212d1876d9f", "tree": "da10ad62009995269dc0733a254a0bc1f6293aa2", "parents": [ "12f44f46bc9c6dc79255e661b085797af395d8da" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Thu Aug 25 16:25:34 2005 -0700" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 23 12:37:58 2005 -0400" }, "message": "NFS: Drop inode after rename\n\n When doing a rename on top of an existing file that is not in use,\n the inode of the overwritten file will remain in the icache.\n\n The fix is to decrement i_nlink of the overwritten inode, like we\n do for unlink, rmdir etc already.\n\n Problem diagnosed by Olaf Kirch. This patch is a slight variation\n on his fix.\n\n Signed-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "01c314a0c0f6367960a7cb1ffb5796560ccaa1c1", "tree": "ae3c0190efa548d47e80e7225a8e8da0b2990317", "parents": [ "cd3716ab40c6049479d29a74b29107fd7e0e1153" ], "author": { "name": "Steve Dickson", "email": "SteveD@redhat.com", "time": "Fri Aug 19 17:57:48 2005 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Fri Aug 19 18:44:56 2005 -0700" }, "message": "[PATCH] NFSv4: unbalanced BKL in nfs_atomic_lookup()\n\nAdded missing unlock_kernel() to NFSv4 atomic lookup.\n\nSigned-off-by: Steve Dickson \u003csteved@redhat.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "dc59250c6ebed099a9bc0a11298e2281dd896657", "tree": "80c294437c0868d90abfa617d873370e6dbe6565", "parents": [ "412d582ec1dd59aab2353f8cb7e74f2c79cd20b9" ], "author": { "name": "Chuck Lever", "email": "cel@citi.umich.edu", "time": "Thu Aug 18 11:24:12 2005 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Thu Aug 18 12:53:57 2005 -0700" }, "message": "[PATCH] NFS: Introduce the use of inode-\u003ei_lock to protect fields in nfsi\n\nDown the road we want to eliminate the use of the global kernel lock entirely\nfrom the NFS client. To do this, we need to protect the fields in the\nnfs_inode structure adequately. Start by serializing updates to the\n\"cache_validity\" field.\n\nNote this change addresses an SMP hang found by njw@osdl.org, where processes\ndeadlock because nfs_end_data_update and nfs_revalidate_mapping update the\n\"cache_validity\" field without proper serialization.\n\nTest plan:\n Millions of fsx ops on SMP clients. Run Nick Wilson\u0027s breaknfs program on\n large SMP clients.\n\nSigned-off-by: Chuck Lever \u003ccel@netapp.com\u003e\nCc: Trond Myklebust \u003ctrond.myklebust@fys.uio.no\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "412d582ec1dd59aab2353f8cb7e74f2c79cd20b9", "tree": "9088c5e29cbd4b33e907a7b5f9a950dd0dce36c1", "parents": [ "5529680981807b44abf3be30fb6d612ff04f68ff" ], "author": { "name": "Chuck Lever", "email": "cel@citi.umich.edu", "time": "Thu Aug 18 11:24:11 2005 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Thu Aug 18 12:53:56 2005 -0700" }, "message": "[PATCH] NFS: use atomic bitops to manipulate flags in nfsi-\u003eflags\n\nIntroduce atomic bitops to manipulate the bits in the nfs_inode structure\u0027s\n\"flags\" field.\n\nUsing bitops means we can use a generic wait_on_bit call instead of an ad hoc\nlocking scheme in fs/nfs/inode.c, so we can remove the \"nfs_i_wait\" field from\nnfs_inode at the same time.\n\nThe other new flags field will continue to use bitmask and logic AND and OR.\nThis permits several flags to be set at the same time efficiently. The\nfollowing patch adds a spin lock to protect these flags, and this spin lock\nwill later cover other fields in the nfs_inode structure, amortizing the cost\nof using this type of serialization.\n\nTest plan:\n Millions of fsx ops on SMP clients.\n\nSigned-off-by: Chuck Lever \u003ccel@netapp.com\u003e\nCc: Trond Myklebust \u003ctrond.myklebust@fys.uio.no\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "5529680981807b44abf3be30fb6d612ff04f68ff", "tree": "57da4e9135c0a85c1f8c6bc797250c0209420b51", "parents": [ "3c7bf1eaee1255315fc7c2c4c300295e556ef768" ], "author": { "name": "Chuck Lever", "email": "cel@citi.umich.edu", "time": "Thu Aug 18 11:24:09 2005 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Thu Aug 18 12:53:56 2005 -0700" }, "message": "[PATCH] NFS: split nfsi-\u003eflags into two fields\n\nCertain bits in nfsi-\u003eflags can be manipulated with atomic bitops, and some\nare better manipulated via logical bitmask operations.\n\nThis patch splits the flags field into two. The next patch introduces atomic\nbitops for one of the fields.\n\nTest plan:\n Millions of fsx ops on SMP clients.\n\nSigned-off-by: Chuck Lever \u003ccel@netapp.com\u003e\nCc: Trond Myklebust \u003ctrond.myklebust@fys.uio.no\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "f0dd2136da6d2070e12bfa6d199b136318e666c7", "tree": "a261d4b4d9e29dbce104cf9515a0a209cff387cf", "parents": [ "00a926422765064cb28e218d4837411c88bf6a3e" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed Jun 22 17:16:29 2005 +0000" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed Jun 22 16:07:34 2005 -0400" }, "message": "[PATCH] NFS: Clean up readdir changes.\n\n Signed-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "00a926422765064cb28e218d4837411c88bf6a3e", "tree": "1e9ad635821c7b037014307d1a2657dc56b17acf", "parents": [ "ae3884621bf5b4caff7785b9a417f262202965b2" ], "author": { "name": "Olivier Galibert", "email": "galibert@pobox.com", "time": "Wed Jun 22 17:16:29 2005 +0000" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed Jun 22 16:07:33 2005 -0400" }, "message": "[PATCH] NFS: Hide NFS server-generated readdir cookies from userland\n\n NFSv3 currently returns the unsigned 64-bit cookie directly to\n userspace. The following patch causes the kernel to generate\n loff_t offsets for the benefit of userland.\n The current server-generated READDIR cookie is cached in the\n nfs_open_context instead of in filp-\u003ef_pos, so we still end up work\n correctly under directory insertions/deletion.\n\n Signed-off-by: Olivier Galibert \u003cgalibert@pobox.com\u003e\n Signed-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "b7fa0554cf1ba6d6895cd0a5b02989a26e0bc704", "tree": "83eb405f3ff78c17695999df38c99484e3aee01f", "parents": [ "a257cdd0e2179630d3201c32ba14d7fcb3c3a055" ], "author": { "name": "Andreas Gruenbacher", "email": "agruen@suse.de", "time": "Wed Jun 22 17:16:27 2005 +0000" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed Jun 22 16:07:24 2005 -0400" }, "message": "[PATCH] NFS: Add support for NFSv3 ACLs\n\n This adds acl support fo nfs clients via the NFSACL protocol extension, by\n implementing the getxattr, listxattr, setxattr, and removexattr iops for the\n system.posix_acl_access and system.posix_acl_default attributes. This patch\n implements a dumb version that uses no caching (and thus adds some overhead).\n (Another patch in this patchset adds caching as well.)\n\n Signed-off-by: Andreas Gruenbacher \u003cagruen@suse.de\u003e\n Acked-by: Olaf Kirch \u003cokir@suse.de\u003e\n Signed-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\n Signed-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" } ], "next": "6b3b5496d7b261d6c9202008dc528e52dbd11e57" }