)]}' { "log": [ { "commit": "8931221411f9ff950de8fd686dc5ab881394cb9a", "tree": "63d97668bae62d1034002cc7778adde2aa18b346", "parents": [ "c14cc63a63e94d490ac6517a555113c30d420db4" ], "author": { "name": "Ian Kent", "email": "raven@themaw.net", "time": "Tue Jan 18 12:06:10 2011 +0800" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Tue Jan 18 01:21:26 2011 -0500" }, "message": "vfs - fix dentry ref count in do_lookup()\n\nThere is a ref count problem in fs/namei.c:do_lookup().\n\nWhen walking in ref-walk mode, if follow_managed() returns a fail we\nneed to drop dentry and possibly vfsmount. Clean up properly,\nas we do in the other caller of follow_managed().\n\nSigned-off-by: Ian Kent \u003craven@themaw.net\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "19a167af7c97248ec646552ebc9140bc6aa3552a", "tree": "a521153c80fa6e40b2b4983c5bba21c2e96d1864", "parents": [ "e78bf5e6cbe837daa6ab628a5f679548742994d3" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Mon Jan 17 01:35:23 2011 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Mon Jan 17 01:35:23 2011 -0500" }, "message": "Take the completion of automount into new helper\n\n... and shift it from namei.c to namespace.c\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "f03c65993b98eeb909a4012ce7833c5857d74755", "tree": "a6dd5e353889b7fe4ab87c54170d09443d788fec", "parents": [ "7b8a53fd815deb39542085897743fa0063f9fe06" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jan 14 22:30:21 2011 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sun Jan 16 13:47:07 2011 -0500" }, "message": "sanitize vfsmount refcounting changes\n\nInstead of splitting refcount between (per-cpu) mnt_count\nand (SMP-only) mnt_longrefs, make all references contribute\nto mnt_count again and keep track of how many are longterm\nones.\n\nAccounting rules for longterm count:\n\t* 1 for each fs_struct.root.mnt\n\t* 1 for each fs_struct.pwd.mnt\n\t* 1 for having non-NULL -\u003emnt_ns\n\t* decrement to 0 happens only under vfsmount lock exclusive\n\nThat allows nice common case for mntput() - since we can\u0027t drop the\nfinal reference until after mnt_longterm has reached 0 due to the rules\nabove, mntput() can grab vfsmount lock shared and check mnt_longterm.\nIf it turns out to be non-zero (which is the common case), we know\nthat this is not the final mntput() and can just blindly decrement\npercpu mnt_count. Otherwise we grab vfsmount lock exclusive and\ndo usual decrement-and-check of percpu mnt_count.\n\nFor fs_struct.c we have mnt_make_longterm() and mnt_make_shortterm();\nnamespace.c uses the latter in places where we don\u0027t already hold\nvfsmount lock exclusive and opencodes a few remaining spots where\nwe need to manipulate mnt_longterm.\n\nNote that we mostly revert the code outside of fs/namespace.c back\nto what we used to have; in particular, normal code doesn\u0027t need\nto care about two kinds of references, etc. And we get to keep\nthe optimization Nick\u0027s variant had bought us...\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": "ab90911ff90cdab59b31c045c3f0ae480d14f29d", "tree": "683450a66eb9dc6bf053e38d63f4740bb53a7b6e", "parents": [ "87556ef19926e97464e0163a7840140527ae6615" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Fri Jan 14 18:46:51 2011 +0000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Jan 15 20:07:47 2011 -0500" }, "message": "Allow d_manage() to be used in RCU-walk mode\n\nAllow d_manage() to be called from pathwalk when it is in RCU-walk mode as well\nas when it is in Ref-walk mode. This permits __follow_mount_rcu() to call\nd_manage() directly. d_manage() needs a parameter to indicate that it is in\nRCU-walk mode as it isn\u0027t allowed to sleep if in that mode (but should return\n-ECHILD instead).\n\nautofs4_d_manage() can then be set to retain RCU-walk mode if the daemon\naccesses it and otherwise request dropping back to ref-walk mode.\n\nSigned-off-by: David Howells \u003cdhowells@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "87556ef19926e97464e0163a7840140527ae6615", "tree": "04f75b49a88523e4700a2cbf73c04c75697c419a", "parents": [ "1972580bb4edea3ed6fe273b2ca72f44f10f8c86" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Fri Jan 14 18:46:46 2011 +0000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Jan 15 20:07:46 2011 -0500" }, "message": "Remove a further kludge from __do_follow_link()\n\nRemove a further kludge from __do_follow_link() as it\u0027s no longer required with\nthe automount code.\n\nThis reverts the non-helper-function parts of\n051d381259eb57d6074d02a6ba6e90e744f1a29f, which breaks union mounts.\n\nReported-by: vaurora@redhat.com\nSigned-off-by: David Howells \u003cdhowells@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "db3729153e82ba3ada89681f26c4f1b6d6807a80", "tree": "55714bb93d9377e5f8762b9030e87fec3e469645", "parents": [ "01c64feac45cea1317263eabc4f7ee1b240f297f" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Fri Jan 14 18:45:53 2011 +0000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Jan 15 20:07:36 2011 -0500" }, "message": "Remove the automount through follow_link() kludge code from pathwalk\n\nRemove the automount through follow_link() kludge code from pathwalk in favour\nof using d_automount().\n\nSigned-off-by: David Howells \u003cdhowells@redhat.com\u003e\nAcked-by: Ian Kent \u003craven@themaw.net\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "6f45b65672c8017d5e210e338bb5858a938ef445", "tree": "b62877c489fb682033c37d30d91c53e4d0c3833a", "parents": [ "cc53ce53c86924bfe98a12ea20b7465038a08792" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Fri Jan 14 18:45:31 2011 +0000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Jan 15 20:07:33 2011 -0500" }, "message": "Add an AT_NO_AUTOMOUNT flag to suppress terminal automount\n\nAdd an AT_NO_AUTOMOUNT flag to suppress terminal automounting of automount\npoint directories. This can be used by fstatat() users to permit the\ngathering of attributes on an automount point and also prevent\nmass-automounting of a directory of automount points by ls.\n\nSigned-off-by: David Howells \u003cdhowells@redhat.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": "9875cf806403fae66b2410a3c2cc820d97731e04", "tree": "6f9546b400716766af95e0f78e3d600e765b2b51", "parents": [ "1a8edf40e7c3eee955e0dd0316a7c9d85e36f597" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Fri Jan 14 18:45:21 2011 +0000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Jan 15 20:05:03 2011 -0500" }, "message": "Add a dentry op to handle automounting rather than abusing follow_link()\n\nAdd a dentry op (d_automount) to handle automounting directories rather than\nabusing the follow_link() inode operation. The operation is keyed off a new\ndentry flag (DCACHE_NEED_AUTOMOUNT).\n\nThis also makes it easier to add an AT_ flag to suppress terminal segment\nautomount during pathwalk and removes the need for the kludge code in the\npathwalk algorithm to handle directories with follow_link() semantics.\n\nThe -\u003ed_automount() dentry operation:\n\n\tstruct vfsmount *(*d_automount)(struct path *mountpoint);\n\ntakes a pointer to the directory to be mounted upon, which is expected to\nprovide sufficient data to determine what should be mounted. If successful, it\nshould return the vfsmount struct it creates (which it should also have added\nto the namespace using do_add_mount() or similar). If there\u0027s a collision with\nanother automount attempt, NULL should be returned. If the directory specified\nby the parameter should be used directly rather than being mounted upon,\n-EISDIR should be returned. In any other case, an error code should be\nreturned.\n\nThe -\u003ed_automount() operation is called with no locks held and may sleep. At\nthis point the pathwalk algorithm will be in ref-walk mode.\n\nWithin fs/namei.c itself, a new pathwalk subroutine (follow_automount()) is\nadded to handle mountpoints. It will return -EREMOTE if the automount flag was\nset, but no d_automount() op was supplied, -ELOOP if we\u0027ve encountered too many\nsymlinks or mountpoints, -EISDIR if the walk point should be used without\nmounting and 0 if successful. The path will be updated to point to the mounted\nfilesystem if a successful automount took place.\n\n__follow_mount() is replaced by follow_managed() which is more generic\n(especially with the patch that adds -\u003ed_manage()). This handles transits from\ndirectories during pathwalk, including automounting and skipping over\nmountpoints (and holding processes with the next patch).\n\n__follow_mount_rcu() will jump out of RCU-walk mode if it encounters an\nautomount point with nothing mounted on it.\n\nfollow_dotdot*() does not handle automounts as you don\u0027t want to trigger them\nwhilst following \"..\".\n\nI\u0027ve also extracted the mount/don\u0027t-mount logic from autofs4 and included it\nhere. It makes the mount go ahead anyway if someone calls open() or creat(),\ntries to traverse the directory, tries to chdir/chroot/etc. into the directory,\nor sticks a \u0027/\u0027 on the end of the pathname. If they do a stat(), however,\nthey\u0027ll only trigger the automount if they didn\u0027t also say O_NOFOLLOW.\n\nI\u0027ve also added an inode flag (S_AUTOMOUNT) so that filesystems can mark their\ninodes as automount points. This flag is automatically propagated to the\ndentry as DCACHE_NEED_AUTOMOUNT by __d_instantiate(). This saves NFS and could\nsave AFS a private flag bit apiece, but is not strictly necessary. It would be\npreferable to do the propagation in d_set_d_op(), but that doesn\u0027t normally\nhave access to the inode.\n\n[AV: fixed breakage in case if __follow_mount_rcu() fails and nameidata_drop_rcu()\nsucceeds in RCU case of do_lookup(); we need to fall through to non-RCU case after\nthat, rather than just returning with ungrabbed *path]\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": "1a8edf40e7c3eee955e0dd0316a7c9d85e36f597", "tree": "e9776058ac678d6f8b29429e77d2de58dcc83608", "parents": [ "d73b388459b1ee2e80f8ff9c1916d75640d7d920" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Jan 15 13:12:53 2011 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Jan 15 20:03:39 2011 -0500" }, "message": "do_lookup() fix\n\ndo_lookup() has a path leading from LOOKUP_RCU case to non-RCU\ncrossing of mountpoints, which breaks things badly. If we\nhit need_revalidate: and do nothing in there, we need to come\nback into LOOKUP_RCU half of things, not to done: in non-RCU\none.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "acda4721ae876dedab3fef04bbd8020bfa67ff0a", "tree": "ec48d554fe48b3915912e2ae62bc962ade0553bd", "parents": [ "822e5215f9eef86c1dd56d5696bf55a212b0e3f0", "32385c7cf60a78375b63afc4f02001df84dfd1a0" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Jan 14 09:08:29 2011 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Jan 14 09:08:29 2011 -0800" }, "message": "Merge branch \u0027vfs-scale-working\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/npiggin/linux-npiggin\n\n* \u0027vfs-scale-working\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/npiggin/linux-npiggin:\n kernel: fix hlist_bl again\n cgroups: Fix a lockdep warning at cgroup removal\n fs: namei fix -\u003eput_link on wrong inode in do_filp_open\n" }, { "commit": "7b9337aaf98f9941d0927a75217d3ff31afec609", "tree": "d61753169b24d521147116717f310dfb65d3d20d", "parents": [ "f20877d94a74557b7c28b4ed8920d834c31e0ea5" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 14 08:42:43 2011 +0000" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 14 08:42:43 2011 +0000" }, "message": "fs: namei fix -\u003eput_link on wrong inode in do_filp_open\n\nJ. R. Okajima noticed that -\u003eput_link is being attempted on the\nwrong inode, and suggested the way to fix it. I changed it a bit\naccording to Al\u0027s suggestion to keep an explicit link path around.\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\n" }, { "commit": "db9effe99adc67c53e6aedadadd2aa9a02342e48", "tree": "efe7ebfe951972568189ef2548130f0abd515210", "parents": [ "9c4bc1c2befbbdce4b9fd526e67a7a2ea143ffa2", "f20877d94a74557b7c28b4ed8920d834c31e0ea5" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jan 13 20:14:13 2011 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jan 13 20:14:13 2011 -0800" }, "message": "Merge branch \u0027vfs-scale-working\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/npiggin/linux-npiggin\n\n* \u0027vfs-scale-working\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/npiggin/linux-npiggin:\n fs: fix do_last error case when need_reval_dot\n nfs: add missing rcu-walk check\n fs: hlist UP debug fixup\n fs: fix dropping of rcu-walk from force_reval_path\n fs: force_reval_path drop rcu-walk before d_invalidate\n fs: small rcu-walk documentation fixes\n\nFixed up trivial conflicts in Documentation/filesystems/porting\n" }, { "commit": "f20877d94a74557b7c28b4ed8920d834c31e0ea5", "tree": "88ea4019a1a8a0a3b8c2f24479ec44b353928f3f", "parents": [ "657e94b673a805b427903c5628e95348235fad06" ], "author": { "name": "J. R. Okajima", "email": "hooanon05@yahoo.co.jp", "time": "Fri Jan 14 03:56:04 2011 +0000" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 14 03:56:04 2011 +0000" }, "message": "fs: fix do_last error case when need_reval_dot\n\nWhen open(2) without O_DIRECTORY opens an existing dir, it should return\nEISDIR. In do_last(), the variable \u0027error\u0027 is initialized EISDIR, but it\nis changed by d_revalidate() which returns any positive to represent\n\u0027the target dir is valid.\u0027\n\nShould we keep and return the initialized \u0027error\u0027 in this case.\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\n" }, { "commit": "90dbb77ba48dddb87445d238e84cd137cf97dd98", "tree": "446772602e5944075a6c614db05dd06681f3f3d8", "parents": [ "bb20c18db6fbb5e6ba499c76473a487d35073467" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 14 02:36:19 2011 +0000" }, "committer": { "name": "Nick Piggin", "email": "npiggin@hera.kernel.org", "time": "Fri Jan 14 02:36:19 2011 +0000" }, "message": "fs: fix dropping of rcu-walk from force_reval_path\n\nAs J. R. Okajima noted, force_reval_path passes in the same dentry to\nd_revalidate as the one in the nameidata structure (other callers pass in a\nchild), so the locking breaks. This can oops with a chrooted nfs mount, for\nexample. Similarly there can be other problems with revalidating a dentry\nwhich is already in nameidata of the path walk.\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\n" }, { "commit": "bb20c18db6fbb5e6ba499c76473a487d35073467", "tree": "ca0426f86e5c2a331bd5006d7f16795711b7ade2", "parents": [ "a82416da83722944d78d933301e32e7c5ad70edd" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 14 02:35:53 2011 +0000" }, "committer": { "name": "Nick Piggin", "email": "npiggin@hera.kernel.org", "time": "Fri Jan 14 02:35:53 2011 +0000" }, "message": "fs: force_reval_path drop rcu-walk before d_invalidate\n\nd_revalidate can return in rcu-walk mode even when it returns 0. We can\u0027t just\ncall any old dcache function on rcu-walk dentry (the dentry is unstable, so\neven through d_lock can safely be taken, the result may no longer be what we\nexpect -- careful re-checks would be required). So just drop rcu in this case.\n\n(I missed this conversion when switching to the rcu-walk convention that Linus\nsuggested)\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\n" }, { "commit": "e1181ee6575d7970bad15aaa852784b4972d2af8", "tree": "cf1010ee6361e8bcd98ea2f29fa899fa52bf801d", "parents": [ "cccb5a1e698535fa5a734ffe21c7061c97f8d8c5" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Dec 07 16:19:50 2010 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Jan 12 20:06:59 2011 -0500" }, "message": "vfs: pass struct file to do_truncate on O_TRUNC opens (try #2)\n\nWhen a file is opened with O_TRUNC, the truncate processing is handled\nby handle_truncate(). This function however doesn\u0027t receive any info\nabout the newly instantiated filp, and therefore can\u0027t pass that info\nalong so that the setattr can use it.\n\nThis makes NFSv4 misbehave. The client does an open and gets a valid\nstateid, and then doesn\u0027t use that stateid on the subsequent truncate.\nIt uses the zero-stateid instead. Most servers ignore this fact and\njust do the truncate anyway, but some don\u0027t like it (notably, RHEL4).\n\nIt seems more correct that since we have a fully instantiated file at\nthe time that handle_truncate is called, that we pass that along so\nthat the truncate operation can properly use it.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "39191628ed169510db2f3f472e1ec14e08f9690f", "tree": "edea4bf852861bc126274b186f2997366ea09a34", "parents": [ "0c21e3aaf6ae85bee804a325aa29c325209180fd" ], "author": { "name": "Randy Dunlap", "email": "randy.dunlap@oracle.com", "time": "Sat Jan 08 19:36:21 2011 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Jan 10 07:38:53 2011 -0800" }, "message": "fs: fix namei.c kernel-doc notation\n\nFix new kernel-doc notation warnings in fs/namei.c and spell\nECHILD correctly.\n\n Warning(fs/namei.c:218): No description found for parameter \u0027flags\u0027\n Warning(fs/namei.c:425): Excess function parameter \u0027Returns\u0027 description in \u0027nameidata_drop_rcu\u0027\n Warning(fs/namei.c:478): Excess function parameter \u0027Returns\u0027 description in \u0027nameidata_dentry_drop_rcu\u0027\n Warning(fs/namei.c:540): Excess function parameter \u0027Returns\u0027 description in \u0027nameidata_drop_rcu_last\u0027\n\nSigned-off-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nCc:\tNick Piggin \u003cnpiggin@kernel.dk\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "b3e19d924b6eaf2ca7d22cba99a517c5171007b6", "tree": "8c1fa4074114a883a4e2de2f7d12eb29ed91bdf1", "parents": [ "c6653a838b1b2738561aff0b8c0f62a9b714bdd9" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:11 2011 +1100" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:33 2011 +1100" }, "message": "fs: scale mntget/mntput\n\nThe problem that this patch aims to fix is vfsmount refcounting scalability.\nWe need to take a reference on the vfsmount for every successful path lookup,\nwhich often go to the same mount point.\n\nThe fundamental difficulty is that a \"simple\" reference count can never be made\nscalable, because any time a reference is dropped, we must check whether that\nwas the last reference. To do that requires communication with all other CPUs\nthat may have taken a reference count.\n\nWe can make refcounts more scalable in a couple of ways, involving keeping\ndistributed counters, and checking for the global-zero condition less\nfrequently.\n\n- check the global sum once every interval (this will delay zero detection\n for some interval, so it\u0027s probably a showstopper for vfsmounts).\n\n- keep a local count and only taking the global sum when local reaches 0 (this\n is difficult for vfsmounts, because we can\u0027t hold preempt off for the life of\n a reference, so a counter would need to be per-thread or tied strongly to a\n particular CPU which requires more locking).\n\n- keep a local difference of increments and decrements, which allows us to sum\n the total difference and hence find the refcount when summing all CPUs. Then,\n keep a single integer \"long\" refcount for slow and long lasting references,\n and only take the global sum of local counters when the long refcount is 0.\n\nThis last scheme is what I implemented here. Attached mounts and process root\nand working directory references are \"long\" references, and everything else is\na short reference.\n\nThis allows scalable vfsmount references during path walking over mounted\nsubtrees and unattached (lazy umounted) mounts with processes still running\nin them.\n\nThis results in one fewer atomic op in the fastpath: mntget is now just a\nper-CPU inc, rather than an atomic inc; and mntput just requires a spinlock\nand non-atomic decrement in the common case. However code is otherwise bigger\nand heavier, so single threaded performance is basically a wash.\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\n" }, { "commit": "b74c79e99389cd79b31fcc08f82c24e492e63c7e", "tree": "763c6b412517306670bc625e90035f2d16bb739f", "parents": [ "34286d6662308d82aed891852d04c7c3a2649b16" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:49:58 2011 +1100" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:29 2011 +1100" }, "message": "fs: provide rcu-walk aware permission i_ops\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\n" }, { "commit": "34286d6662308d82aed891852d04c7c3a2649b16", "tree": "c4b7311404d302e7cb94df7a4690298e1059910a", "parents": [ "44a7d7a878c9cbb74f236ea755b25b6b2e26a9a9" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:49:57 2011 +1100" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:29 2011 +1100" }, "message": "fs: rcu-walk aware d_revalidate method\n\nRequire filesystems be aware of .d_revalidate being called in rcu-walk\nmode (nd-\u003eflags \u0026 LOOKUP_RCU). For now do a simple push down, returning\n-ECHILD from all implementations.\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\n" }, { "commit": "fb045adb99d9b7c562dc7fef834857f78249daa1", "tree": "1fd6a4024fffeec568abe100d730589bfdb81c38", "parents": [ "5f57cbcc02cf18f6b22ef4066bb10afeb8f930ff" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:49:55 2011 +1100" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:28 2011 +1100" }, "message": "fs: dcache reduce branches in lookup path\n\nReduce some branches and memory accesses in dcache lookup by adding dentry\nflags to indicate common d_ops are set, rather than having to check them.\nThis saves a pointer memory access (dentry-\u003ed_op) in common path lookup\nsituations, and saves another pointer load and branch in cases where we\nhave d_op but not the particular operation.\n\nPatched with:\n\ngit grep -E \u0027[.\u003e]([[:space:]])*d_op([[:space:]])*\u003d\u0027 | xargs sed -e \u0027s/\\([^\\t ]*\\)-\u003ed_op \u003d \\(.*\\);/d_set_d_op(\\1, \\2);/\u0027 -e \u0027s/\\([^\\t ]*\\)\\.d_op \u003d \\(.*\\);/d_set_d_op(\\\u0026\\1, \\2);/\u0027 -i\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\n" }, { "commit": "c28cc36469554dc55540f059fbdc7fa22a2c31fc", "tree": "6b867456be48b8633a2d56a99e00bb3faf9dccc7", "parents": [ "31e6b01f4183ff419a6d1f86177cbf4662347cec" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:49:53 2011 +1100" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:27 2011 +1100" }, "message": "fs: fs_struct use seqlock\n\nUse a seqlock in the fs_struct to enable us to take an atomic copy of the\ncomplete cwd and root paths. Use this in the RCU lookup path to avoid a\nthread-shared spinlock in RCU lookup operations.\n\nMulti-threaded apps may now perform path lookups with scalability matching\nmulti-process apps. Operations such as stat(2) become very scalable for\nmulti-threaded workload.\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\n" }, { "commit": "31e6b01f4183ff419a6d1f86177cbf4662347cec", "tree": "e215ec9af88352c55e024f784f3d9f8eb13fab85", "parents": [ "3c22cd5709e8143444a6d08682a87f4c57902df3" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:49:52 2011 +1100" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:27 2011 +1100" }, "message": "fs: rcu-walk for path lookup\n\nPerform common cases of path lookups without any stores or locking in the\nancestor dentry elements. This is called rcu-walk, as opposed to the current\nalgorithm which is a refcount based walk, or ref-walk.\n\nThis results in far fewer atomic operations on every path element,\nsignificantly improving path lookup performance. It also avoids cacheline\nbouncing on common dentries, significantly improving scalability.\n\nThe overall design is like this:\n* LOOKUP_RCU is set in nd-\u003eflags, which distinguishes rcu-walk from ref-walk.\n* Take the RCU lock for the entire path walk, starting with the acquiring\n of the starting path (eg. root/cwd/fd-path). So now dentry refcounts are\n not required for dentry persistence.\n* synchronize_rcu is called when unregistering a filesystem, so we can\n access d_ops and i_ops during rcu-walk.\n* Similarly take the vfsmount lock for the entire path walk. So now mnt\n refcounts are not required for persistence. Also we are free to perform mount\n lookups, and to assume dentry mount points and mount roots are stable up and\n down the path.\n* Have a per-dentry seqlock to protect the dentry name, parent, and inode,\n so we can load this tuple atomically, and also check whether any of its\n members have changed.\n* Dentry lookups (based on parent, candidate string tuple) recheck the parent\n sequence after the child is found in case anything changed in the parent\n during the path walk.\n* inode is also RCU protected so we can load d_inode and use the inode for\n limited things.\n* i_mode, i_uid, i_gid can be tested for exec permissions during path walk.\n* i_op can be loaded.\n\nWhen we reach the destination dentry, we lock it, recheck lookup sequence,\nand increment its refcount and mountpoint refcount. RCU and vfsmount locks\nare dropped. This is termed \"dropping rcu-walk\". If the dentry refcount does\nnot match, we can not drop rcu-walk gracefully at the current point in the\nlokup, so instead return -ECHILD (for want of a better errno). This signals the\npath walking code to re-do the entire lookup with a ref-walk.\n\nAside from the final dentry, there are other situations that may be encounted\nwhere we cannot continue rcu-walk. In that case, we drop rcu-walk (ie. take\na reference on the last good dentry) and continue with a ref-walk. Again, if\nwe can drop rcu-walk gracefully, we return -ECHILD and do the whole lookup\nusing ref-walk. But it is very important that we can continue with ref-walk\nfor most cases, particularly to avoid the overhead of double lookups, and to\ngain the scalability advantages on common path elements (like cwd and root).\n\nThe cases where rcu-walk cannot continue are:\n* NULL dentry (ie. any uncached path element)\n* parent with d_inode-\u003ei_op-\u003epermission or ACLs\n* dentries with d_revalidate\n* Following links\n\nIn future patches, permission checks and d_revalidate become rcu-walk aware. It\nmay be possible eventually to make following links rcu-walk aware.\n\nUncached path elements will always require dropping to ref-walk mode, at the\nvery least because i_mutex needs to be grabbed, and objects allocated.\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\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": "b7ab39f631f505edc2bbdb86620d5493f995c9da", "tree": "62be97ebc7fc69ceb601f23312d335ebb8038ee7", "parents": [ "2304450783dfde7b0b94ae234edd0dbffa865073" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:49:32 2011 +1100" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:21 2011 +1100" }, "message": "fs: dcache scale dentry refcount\n\nMake d_count non-atomic and protect it with d_lock. This allows us to ensure a\n0 refcount dentry remains 0 without dcache_lock. It is also fairly natural when\nwe start protecting many other dentry members with d_lock.\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\n" }, { "commit": "b1e6a015a580ad145689ad1d6b4aa0e03e6c868b", "tree": "57a10ef164e4d2f798d9b832dbeaf973aca2ab83", "parents": [ "621e155a3591962420eacdd39f6f0aa29ceb221e" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:49:28 2011 +1100" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:20 2011 +1100" }, "message": "fs: change d_hash for rcu-walk\n\nChange d_hash so it may be called from lock-free RCU lookups. See similar\npatch for d_compare for details.\n\nFor in-tree filesystems, this is just a mechanical change.\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\n" }, { "commit": "b1085ba80cd2784400a7beec3fda5099198ed01c", "tree": "c83530f7d7f161c898d464fc1ce65be38a0268c1", "parents": [ "88d60c32765716289abeb362c44adf6c35c6824c" ], "author": { "name": "Lino Sanfilippo", "email": "LinoSanfilippo@gmx.de", "time": "Fri Nov 05 17:05:27 2010 +0100" }, "committer": { "name": "Eric Paris", "email": "eparis@redhat.com", "time": "Tue Dec 07 16:14:21 2010 -0500" }, "message": "fanotify: if set by user unset FMODE_NONOTIFY before fsnotify_perm() is called\n\nUnsetting FMODE_NONOTIFY in fsnotify_open() is too late, since fsnotify_perm()\nis called before. If FMODE_NONOTIFY is set fsnotify_perm() will skip permission\nchecks, so a user can still disable permission checks by setting this flag\nin an open() call.\nThis patch corrects this by unsetting the flag before fsnotify_perm is called.\n\nSigned-off-by: Lino Sanfilippo \u003cLinoSanfilippo@gmx.de\u003e\nSigned-off-by: Eric Paris \u003ceparis@redhat.com\u003e\n" }, { "commit": "d893f1bc2a9f0f7dcb4b433452c59f9bedac0d7d", "tree": "b3cf84a271ccb19529d83a544b6024bbb23a7801", "parents": [ "a4118ee1d80b527c385cadd14db79559efb8a493" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Oct 29 03:30:42 2010 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Oct 29 04:14:56 2010 -0400" }, "message": "fix open/umount race\n\nnameidata_to_filp() drops nd-\u003epath or transfers it to opened\nfile. In the former case it\u0027s a Bad Idea(tm) to do mnt_drop_write()\non nd-\u003epath.mnt, since we might race with umount and vfsmount in\nquestion might be gone already.\n\nFix: don\u0027t drop it, then... IOW, have nameidata_to_filp() grab nd-\u003epath\nin case it transfers it to file and do path_drop() in callers. After\nthey are through with accessing nd-\u003epath...\n\nReported-by: Miklos Szeredi \u003cmiklos@szeredi.hu\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "7de9c6ee3ecffd99e1628e81a5ea5468f7581a1f", "tree": "88787e77ba8a253d0a26aeda4bd5e58532d592e0", "parents": [ "646ec4615cd05972581c9c5342ed7a1e77df17bb" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Oct 23 11:11:40 2010 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Mon Oct 25 21:26:11 2010 -0400" }, "message": "new helper: ihold()\n\nClones an existing reference to inode; caller must already hold one.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "81fca444001e5a41ab80ce8cf9a5734c00ec6546", "tree": "1d345a633c666686830a679a0a90c1f7c69470e6", "parents": [ "72e58063d63c5f0a7bf65312f1e3a5ed9bb5c2ff" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Wed Oct 06 10:47:47 2010 +0200" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Mon Oct 25 21:18:19 2010 -0400" }, "message": "fs: move permission check back into __lookup_hash\n\nThe caller that didn\u0027t need it is gone.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "99b7db7b8ffd6bb755eb0a175596421a0b581cb2", "tree": "cbaf57d252f0852f967d3fd5a5f87472964a01fe", "parents": [ "6416ccb7899960868f5016751fb81bf25213d24f" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Wed Aug 18 04:37:39 2010 +1000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Aug 18 08:35:48 2010 -0400" }, "message": "fs: brlock vfsmount_lock\n\nfs: brlock vfsmount_lock\n\nUse a brlock for the vfsmount lock. It must be taken for write whenever\nmodifying the mount hash or associated fields, and may be taken for read when\nperforming mount hash lookups.\n\nA new lock is added for the mnt-id allocator, so it doesn\u0027t need to take\nthe heavy vfsmount write-lock.\n\nThe number of atomics should remain the same for fastpath rlock cases, though\ncode would be slightly slower due to per-cpu access. Scalability is not not be\nmuch improved in common cases yet, due to other locks (ie. dcache_lock) getting\nin the way. However path lookups crossing mountpoints should be one case where\nscalability is improved (currently requiring the global lock).\n\nThe slowpath is slower due to use of brlock. On a 64 core, 64 socket, 32 node\nAltix system (high latency to remote nodes), a simple umount microbenchmark\n(mount --bind mnt mnt2 ; umount mnt2 loop 1000 times), before this patch it\ntook 6.8s, afterwards took 7.1s, about 5% slower.\n\nCc: Al Viro \u003cviro@ZenIV.linux.org.uk\u003e\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "b04f784e5d19ed58892833dae845738972cea260", "tree": "6060e063b5a51461fd60630d57318778fe987148", "parents": [ "2a4419b5b2a77f3f4537c14f7ad7df95770655dd" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Wed Aug 18 04:37:34 2010 +1000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Aug 18 08:35:47 2010 -0400" }, "message": "fs: remove extra lookup in __lookup_hash\n\nfs: remove extra lookup in __lookup_hash\n\nOptimize lookup for create operations, where no dentry should often be\ncommon-case. In cases where it is not, such as unlink, the added overhead\nis much smaller than the removed.\n\nAlso, move comments about __d_lookup racyness to the __d_lookup call site.\nd_lookup is intuitive; __d_lookup is what needs commenting. So in that same\nvein, add kerneldoc comments to __d_lookup and clean up some of the comments:\n\n- We are interested in how the RCU lookup works here, particularly with\n renames. Make that explicit, and point to the document where it is explained\n in more detail.\n- RCU is pretty standard now, and macros make implementations pretty mindless.\n If we want to know about RCU barrier details, we look in RCU code.\n- Delete some boring legacy comments because we don\u0027t care much about how the\n code used to work, more about the interesting parts of how it works now. So\n comments about lazy LRU may be interesting, but would better be done in the\n LRU or refcount management code.\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "baa0389073eb7beb9d36f6d13df97e16c1bfa626", "tree": "38567c4e315f74fe086e4f7459df2a1cadd9add6", "parents": [ "2e2e88ea8c3bd9e1bd6e42faf047a4ac3fbb3b2f" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Wed Aug 18 04:37:31 2010 +1000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Aug 18 08:35:45 2010 -0400" }, "message": "fs: dentry allocation consolidation\n\nfs: dentry allocation consolidation\n\nThere are 2 duplicate copies of code in dentry allocation in path lookup.\nConsolidate them into a single function.\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "2e2e88ea8c3bd9e1bd6e42faf047a4ac3fbb3b2f", "tree": "398f6de92d9e41419297354a26e25e0db8a693f6", "parents": [ "3a48ee8a4ad26c3a538b6fc11a86a8f80c3dce18" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Wed Aug 18 04:37:30 2010 +1000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Aug 18 08:35:45 2010 -0400" }, "message": "fs: fix do_lookup false negative\n\nfs: fix do_lookup false negative\n\nIn do_lookup, if we initially find no dentry, we take the directory i_mutex and\nre-check the lookup. If we find a dentry there, then we revalidate it if\nneeded. However if that revalidate asks for the dentry to be invalidated, we\nreturn -ENOENT from do_lookup. What should happen instead is an attempt to\nallocate and lookup a new dentry.\n\nThis is probably not noticed because it is rare. It is only reached if a\nconcurrent create races in first (in which case, the dentry probably won\u0027t be\ninvalidated anyway), or if the racy __d_lookup has failed due to a\nfalse-negative (which is very rare).\n\nFix this by removing code and have it use the normal reval path.\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "f7ad3c6be90809b53b7f0ae9d4eaa45ce2564a79", "tree": "dc9b09188bab35320200f318b5e7b52f24dc43ad", "parents": [ "542ce7a9bc6b3838832ae0f4f8de30c667af8ff3" ], "author": { "name": "Miklos Szeredi", "email": "mszeredi@suse.cz", "time": "Tue Aug 10 11:41:36 2010 +0200" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Aug 11 00:28:20 2010 -0400" }, "message": "vfs: add helpers to get root and pwd\n\nAdd three helpers that retrieve a refcounted copy of the root and cwd\nfrom the supplied fs_struct.\n\n get_fs_root()\n get_fs_pwd()\n get_fs_root_and_pwd()\n\nSigned-off-by: Miklos Szeredi \u003cmszeredi@suse.cz\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "8c8946f509a494769a8c602b5ed189df01917d39", "tree": "dfd96bd6ca5ea6803c6d77f65ba37e04f78b2d3b", "parents": [ "5f248c9c251c60af3403902b26e08de43964ea0b", "1968f5eed54ce47bde488fd9a450912e4a2d7138" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Aug 10 11:39:13 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Aug 10 11:39:13 2010 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.infradead.org/users/eparis/notify\n\n* \u0027for-linus\u0027 of git://git.infradead.org/users/eparis/notify: (132 commits)\n fanotify: use both marks when possible\n fsnotify: pass both the vfsmount mark and inode mark\n fsnotify: walk the inode and vfsmount lists simultaneously\n fsnotify: rework ignored mark flushing\n fsnotify: remove global fsnotify groups lists\n fsnotify: remove group-\u003emask\n fsnotify: remove the global masks\n fsnotify: cleanup should_send_event\n fanotify: use the mark in handler functions\n audit: use the mark in handler functions\n dnotify: use the mark in handler functions\n inotify: use the mark in handler functions\n fsnotify: send fsnotify_mark to groups in event handling functions\n fsnotify: Exchange list heads instead of moving elements\n fsnotify: srcu to protect read side of inode and vfsmount locks\n fsnotify: use an explicit flag to indicate fsnotify_destroy_mark has been called\n fsnotify: use _rcu functions for mark list traversal\n fsnotify: place marks on object in order of group memory address\n vfs/fsnotify: fsnotify_close can delay the final work in fput\n fsnotify: store struct file not struct path\n ...\n\nFix up trivial delete/modify conflict in fs/notify/inotify/inotify.c.\n" }, { "commit": "d09ca73979460b96d5d4684d588b188be9a1f57d", "tree": "217543affc5c1c76181ffca00c23cfa69f1dd4f6", "parents": [ "9cfcac810e8993fa7a5bfd24b1a21f1dbbb03a7b" ], "author": { "name": "Eric Paris", "email": "eparis@redhat.com", "time": "Fri Jul 23 11:43:57 2010 -0400" }, "committer": { "name": "James Morris", "email": "jmorris@namei.org", "time": "Mon Aug 02 15:35:07 2010 +1000" }, "message": "security: make LSMs explicitly mask off permissions\n\nSELinux needs to pass the MAY_ACCESS flag so it can handle auditting\ncorrectly. Presently the masking of MAY_* flags is done in the VFS. In\norder to allow LSMs to decide what flags they care about and what flags\nthey don\u0027t just pass them all and the each LSM mask off what they don\u0027t\nneed. This patch should contain no functional changes to either the VFS or\nany LSM.\n\nSigned-off-by: Eric Paris \u003ceparis@redhat.com\u003e\nAcked-by: Stephen D. Smalley \u003csds@tycho.nsa.gov\u003e\nSigned-off-by: James Morris \u003cjmorris@namei.org\u003e\n" }, { "commit": "ea0d3ab239fba48d6e998b19c28d78f765963007", "tree": "c1e20273bf121a4f404ca7ac2a012161b0e0201e", "parents": [ "3e62cbb8436f6c0cb799c8b7f106de7f662a7b8d" ], "author": { "name": "Tetsuo Handa", "email": "penguin-kernel@I-love.SAKURA.ne.jp", "time": "Wed Jun 02 13:24:43 2010 +0900" }, "committer": { "name": "James Morris", "email": "jmorris@namei.org", "time": "Mon Aug 02 15:33:40 2010 +1000" }, "message": "LSM: Remove unused arguments from security_path_truncate().\n\nWhen commit be6d3e56a6b9b3a4ee44a0685e39e595073c6f0d \"introduce new LSM hooks\nwhere vfsmount is available.\" was proposed, regarding security_path_truncate(),\nonly \"struct file *\" argument (which AppArmor wanted to use) was removed.\nBut length and time_attrs arguments are not used by TOMOYO nor AppArmor.\nThus, let\u0027s remove these arguments.\n\nSigned-off-by: Tetsuo Handa \u003cpenguin-kernel@I-love.SAKURA.ne.jp\u003e\nAcked-by: Nick Piggin \u003cnpiggin@suse.de\u003e\nSigned-off-by: James Morris \u003cjmorris@namei.org\u003e\n" }, { "commit": "59b0df211bd9699d7e0d01fcf9345a149f75b033", "tree": "0f6e8bebfa090aa5c1d91d9efc5c2b04aeeed05c", "parents": [ "43ed7e16a8b47059d7f6ff67ba76f383a2421de3" ], "author": { "name": "Eric Paris", "email": "eparis@redhat.com", "time": "Mon Feb 08 12:53:52 2010 -0500" }, "committer": { "name": "Eric Paris", "email": "eparis@redhat.com", "time": "Wed Jul 28 09:59:01 2010 -0400" }, "message": "fsnotify: use unsigned char * for dentry-\u003ed_name.name\n\nfsnotify was using char * when it passed around the d_name.name string\ninternally but it is actually an unsigned char *. This patch switches\nfsnotify to use unsigned and should silence some pointer signess warnings\nwhich have popped out of xfs. I do not add -Wpointer-sign to the fsnotify\ncode as there are still issues with kstrdup and strlen which would pop\nout needless warnings.\n\nSigned-off-by: Eric Paris \u003ceparis@redhat.com\u003e\n" }, { "commit": "176306f59ac7a35369cbba87aff13e14c5916074", "tree": "9a507a050923d56ee5936a222ebc833d21cfda75", "parents": [ "1eb2cbb6d5efe129cd006691267ce513c0aa59da" ], "author": { "name": "Neil Brown", "email": "neilb@suse.de", "time": "Mon May 24 16:57:56 2010 +1000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu May 27 22:03:06 2010 -0400" }, "message": "VFS: fix recent breakage of FS_REVAL_DOT\n\nCommit 1f36f774b22a0ceb7dd33eca626746c81a97b6a5 broke FS_REVAL_DOT semantics.\n\nIn particular, before this patch, the command\n ls -l\nin an NFS mounted directory would always check if the directory on the server\nhad changed and if so would flush and refill the pagecache for the dir.\nAfter this patch, the same \"ls -l\" will repeatedly return stale date until\nthe cached attributes for the directory time out.\n\nThe following patch fixes this by ensuring the d_revalidate is called by\ndo_last when \".\" is being looked-up.\nlink_path_walk has already called d_revalidate, but in that case LOOKUP_OPEN\nis not set so nfs_lookup_verify_inode chooses not to do any validation.\n\nThe following patch restores the original behaviour.\n\nCc: stable@kernel.org\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "9a2296832c43da93a88e1edd59f3d17acffca36c", "tree": "e55b77c403c60291344d0961d2f0e4f2e9a2bb7f", "parents": [ "51ee049e771c14a29aaee8ecd6cbbe14db088f3a" ], "author": { "name": "Huang Shijie", "email": "shijie8@gmail.com", "time": "Fri Apr 02 17:37:13 2010 +0800" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri May 21 18:31:22 2010 -0400" }, "message": "namei.c : update mnt when it needed\n\nupdate the mnt of the path when it is not equal to the new one.\n\nSigned-off-by: Huang Shijie \u003cshijie8@gmail.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "d83c49f3e36cecd2e8823b6c48ffba083b8a5704", "tree": "a304de4eb43652d2a9528d4b43f798ab821c8b93", "parents": [ "6a251b0ab67989f468f4cb65179e0cf40cf8c295" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Apr 30 17:17:09 2010 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat May 15 07:16:33 2010 -0400" }, "message": "Fix the regression created by \"set S_DEAD on unlink()...\" commit\n\n1) i_flags simply doesn\u0027t work for mount/unlink race prevention;\nwe may have many links to file and rm on one of those obviously\nshouldn\u0027t prevent bind on top of another later on. To fix it\nright way we need to mark _dentry_ as unsuitable for mounting\nupon; new flag (DCACHE_CANT_MOUNT) is protected by d_flags and\ni_mutex on the inode in question. Set it (with dont_mount(dentry))\nin unlink/rmdir/etc., check (with cant_mount(dentry)) in places\nin namespace.c that used to check for S_DEAD. Setting S_DEAD\nis still needed in places where we used to set it (for directories\ngetting killed), since we rely on it for readdir/rmdir race\nprevention.\n\n2) rename()/mount() protection has another bogosity - we unhash\nthe target before we\u0027d checked that it\u0027s not a mountpoint. Fixed.\n\n3) ancient bogosity in pivot_root() - we locked i_mutex on the\nright directory, but checked S_DEAD on the different (and wrong)\none. Noticed and fixed.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "002baeecf53677d2034113e34197ec221f42e037", "tree": "081c54c4a9a266a894a00165e97dabfb212a2470", "parents": [ "fc8e38f142dd59f81306ff6a81c1351e9eb0e014" ], "author": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Thu May 13 12:52:57 2010 +0200" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 13 08:46:04 2010 -0700" }, "message": "vfs: Fix O_NOFOLLOW behavior for paths with trailing slashes\n\nAccording to specification\n\n\tmkdir d; ln -s d a; open(\"a/\", O_NOFOLLOW | O_RDONLY)\n\nshould return success but currently it returns ELOOP. This is a\nregression caused by path lookup cleanup patch series.\n\nFix the code to ignore O_NOFOLLOW in case the provided path has trailing\nslashes.\n\nCc: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nCc: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nReported-by: Marius Tolzmann \u003ctolzmann@molgen.mpg.de\u003e\nAcked-by: Miklos Szeredi \u003cmszeredi@suse.cz\u003e\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "3e297b613491f0d4928aa652a2cd266aa06dc409", "tree": "df688c6226e84768a0748d4a19cb893255f0bebf", "parents": [ "01e77706cdde7c0b47e5ca1f4284a795504c7c40" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 26 12:40:13 2010 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 26 12:41:05 2010 -0400" }, "message": "Restore LOOKUP_DIRECTORY hint handling in final lookup on open()\n\n\tLose want_dir argument, while we are at it - since now\nnd-\u003eflags \u0026 LOOKUP_DIRECTORY is equivalent to it.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "318ae2edc3b29216abd8a2510f3f80b764f06858", "tree": "ce595adde342f57f379d277b25e4dd206988a052", "parents": [ "25cf84cf377c0aae5dbcf937ea89bc7893db5176", "3e58974027b04e84f68b964ef368a6cd758e2f84" ], "author": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Mon Mar 08 16:55:37 2010 +0100" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Mon Mar 08 16:55:37 2010 +0100" }, "message": "Merge branch \u0027for-next\u0027 into for-linus\n\nConflicts:\n\tDocumentation/filesystems/proc.txt\n\tarch/arm/mach-u300/include/mach/debug-macro.S\n\tdrivers/net/qlge/qlge_ethtool.c\n\tdrivers/net/qlge/qlge_main.c\n\tdrivers/net/typhoon.c\n" }, { "commit": "781b16775ba0bb55fac0e1757bf0bd87c8879632", "tree": "f081f67fe98a50a217a8c9be8d55e64854e46193", "parents": [ "64096c17417380d8a472d096645f4cbc9406c987" ], "author": { "name": "Al Viro", "email": "viro@ZenIV.linux.org.uk", "time": "Sat Mar 06 18:41:07 2010 +0000" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sat Mar 06 10:54:48 2010 -0800" }, "message": "Fix a dumb typo - use of \u0026 instead of \u0026\u0026\n\nWe managed to lose O_DIRECTORY testing due to a stupid typo in commit\n1f36f774b2 (\"Switch !O_CREAT case to use of do_last()\")\n\nReported-by: Walter Sheets \u003cw41ter@gmail.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "e213e26ab3988c516c06eba4dcd030ac052f6dc9", "tree": "6e26fbdbb842b387697d73daf6e70cf718269a77", "parents": [ "c812a51d11bbe983f4c24e32b59b265705ddd3c2", "efd8f0e6f6c1faa041f228d7113bd3a9db802d49" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Mar 05 13:20:53 2010 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Mar 05 13:20:53 2010 -0800" }, "message": "Merge branch \u0027for_linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jack/linux-fs-2.6\n\n* \u0027for_linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jack/linux-fs-2.6: (33 commits)\n quota: stop using QUOTA_OK / NO_QUOTA\n dquot: cleanup dquot initialize routine\n dquot: move dquot initialization responsibility into the filesystem\n dquot: cleanup dquot drop routine\n dquot: move dquot drop responsibility into the filesystem\n dquot: cleanup dquot transfer routine\n dquot: move dquot transfer responsibility into the filesystem\n dquot: cleanup inode allocation / freeing routines\n dquot: cleanup space allocation / freeing routines\n ext3: add writepage sanity checks\n ext3: Truncate allocated blocks if direct IO write fails to update i_size\n quota: Properly invalidate caches even for filesystems with blocksize \u003c pagesize\n quota: generalize quota transfer interface\n quota: sb_quota state flags cleanup\n jbd: Delay discarding buffers in journal_unmap_buffer\n ext3: quota_write cross block boundary behaviour\n quota: drop permission checks from xfs_fs_set_xstate/xfs_fs_set_xquota\n quota: split out compat_sys_quotactl support from quota.c\n quota: split out netlink notification support from quota.c\n quota: remove invalid optimization from quota_sync_all\n ...\n\nFixed trivial conflicts in fs/namei.c and fs/ufs/inode.c\n" }, { "commit": "1f36f774b22a0ceb7dd33eca626746c81a97b6a5", "tree": "cfc2757bb3e21d484ce28bd3030e649b5767b5a1", "parents": [ "def4af30cf945a3735ffca865788ea84b30b25d9" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Dec 26 10:56:19 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:22:25 2010 -0500" }, "message": "Switch !O_CREAT case to use of do_last()\n\n... and now we have all intents crap well localized\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "def4af30cf945a3735ffca865788ea84b30b25d9", "tree": "adeac07e1675c51e8d9fa42ccf284b7b893041f1", "parents": [ "3866248e5f86d74960a3d1592882490ec3021675" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Dec 26 08:37:05 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:40 2010 -0500" }, "message": "Get rid of symlink body copying\n\nNow that nd-\u003elast stays around until -\u003eput_link() is called, we can\njust postpone that -\u003eput_link() in do_filp_open() a bit and don\u0027t\nbother with copying.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "3866248e5f86d74960a3d1592882490ec3021675", "tree": "af44e08612f8895a81aea230739439d4ef09cfaf", "parents": [ "806b681cbe588bebe8fe47dd24da62f2d1c55851" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Dec 26 07:21:48 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:38 2010 -0500" }, "message": "Finish pulling of -ESTALE handling to upper level in do_filp_open()\n\nDon\u0027t bother with path_walk() (and its retry loop); link_path_walk()\nwill do it.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "806b681cbe588bebe8fe47dd24da62f2d1c55851", "tree": "030d0e479f0c3e462930d18134a15092803d1ad3", "parents": [ "10fa8e62f2bc33c452516585911f151d88389e4c" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Dec 26 07:16:40 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:36 2010 -0500" }, "message": "Turn do_link spaghetty into a normal loop\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "10fa8e62f2bc33c452516585911f151d88389e4c", "tree": "9a69dd68ffdf419cd6391f198ff40ac0943b9a35", "parents": [ "9e67f36169117e07daf16dc7ca314f1db9e2050a" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Dec 26 07:09:49 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:35 2010 -0500" }, "message": "Unify exits in O_CREAT handling\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "9e67f36169117e07daf16dc7ca314f1db9e2050a", "tree": "7090b0fdafa07c91afa3fcc1eede94d0df74942f", "parents": [ "67ee3ad21d0d0b2cc0b70708de8aed860fadda44" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Dec 26 07:04:50 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:33 2010 -0500" }, "message": "Kill is_link argument of do_last()\n\nWe set it to 1 iff we return NULL\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "67ee3ad21d0d0b2cc0b70708de8aed860fadda44", "tree": "842c575f8d7c06b4ebca8c418270da7207359767", "parents": [ "4296e2cbf2138b5831b83f03e81de916ce1a967d" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Dec 26 07:01:01 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:31 2010 -0500" }, "message": "Pull handling of LAST_BIND into do_last(), clean up ok: part in do_filp_open()\n\nNote that in case of !O_CREAT we know that nd.root has already been given up\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "4296e2cbf2138b5831b83f03e81de916ce1a967d", "tree": "395b8422f7555c79c0251be27cff715adf9ec11e", "parents": [ "5b369df8263fe7ab4dac2bb08b8f423dc5e33752" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Dec 24 07:15:41 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:29 2010 -0500" }, "message": "Leave mangled flag only for setting nd.intent.open.flag\n\nNothing else uses it anymore\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "5b369df8263fe7ab4dac2bb08b8f423dc5e33752", "tree": "0b127915244593c1ed2494454d191a4fedb4d661", "parents": [ "9a66179e13504c676f891908a1e94912ec5cdefb" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Dec 24 06:51:13 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:27 2010 -0500" }, "message": "Get rid of passing mangled flag to do_last()\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "9a66179e13504c676f891908a1e94912ec5cdefb", "tree": "ddfa9f767ba393744adc6714a4c7ef0966222d81", "parents": [ "a2c36b450ee68470836cb858c58a6ba3a52c5ec5" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Dec 24 06:49:47 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:25 2010 -0500" }, "message": "Don\u0027t pass mangled open_flag to finish_open()\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "a2c36b450ee68470836cb858c58a6ba3a52c5ec5", "tree": "5c1aff4738fd679efd6d96f2ae224ea97121b019", "parents": [ "c99658fe970f442199733bcace1a00b087336a0d" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Dec 24 03:39:50 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:24 2010 -0500" }, "message": "pull more into do_last()\n\nHandling of LAST_DOT/LAST_ROOT/LAST_DOTDOT/terminating slash\ncan be pulled in as well\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "c99658fe970f442199733bcace1a00b087336a0d", "tree": "b9065b274d8f00639af43bbb8e4eaabcda5b559d", "parents": [ "a1e28038df98e186807ff55a49c1c26d33d530a5" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Dec 24 02:27:30 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:22 2010 -0500" }, "message": "bail out with ELOOP earlier in do_link loop\n\nIf we\u0027d passed through 32 trailing symlinks already, there\u0027s\nno sense following the 33rd - we\u0027ll bail out anyway. Better\nbugger off earlier.\n\nIt *does* change behaviour, after a fashion - if the 33rd happens\nto be a procfs-style symlink, original code *would* allow it.\nThis one will not. Cry me a river if that hurts you. Please, do.\nAnd post a video of that, while you are at it.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "a1e28038df98e186807ff55a49c1c26d33d530a5", "tree": "a72a14858ae883b333b4a0cf6bbf52d33a6bdb82", "parents": [ "c41c14056210e4a328659c82b1edaccb0910d18c" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Dec 24 02:12:06 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:20 2010 -0500" }, "message": "pull the common predecessors into do_last()\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "c41c14056210e4a328659c82b1edaccb0910d18c", "tree": "0ae7f7e9f952feb4577bb1908b063997232171bb", "parents": [ "27bff34300482632caf52ff589a4e7d755b32539" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Dec 24 02:08:19 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:18 2010 -0500" }, "message": "postpone __putname() until after do_last()\n\nSince do_last() doesn\u0027t mangle nd-\u003elast_name, we can safely postpone\n__putname() done in handling of trailing symlinks until after the\ncall of do_last()\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "27bff34300482632caf52ff589a4e7d755b32539", "tree": "49c6b38dd0c6ff2eebf0f0dad4c73a6539d52bbc", "parents": [ "3343eb8209cc69f0d2059f8c484ad7a3e1834c0b" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Dec 24 02:05:43 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:16 2010 -0500" }, "message": "unroll do_last: loop in do_filp_open()\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "3343eb8209cc69f0d2059f8c484ad7a3e1834c0b", "tree": "25d72fd720d5dcbf0f0be40832c76759e49413cc", "parents": [ "fb1cc555d533869910e20de4b8d5147570afdfad" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Dec 24 02:02:38 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:15 2010 -0500" }, "message": "Shift releasing nd-\u003eroot from do_last() to its caller\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "fb1cc555d533869910e20de4b8d5147570afdfad", "tree": "99ee86c8370e28df0991a4ecd03677cb65704f01", "parents": [ "648fa8611de3d4d43bbd64af3226679d2d0eb609" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Dec 24 01:58:28 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:13 2010 -0500" }, "message": "gut do_filp_open() a bit more (do_last separation)\n\nBrute-force separation of stuff reachable from do_last: with\nthe exception of do_link:; just take all that crap to a helper\nfunction as-is and have it tell the caller if it has to go\nto do_link.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "648fa8611de3d4d43bbd64af3226679d2d0eb609", "tree": "96d134de1d935d6d54daac678dc07346bbbda940", "parents": [ "64ba9926759792cf7b95f823402e2781edd1b5d4" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Dec 24 01:26:48 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 09:01:11 2010 -0500" }, "message": "beginning to untangle do_filp_open()\n\nThat\u0027s going to be a long and painful series. The first step:\ntake the stuff reachable from \u0027ok\u0027 label in do_filp_open() into\na new helper (finish_open()).\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "907f4554e2521cb28b0009d17167760650a9561c", "tree": "68dc49163fd34331f8efbd63592c8f1baa387031", "parents": [ "9f7547580263d4a55efe06ce5cfd567f568be6e8" ], "author": { "name": "Christoph Hellwig", "email": "hch@infradead.org", "time": "Wed Mar 03 09:05:06 2010 -0500" }, "committer": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Fri Mar 05 00:20:30 2010 +0100" }, "message": "dquot: move dquot initialization responsibility into the filesystem\n\nCurrently various places in the VFS call vfs_dq_init directly. This means\nwe tie the quota code into the VFS. Get rid of that and make the\nfilesystem responsible for the initialization. For most metadata operations\nthis is a straight forward move into the methods, but for truncate and\nopen it\u0027s a bit more complicated.\n\nFor truncate we currently only call vfs_dq_init for the sys_truncate case\nbecause open already takes care of it for ftruncate and open(O_TRUNC) - the\nnew code causes an additional vfs_dq_init for those which is harmless.\n\nFor open the initialization is moved from do_filp_open into the open method,\nwhich means it happens slightly earlier now, and only for regular files.\nThe latter is fine because we don\u0027t need to initialize it for operations\non special files, and we already do it as part of the namespace operations\nfor directories.\n\nAdd a dquot_file_open helper that filesystems that support generic quotas\ncan use to fill in -\u003eopen.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\n" }, { "commit": "9643f5d94aadd47a5fa9754fb60f2c957de05903", "tree": "c299736cbcdd7f971c1c308433e248198060fefa", "parents": [ "2bd3a997befc226ab4b504f05c5cbba305f3e0e6", "c44dcc56d2b5c79ba3063d20f76e5347e2e418f6" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Mar 03 17:12:40 2010 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Mar 03 17:12:40 2010 -0500" }, "message": "Merge branch \u0027for-fsnotify\u0027 into for-linus\n" }, { "commit": "bec1052e5be6a70f03f6adc650f3a6e4c2f44ddf", "tree": "e73e73b59d5b8c87f7cc50b0e17a89eaa0e704dc", "parents": [ "db1f05bb85d7966b9176e293f3ceead1cb8b5d79" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Mar 03 14:12:08 2010 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Mar 03 14:12:08 2010 -0500" }, "message": "set S_DEAD on unlink() and non-directory rename() victims\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "3088dd7080d1ecc6d18c27ef9e617cbbd2a2e51e", "tree": "4c31fc5f1710bafd79faa153839bbc617d1f715a", "parents": [ "de27a5bf9caef3f1fca1f315aa58eee54fbf929a" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Jan 30 15:47:29 2010 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Mar 03 14:07:56 2010 -0500" }, "message": "Clean follow_dotdot() up a bit\n\nNo need to open-code follow_up() in it and locking can be lighter.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "8737c9305bd5602b11f7eb4655d5695d4a42a0c6", "tree": "54038cac1135b039a292151ebe9b156f80904843", "parents": [ "d208bbdda991b8808d9c033ce4d31cb1bd87dcfc" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Dec 24 06:47:55 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Mar 03 13:00:21 2010 -0500" }, "message": "Switch may_open() and break_lease() to passing O_...\n\n... instead of mixing FMODE_ and O_\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "ac278a9c505092dd82077a2446af8f9fc0d9c095", "tree": "227d4ac77de3b909e70cbe10b2a1a6293037c8c0", "parents": [ "f8b55f251012e104093e105483c45c5d85ad3040" ], "author": { "name": "Al Viro", "email": "viro@ZenIV.linux.org.uk", "time": "Tue Feb 16 18:09:36 2010 +0000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Feb 19 03:56:42 2010 -0500" }, "message": "fix LOOKUP_FOLLOW on automount \"symlinks\"\n\nMake sure that automount \"symlinks\" are followed regardless of LOOKUP_FOLLOW;\nit should have no effect on them.\n\nCc: stable@kernel.org\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "cccc6bba3f771ef29b33e4f79e70ebc3dba245b0", "tree": "0abfed21a68d0ae54217a6f4308046fd30a70186", "parents": [ "123df2944c436c80640c4281c5bc9c7950b18687" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Dec 25 05:07:33 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Mon Feb 08 14:38:36 2010 -0500" }, "message": "Lose the first argument of audit_inode_child()\n\nit\u0027s always equal to -\u003ed_name.name of the second argument\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "123df2944c436c80640c4281c5bc9c7950b18687", "tree": "a4fe67a9ce7cd4ae7d54ec94577916d38a0603e6", "parents": [ "6339204ecc2aa2067a99595522de0403f0854bb8" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Dec 25 04:57:57 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Mon Feb 08 14:38:36 2010 -0500" }, "message": "Lose the new_name argument of fsnotify_move()\n\nit\u0027s always new_dentry-\u003ed_name.name\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "9bbb6cad0173e6220f3ac609e26beb48dab3b7cd", "tree": "680e0de3071c938ca9858fa9ed5bd5ca8ff2f20f", "parents": [ "54bb6552bd9405dc7685653157a4ec260c77a71c" ], "author": { "name": "Mimi Zohar", "email": "zohar@linux.vnet.ibm.com", "time": "Tue Jan 26 17:02:40 2010 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sun Feb 07 03:06:22 2010 -0500" }, "message": "ima: rename ima_path_check to ima_file_check\n\nima_path_check actually deals with files! call it ima_file_check instead.\n\nSigned-off-by: Eric Paris \u003ceparis@redhat.com\u003e\nAcked-by: Mimi Zohar \u003czohar@linux.vnet.ibm.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "8eb988c70e7709b7bd1a69f0ec53d19ac20dea84", "tree": "6d0283a9fbca5cc104f591b9cc628edf39bc0b05", "parents": [ "1e41568d7378d1ba8c64ba137b9ddd00b59f893a" ], "author": { "name": "Mimi Zohar", "email": "zohar@linux.vnet.ibm.com", "time": "Wed Jan 20 15:35:41 2010 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sun Feb 07 03:06:22 2010 -0500" }, "message": "fix ima breakage\n\nThe \"Untangling ima mess, part 2 with counters\" patch messed\nup the counters. Based on conversations with Al Viro, this patch\nstreamlines ima_path_check() by removing the counter maintaince.\nThe counters are now updated independently, from measuring the file,\nin __dentry_open() and alloc_file() by calling ima_counts_get().\nima_path_check() is called from nfsd and do_filp_open().\nIt also did not measure all files that should have been measured.\nReason: ima_path_check() got bogus value passed as mask.\n[AV: mea culpa]\n[AV: add missing nfsd bits]\n\nSigned-off-by: Mimi Zohar \u003czohar@us.ibm.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "c41b20e721ea4f6f20f66a66e7f0c3c97a2ca9c2", "tree": "92cd1b2b1cdb871530bb07bbbcbfeaffe1170830", "parents": [ "a089377f29d3af0f62f3bdc6db0c5042513fc3f3" ], "author": { "name": "Adam Buchbinder", "email": "adam.buchbinder@gmail.com", "time": "Fri Dec 11 16:35:39 2009 -0500" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Thu Feb 04 11:55:45 2010 +0100" }, "message": "Fix misspellings of \"truly\" in comments.\n\nSome comments misspell \"truly\"; this fixes them. No code changes.\n\nSigned-off-by: Adam Buchbinder \u003cadam.buchbinder@gmail.com\u003e\nSigned-off-by: Jiri Kosina \u003cjkosina@suse.cz\u003e\n" }, { "commit": "9850c056559f3633a32d810aaf00ced39437b364", "tree": "30e80de94e12ae758736cfb586db39cfa77d69df", "parents": [ "806892e9e12e731a0ca76c8f62ad95cf8eea9614" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Jan 13 15:01:15 2010 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Jan 14 09:05:26 2010 -0500" }, "message": "Fix the -ESTALE handling in do_filp_open()\n\nInstead of playing sick games with path saving, cleanups, just retry\nthe entire thing once with LOOKUP_REVAL added. Post-.34 we\u0027ll convert\nall -ESTALE handling in there to that style, rather than playing with\nmany retry loops deep in the call chain.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "6d125529c6cbfe570ce3bf9a0728548f087499da", "tree": "89ba434f76d224741bd0e0b0ef02b10a4ff95136", "parents": [ "4ecf09fd3a7c8858198875171b684c73338fad83" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Dec 24 06:58:56 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Jan 14 09:05:26 2010 -0500" }, "message": "Fix ACC_MODE() for real\n\ncommit 5300990c0370e804e49d9a59d928c5d53fb73487 had stepped on a rather\nnasty mess: definitions of ACC_MODE used to be different. Fixed the\nresulting breakage, converting them to variant that takes O_... value;\nall callers have that and it actually simplifies life (see tomoyo part\nof changes).\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "86acdca1b63e6890540fa19495cfc708beff3d8b", "tree": "f589566363ef4f77098eba9ced48325f410168e6", "parents": [ "004b35063296b6772fa72404a35b498f1e71e87e" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Tue Dec 22 23:45:11 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Jan 14 09:05:25 2010 -0500" }, "message": "fix autofs/afs/etc. magic mountpoint breakage\n\nWe end up trying to kfree() nd.last.name on open(\"/mnt/tmp\", O_CREAT)\nif /mnt/tmp is an autofs direct mount. The reason is that nd.last_type\nis bogus here; we want LAST_BIND for everything of that kind and we\nget LAST_NORM left over from finding parent directory.\n\nSo make sure that it *is* set properly; set to LAST_BIND before\ndoing -\u003efollow_link() - for normal symlinks it will be changed\nby __vfs_follow_link() and everything else needs it set that way.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "7ea6600148c265b1fd53e521022b1d7aec81d974", "tree": "aeb16e15ff5e6f88cc7137847fb5610377bf7395", "parents": [ "9a7262a0563da6b91019156abf487bcdf1a41526" ], "author": { "name": "Serge E. Hallyn", "email": "serue@us.ibm.com", "time": "Tue Dec 29 14:50:19 2009 -0600" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Dec 30 12:35:44 2009 -0800" }, "message": "generic_permission: MAY_OPEN is not write access\n\ngeneric_permission was refusing CAP_DAC_READ_SEARCH-enabled\nprocesses from opening DAC-protected files read-only, because\ndo_filp_open adds MAY_OPEN to the open mask.\n\nIgnore MAY_OPEN. After this patch, CAP_DAC_READ_SEARCH is\nagain sufficient to open(fname, O_RDONLY) on a file to which\nDAC otherwise refuses us read permission.\n\nReported-by: Mike Kazantsev \u003cmk.fraggod@gmail.com\u003e\nSigned-off-by: Serge E. Hallyn \u003cserue@us.ibm.com\u003e\nTested-by: Mike Kazantsev \u003cmk.fraggod@gmail.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "5300990c0370e804e49d9a59d928c5d53fb73487", "tree": "08ed922afd172662039c082ec9e9410070f4afe8", "parents": [ "482928d59db668b8d82a48717f78986d8cea72e9" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Dec 19 10:15:07 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Tue Dec 22 12:27:34 2009 -0500" }, "message": "Sanitize f_flags helpers\n\n* pull ACC_MODE to fs.h; we have several copies all over the place\n* nightmarish expression calculating f_mode by f_flags deserves a helper\ntoo (OPEN_FMODE(flags))\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "482928d59db668b8d82a48717f78986d8cea72e9", "tree": "6985c64474172ddfb67c737bc4a49e588c49d055", "parents": [ "628ff7c1d8d8466a5ad8078bd0206a130f8b8a51" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Dec 19 10:10:39 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Tue Dec 22 12:27:34 2009 -0500" }, "message": "Fix f_flags/f_mode in case of lookup_instantiate_filp() from open(pathname, 3)\n\nJust set f_flags when shoving struct file into nameidata; don\u0027t\npostpone that until __dentry_open(). do_filp_open() has correct\nvalue; lookup_instantiate_filp() doesn\u0027t - we lose the difference\nbetween O_RDWR and 3 by that point.\n\nWe still set .intent.open.flags, so no fs code needs to be changed.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "cb59861f03a626196a23fdef5e20ddbb8cca6466", "tree": "777dda2c1764e74cf6e7e5bda27314bb99c5b4d1", "parents": [ "a3a065e3f13da8a3470ed09c7f38aad256083726" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Nov 16 12:05:20 2009 -0800" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Dec 17 10:52:34 2009 -0500" }, "message": "vfs: remove extraneous NULL d_inode check from do_filp_open\n\nWe can\u0027t get to this point unless it\u0027s a valid pointer.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nCc: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nCc: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "39159de2a091a35ea86b188ebdc5e642f5cfc832", "tree": "793c0c95fe22ddb06c4cc4271428ad41386ccf6b", "parents": [ "d1625436b4fe526fa463bc0519ba37d7e4b37bbc" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Dec 07 12:01:50 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Dec 16 12:16:48 2009 -0500" }, "message": "vfs: force reval of target when following LAST_BIND symlinks (try #7)\n\nprocfs-style symlinks return a last_type of LAST_BIND without an actual\npath string. This causes __follow_link to skip calling __vfs_follow_link\nand so the dentry isn\u0027t revalidated.\n\nThis is a problem when the link target sits on NFSv4 as it depends on\nthe VFS to revalidate the dentry before using it on an open call. Ensure\nthat this occurs by forcing a revalidation of the target dentry of\nLAST_BIND symlinks.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nAcked-by: \"Eric W. Biederman\" \u003cebiederm@xmission.com\u003e\nAcked-by: Miklos Szeredi \u003cmszeredi@suse.cz\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "1429b3eca23818f87f9fa569a15d9816de81f698", "tree": "3100f009ec8863ee4692ee197b8e0c16c11258e6", "parents": [ "b65a9cfc2c38eebc33533280b8ad5841caee8b6e" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Dec 16 06:38:01 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Dec 16 12:16:47 2009 -0500" }, "message": "Untangling ima mess, part 3: kill dead code in ima\n\nKill the \u0027update\u0027 argument of ima_path_check(), kill\ndead code in ima.\n\nCurrent rules: ima counters are bumped at the same time\nwhen the file switches from put_filp() fodder to fput()\none. Which happens exactly in two places - alloc_file()\nand __dentry_open(). Nothing else needs to do that at\nall.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "b65a9cfc2c38eebc33533280b8ad5841caee8b6e", "tree": "d6e5b713615cc5e65c900162ab09235ae4847909", "parents": [ "0552f879d45cecc35d8e372a591fc5ed863bca58" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Dec 16 06:27:40 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Dec 16 12:16:47 2009 -0500" }, "message": "Untangling ima mess, part 2: deal with counters\n\n* do ima_get_count() in __dentry_open()\n* stop doing that in followups\n* move ima_path_check() to right after nameidata_to_filp()\n* don\u0027t bump counters on it\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "7715b521222b6ebb6e927fa261ed91ed687fe454", "tree": "33c6f8179d405974ed6763df331f731044c16072", "parents": [ "85a17f552dfe77efb44b971615e4f221a5f28f37" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Dec 16 03:54:00 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Dec 16 12:16:47 2009 -0500" }, "message": "O_TRUNC open shouldn\u0027t fail after file truncation\n\n* take truncate logics into a helper (handle_truncate())\n* rip it out of may_open()\n* call it from the only caller of may_open() that might pass\nO_TRUNC\n* and do that after we\u0027d finished with opening.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "e81e3f4dca6c54116a24aec217d2c15c6f58ada5", "tree": "4cbaf8b4573a4f4dac2dafc2bebffe398a984b3d", "parents": [ "b75b5086be6504132eadb22a907854e0bf52e365" ], "author": { "name": "Eric Paris", "email": "eparis@redhat.com", "time": "Fri Dec 04 15:47:36 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Dec 16 12:16:45 2009 -0500" }, "message": "fs: move get_empty_filp() deffinition to internal.h\n\nAll users outside of fs/ of get_empty_filp() have been removed. This patch\nmoves the definition from the include/ directory to internal.h so no new\nusers crop up and removes the EXPORT_SYMBOL. I\u0027d love to see open intents\nstop using it too, but that\u0027s a problem for another day and a smarter\ndeveloper!\n\nSigned-off-by: Eric Paris \u003ceparis@redhat.com\u003e\nAcked-by: Miklos Szeredi \u003cmiklos@szeredi.hu\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "b75b5086be6504132eadb22a907854e0bf52e365", "tree": "b9859bcb09d12bb2a1dcfd5c3cecf303c2c02d1b", "parents": [ "6e6b1bd1e739faf4fa259fc0d8bfcadea2493222" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Dec 16 01:01:38 2009 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Dec 16 12:16:45 2009 -0500" }, "message": "Sanitize exec_permission_lite()\n\nUse the sucker in other places in pathname resolution\nthat check MAY_EXEC for directories; lose the _lite\nfrom name, it\u0027s equivalent of full-blown inode_permission()\nfor its callers (albeit still lighter, since large parts\nof generic_permission() do not apply for pure MAY_EXEC).\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "6e6b1bd1e739faf4fa259fc0d8bfcadea2493222", "tree": "de20d84d0051852c4ff5b6200ab8c9bd9e932b75", "parents": [ "2dd6d1f41852bbb1f66d66b3634ddfdaeddcf9bc" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Aug 13 23:38:37 2009 +0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Dec 16 12:16:45 2009 -0500" }, "message": "Kill cached_lookup() and real_lookup()\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "2dd6d1f41852bbb1f66d66b3634ddfdaeddcf9bc", "tree": "82a4e934af91a28fb2be1501b9b5392b892f40e7", "parents": [ "3cac260ad88f4e37637d6e4b33e6a6a849d273c7" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Aug 13 20:40:45 2009 +0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Dec 16 12:16:45 2009 -0500" }, "message": "Kill path_lookup_open()\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "3cac260ad88f4e37637d6e4b33e6a6a849d273c7", "tree": "2cde342e32fccadfcc900c10b4a8e49398fb24cf", "parents": [ "e9496ff46a20a8592fdc7bdaaf41b45eb808d310" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Aug 13 18:27:43 2009 +0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Dec 16 12:16:44 2009 -0500" }, "message": "Take hash recalculation into do_lookup()\n\nBoth callers of do_lookup() do the same thing before it\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "6de88d72927dc85297b3075024487313c4ba3a2e", "tree": "278609d97c5635bb3922634b274689739fbf09d6", "parents": [ "258fa99905f704afed1a43f195bc5235a56fb895" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sun Aug 09 01:41:57 2009 +0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Dec 16 12:16:43 2009 -0500" }, "message": "kill __link_path_walk()/link_path_walk() distinction\n\nput retry logics into path_walk() and do_filp_open()\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "258fa99905f704afed1a43f195bc5235a56fb895", "tree": "434bd55b08ab3f635e66cce89e3ab931dbc9f2ef", "parents": [ "d231412db66355e72d606f8403ee8b6bd8ad4f9a" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sun Aug 09 01:32:02 2009 +0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Dec 16 12:16:43 2009 -0500" }, "message": "lift path_put(path) to callers of __do_follow_link()\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "3126c136bc30225d7a43af741778aa50e95e467a", "tree": "71e6f0de6e1f4cde200dd632da4d2f61180289bf", "parents": [ "f4d544ee5720d336a8c64f9fd33efb888c302309", "8e0eb4011bd73d5f91b215b532f74eef478ef795" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Dec 11 15:31:13 2009 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Dec 11 15:31:13 2009 -0800" }, "message": "Merge branch \u0027for_linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jack/linux-fs-2.6\n\n* \u0027for_linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jack/linux-fs-2.6: (21 commits)\n ext3: PTR_ERR return of wrong pointer in setup_new_group_blocks()\n ext3: Fix data / filesystem corruption when write fails to copy data\n ext4: Support for 64-bit quota format\n ext3: Support for vfsv1 quota format\n quota: Implement quota format with 64-bit space and inode limits\n quota: Move definition of QFMT_OCFS2 to linux/quota.h\n ext2: fix comment in ext2_find_entry about return values\n ext3: Unify log messages in ext3\n ext2: clear uptodate flag on super block I/O error\n ext2: Unify log messages in ext2\n ext3: make \"norecovery\" an alias for \"noload\"\n ext3: Don\u0027t update the superblock in ext3_statfs()\n ext3: journal all modifications in ext3_xattr_set_handle\n ext2: Explicitly assign values to on-disk enum of filetypes\n quota: Fix WARN_ON in lookup_one_len\n const: struct quota_format_ops\n ubifs: remove manual O_SYNC handling\n afs: remove manual O_SYNC handling\n kill wait_on_page_writeback_range\n vfs: Implement proper O_SYNC semantics\n ...\n" }, { "commit": "832b6af198aefe6034310e124594cc8b833c0ef9", "tree": "853eec8f7449dd7fc51c6d4c9b70ea44b81cf0d9", "parents": [ "a16bbc3430ed94b543222f4c8ef68025f8493e93" ], "author": { "name": "Eric W. Biederman", "email": "ebiederm@xmission.com", "time": "Fri Nov 20 16:08:56 2009 -0800" }, "committer": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Fri Dec 11 11:24:54 2009 -0800" }, "message": "sysfs: Propagate renames to the vfs on demand\n\nBy teaching sysfs_revalidate to hide a dentry for\na sysfs_dirent if the sysfs_dirent has been renamed,\nand by teaching sysfs_lookup to return the original\ndentry if the sysfs dirent has been renamed. I can\nshow the results of renames correctly without having to\nupdate the dcache during the directory rename.\n\nThis massively simplifies the rename logic allowing a lot\nof weird sysfs special cases to be removed along with\na lot of now unnecesary helper code.\n\nAcked-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Eric W. Biederman \u003cebiederm@aristanetworks.com\u003e\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\n\n" }, { "commit": "6b2f3d1f769be5779b479c37800229d9a4809fc3", "tree": "046ef6736ec6c25ab1c68741ba715d13645af336", "parents": [ "59bc055211b8d266ab6089158058bf8268e02006" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Tue Oct 27 11:05:28 2009 +0100" }, "committer": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Thu Dec 10 15:02:50 2009 +0100" }, "message": "vfs: Implement proper O_SYNC semantics\n\nWhile Linux provided an O_SYNC flag basically since day 1, it took until\nLinux 2.4.0-test12pre2 to actually get it implemented for filesystems,\nsince that day we had generic_osync_around with only minor changes and the\ngreat \"For now, when the user asks for O_SYNC, we\u0027ll actually give\nO_DSYNC\" comment. This patch intends to actually give us real O_SYNC\nsemantics in addition to the O_DSYNC semantics. After Jan\u0027s O_SYNC\npatches which are required before this patch it\u0027s actually surprisingly\nsimple, we just need to figure out when to set the datasync flag to\nvfs_fsync_range and when not.\n\nThis patch renames the existing O_SYNC flag to O_DSYNC while keeping it\u0027s\nnumerical value to keep binary compatibility, and adds a new real O_SYNC\nflag. To guarantee backwards compatiblity it is defined as expanding to\nboth the O_DSYNC and the new additional binary flag (__O_SYNC) to make\nsure we are backwards-compatible when compiled against the new headers.\n\nThis also means that all places that don\u0027t care about the differences can\njust check O_DSYNC and get the right behaviour for O_SYNC, too - only\nplaces that actuall care need to check __O_SYNC in addition. Drivers and\nnetwork filesystems have been updated in a fail safe way to always do the\nfull sync magic if O_DSYNC is set. The few places setting O_SYNC for\nlower layers are kept that way for now to stay failsafe.\n\nWe enforce that O_DSYNC is set when __O_SYNC is set early in the open path\nto make sure we always get these sane options.\n\nNote that parisc really screwed up their headers as they already define a\nO_DSYNC that has always been a no-op. We try to repair it by using it for\nthe new O_DSYNC and redefinining O_SYNC to send both the traditional\nO_SYNC numerical value _and_ the O_DSYNC one.\n\nCc: Richard Henderson \u003crth@twiddle.net\u003e\nCc: Ivan Kokshaysky \u003cink@jurassic.park.msu.ru\u003e\nCc: Grant Grundler \u003cgrundler@parisc-linux.org\u003e\nCc: \"David S. Miller\" \u003cdavem@davemloft.net\u003e\nCc: Ingo Molnar \u003cmingo@elte.hu\u003e\nCc: \"H. Peter Anvin\" \u003chpa@zytor.com\u003e\nCc: Thomas Gleixner \u003ctglx@linutronix.de\u003e\nCc: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nCc: Andreas Dilger \u003cadilger@sun.com\u003e\nAcked-by: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\nAcked-by: Kyle McMartin \u003ckyle@mcmartin.ca\u003e\nAcked-by: Ulrich Drepper \u003cdrepper@redhat.com\u003e\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\n" }, { "commit": "f6f79190866d5b2d06a2114d673f91f54e7c7ce4", "tree": "025bc2ff00351c71a719cb5bc2aa3f59133400f6", "parents": [ "0d03d59d9b31cd1e33b7e46a80b6fef66244b1f2", "a3c8b97396ef42edfb845788ba6f53b2a93ce980" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Sep 11 08:55:49 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Sep 11 08:55:49 2009 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/security-testing-2.6\n\n* \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/security-testing-2.6: (57 commits)\n binfmt_elf: fix PT_INTERP bss handling\n TPM: Fixup boot probe timeout for tpm_tis driver\n sysfs: Add labeling support for sysfs\n LSM/SELinux: inode_{get,set,notify}secctx hooks to access LSM security context information.\n VFS: Factor out part of vfs_setxattr so it can be called from the SELinux hook for inode_setsecctx.\n KEYS: Add missing linux/tracehook.h #inclusions\n KEYS: Fix default security_session_to_parent()\n Security/SELinux: includecheck fix kernel/sysctl.c\n KEYS: security_cred_alloc_blank() should return int under all circumstances\n IMA: open new file for read\n KEYS: Add a keyctl to install a process\u0027s session keyring on its parent [try #6]\n KEYS: Extend TIF_NOTIFY_RESUME to (almost) all architectures [try #6]\n KEYS: Do some whitespace cleanups [try #6]\n KEYS: Make /proc/keys use keyid not numread as file position [try #6]\n KEYS: Add garbage collection for dead, revoked and expired keys. [try #6]\n KEYS: Flag dead keys to induce EKEYREVOKED [try #6]\n KEYS: Allow keyctl_revoke() on keys that have SETATTR but not WRITE perm [try #6]\n KEYS: Deal with dead-type keys appropriately [try #6]\n CRED: Add some configurable debugging [try #6]\n selinux: Support for the new TUN LSM hooks\n ...\n" } ], "next": "a3c8b97396ef42edfb845788ba6f53b2a93ce980" }