)]}' { "log": [ { "commit": "80a16b21a81eb639f0b726549f4c46c0e9aff92e", "tree": "5095cb05498d16e64976b03c38b3fcdaa83e4ecb", "parents": [ "281cad46b34db4dbb1d1e603f7b9cfe25d1ae7c9" ], "author": { "name": "Bryan Schumaker", "email": "bjschuma@netapp.com", "time": "Fri Apr 27 13:27:46 2012 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Apr 27 14:10:39 2012 -0400" }, "message": "NFS: Remove extra rpc_clnt argument to proc_lookup\n\nNow that I\u0027m doing secinfo automatically in the v4 code this extra\nargument isn\u0027t needed.\n\nSigned-off-by: Bryan Schumaker \u003cbjschuma@netapp.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "281cad46b34db4dbb1d1e603f7b9cfe25d1ae7c9", "tree": "f13917030755776a4f80ffcc0d07c2b5da60066d", "parents": [ "2671bfc3beb44e70636bd0208274426db57f73b5" ], "author": { "name": "Bryan Schumaker", "email": "bjschuma@netapp.com", "time": "Fri Apr 27 13:27:45 2012 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Apr 27 14:10:39 2012 -0400" }, "message": "NFS: Create a submount rpc_op\n\nThis simplifies the code for v2 and v3 and gives v4 a chance to decide\non referrals without needing to modify the generic client.\n\nSigned-off-by: Bryan Schumaker \u003cbjschuma@netapp.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "2671bfc3beb44e70636bd0208274426db57f73b5", "tree": "5aeec6224c390a046bfa5758a958174ebf542772", "parents": [ "df0117481cd94dbb8970f4be9d05b0568fa09ab1" ], "author": { "name": "Bryan Schumaker", "email": "bjschuma@netapp.com", "time": "Fri Apr 27 13:27:44 2012 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Apr 27 14:10:39 2012 -0400" }, "message": "NFS: Remove secinfo knowledge out of the generic client\n\nAnd also remove the unneeded rpc_op.\n\nSigned-off-by: Bryan Schumaker \u003cbjschuma@netapp.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "e245d4250d0326cfcf7c816a2081b6ab2ea810be", "tree": "1424c497bdcd1305137b18201a3086a365595a72", "parents": [ "7e6eb683d260d9325f0d1bd911518d5ed3cb4f0c" ], "author": { "name": "Bryan Schumaker", "email": "bjschuma@netapp.com", "time": "Fri Apr 27 13:27:43 2012 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Apr 27 14:10:03 2012 -0400" }, "message": "NFS: Remove unused function nfs_lookup_with_sec()\n\nThis fixes a compiler warning.\n\nSigned-off-by: Bryan Schumaker \u003cbjschuma@netapp.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "7e6eb683d260d9325f0d1bd911518d5ed3cb4f0c", "tree": "51b22645b2e55cff65acf9feb0d9f43f44ed70d3", "parents": [ "f05d147f7e3cf0d86b3a4bd5603029a7cb109633" ], "author": { "name": "Bryan Schumaker", "email": "bjschuma@netapp.com", "time": "Fri Apr 27 13:27:42 2012 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Apr 27 14:10:03 2012 -0400" }, "message": "NFS: Honor the authflavor set in the clone mount data\n\nThe authflavor is set in an nfs_clone_mount structure and passed to the\nxdev_mount() functions where it was promptly ignored. Instead, use it\nto initialize an rpc_clnt for the cloned server.\n\nSigned-off-by: Bryan Schumaker \u003cbjschuma@netapp.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "f05d147f7e3cf0d86b3a4bd5603029a7cb109633", "tree": "e351ef7dd3d541626f5cccef86f50906bfb3a649", "parents": [ "72de53ec4bca39c26709122a8f78bfefe7b6bca4" ], "author": { "name": "Bryan Schumaker", "email": "bjschuma@netapp.com", "time": "Fri Apr 27 13:27:41 2012 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Apr 27 14:10:02 2012 -0400" }, "message": "NFS: Fix following referral mount points with different security\n\nI create a new proc_lookup_mountpoint() to use when submounting an NFS\nv4 share. This function returns an rpc_clnt to use for performing an\nfs_locations() call on a referral\u0027s mountpoint.\n\nSigned-off-by: Bryan Schumaker \u003cbjschuma@netapp.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "d7c32675021bd750d8e0e726f2f81f746e8cab01", "tree": "873a67e2006ea589d44ca7d4c9514ece4f34a133", "parents": [ "dbb9c2a22d32492544765e798386daa2d9da27d2" ], "author": { "name": "Chuck Lever", "email": "chuck.lever@oracle.com", "time": "Wed Feb 15 16:35:17 2012 -0500" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Thu Feb 16 15:05:16 2012 -0500" }, "message": "nfs: Clean up debugging in nfs_follow_mountpoint()\n\nClean up: Fix a debugging message which had an obsolete function name\nin it (nfs_follow_mountpoint).\n\nIntroduced by commit 36d43a43 \"NFS: Use d_automount() rather than\nabusing follow_link()\" (January 14, 2011)\n\nSigned-off-by: Chuck Lever \u003cchuck.lever@oracle.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "fca78d6d2c77f87d7dbee89bbe4836a44da881e2", "tree": "5378bd6ad0917cf11ad7f7b739b71794df230c85", "parents": [ "6382a44138e7aa40bf52170e7afc014443a24806" ], "author": { "name": "Bryan Schumaker", "email": "bjschuma@netapp.com", "time": "Thu Jun 02 14:59:07 2011 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Jul 12 13:40:27 2011 -0400" }, "message": "NFS: Add SECINFO_NO_NAME procedure\n\nIf the client is using NFS v4.1, then we can use SECINFO_NO_NAME to find\nthe secflavor for the initial mount. If the server doesn\u0027t support\nSECINFO_NO_NAME then I fall back on the \"guess and check\" method used\nfor v4.0 mounts.\n\nSigned-off-by: Bryan Schumaker \u003cbjschuma@netapp.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "561f0b0ad073859a87d22845ddfd9df149b22e5f", "tree": "14ac040cb0f709854ba71513332b5b3871b87b4b", "parents": [ "4b38a6db01b09198f4045661815a0039c3d80660" ], "author": { "name": "Bryan Schumaker", "email": "bjschuma@netapp.com", "time": "Tue Apr 12 08:47:15 2011 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Apr 12 19:34:23 2011 -0400" }, "message": "NFS: Remove unused argument from nfs_find_best_sec()\n\nThe inode was used in an earlier version of the code, but it isn\u0027t\nused anymore.\n\nSigned-off-by: Bryan Schumaker \u003cbjschuma@netapp.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "418875900e3de4831c84f86ae4756690dac5be77", "tree": "d0609f12b62962806adc092867350b7bfcfed5a3", "parents": [ "0867659fa3c245bf203d837a82e0f6ea5079c2c5" ], "author": { "name": "Bryan Schumaker", "email": "bjschuma@netapp.com", "time": "Wed Apr 06 14:33:28 2011 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed Apr 06 13:25:04 2011 -0700" }, "message": "NFS: Fix a signed vs. unsigned secinfo bug\n\nrpc_authflavor_t is cast from an unsigned int, but the\ninitial code tried to use it as a signed int. I fix\nthis by passing an rpc_authflavor_t pointer around, and\nreturning signed integers from functions.\n\nSigned-off-by: Bryan Schumaker \u003cbjschuma@netapp.com\u003e\nReported-by: Dan Carpenter \u003cerror27@gmail.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "a0e7e3cf7932d6c77de0dc79a40dbaeb8060b544", "tree": "fcc8ab44418afe356b683674fcd2c1589b7c6efa", "parents": [ "4d65c520fb4abed970069d18c119cfe85624f46d" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Mar 26 02:24:35 2011 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sun Mar 27 17:48:17 2011 +0200" }, "message": "NFS: Don\u0027t leak RPC clients in NFSv4 secinfo negotiation\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "7ebb931598cd95cccea10d4bc4c0123a464ea565", "tree": "dec1af6eefdbbc3ce42346d7c6d407088eaf509e", "parents": [ "5a5ea0d485c9715c86bf858bbdc5f6d373b3db88" ], "author": { "name": "Bryan Schumaker", "email": "bjschuma@netapp.com", "time": "Thu Mar 24 17:12:30 2011 +0000" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Thu Mar 24 13:52:42 2011 -0400" }, "message": "NFS: use secinfo when crossing mountpoints\n\nA submount may use different security than the parent\nmount does. We should figure out what sec flavor the\nsubmount uses at mount time.\n\nSigned-off-by: Bryan Schumaker \u003cbjschuma@netapp.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "7c5130588d691a3b34d02312f1bd1b6d56fe0100", "tree": "154ccea3a837ca1aae2c5676e0946b79fb4bb671", "parents": [ "e73b83f270828630a9ce33728f6ef61c37a82340" ], "author": { "name": "Bryan Schumaker", "email": "bjschuma@netapp.com", "time": "Thu Mar 24 17:12:24 2011 +0000" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Thu Mar 24 13:52:41 2011 -0400" }, "message": "NFS: lookup supports alternate client\n\nA later patch will need to perform a lookup using an\nalternate client with a different security flavor.\nThis patch adds support for doing that on NFS v4.\n\nSigned-off-by: Bryan Schumaker \u003cbjschuma@netapp.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "1c34092adf1feaba25b7c739cc4def2751f4fa05", "tree": "4950fc9c2476eb07c590bb9156906004b6a3c1ad", "parents": [ "a872d5101008b091035d579897bccefdeff70def" ], "author": { "name": "Dan Carpenter", "email": "error27@gmail.com", "time": "Sun Mar 20 14:22:07 2011 +0300" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Mon Mar 21 00:45:50 2011 -0400" }, "message": "nfs: lock() vs unlock() typo\n\nThese should be spin_unlock() instead of spin_lock(). It\u0027s a typo.\n\nSigned-off-by: Dan Carpenter \u003cerror27@gmail.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "f8ad9c4bae99854c961ca79ed130a0d11d9ab53c", "tree": "de466d67c6413ba5a0fed1733768f21e1ffc2acf", "parents": [ "b514f872f86d4b0c13fed74a1fe1f7ab500c4fd0" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Mar 16 06:32:07 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Mar 16 16:48:06 2011 -0400" }, "message": "nfs: nfs_do_{ref,sub}mount() superblock argument is redundant\n\nIt\u0027s always equal to dentry-\u003ed_sb\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "b514f872f86d4b0c13fed74a1fe1f7ab500c4fd0", "tree": "a54821b3e2363150e4e9b180e097a8092f884dda", "parents": [ "b1942c5f8cf3bea3a3c88a7498ae4c4361f31afe" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Mar 16 06:26:11 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Mar 16 16:47:55 2011 -0400" }, "message": "nfs: make nfs_path() work without vfsmount\n\npart 3: now we have everything to get nfs_path() just by dentry -\njust follow to (disconnected) root and pick the rest of the thing\nthere.\n\nStart killing propagation of struct vfsmount * on the paths that\nused to bring it to nfs_path().\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "ea5b778a8b98c85a87d66bf844904f9c3802b869", "tree": "baa56cbe1a907d76341f2cad53e16569cc1d3288", "parents": [ "ab90911ff90cdab59b31c045c3f0ae480d14f29d" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Fri Jan 14 19:10:03 2011 +0000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Jan 15 20:07:48 2011 -0500" }, "message": "Unexport do_add_mount() and add in follow_automount(), not -\u003ed_automount()\n\nUnexport do_add_mount() and make -\u003ed_automount() return the vfsmount to be\nadded rather than calling do_add_mount() itself. follow_automount() will then\ndo the addition.\n\nThis slightly complicates things as -\u003ed_automount() normally wants to add the\nnew vfsmount to an expiration list and start an expiration timer. The problem\nwith that is that the vfsmount will be deleted if it has a refcount of 1 and\nthe timer will not repeat if the expiration list is empty.\n\nTo this end, we require the vfsmount to be returned from d_automount() with a\nrefcount of (at least) 2. One of these refs will be dropped unconditionally.\nIn addition, follow_automount() must get a 3rd ref around the call to\ndo_add_mount() lest it eat a ref and return an error, leaving the mount we\nhave open to being expired as we would otherwise have only 1 ref on it.\n\nd_automount() should also add the the vfsmount to the expiration list (by\ncalling mnt_set_expiry()) and start the expiration timer before returning, if\nthis mechanism is to be used. The vfsmount will be unlinked from the\nexpiration list by follow_automount() if do_add_mount() fails.\n\nThis patch also fixes the call to do_add_mount() for AFS to propagate the mount\nflags from the parent vfsmount.\n\nSigned-off-by: David Howells \u003cdhowells@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "36d43a43761b004ad1879ac21471d8fc5f3157ec", "tree": "6cb6c6d978f4e58de7f9bf901707d6929f098345", "parents": [ "d18610b0ce9eb48c60649d8fcbf68374c84349d3" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Fri Jan 14 18:45:42 2011 +0000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Jan 15 20:07:34 2011 -0500" }, "message": "NFS: Use d_automount() rather than abusing follow_link()\n\nMake NFS use the new d_automount() dentry operation rather than abusing\nfollow_link() on directories.\n\nSigned-off-by: David Howells \u003cdhowells@redhat.com\u003e\nAcked-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\nAcked-by: Ian Kent \u003craven@themaw.net\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "cc53ce53c86924bfe98a12ea20b7465038a08792", "tree": "3c9a4923dd9f413c46bfa83a20cb579446df6deb", "parents": [ "9875cf806403fae66b2410a3c2cc820d97731e04" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Fri Jan 14 18:45:26 2011 +0000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Jan 15 20:07:31 2011 -0500" }, "message": "Add a dentry op to allow processes to be held during pathwalk transit\n\nAdd a dentry op (d_manage) to permit a filesystem to hold a process and make it\nsleep when it tries to transit away from one of that filesystem\u0027s directories\nduring a pathwalk. The operation is keyed off a new dentry flag\n(DCACHE_MANAGE_TRANSIT).\n\nThe filesystem is allowed to be selective about which processes it holds and\nwhich it permits to continue on or prohibits from transiting from each flagged\ndirectory. This will allow autofs to hold up client processes whilst letting\nits userspace daemon through to maintain the directory or the stuff behind it\nor mounted upon it.\n\nThe -\u003ed_manage() dentry operation:\n\n\tint (*d_manage)(struct path *path, bool mounting_here);\n\ntakes a pointer to the directory about to be transited away from and a flag\nindicating whether the transit is undertaken by do_add_mount() or\ndo_move_mount() skipping through a pile of filesystems mounted on a mountpoint.\n\nIt should return 0 if successful and to let the process continue on its way;\n-EISDIR to prohibit the caller from skipping to overmounted filesystems or\nautomounting, and to use this directory; or some other error code to return to\nthe user.\n\n-\u003ed_manage() is called with namespace_sem writelocked if mounting_here is true\nand no other locks held, so it may sleep. However, if mounting_here is true,\nit may not initiate or wait for a mount or unmount upon the parameter\ndirectory, even if the act is actually performed by userspace.\n\nWithin fs/namei.c, follow_managed() is extended to check with d_manage() first\non each managed directory, before transiting away from it or attempting to\nautomount upon it.\n\nfollow_down() is renamed follow_down_one() and should only be used where the\nfilesystem deliberately intends to avoid management steps (e.g. autofs).\n\nA new follow_down() is added that incorporates the loop done by all other\ncallers of follow_down() (do_add/move_mount(), autofs and NFSD; whilst AFS, NFS\nand CIFS do use it, their use is removed by converting them to use\nd_automount()). The new follow_down() calls d_manage() as appropriate. It\nalso takes an extra parameter to indicate if it is being called from mount code\n(with namespace_sem writelocked) which it passes to d_manage(). follow_down()\nignores automount points so that it can be used to mount on them.\n\n__follow_mount_rcu() is made to abort rcu-walk mode if it hits a directory with\nDCACHE_MANAGE_TRANSIT set on the basis that we\u0027re probably going to have to\nsleep. It would be possible to enter d_manage() in rcu-walk mode too, and have\nthat determine whether to abort or not itself. That would allow the autofs\ndaemon to continue on in rcu-walk mode.\n\nNote that DCACHE_MANAGE_TRANSIT on a directory should be cleared when it isn\u0027t\nrequired as every tranist from that directory will cause d_manage() to be\ninvoked. It can always be set again when necessary.\n\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nWHAT THIS MEANS FOR AUTOFS\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\nAutofs currently uses the lookup() inode op and the d_revalidate() dentry op to\ntrigger the automounting of indirect mounts, and both of these can be called\nwith i_mutex held.\n\nautofs knows that the i_mutex will be held by the caller in lookup(), and so\ncan drop it before invoking the daemon - but this isn\u0027t so for d_revalidate(),\nsince the lock is only held on _some_ of the code paths that call it. This\nmeans that autofs can\u0027t risk dropping i_mutex from its d_revalidate() function\nbefore it calls the daemon.\n\nThe bug could manifest itself as, for example, a process that\u0027s trying to\nvalidate an automount dentry that gets made to wait because that dentry is\nexpired and needs cleaning up:\n\n\tmkdir S ffffffff8014e05a 0 32580 24956\n\tCall Trace:\n\t [\u003cffffffff885371fd\u003e] :autofs4:autofs4_wait+0x674/0x897\n\t [\u003cffffffff80127f7d\u003e] avc_has_perm+0x46/0x58\n\t [\u003cffffffff8009fdcf\u003e] autoremove_wake_function+0x0/0x2e\n\t [\u003cffffffff88537be6\u003e] :autofs4:autofs4_expire_wait+0x41/0x6b\n\t [\u003cffffffff88535cfc\u003e] :autofs4:autofs4_revalidate+0x91/0x149\n\t [\u003cffffffff80036d96\u003e] __lookup_hash+0xa0/0x12f\n\t [\u003cffffffff80057a2f\u003e] lookup_create+0x46/0x80\n\t [\u003cffffffff800e6e31\u003e] sys_mkdirat+0x56/0xe4\n\nversus the automount daemon which wants to remove that dentry, but can\u0027t\nbecause the normal process is holding the i_mutex lock:\n\n\tautomount D ffffffff8014e05a 0 32581 1 32561\n\tCall Trace:\n\t [\u003cffffffff80063c3f\u003e] __mutex_lock_slowpath+0x60/0x9b\n\t [\u003cffffffff8000ccf1\u003e] do_path_lookup+0x2ca/0x2f1\n\t [\u003cffffffff80063c89\u003e] .text.lock.mutex+0xf/0x14\n\t [\u003cffffffff800e6d55\u003e] do_rmdir+0x77/0xde\n\t [\u003cffffffff8005d229\u003e] tracesys+0x71/0xe0\n\t [\u003cffffffff8005d28d\u003e] tracesys+0xd5/0xe0\n\nwhich means that the system is deadlocked.\n\nThis patch allows autofs to hold up normal processes whilst the daemon goes\nahead and does things to the dentry tree behind the automouter point without\nrisking a deadlock as almost no locks are held in d_manage() and none in\nd_automount().\n\nSigned-off-by: David Howells \u003cdhowells@redhat.com\u003e\nWas-Acked-by: Ian Kent \u003craven@themaw.net\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "b5c84bf6f6fa3a7dfdcb556023a62953574b60ee", "tree": "7a2c299a180713e21d5cb653cb933121adf53c31", "parents": [ "949854d02455080d20cd3e1db28a3a18daf7599d" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:49:38 2011 +1100" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:23 2011 +1100" }, "message": "fs: dcache remove dcache_lock\n\ndcache_lock no longer protects anything. remove it.\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\n" }, { "commit": "949854d02455080d20cd3e1db28a3a18daf7599d", "tree": "9b13a6f86c1d0b91e462a471e53b0e717036b18e", "parents": [ "9abca36087288fe28de4749c71ca003d4b9e3ed0" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:49:37 2011 +1100" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:22 2011 +1100" }, "message": "fs: Use rename lock and RCU for multi-step operations\n\nThe remaining usages for dcache_lock is to allow atomic, multi-step read-side\noperations over the directory tree by excluding modifications to the tree.\nAlso, to walk in the leaf-\u003eroot direction in the tree where we don\u0027t have\na natural d_lock ordering.\n\nThis could be accomplished by taking every d_lock, but this would mean a\nhuge number of locks and actually gets very tricky.\n\nSolve this instead by using the rename seqlock for multi-step read-side\noperations, retry in case of a rename so we don\u0027t walk up the wrong parent.\nConcurrent dentry insertions are not serialised against. Concurrent deletes\nare tricky when walking up the directory: our parent might have been deleted\nwhen dropping locks so also need to check and retry for that.\n\nWe can also use the rename lock in cases where livelock is a worry (and it\nis introduced in subsequent patch).\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\n" }, { "commit": "a4d7f16806e98cee752006d3a8c10067a7c2aa6b", "tree": "b7acd9998abaa6f25edc2f40fd5ce0cd996a3934", "parents": [ "815409d22df870ea0b0d86f2a3bf33c35bcef55c" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Apr 16 16:22:46 2010 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri May 14 15:09:22 2010 -0400" }, "message": "NFS: Reduce the stack footprint of nfs_follow_mountpoint()\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "5a0e3ad6af8660be21ca98a971cd00f331318c05", "tree": "5bfb7be11a03176a87296a43ac6647975c00a1d1", "parents": [ "ed391f4ebf8f701d3566423ce8f17e614cde9806" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Mar 24 17:04:11 2010 +0900" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Tue Mar 30 22:02:32 2010 +0900" }, "message": "include cleanup: Update gfp.h and slab.h includes to prepare for breaking implicit slab.h inclusion from percpu.h\n\npercpu.h is included by sched.h and module.h and thus ends up being\nincluded when building most .c files. percpu.h includes slab.h which\nin turn includes gfp.h making everything defined by the two files\nuniversally available and complicating inclusion dependencies.\n\npercpu.h -\u003e slab.h dependency is about to be removed. Prepare for\nthis change by updating users of gfp and slab facilities include those\nheaders directly instead of assuming availability. As this conversion\nneeds to touch large number of source files, the following script is\nused as the basis of conversion.\n\n http://userweb.kernel.org/~tj/misc/slabh-sweep.py\n\nThe script does the followings.\n\n* Scan files for gfp and slab usages and update includes such that\n only the necessary includes are there. ie. if only gfp is used,\n gfp.h, if slab is used, slab.h.\n\n* When the script inserts a new include, it looks at the include\n blocks and try to put the new include such that its order conforms\n to its surrounding. It\u0027s put in the include block which contains\n core kernel includes, in the same order that the rest are ordered -\n alphabetical, Christmas tree, rev-Xmas-tree or at the end if there\n doesn\u0027t seem to be any matching order.\n\n* If the script can\u0027t find a place to put a new include (mostly\n because the file doesn\u0027t have fitting include block), it prints out\n an error message indicating which .h file needs to be added to the\n file.\n\nThe conversion was done in the following steps.\n\n1. The initial automatic conversion of all .c files updated slightly\n over 4000 files, deleting around 700 includes and adding ~480 gfp.h\n and ~3000 slab.h inclusions. The script emitted errors for ~400\n files.\n\n2. Each error was manually checked. Some didn\u0027t need the inclusion,\n some needed manual addition while adding it to implementation .h or\n embedding .c file was more appropriate for others. This step added\n inclusions to around 150 files.\n\n3. The script was run again and the output was compared to the edits\n from #2 to make sure no file was left behind.\n\n4. Several build tests were done and a couple of problems were fixed.\n e.g. lib/decompress_*.c used malloc/free() wrappers around slab\n APIs requiring slab.h to be added manually.\n\n5. The script was run on all .h files but without automatically\n editing them as sprinkling gfp.h and slab.h inclusions around .h\n files could easily lead to inclusion dependency hell. Most gfp.h\n inclusion directives were ignored as stuff from gfp.h was usually\n wildly available and often used in preprocessor macros. Each\n slab.h inclusion directive was examined and added manually as\n necessary.\n\n6. percpu.h was updated not to include slab.h.\n\n7. Build test were done on the following configurations and failures\n were fixed. CONFIG_GCOV_KERNEL was turned off for all tests (as my\n distributed build env didn\u0027t work with gcov compiles) and a few\n more options had to be turned off depending on archs to make things\n build (like ipr on powerpc/64 which failed due to missing writeq).\n\n * x86 and x86_64 UP and SMP allmodconfig and a custom test config.\n * powerpc and powerpc64 SMP allmodconfig\n * sparc and sparc64 SMP allmodconfig\n * ia64 SMP allmodconfig\n * s390 SMP allmodconfig\n * alpha SMP allmodconfig\n * um on x86_64 SMP allmodconfig\n\n8. percpu.h modifications were reverted so that it could be applied as\n a separate patch and serve as bisection point.\n\nGiven the fact that I had only a couple of failures from tests on step\n6, I\u0027m fairly confident about the coverage of this conversion patch.\nIf there is a breakage, it\u0027s likely to be something in one of the arch\nheaders which should be easily discoverable easily on most builds of\nthe specific arch.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nGuess-its-ok-by: Christoph Lameter \u003ccl@linux-foundation.org\u003e\nCc: Ingo Molnar \u003cmingo@redhat.com\u003e\nCc: Lee Schermerhorn \u003cLee.Schermerhorn@hp.com\u003e\n" }, { "commit": "0b75b35c7cad33e7613f5adf28fa10fe8b09b1c3", "tree": "92b9987a33dc791aed752469ea7aed6e00a63b36", "parents": [ "c02d7adf8c5429727a98bad1d039bccad4c61c50" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon Jun 22 15:09:14 2009 -0400" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Jun 22 21:28:25 2009 -0700" }, "message": "NFS: Fix nfs_path() to always return a \u0027/\u0027 at the beginning of the path\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "9393bd07cf218ca51d0e627653f906a9d76a9131", "tree": "402205fe6336028df48cd65b31da0482f0b0bb41", "parents": [ "589ff870ed60a9ebdd5ec99ec3f5afe1282fe151" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Apr 18 13:58:15 2009 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Jun 11 21:36:01 2009 -0400" }, "message": "switch follow_down()\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "44d5759d3fdad660f000ef319f0ec33a6ac6ae28", "tree": "2d6508f919c21a3f25254567d88606438f339c38", "parents": [ "c9f6cde6e26ef98ee9c4b6288b126ac9c580d88b" ], "author": { "name": "Denis V. Lunev", "email": "den@openvz.org", "time": "Mon Aug 11 12:02:34 2008 +0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 07 18:15:16 2008 -0400" }, "message": "nfs: BUG_ON in nfs_follow_mountpoint\n\nUnfortunately, BUG_ON(IS_ROOT(dentry)) can happen inside\nnfs_follow_mountpoint with NFS running Fedora 8 using a\nspecific setup.\nhttps://bugzilla.redhat.com/show_bug.cgi?id\u003d458622\n\nSo, the situation should be handled on NFS client gracefully.\n\nSigned-off-by: Denis V. Lunev \u003cden@openvz.org\u003e\nCC: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\nCC: J. Bruce Fields \u003cbfields@fieldses.org\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "fd08d7e9d196ca49afcce0181f1f0ca68f241aa2", "tree": "445a2d883332274b3ec01c6cf30f840a6d1d4d9b", "parents": [ "bb8a3b53c20f2c07164a23ff6c320794fee8b95f" ], "author": { "name": "Denis V. Lunev", "email": "den@openvz.org", "time": "Thu Jul 31 09:38:55 2008 +0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Oct 07 18:14:34 2008 -0400" }, "message": "nfs: ERR_PTR is expected on failure from nfs_do_clone_mount\n\nReplace NULL with ERR_PTR(-EINVAL).\n\nSigned-off-by: Denis V. Lunev \u003cden@openvz.org\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "8d66bf5481002b0960aa49aed0987c73f5d7816c", "tree": "7beac720e1df882a1cd4d6d09e6ac25ddd04b399", "parents": [ "d5686b444ff3f72808d2b3fbd58672a86cdf38e7" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Aug 01 09:05:54 2008 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Aug 01 11:25:32 2008 -0400" }, "message": "[PATCH] pass struct path * to do_add_mount()\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "31f31db1a15671513df9cd9fbe56ef45ee1e9a2a", "tree": "ad4015fb8159aaa18041791c2c85e272c5565dd7", "parents": [ "3110ff8048fb757b36112b044b384aea9c44d6e4" ], "author": { "name": "Jan Blunck", "email": "jblunck@suse.de", "time": "Fri May 02 13:42:45 2008 -0700" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri May 16 09:43:30 2008 -0700" }, "message": "nfs: path_{get,put}() cleanups\n\nHere are some more places where path_{get,put}() can be used instead of\ndput()/mntput() pair.\n\nSigned-off-by: Jan Blunck \u003cjblunck@suse.de\u003e\nCc: Trond Myklebust \u003ctrond.myklebust@fys.uio.no\u003e\nCc: \"J. Bruce Fields\" \u003cbfields@fieldses.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "3110ff8048fb757b36112b044b384aea9c44d6e4", "tree": "5260f4973bd0848ddef3fb3c111386e59616abbc", "parents": [ "46c8ac74250a396aca855e494f49a960797a6b5e" ], "author": { "name": "Harvey Harrison", "email": "harvey.harrison@gmail.com", "time": "Fri May 02 13:42:44 2008 -0700" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri May 16 09:43:29 2008 -0700" }, "message": "nfs: replace remaining __FUNCTION__ occurrences\n\n__FUNCTION__ is gcc-specific, use __func__\n\nSigned-off-by: Harvey Harrison \u003charvey.harrison@gmail.com\u003e\nCc: Trond Myklebust \u003ctrond.myklebust@fys.uio.no\u003e\nCc: \"J. Bruce Fields\" \u003cbfields@fieldses.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "a3dab293539031b0970585b9b355cebbc91ecbd4", "tree": "2ad3d7e8f051ab6cf7cefa3872eedf3eeed1247b", "parents": [ "daa7da5fd3040e08e3d7878cd0667c1f4cfd338a" ], "author": { "name": "Adrian Bunk", "email": "bunk@kernel.org", "time": "Mon Apr 14 21:41:32 2008 +0300" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Apr 19 16:55:29 2008 -0400" }, "message": "make nfs_automount_list static\n\nnfs_automount_list can now become static.\n\nSigned-off-by: Adrian Bunk \u003cbunk@kernel.org\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "1d957f9bf87da74f420424d16ece005202bbebd3", "tree": "363d4770c0c74a536524c99ccd2762ce96ee9bbe", "parents": [ "4ac9137858e08a19f29feac4e1f4df7c268b0ba5" ], "author": { "name": "Jan Blunck", "email": "jblunck@suse.de", "time": "Thu Feb 14 19:34:35 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Thu Feb 14 21:13:33 2008 -0800" }, "message": "Introduce path_put()\n\n* Add path_put() functions for releasing a reference to the dentry and\n vfsmount of a struct path in the right order\n\n* Switch from path_release(nd) to path_put(\u0026nd-\u003epath)\n\n* Rename dput_path() to path_put_conditional()\n\n[akpm@linux-foundation.org: fix cifs]\nSigned-off-by: Jan Blunck \u003cjblunck@suse.de\u003e\nSigned-off-by: Andreas Gruenbacher \u003cagruen@suse.de\u003e\nAcked-by: Christoph Hellwig \u003chch@lst.de\u003e\nCc: \u003clinux-fsdevel@vger.kernel.org\u003e\nCc: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nCc: Steven French \u003csfrench@us.ibm.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "4ac9137858e08a19f29feac4e1f4df7c268b0ba5", "tree": "f5b5d84fd12fcc2b0ba0e7ce1a79ff381ad8f5dd", "parents": [ "c5e725f33b733a77de622e91b6ba5645fcf070be" ], "author": { "name": "Jan Blunck", "email": "jblunck@suse.de", "time": "Thu Feb 14 19:34:32 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Thu Feb 14 21:13:33 2008 -0800" }, "message": "Embed a struct path into struct nameidata instead of nd-\u003e{dentry,mnt}\n\nThis is the central patch of a cleanup series. In most cases there is no good\nreason why someone would want to use a dentry for itself. This series reflects\nthat fact and embeds a struct path into nameidata.\n\nTogether with the other patches of this series\n- it enforced the correct order of getting/releasing the reference count on\n \u003cdentry,vfsmount\u003e pairs\n- it prepares the VFS for stacking support since it is essential to have a\n struct path in every place where the stack can be traversed\n- it reduces the overall code size:\n\nwithout patch series:\n text data bss dec hex filename\n5321639 858418 715768 6895825 6938d1 vmlinux\n\nwith patch series:\n text data bss dec hex filename\n5320026 858418 715768 6894212 693284 vmlinux\n\nThis patch:\n\nSwitch from nd-\u003e{dentry,mnt} to nd-\u003epath.{dentry,mnt} everywhere.\n\n[akpm@linux-foundation.org: coding-style fixes]\n[akpm@linux-foundation.org: fix cifs]\n[akpm@linux-foundation.org: fix smack]\nSigned-off-by: Jan Blunck \u003cjblunck@suse.de\u003e\nSigned-off-by: Andreas Gruenbacher \u003cagruen@suse.de\u003e\nAcked-by: Christoph Hellwig \u003chch@lst.de\u003e\nCc: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nCc: Casey Schaufler \u003ccasey@schaufler-ca.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "40c553193df41920de659f0446e5d214c862e827", "tree": "4ee62a79fae0ebba26d119a1aae89bfd1a552c10", "parents": [ "c81468a1a766921f11ae44e8a99816ac8dc7b015" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Dec 14 14:56:07 2007 -0500" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Wed Jan 30 02:05:49 2008 -0500" }, "message": "NFS: Remove the redundant nfs_client-\u003ecl_nfsversion\n\nWe can get the same information from the rpc_ops structure instead.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "560aef74503e928f44ddbf481b8b02d9cef37dbf", "tree": "739c3f1d806d2f34b5573a5ba80d25765e38c887", "parents": [ "40ffbfad6bb79a99cc7627bdaca0ee22dec526f6" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Mon Aug 27 09:14:56 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Sat Sep 01 10:14:36 2007 -0400" }, "message": "NFS: Fix use of cancel_delayed_work_sync in nfs_release_automount_timer\n\nDoh! We can\u0027t use cancel_delayed_work_sync because we may have been called\nfrom an unmount that was being performed by nfs_automount_task.\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "3d39c691ff486142dd9aaeac12f553f4476b7a62", "tree": "78ad555ea06be23933a93ec9c7c68a788f979085", "parents": [ "a4deb81ba8ece75af5560d40d9bb8d242c48a111" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Aug 07 15:28:33 2007 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Tue Aug 07 16:12:50 2007 -0400" }, "message": "NFS: Replace flush_scheduled_work with cancel_work_sync() and friends\n\nThis will avoid deadlocks of the form:\n\nstack backtrace:\n [\u003cc0104fda\u003e] show_trace_log_lvl+0x1a/0x30\n [\u003cc0105c02\u003e] show_trace+0x12/0x20\n [\u003cc0105d15\u003e] dump_stack+0x15/0x20\n [\u003cc013ee42\u003e] __lock_acquire+0xc22/0x1030\n [\u003cc013f2b1\u003e] lock_acquire+0x61/0x80\n [\u003cc012edd9\u003e] flush_workqueue+0x49/0x70\n [\u003cc012ee0d\u003e] flush_scheduled_work+0xd/0x10\n [\u003cdcf55c0c\u003e] nfs_release_automount_timer+0x2c/0x30 [nfs]\n [\u003cdcf45d8e\u003e] nfs_free_server+0x9e/0xd0 [nfs]\n [\u003cdcf4e626\u003e] nfs_kill_super+0x16/0x20 [nfs]\n [\u003cc017b38d\u003e] deactivate_super+0x7d/0xa0\n [\u003cc018f94b\u003e] mntput_no_expire+0x4b/0x80\n [\u003cc018fd94\u003e] expire_mount_list+0xe4/0x140\n [\u003cc0191219\u003e] mark_mounts_for_expiry+0x99/0xb0\n [\u003cdcf55d1d\u003e] nfs_expire_automounts+0xd/0x40 [nfs]\n [\u003cc012e61b\u003e] run_workqueue+0x12b/0x1e0\n [\u003cc012f05b\u003e] worker_thread+0x9b/0x100\n [\u003cc0131c72\u003e] kthread+0x42/0x70\n [\u003cc0104c0f\u003e] kernel_thread_helper+0x7/0x18\n \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\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": "65f27f38446e1976cc98fd3004b110fedcddd189", "tree": "68f8be93feae31dfa018c22db392a05546b63ee1", "parents": [ "365970a1ea76d81cb1ad2f652acb605f06dae256" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Wed Nov 22 14:55:48 2006 +0000" }, "committer": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Wed Nov 22 14:55:48 2006 +0000" }, "message": "WorkStruct: Pass the work_struct pointer instead of context data\n\nPass the work_struct pointer to the work function rather than context data.\nThe work function can use container_of() to work out the data.\n\nFor the cases where the container of the work_struct may go away the moment the\npending bit is cleared, it is made possible to defer the release of the\nstructure by deferring the clearing of the pending bit.\n\nTo make this work, an extra flag is introduced into the management side of the\nwork_struct. This governs auto-release of the structure upon execution.\n\nOrdinarily, the work queue executor would release the work_struct for further\nscheduling or deallocation by clearing the pending bit prior to jumping to the\nwork function. This means that, unless the driver makes some guarantee itself\nthat the work_struct won\u0027t go away, the work function may not access anything\nelse in the work_struct or its container lest they be deallocated.. This is a\nproblem if the auxiliary data is taken away (as done by the last patch).\n\nHowever, if the pending bit is *not* cleared before jumping to the work\nfunction, then the work function *may* access the work_struct and its container\nwith no problems. But then the work function must itself release the\nwork_struct by calling work_release().\n\nIn most cases, automatic release is fine, so this is the default. Special\ninitiators exist for the non-auto-release case (ending in _NAR).\n\n\nSigned-Off-By: David Howells \u003cdhowells@redhat.com\u003e\n" }, { "commit": "52bad64d95bd89e08c49ec5a071fa6dcbe5a1a9c", "tree": "5849b4e3c17daa70a7e81cfdeaddac9ac8a0e953", "parents": [ "0f9005a6f7a82f4aacbd72f7b92322a8ca1c3f97" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Wed Nov 22 14:54:01 2006 +0000" }, "committer": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Wed Nov 22 14:54:01 2006 +0000" }, "message": "WorkStruct: Separate delayable and non-delayable events.\n\nSeparate delayable work items from non-delayable work items be splitting them\ninto a separate structure (delayed_work), which incorporates a work_struct and\nthe timer_list removed from work_struct.\n\nThe work_struct struct is huge, and this limits it\u0027s usefulness. On a 64-bit\narchitecture it\u0027s nearly 100 bytes in size. This reduces that by half for the\nnon-delayable type of event.\n\nSigned-Off-By: David Howells \u003cdhowells@redhat.com\u003e\n" }, { "commit": "038b0a6d8d32db934bba6a24e74e76e4e327a94f", "tree": "5fbeb3e8f140e20f8ce0e33e12b32ec5b0724cd6", "parents": [ "78b656b8bf933101b42409b4492734b23427bfc3" ], "author": { "name": "Dave Jones", "email": "davej@redhat.com", "time": "Wed Oct 04 03:38:54 2006 -0400" }, "committer": { "name": "Dave Jones", "email": "davej@redhat.com", "time": "Wed Oct 04 03:38:54 2006 -0400" }, "message": "Remove all inclusions of \u003clinux/config.h\u003e\nkbuild explicitly includes this at build time.\n\nSigned-off-by: Dave Jones \u003cdavej@redhat.com\u003e\n" }, { "commit": "66f37509fc7191df468a8d183374f48b13bacb73", "tree": "6e9d7d2679297f0c6ebe933e138d4aa07c9caaa3", "parents": [ "b7b52630deca785c4fc2bf4c53500cee066786af" ], "author": { "name": "Adrian Bunk", "email": "bunk@stusta.de", "time": "Wed Sep 27 01:51:13 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Wed Sep 27 08:26:20 2006 -0700" }, "message": "[PATCH] fs/nfs/: make code static\n\nSigned-off-by: Adrian Bunk \u003cbunk@stusta.de\u003e\nAcked-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": "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": "509de8111656a7d89b4a1a5f430f4460ce510f0f", "tree": "94ec0130ab74a5c80956ca3edd3298999b3b9897", "parents": [ "0c7d90cfed91a283228017ba6faf37ee0bcd32b1" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Tue Aug 22 20:06:11 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Sep 22 23:24:34 2006 -0400" }, "message": "NFS: Add extra const qualifiers\n\nAdd some extra const qualifiers into NFS.\n\nSigned-Off-By: David Howells \u003cdhowells@redhat.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "ce510193272c295b891e45525a83b543ae3207c1", "tree": "4b343858aa43e2a1e549ea23c5cff4cf86aeff47", "parents": [ "3f2792ffbd88dc1cd41d226674cc428914981e98" ], "author": { "name": "Josh Triplett", "email": "josht@us.ibm.com", "time": "Mon Jul 24 16:30:00 2006 -0700" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Thu Aug 03 16:55:01 2006 -0400" }, "message": "NFS: Release dcache_lock in an error path of nfs_path\n\nIn one of the error paths of nfs_path, it may return with dcache_lock still\nheld; fix this by adding and using a new error path Elong_unlock which unlocks\ndcache_lock.\n\nSigned-off-by: Josh Triplett \u003cjosh@freedesktop.org\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n(cherry picked from f4b90b43677fb23297c56802c3056fc304f988d9 commit)\n" }, { "commit": "f7b422b17ee5ee4920e8ae24a6ad04bf3481ce72", "tree": "4ae8372762efc092ceb4f884b57cad1efe6594de", "parents": [ "4e5ccf60c5aa79d325c123f47d288a068166f389" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Fri Jun 09 09:34:33 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Jun 09 09:34:33 2006 -0400" }, "message": "NFS: Split fs/nfs/inode.c\n\nAs fs/nfs/inode.c is rather large, heterogenous and unwieldy, the attached\npatch splits it up into a number of files:\n\n (*) fs/nfs/inode.c\n\n Strictly inode specific functions.\n\n (*) fs/nfs/super.c\n\n Superblock management functions for NFS and NFS4, normal access, clones\n and referrals. The NFS4 superblock functions _could_ move out into a\n separate conditionally compiled file, but it\u0027s probably not worth it as\n there\u0027re so many common bits.\n\n (*) fs/nfs/namespace.c\n\n Some namespace-specific functions have been moved here.\n\n (*) fs/nfs/nfs4namespace.c\n\n NFS4-specific namespace functions (this could be merged into the previous\n file). This file is conditionally compiled.\n\n (*) fs/nfs/internal.h\n\n Inter-file declarations, plus a few simple utility functions moved from\n fs/nfs/inode.c.\n\n Additionally, all the in-.c-file externs have been moved here, and those\n files they were moved from now includes this file.\n\nFor the most part, the functions have not been changed, only some multiplexor\nfunctions have changed significantly.\n\nI\u0027ve also:\n\n (*) Added some extra banner comments above some functions.\n\n (*) Rearranged the function order within the files to be more logical and\n better grouped (IMO), though someone may prefer a different order.\n\n (*) Reduced the number of #ifdefs in .c files.\n\n (*) Added missing __init and __exit directives.\n\nSigned-Off-By: David Howells \u003cdhowells@redhat.com\u003e\n" }, { "commit": "6b97fd3da1eab2cc490cfe884c7d4956522eaf8b", "tree": "055f5f603101a15f783ae52081e4714ddc3edc96", "parents": [ "9cdb3883c38f883436a84c2353a4cf964ff890a2" ], "author": { "name": "Manoj Naik", "email": "manoj@almaden.ibm.com", "time": "Fri Jun 09 09:34:29 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Jun 09 09:34:29 2006 -0400" }, "message": "NFSv4: Follow a referral\n\nRespond to a moved error on NFS lookup by setting up the referral.\nNote: We don\u0027t actually follow the referral during lookup/getattr, but\nlater when we detect fsid mismatch in inode revalidation (similar to the\nprocessing done for cloning submounts). Referrals will have fake attributes\nuntil they are actually followed or traversed.\n\nSigned-off-by: Manoj Naik \u003cmanoj@almaden.ibm.com\u003e\nSigned-off-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\n" }, { "commit": "51d8fa6a109589d522c18a8e9bf3fb167a91b1bc", "tree": "a9ec5e642bfa64f7cf1fde01617b30bac385bac6", "parents": [ "55a975937d40cac582e981ddc8ed783b3dcc043c" ], "author": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Jun 09 09:34:20 2006 -0400" }, "committer": { "name": "Trond Myklebust", "email": "Trond.Myklebust@netapp.com", "time": "Fri Jun 09 09:34:20 2006 -0400" }, "message": "NFS: Add timeout to submounts\n\nMake automounted partitions expire using the mark_mounts_for_expiry()\nfunction. The timeout is controlled via a sysctl.\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" } ] }