)]}' { "log": [ { "commit": "873feea09ebc980cbd3631b767356ce1eee65ec1", "tree": "59a8fce9b138086abee7cf845f62ff70a390cf81", "parents": [ "ceb5bdc2d246f6d81cf61ed70f325308a11821d2" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:06 2011 +1100" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:31 2011 +1100" }, "message": "fs: dcache per-inode inode alias locking\n\ndcache_inode_lock can be replaced with per-inode locking. Use existing\ninode-\u003ei_lock for this. This is slightly non-trivial because we sometimes\nneed to find the inode from the dentry, which requires d_inode to be\nstabilised (either with refcount or d_lock).\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": "fa0d7e3de6d6fc5004ad9dea0dd6b286af8f03e9", "tree": "203e0f73883e4c26b5597e36042386a1237dab35", "parents": [ "77812a1ef139d84270d27faacc0630c887411013" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:49:49 2011 +1100" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:26 2011 +1100" }, "message": "fs: icache RCU free inodes\n\nRCU free the struct inode. This will allow:\n\n- Subsequent store-free path walking patch. The inode must be consulted for\n permissions when walking, so an RCU inode reference is a must.\n- sb_inode_list_lock to be moved inside i_lock because sb list walkers who want\n to take i_lock no longer need to take sb_inode_list_lock to walk the list in\n the first place. This will simplify and optimize locking.\n- Could remove some nested trylock loops in dcache code\n- Could potentially simplify things a bit in VM land. Do not need to take the\n page lock to follow page-\u003emapping.\n\nThe downsides of this is the performance cost of using RCU. In a simple\ncreat/unlink microbenchmark, performance drops by about 10% due to inability to\nreuse cache-hot slab objects. As iterations increase and RCU freeing starts\nkicking over, this increases to about 20%.\n\nIn cases where inode lifetimes are longer (ie. many inodes may be allocated\nduring the average life span of a single inode), a lot of this cache reuse is\nnot applicable, so the regression caused by this patch is smaller.\n\nThe cache-hot regression could largely be avoided by using SLAB_DESTROY_BY_RCU,\nhowever this adds some complexity to list walking and store-free path walking,\nso I prefer to implement this at a later date, if it is shown to be a win in\nreal situations. I haven\u0027t found a regression in any non-micro benchmark so I\ndoubt it will be a problem.\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": "b23fb0a60379a95e10c671f646b259ea2558421e", "tree": "7c3644b91241d32fda502a7be0b78e4c225f8091", "parents": [ "2fd6b7f50797f2e993eea59e0a0b8c6399c811dc" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:49:35 2011 +1100" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:22 2011 +1100" }, "message": "fs: scale inode alias list\n\nAdd a new lock, dcache_inode_lock, to protect the inode\u0027s i_dentry list\nfrom concurrent modification. d_alias is also protected by 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": "621e155a3591962420eacdd39f6f0aa29ceb221e", "tree": "387a9fb396f1bf24514b712c294182e36ba51076", "parents": [ "fb2d5b86aff355a27ebfc132d3c99f4a940cc3fe" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:49:27 2011 +1100" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:19 2011 +1100" }, "message": "fs: change d_compare for rcu-walk\n\nChange d_compare so it may be called from lock-free RCU lookups. This\ndoes put significant restrictions on what may be done from the callback,\nhowever there don\u0027t seem to have been any problems with in-tree fses.\nIf some strange use case pops up that _really_ cannot cope with the\nrcu-walk rules, we can just add new rcu-unaware callbacks, which would\ncause name lookup to drop out of rcu-walk mode.\n\nFor in-tree filesystems, this is just a mechanical change.\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\n" }, { "commit": "79eb4dde742fe2e9c9e301432b894a7410261ce7", "tree": "89150e92450fd10f7bd55348459d9f36be544ec0", "parents": [ "fe15ce446beb3a33583af81ffe6c9d01a75314ed" ], "author": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:49:24 2011 +1100" }, "committer": { "name": "Nick Piggin", "email": "npiggin@kernel.dk", "time": "Fri Jan 07 17:50:18 2011 +1100" }, "message": "cifs: dont overwrite dentry name in d_revalidate\n\nUse vfat\u0027s method for dealing with negative dentries to preserve case,\nrather than overwrite dentry name in d_revalidate, which is a bit ugly\nand also gets in the way of doing lock-free path walking.\n\nSigned-off-by: Nick Piggin \u003cnpiggin@kernel.dk\u003e\n" }, { "commit": "545c988b2018a593f24d291b66776a0d08525acd", "tree": "e2a0b2f523bf3c51e361c3735db359878e1d868f", "parents": [ "7d161b7f41e24203b54b0f18ae0d26c18a6c6fab" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Mon Dec 06 13:45:50 2010 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Dec 08 18:47:54 2010 +0000" }, "message": "cifs: remove bogus remapping of error in cifs_filldir()\n\nAs the FIXME points out correctly, now filldir() itself returns -EOVERFLOW if\nit not possible to represent the inode number supplied by the filesystem in\nthe field provided by userspace.\n\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "7d161b7f41e24203b54b0f18ae0d26c18a6c6fab", "tree": "10219d15bed51128f72e2e45fc58947071fed0c9", "parents": [ "03ceace5c6923ffbcf2b4d2e37afbffbdb5d4a67" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Dec 07 02:10:35 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Dec 07 19:25:37 2010 +0000" }, "message": "cifs: allow calling cifs_build_path_to_root on incomplete cifs_sb\n\nIt\u0027s possible that cifs_mount will call cifs_build_path_to_root on a\nnewly instantiated cifs_sb. In that case, it\u0027s likely that the\nmaster_tlink pointer has not yet been instantiated.\n\nFix this by having cifs_build_path_to_root take a cifsTconInfo pointer\nas well, and have the caller pass that in.\n\nReported-and-Tested-by: Robbert Kouprie \u003crobbert@exx.nl\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "03ceace5c6923ffbcf2b4d2e37afbffbdb5d4a67", "tree": "5f8630682dbc8e7e0b38c47e1403945b0b5d07d6", "parents": [ "884639996814585ef7079daa9e03a1eb562e235c" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Dec 06 21:07:33 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Dec 07 19:17:59 2010 +0000" }, "message": "cifs: fix check of error return from is_path_accessable\n\nThis function will return 0 if everything went ok. Commit 9d002df4\nhowever added a block of code after the following check for\nrc \u003d\u003d -EREMOTE. With that change and when rc \u003d\u003d 0, doing the\n\"goto mount_fail_check\" here skips that code, leaving the tlink_tree\nand master_tlink pointer unpopulated. That causes an oops later\nin cifs_root_iget.\n\nReported-and-Tested-by: Robbert Kouprie \u003crobbert@exx.nl\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "884639996814585ef7079daa9e03a1eb562e235c", "tree": "652d60d1716be67db2c21aaeaf643dc00d65e392", "parents": [ "79df1baeec29022e4181f2964187b88661ef5517" ], "author": { "name": "Jeff Layton", "email": "jlayton@poochiereds.net", "time": "Mon Nov 22 15:31:03 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Dec 06 22:45:19 2010 +0000" }, "message": "cifs: remove Local_System_Name\n\n...this string is zeroed out and nothing ever changes it.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "79df1baeec29022e4181f2964187b88661ef5517", "tree": "2692721239eeb8bbfe591fc1ef038c90e61e8826", "parents": [ "ebb27386ffd2fcd0fad141e0eee97b8070f94ee2" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Dec 06 12:52:08 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Dec 06 20:22:39 2010 +0000" }, "message": "cifs: fix use of CONFIG_CIFS_ACL\n\nSome of the code under CONFIG_CIFS_ACL is dependent upon code under\nCONFIG_CIFS_EXPERIMENTAL, but the Kconfig options don\u0027t reflect that\ndependency. Move more of the ACL code out from under\nCONFIG_CIFS_EXPERIMENTAL and under CONFIG_CIFS_ACL.\n\nAlso move find_readable_file out from other any sort of Kconfig\noption and make it a function normally compiled in.\n\nReported-and-Acked-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6d20e8406f0942228a73000663c2b33f488103ea", "tree": "2469267c2ee10c4c723eaa01b1f24c8d0f704870", "parents": [ "8cb280c90f9cfaab3ba3afbace0b1711dee80d0c" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Wed Dec 01 14:42:28 2010 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Dec 02 19:32:11 2010 +0000" }, "message": "cifs: add attribute cache timeout (actimeo) tunable\n\nCurrently, the attribute cache timeout for CIFS is hardcoded to 1 second. This\nmeans that the client might have to issue a QPATHINFO/QFILEINFO call every 1\nsecond to verify if something has changes, which seems too expensive. On the\nother hand, if the timeout is hardcoded to a higher value, workloads that\nexpect strict cache coherency might see unexpected results.\n\nMaking attribute cache timeout as a tunable will allow us to make a tradeoff\nbetween performance and cache metadata correctness depending on the\napplication/workload needs.\n\nAdd \u0027actimeo\u0027 tunable that can be used to tune the attribute cache timeout.\nThe default timeout is set to 1 second. Also, display actimeo option value in\n/proc/mounts.\n\nIt appears to me that \u0027actimeo\u0027 and the proposed (but not yet merged)\n\u0027strictcache\u0027 option cannot coexist, so care must be taken that we reset the\nother option if one of them is set.\n\nChanges since last post:\n - fix option parsing and handle possible values correcly\n\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ba03864872691c0bb580a7fb47388da337ef4aa2", "tree": "05584dc9061390fedf5caa749a2f7d728324b232", "parents": [ "476428f8c3bb6679f8f52bf2b935ac40bc9c7358" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Nov 30 15:14:48 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Nov 30 20:44:05 2010 +0000" }, "message": "cifs: fix parsing of hostname in dfs referrals\n\nThe DFS referral parsing code does a memchr() call to find the \u0027\\\\\u0027\ndelimiter that separates the hostname in the referral UNC from the\nsharename. It then uses that value to set the length of the hostname via\npointer subtraction. Instead of subtracting the start of the hostname\nhowever, it subtracts the start of the UNC, which causes the code to\npass in a hostname length that is 2 bytes too long.\n\nRegression introduced in commit 1a4240f4.\n\nReported-and-Tested-by: Robbert Kouprie \u003crobbert@exx.nl\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nCc: Wang Lei \u003cwang840925@gmail.com\u003e\nCc: David Howells \u003cdhowells@redhat.com\u003e\nCc: stable@kernel.org\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "476428f8c3bb6679f8f52bf2b935ac40bc9c7358", "tree": "64957af79b5f0f5ba5111381650bc3854be18b69", "parents": [ "b81209de2455c6051cf67f2acd8ec26ccb8caf83" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Wed Nov 24 17:49:07 2010 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Nov 30 05:51:49 2010 +0000" }, "message": "cifs: display fsc in /proc/mounts\n\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "b81209de2455c6051cf67f2acd8ec26ccb8caf83", "tree": "2f2b3ffcdbd2448342615bbd5bd45de3eafaa49f", "parents": [ "607a569da4cf289fd8eb3887080ed3b212e3112d" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Wed Nov 24 17:49:06 2010 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Nov 30 05:49:32 2010 +0000" }, "message": "cifs: enable fscache iff fsc mount option is used explicitly\n\nCurrently, if CONFIG_CIFS_FSCACHE is set, fscache is enabled on files opened\nas read-only irrespective of the \u0027fsc\u0027 mount option. Fix this by enabling\nfscache only if \u0027fsc\u0027 mount option is specified explicitly.\n\nRemove an extraneous cFYI debug message and fix a typo while at it.\n\nReported-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "607a569da4cf289fd8eb3887080ed3b212e3112d", "tree": "9e8e54738ebe1c80718ba67d44a7f736ba0af879", "parents": [ "fbeba8bb16d7c50362e28f3f91a79fb414903199" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Wed Nov 24 17:49:05 2010 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Nov 30 05:49:28 2010 +0000" }, "message": "cifs: allow fsc mount option only if CONFIG_CIFS_FSCACHE is set\n\nCurrently, it is possible to specify \u0027fsc\u0027 mount option even if\nCONFIG_CIFS_FSCACHE has not been set. The option is being ignored silently\nwhile the user fscache functionality to work. Fix this by raising error when\nthe CONFIG option is not set.\n\nReported-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "fbeba8bb16d7c50362e28f3f91a79fb414903199", "tree": "8ec770c15edfd25c83c70434cbf1c966b91e325d", "parents": [ "78415d2d306bfed0a0ac351aec6c69759d007224" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Sat Nov 27 11:37:54 2010 -0600" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Nov 30 05:49:24 2010 +0000" }, "message": "cifs: Handle extended attribute name cifs_acl to generate cifs acl blob (try #4)\n\nAdd extended attribute name system.cifs_acl\n\nGet/generate cifs/ntfs acl blob and hand over to the invoker however\nit wants to parse/process it under experimental configurable option CIFS_ACL.\n\nDo not get CIFS/NTFS ACL for xattr for attribute system.posix_acl_access\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "78415d2d306bfed0a0ac351aec6c69759d007224", "tree": "761fd93aafa6d90f9b3b18af0a4157072190519f", "parents": [ "523fb8c867650196bef830d5dd9315d9975a9b7e" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Sat Nov 27 11:37:26 2010 -0600" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Nov 30 05:49:17 2010 +0000" }, "message": "cifs: Misc. cleanup in cifsacl handling [try #4]\n\nChange the name of function mode_to_acl to mode_to_cifs_acl.\n\nHandle return code in functions mode_to_cifs_acl and\ncifs_acl_to_fattr.\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "523fb8c867650196bef830d5dd9315d9975a9b7e", "tree": "7697f22640a3281649e2445a58f1e0d5f431d1f2", "parents": [ "362d31297fafb150676f4d564ecc7f7f3e3b7fd4" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Mon Nov 29 22:39:47 2010 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Nov 29 17:48:16 2010 +0000" }, "message": "cifs: trivial comment fix for cifs_invalidate_mapping\n\nOnly the callers check whether the invalid_mapping flag is set and not\ncifs_invalidate_mapping().\n\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "362d31297fafb150676f4d564ecc7f7f3e3b7fd4", "tree": "b210a4dc7c36b3177a0677e2d47e57fadd87cf4d", "parents": [ "59c55ba1fba61ae13fbfff094a674e73407201f1" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sun Nov 14 03:34:30 2010 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sun Nov 14 03:34:30 2010 +0000" }, "message": "[CIFS] fs/cifs/Kconfig: CIFS depends on CRYPTO_HMAC\n\nlinux-2.6.37-rc1: I compiled a kernel with CIFS which subsequently\nfailed with an error indicating it couldn\u0027t initialize crypto module\n\"hmacmd5\". CONFIG_CRYPTO_HMAC\u003dy fixed the problem.\n\nThis patch makes CIFS depend on CRYPTO_HMAC in kconfig.\n\nSigned-off-by: Jody Bruchon\u003cjody@nctritech.com\u003e\nCC: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "59c55ba1fba61ae13fbfff094a674e73407201f1", "tree": "0ab51709f8115e4bbd8bed8302175e7d4be5ce3c", "parents": [ "987b21d7d91d0335e164a41a312174851c58f333" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Nov 12 06:30:29 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat Nov 13 03:26:17 2010 +0000" }, "message": "cifs: don\u0027t take extra tlink reference in initiate_cifs_search\n\nIt\u0027s possible for initiate_cifs_search to be called on a filp that\nalready has private_data attached. If this happens, we\u0027ll end up\ncalling cifs_sb_tlink, taking an extra reference to the tlink and\nattaching that to the cifsFileInfo. This leads to refcount leaks\nthat manifest as a \"stuck\" cifsd at umount time.\n\nFix this by only looking up the tlink for the cifsFile on the filp\u0027s\nfirst pass through this function. When called on a filp that already\nhas cifsFileInfo associated with it, just use the tlink reference\nthat it already owns.\n\nThis patch fixes samba.org bug 7792:\n\n https://bugzilla.samba.org/show_bug.cgi?id\u003d7792\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-and-Tested-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "987b21d7d91d0335e164a41a312174851c58f333", "tree": "2e73097480e5482ef0fdc50db19c162e6fea78cf", "parents": [ "a7851ce73b9fdef53f251420e6883cf4f3766534" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Wed Nov 10 07:50:35 2010 -0600" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Nov 11 03:54:36 2010 +0000" }, "message": "cifs: Percolate error up to the caller during get/set acls [try #4]\n\nModify get/set_cifs_acl* calls to reutrn error code and percolate the\nerror code up to the caller.\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "a7851ce73b9fdef53f251420e6883cf4f3766534", "tree": "cbf63df689b3e28769555f294e69b86bbac9981c", "parents": [ "ebe2e91e000c59aed0300d81815f451c85e0bda6" ], "author": { "name": "Oskar Schirmer", "email": "oskar@scara.com", "time": "Wed Nov 10 21:06:13 2010 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Nov 11 03:40:13 2010 +0000" }, "message": "cifs: fix another memleak, in cifs_root_iget\n\ncifs_root_iget allocates full_path through\ncifs_build_path_to_root, but fails to kfree it upon\ncifs_get_inode_info* failure.\n\nMake all failure exit paths traverse clean up\nhandling at the end of the function.\n\nSigned-off-by: Oskar Schirmer \u003coskar@scara.com\u003e\nReviewed-by: Jesper Juhl \u003cjj@chaosbits.net\u003e\nCc: stable@kernel.org\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ebe2e91e000c59aed0300d81815f451c85e0bda6", "tree": "fc3f70522b873aba4cae89c963386a890d24fa5c", "parents": [ "f6614b7bb405a9b35dd28baea989a749492c46b2" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Nov 10 10:19:10 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Nov 10 15:37:17 2010 +0000" }, "message": "cifs: fix potential use-after-free in cifs_oplock_break_put\n\ncfile may very well be freed after the cifsFileInfo_put. Make sure we\nhave a valid pointer to the superblock for cifs_sb_deactive.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "3565bd46b1c6a3dbf1f670d3275aa4018a4c65ae", "tree": "76e5a4468175b5ca45e674abf5bd87d4fc6ec704", "parents": [ "618763958b2291a09057dbfa553da6ded93dcfad" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Tue Nov 09 12:27:41 2010 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Nov 09 15:17:53 2010 +0000" }, "message": "cifs: fix a memleak in cifs_setattr_nounix()\n\nAndrew Hendry reported a kmemleak warning in 2.6.37-rc1 while editing a\ntext file with gedit over cifs.\n\nunreferenced object 0xffff88022ee08b40 (size 32):\n comm \"gedit\", pid 2524, jiffies 4300160388 (age 2633.655s)\n hex dump (first 32 bytes):\n 5c 2e 67 6f 75 74 70 75 74 73 74 72 65 61 6d 2d \\.goutputstream-\n 35 42 41 53 4c 56 00 de 09 00 00 00 2c 26 78 ee 5BASLV......,\u0026x.\n backtrace:\n [\u003cffffffff81504a4d\u003e] kmemleak_alloc+0x2d/0x60\n [\u003cffffffff81136e13\u003e] __kmalloc+0xe3/0x1d0\n [\u003cffffffffa0313db0\u003e] build_path_from_dentry+0xf0/0x230 [cifs]\n [\u003cffffffffa031ae1e\u003e] cifs_setattr+0x9e/0x770 [cifs]\n [\u003cffffffff8115fe90\u003e] notify_change+0x170/0x2e0\n [\u003cffffffff81145ceb\u003e] sys_fchmod+0x10b/0x140\n [\u003cffffffff8100c172\u003e] system_call_fastpath+0x16/0x1b\n [\u003cffffffffffffffff\u003e] 0xffffffffffffffff\n\nThe commit 1025774c that removed inode_setattr() seems to have introduced this\nmemleak by returning early without freeing \u0027full_path\u0027.\n\nReported-by: Andrew Hendry \u003candrew.hendry@gmail.com\u003e\nCc: Christoph Hellwig \u003chch@lst.de\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "618763958b2291a09057dbfa553da6ded93dcfad", "tree": "985c2e8eba7e0b63f0c9ac314fa1bf6c311aee73", "parents": [ "151f52f09c5728ecfdd0c289da1a4b30bb416f2c" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Nov 08 07:28:32 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Nov 08 18:56:36 2010 +0000" }, "message": "cifs: make cifs_ioctl handle NULL filp-\u003eprivate_data correctly\n\nCommit 13cfb7334e made cifs_ioctl use the tlink attached to the\ncifsFileInfo for a filp. This ignores the case of an open directory\nhowever, which in CIFS can have a NULL private_data until a readdir\nis done on it.\n\nThis patch re-adds the NULL pointer checks that were removed in commit\n50ae28f01 and moves the setting of tcon and \"caps\" variables lower.\n\nLong term, a better fix would be to establish a f_op-\u003eopen routine for\ndirectories that populates that field at open time, but that requires\nsome other changes to how readdir calls are handled.\n\nReported-by: Kjell Rune Skaaraas \u003ckjella79@yahoo.no\u003e\nReviewed-and-Tested-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "c67236281c5d749741f5414103903a7c1b9c4636", "tree": "ddc93fd5fcf47db18ce067309475a12a32b4c00c", "parents": [ "d38922949d377da7d47473c7868334408ae3b373" ], "author": { "name": "Pavel Shilovsky", "email": "piastryyy@gmail.com", "time": "Wed Nov 03 10:58:57 2010 +0300" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Nov 05 17:39:01 2010 +0000" }, "message": "cifs: make cifs_set_oplock_level() take a cifsInodeInfo pointer\n\nAll the callers already have a pointer to struct cifsInodeInfo. Use it.\n\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "d38922949d377da7d47473c7868334408ae3b373", "tree": "322cf984f274c45da08d57acc74d29bb5c5e5048", "parents": [ "6ef933a38ade555a175ecab9d803e6bb73399763" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Nov 02 16:22:50 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Nov 04 19:39:07 2010 +0000" }, "message": "cifs: dereferencing first then checking\n\nThis patch is based on Dan\u0027s original patch. His original description is\nbelow:\n\nSmatch complained about a couple checking for NULL after dereferencing\nbugs. I\u0027m not super familiar with the code so I did the conservative\nthing and move the dereferences after the checks.\n\nThe dereferences in cifs_lock() and cifs_fsync() were added in\nba00ba64cf0 \"cifs: make various routines use the cifsFileInfo-\u003etcon\npointer\". The dereference in find_writable_file() was added in\n6508d904e6f \"cifs: have find_readable/writable_file filter by fsuid\".\nThe comments there say it\u0027s possible to trigger the NULL dereference\nunder stress.\n\nSigned-off-by: Dan Carpenter \u003cerror27@gmail.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6ef933a38ade555a175ecab9d803e6bb73399763", "tree": "4def2e603604ce790709748e7729688bee3f2a3a", "parents": [ "54eeafe1e4fb7b11da17adacacb1fbe279e0cf6e" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Wed Nov 03 10:53:49 2010 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Nov 04 19:35:30 2010 +0000" }, "message": "cifs: trivial comment fix: tlink_tree is now a rbtree\n\nNoticed while reviewing (late) the rbtree conversion patchset (which has been merged\nalready).\n\nCc: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "54eeafe1e4fb7b11da17adacacb1fbe279e0cf6e", "tree": "834e7946a6518745ce168dfb6cbd7ffc6c21739c", "parents": [ "b647c35f77af9c07d336247b23014596e9f0a593" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Nov 02 19:22:45 2010 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Nov 02 19:22:45 2010 +0000" }, "message": "[CIFS] Cleanup unused variable build warning\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "b647c35f77af9c07d336247b23014596e9f0a593", "tree": "653949250681fd1c23ad529b631da793f95778a6", "parents": [ "413e661c136c52290de1ee19a1b049a4da9dbf51" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Oct 28 11:16:44 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Nov 02 19:20:23 2010 +0000" }, "message": "cifs: convert tlink_tree to a rbtree\n\nRadix trees are ideal when you want to track a bunch of pointers and\ncan\u0027t embed a tracking structure within the target of those pointers.\nThe tradeoff is an increase in memory, particularly if the tree is\nsparse.\n\nIn CIFS, we use the tlink_tree to track tcon_link structs. A tcon_link\ncan never be in more than one tlink_tree, so there\u0027s no impediment to\nusing a rb_tree here instead of a radix tree.\n\nConvert the new multiuser mount code to use a rb_tree instead. This\nshould reduce the memory required to manage the tlink_tree.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "413e661c136c52290de1ee19a1b049a4da9dbf51", "tree": "cd875b7b3cab4c9df89b360734a2d55d220acd43", "parents": [ "df098db12ada832c0232ee1f91eff21a8701889c" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Oct 28 13:33:38 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Nov 02 19:15:09 2010 +0000" }, "message": "cifs: store pointer to master tlink in superblock (try #2)\n\nThis is the second version of this patch, the only difference between\nit and the first one is that this explicitly makes cifs_sb_master_tlink\na static inline.\n\nInstead of keeping a tag on the master tlink in the tree, just keep a\npointer to the master in the superblock. That eliminates the need for\nusing the radix tree to look up a tagged entry.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "df098db12ada832c0232ee1f91eff21a8701889c", "tree": "272b78491e0c78e8173f5fe2aef3c371b375cf84", "parents": [ "e66673e39ac9d4749bd9676dd1caf928095409f5" ], "author": { "name": "J. Bruce Fields", "email": "bfields@fieldses.org", "time": "Sat Oct 30 17:06:21 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Nov 02 18:48:14 2010 +0000" }, "message": "cifs: trivial doc fix: note setlease implemented\n\nSigned-off-by: J. Bruce Fields \u003cbfields@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "e66673e39ac9d4749bd9676dd1caf928095409f5", "tree": "f16191d689910a4f939992d8836e575195633116", "parents": [ "ce2f6fb8bd8c1e85f288033020dfc31c9fc2be94" ], "author": { "name": "Pavel Shilovsky", "email": "piastryyy@gmail.com", "time": "Tue Nov 02 12:00:42 2010 +0300" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Nov 02 18:40:54 2010 +0000" }, "message": "CIFS: Add cifs_set_oplock_level\n\nSimplify many places when we need to set oplock level on an inode.\n\nSigned-off-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "50ae28f0144a790fc63a5b89b9aca3ffa9f88522", "tree": "b0e77798ad82428f4f0026cf8f72cc0608ab3223", "parents": [ "c8ddb2713c624f432fa5fe3c7ecffcdda46ea0d4" ], "author": { "name": "Jiri Slaby", "email": "jslaby@suse.cz", "time": "Mon Nov 01 16:08:55 2010 +0100" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Nov 02 03:47:21 2010 +0000" }, "message": "FS: cifs, remove unneeded NULL tests\n\nStanse found that pSMBFile in cifs_ioctl and file-\u003ef_path.dentry in\ncifs_user_write are dereferenced prior their test to NULL.\n\nThe alternative is not to dereference them before the tests. The patch is\nto point out the problem, you have to decide.\n\nWhile at it we cache the inode in cifs_user_write to a local variable\nand use all over the function.\n\nSigned-off-by: Jiri Slaby \u003cjslaby@suse.cz\u003e\nCc: Steve French \u003csfrench@samba.org\u003e\nCc: linux-cifs@vger.kernel.org\nCc: Jeff Layton \u003cjlayton@redhat.com\u003e\nCc: Christoph Hellwig \u003chch@infradead.org\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "51ee4b84f5c86935b438d6636f34b523edb415a8", "tree": "c60e0da8f8b6393477d79ef6d6ce321ee3b8ecaa", "parents": [ "96f935934591b72f5b05fd6923bc8cdcae92f2e5" ], "author": { "name": "Christoph Hellwig", "email": "hch@infradead.org", "time": "Sun Oct 31 08:35:10 2010 -0400" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun Oct 31 06:35:15 2010 -0700" }, "message": "locks: let the caller free file_lock on -\u003esetlease failure\n\nThe caller allocated it, the caller should free it.\n\nThe only issue so far is that we could change the flp pointer even on an\nerror return if the fl_change callback failed. But we can simply move\nthe flp assignment after the fl_change invocation, as the callers don\u0027t\ncare about the flp return value if the setlease call failed.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "05fa3135fdc7b9b510b502a35b6b97d2b38c6f48", "tree": "d533e8112111202ae890b2061c0386669002d080", "parents": [ "096657b65e1ac197e20be5ce7cff6b6ca2532787" ], "author": { "name": "J. Bruce Fields", "email": "bfields@redhat.com", "time": "Sat Oct 30 17:31:15 2010 -0400" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sat Oct 30 18:08:15 2010 -0700" }, "message": "locks: fix setlease methods to free passed-in lock\n\nWe modified setlease to require the caller to allocate the new lease in\nthe case of creating a new lease, but forgot to fix up the filesystem\nmethods.\n\nCc: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nCc: Steve French \u003csfrench@samba.org\u003e\nCc: Trond Myklebust \u003cTrond.Myklebust@netapp.com\u003e\nSigned-off-by: J. Bruce Fields \u003cbfields@redhat.com\u003e\nAcked-by: Arnd Bergmann \u003carnd@arndb.de\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "b4020c1b198c0f0c0b0ff0cfdd824a26b93edd6f", "tree": "f2d0dcca7bae865606e90ffc2599c993da1a888f", "parents": [ "435f49a518c78eec8e2edbbadd912737246cbe20", "d3686d54c7902a303bd65d751226aa1647319863" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Oct 29 10:37:27 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Oct 29 10:37:27 2010 -0700" }, "message": "Merge git://git.kernel.org/pub/scm/linux/kernel/git/sfrench/cifs-2.6\n\n* git://git.kernel.org/pub/scm/linux/kernel/git/sfrench/cifs-2.6:\n cifs: Cleanup and thus reduce smb session structure and fields used during authentication\n NTLM auth and sign - Use appropriate server challenge\n cifs: add kfree() on error path\n NTLM auth and sign - minor error corrections and cleanup\n NTLM auth and sign - Use kernel crypto apis to calculate hashes and smb signatures\n NTLM auth and sign - Define crypto hash functions and create and send keys needed for key exchange\n cifs: cifs_convert_address() returns zero on error\n NTLM auth and sign - Allocate session key/client response dynamically\n cifs: update comments - [s/GlobalSMBSesLock/cifs_file_list_lock/g]\n cifs: eliminate cifsInodeInfo-\u003ewrite_behind_rc (try #6)\n [CIFS] Fix checkpatch warnings and bump cifs version number\n cifs: wait for writeback to complete in cifs_flush\n cifs: convert cifsFileInfo-\u003ecount to non-atomic counter\n" }, { "commit": "d753ed975953a4e97a356bcd59ff146919f29235", "tree": "4fe98f79583f74794e64174db7da3f72d92f3246", "parents": [ "e4c59d61e80529aebca4a3690b4378f2c6fc4e82" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Mon Jul 26 12:52:33 2010 +0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Oct 29 04:16:56 2010 -0400" }, "message": "convert cifs\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "d3686d54c7902a303bd65d751226aa1647319863", "tree": "b4acd7dfc6c5ec2f254608a2f0ef11ef274861b8", "parents": [ "d3ba50b17aa7a391bb5b3dcd8d6ba7a02c4f031c" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Thu Oct 28 09:53:07 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Oct 29 01:47:33 2010 +0000" }, "message": "cifs: Cleanup and thus reduce smb session structure and fields used during authentication\n\nRemoved following fields from smb session structure\n cryptkey, ntlmv2_hash, tilen, tiblob\nand ntlmssp_auth structure is allocated dynamically only if the auth mech\nin NTLMSSP.\n\nresponse field within a session_key structure is used to initially store the\ntarget info (either plucked from type 2 challenge packet in case of NTLMSSP\nor fabricated in case of NTLMv2 without extended security) and then to store\nMessage Authentication Key (mak) (session key + client response).\n\nServer challenge or cryptkey needed during a NTLMSSP authentication\nis now part of ntlmssp_auth structure which gets allocated and freed\nonce authenticaiton process is done.\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "d3ba50b17aa7a391bb5b3dcd8d6ba7a02c4f031c", "tree": "452ddaeb1d52387a852d0c3cf46c4253347d0a27", "parents": [ "6b03590412c977ae8fa1635c9b80854ab19a5b78" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Wed Oct 27 15:20:36 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Oct 29 01:47:30 2010 +0000" }, "message": "NTLM auth and sign - Use appropriate server challenge\n\nNeed to have cryptkey or server challenge in smb connection\n(struct TCP_Server_Info) for ntlm and ntlmv2 auth types for which\ncryptkey (Encryption Key) is supplied just once in Negotiate Protocol\nresponse during an smb connection setup for all the smb sessions over\nthat smb connection.\n\nFor ntlmssp, cryptkey or server challenge is provided for every\nsmb session in type 2 packet of ntlmssp negotiation, the cryptkey\nprovided during Negotiation Protocol response before smb connection\ndoes not count.\n\nRename cryptKey to cryptkey and related changes.\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6b03590412c977ae8fa1635c9b80854ab19a5b78", "tree": "b2a6d50f52cbb26241fa9bc7a951488e3b645847", "parents": [ "f7c5445a9deecffea8a4fffc0163bf582411ac8a" ], "author": { "name": "Dan Carpenter", "email": "error27@gmail.com", "time": "Wed Oct 27 23:19:32 2010 +0200" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Oct 28 00:55:45 2010 +0000" }, "message": "cifs: add kfree() on error path\n\nWe leak 256 bytes here on this error path.\n\nSigned-off-by: Dan Carpenter \u003cerror27@gmail.com\u003e\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "f7c5445a9deecffea8a4fffc0163bf582411ac8a", "tree": "efea64611599a034510be60077ca20bed68a48e8", "parents": [ "307fbd31b61623ad1b5388b452118f8aea99f9d0" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Tue Oct 26 18:10:24 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Oct 27 02:04:30 2010 +0000" }, "message": "NTLM auth and sign - minor error corrections and cleanup\n\nMinor cleanup - Fix spelling mistake, make meaningful (goto) label\n\nIn function setup_ntlmv2_rsp(), do not return 0 and leak memory,\nlet the tiblob get freed.\n\nFor function find_domain_name(), pass already available nls table pointer\ninstead of loading and unloading the table again in this function.\n\nFor ntlmv2, the case sensitive password length is the length of the\nresponse, so subtract session key length (16 bytes) from the .len.\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "1b430beee5e388605dfb092b214ef0320f752cf6", "tree": "c1b1ece282aab771fd1386a3fe0c6e82cb5c5bfe", "parents": [ "d19d5476f4b9f91d2de92b91588bb118beba6c0d" ], "author": { "name": "Wu Fengguang", "email": "fengguang.wu@intel.com", "time": "Tue Oct 26 14:21:26 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Oct 26 16:52:05 2010 -0700" }, "message": "writeback: remove nonblocking/encountered_congestion references\n\nThis removes more dead code that was somehow missed by commit 0d99519efef\n(writeback: remove unused nonblocking and congestion checks). There are\nno behavior change except for the removal of two entries from one of the\next4 tracing interface.\n\nThe nonblocking checks in -\u003ewritepages are no longer used because the\nflusher now prefer to block on get_request_wait() than to skip inodes on\nIO congestion. The latter will lead to more seeky IO.\n\nThe nonblocking checks in -\u003ewritepage are no longer used because it\u0027s\nredundant with the WB_SYNC_NONE check.\n\nWe no long set -\u003enonblocking in VM page out and page migration, because\na) it\u0027s effectively redundant with WB_SYNC_NONE in current code\nb) it\u0027s old semantic of \"Don\u0027t get stuck on request queues\" is mis-behavior:\n that would skip some dirty inodes on congestion and page out others, which\n is unfair in terms of LRU age.\n\nInspired by Christoph Hellwig. Thanks!\n\nSigned-off-by: Wu Fengguang \u003cfengguang.wu@intel.com\u003e\nCc: Theodore Ts\u0027o \u003ctytso@mit.edu\u003e\nCc: David Howells \u003cdhowells@redhat.com\u003e\nCc: Sage Weil \u003csage@newdream.net\u003e\nCc: Steve French \u003csfrench@samba.org\u003e\nCc: Chris Mason \u003cchris.mason@oracle.com\u003e\nCc: Jens Axboe \u003caxboe@kernel.dk\u003e\nCc: Christoph Hellwig \u003chch@infradead.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "307fbd31b61623ad1b5388b452118f8aea99f9d0", "tree": "71231b840aac77531d5dc26021ecb0451feab7ae", "parents": [ "d2b915210b5ec01409f581421d633eca6c38d444" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Thu Oct 21 14:25:17 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Oct 26 18:38:06 2010 +0000" }, "message": "NTLM auth and sign - Use kernel crypto apis to calculate hashes and smb signatures\n\nUse kernel crypto sync hash apis insetead of cifs crypto functions.\nThe calls typically corrospond one to one except that insead of\nkey init, setkey is used.\n\nUse crypto apis to generate smb signagtures also.\nUse hmac-md5 to genereate ntlmv2 hash, ntlmv2 response, and HMAC (CR1 of\nntlmv2 auth blob.\nUser crypto apis to genereate signature and to verify signature.\nmd5 hash is used to calculate signature.\nUse secondary key to calculate signature in case of ntlmssp.\n\nFor ntlmv2 within ntlmssp, during signature calculation, only 16 bytes key\n(a nonce) stored within session key is used. during smb signature calculation.\nFor ntlm and ntlmv2 without extended security, 16 bytes key\nas well as entire response (24 bytes in case of ntlm and variable length\nin case of ntlmv2) is used for smb signature calculation.\nFor kerberos, there is no distinction between key and response.\n\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "d2b915210b5ec01409f581421d633eca6c38d444", "tree": "5e4467f4888eb8c9936fbfd7b5bd7b9058af26e9", "parents": [ "b235f371a2572d7c86a121d96d889eee02ed00e2" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Thu Oct 21 14:25:08 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Oct 26 18:35:31 2010 +0000" }, "message": "NTLM auth and sign - Define crypto hash functions and create and send keys needed for key exchange\n\nMark dependency on crypto modules in Kconfig.\n\nDefining per structures sdesc and cifs_secmech which are used to store\ncrypto hash functions and contexts. They are stored per smb connection\nand used for all auth mechs to genereate hash values and signatures.\n\nAllocate crypto hashing functions, security descriptiors, and respective\ncontexts when a smb/tcp connection is established.\nRelease them when a tcp/smb connection is taken down.\n\nmd5 and hmac-md5 are two crypto hashing functions that are used\nthrought the life of an smb/tcp connection by various functions that\ncalcualte signagure and ntlmv2 hash, HMAC etc.\n\nstructure ntlmssp_auth is defined as per smb connection.\n\nntlmssp_auth holds ciphertext which is genereated by rc4/arc4 encryption of\nsecondary key, a nonce using ntlmv2 session key and sent in the session key\nfield of the type 3 message sent by the client during ntlmssp\nnegotiation/exchange\n\nA key is exchanged with the server if client indicates so in flags in\ntype 1 messsage and server agrees in flag in type 2 message of ntlmssp\nnegotiation. If both client and agree, a key sent by client in\ntype 3 message of ntlmssp negotiation in the session key field.\nThe key is a ciphertext generated off of secondary key, a nonce, using\nntlmv2 hash via rc4/arc4.\n\nSigning works for ntlmssp in this patch. The sequence number within\nthe server structure needs to be zero until session is established\ni.e. till type 3 packet of ntlmssp exchange of a to be very first\nsmb session on that smb connection is sent.\n\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "b235f371a2572d7c86a121d96d889eee02ed00e2", "tree": "72ae25c83a6ba59926a599e28d47b67bfa0ad863", "parents": [ "21e733930be6458e0c33482b6783e7c15ba984eb" ], "author": { "name": "Dan Carpenter", "email": "error27@gmail.com", "time": "Mon Oct 25 07:02:56 2010 +0200" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Oct 26 18:22:38 2010 +0000" }, "message": "cifs: cifs_convert_address() returns zero on error\n\nThe cifs_convert_address() returns zero on error but this caller is\ntesting for negative returns.\n\nBtw. \"i\" is unsigned here, so it\u0027s never negative.\n\nReviewed-by: Jeff Layton \u003cjlayton@samba.org\u003e\nSigned-off-by: Dan Carpenter \u003cerror27@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "21e733930be6458e0c33482b6783e7c15ba984eb", "tree": "1458ccf45529649267451a7e06fcd25d9c7ab0ac", "parents": [ "6573e9b73e19c0f6b9dfa2b399267ea0f42d6c6b" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Thu Oct 21 06:42:55 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Oct 26 18:20:10 2010 +0000" }, "message": "NTLM auth and sign - Allocate session key/client response dynamically\n\nStart calculating auth response within a session. Move/Add pertinet\ndata structures like session key, server challenge and ntlmv2_hash in\na session structure. We should do the calculations within a session\nbefore copying session key and response over to server data\nstructures because a session setup can fail.\n\nOnly after a very first smb session succeeds, it copy/make its\nsession key, session key of smb connection. This key stays with\nthe smb connection throughout its life.\nsequence_number within server is set to 0x2.\n\nThe authentication Message Authentication Key (mak) which consists\nof session key followed by client response within structure session_key\nis now dynamic. Every authentication type allocates the key + response\nsized memory within its session structure and later either assigns or\nfrees it once the client response is sent and if session\u0027s session key\nbecomes connetion\u0027s session key.\n\nntlm/ntlmi authentication functions are rearranged. A function\nnamed setup_ntlm_resp(), similar to setup_ntlmv2_resp(), replaces\nfunction cifs_calculate_session_key().\n\nsize of CIFS_SESS_KEY_SIZE is changed to 16, to reflect the byte size\nof the key it holds.\n\nReviewed-by: Jeff Layton \u003cjlayton@samba.org\u003e\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6573e9b73e19c0f6b9dfa2b399267ea0f42d6c6b", "tree": "7752091c8b066be2c792164f9dd1a3ea4354c4a8", "parents": [ "eb4b756b1e60b66e54932619088b645c712414a3" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Mon Oct 18 23:52:18 2010 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Oct 25 00:19:01 2010 +0000" }, "message": "cifs: update comments - [s/GlobalSMBSesLock/cifs_file_list_lock/g]\n\nGlobalSMBSesLock is now cifs_file_list_lock. Update comments to reflect this.\n\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "eb4b756b1e60b66e54932619088b645c712414a3", "tree": "c2a10df2f33be6bd9a8b5491cc1815515dc59e84", "parents": [ "6c0f6218ba04a5d6e61d0c55b68e6c4ef0736531" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Oct 22 14:52:29 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Oct 25 00:19:00 2010 +0000" }, "message": "cifs: eliminate cifsInodeInfo-\u003ewrite_behind_rc (try #6)\n\nwrite_behind_rc is redundant and just adds complexity to the code. What\nwe really want to do instead is to use mapping_set_error to reset the\nflags on the mapping when we find a writeback error and can\u0027t report it\nto userspace yet.\n\nFor cifs_flush and cifs_fsync, we shouldn\u0027t reset the flags since errors\nreturned there do get reported to userspace.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nReviewed-by: Dave Kleikamp \u003cshaggy@linux.vnet.ibm.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6c0f6218ba04a5d6e61d0c55b68e6c4ef0736531", "tree": "20575207bcaa90572a619483e763a3e4275845b3", "parents": [ "d3f1322af8f866be54ea79a4002fc30c98290411" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat Oct 23 16:51:40 2010 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Oct 25 00:19:00 2010 +0000" }, "message": "[CIFS] Fix checkpatch warnings and bump cifs version number\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "d3f1322af8f866be54ea79a4002fc30c98290411", "tree": "b043a2c269a0e2b8b64c34644dfe8586f114ead9", "parents": [ "5f6dbc9e4afe4d1d39e85de3ac2720a2042ad191" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Oct 15 15:34:07 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Oct 25 00:18:59 2010 +0000" }, "message": "cifs: wait for writeback to complete in cifs_flush\n\nThe f_op-\u003eflush operation is the last chance to return a writeback\nrelated error when closing a file. Ensure that we don\u0027t miss reporting\nany errors by waiting for writeback to complete in cifs_flush before\nproceeding.\n\nThere\u0027s no reason to do this when the file isn\u0027t open for write\nhowever.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nReviewed-by: David Kleikamp \u003cshaggy@linux.vnet.ibm.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "5f6dbc9e4afe4d1d39e85de3ac2720a2042ad191", "tree": "8e97fe18b6d0824981fd377b2ff35042da264bb4", "parents": [ "229aebb873e29726b91e076161649cf45154b0bf" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Oct 15 15:34:06 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Oct 25 00:18:59 2010 +0000" }, "message": "cifs: convert cifsFileInfo-\u003ecount to non-atomic counter\n\nThe count for cifsFileInfo is currently an atomic, but that just adds\ncomplexity for little value. We generally need to hold cifs_file_list_lock\nto traverse the lists anyway so we might as well make this counter\nnon-atomic and simply use the cifs_file_list_lock to protect it.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "d2ecad9faca2221ae6f664f146f0dcae661bf39d", "tree": "9a13aabd0925e41586a292d34990ac2df73c34b2", "parents": [ "c70b5296e775cde46cfcb2d860ba160108a5ec7a", "cdff08e76612e53580139653403aedea979aa639" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Oct 22 17:52:29 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Oct 22 17:52:29 2010 -0700" }, "message": "Merge git://git.kernel.org/pub/scm/linux/kernel/git/sfrench/cifs-2.6\n\n* git://git.kernel.org/pub/scm/linux/kernel/git/sfrench/cifs-2.6: (56 commits)\n [CIFS] move close processing from cifs_close to cifsFileInfo_put\n cifs: convert cifs_tcp_ses_lock from a rwlock to a spinlock\n cifs: cancel_delayed_work() + flush_scheduled_work() -\u003e cancel_delayed_work_sync()\n Clean up two declarations of blob_len\n cifs: move cifsFileInfo_put to file.c\n cifs: convert GlobalSMBSeslock from a rwlock to regular spinlock\n [CIFS] Fix minor checkpatch warning and update cifs version\n cifs: move cifs_new_fileinfo to file.c\n cifs: eliminate pfile pointer from cifsFileInfo\n cifs: cifs_write argument change and cleanup\n cifs: clean up cifs_reopen_file\n cifs: eliminate the inode argument from cifs_new_fileinfo\n cifs: eliminate oflags option from cifs_new_fileinfo\n cifs: fix flags handling in cifs_posix_open\n cifs: eliminate cifs_posix_open_inode_helper\n cifs: handle FindFirst failure gracefully\n NTLM authentication and signing - Calculate auth response per smb session\n cifs: don\u0027t use vfsmount to pin superblock for oplock breaks\n cifs: keep dentry reference in cifsFileInfo instead of inode reference\n cifs: on multiuser mount, set ownership to current_fsuid/current_fsgid (try #7)\n ...\n\nFix up trivial conflict in fs/cifs/cifsfs.c due to added/removed header files\n" }, { "commit": "79f14b7c56d3b3ba58f8b43d1f70b9b71477a800", "tree": "3bb53b7806c1baba6cc24b91724a9264cceccd39", "parents": [ "c37927d4359e81b85de644f8fb08878717cf5f3f", "6d7bccc2215c37205ede6c9cf84db64e7c4f9443" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Oct 22 10:52:01 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Oct 22 10:52:01 2010 -0700" }, "message": "Merge branch \u0027vfs\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/arnd/bkl\n\n* \u0027vfs\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/arnd/bkl: (30 commits)\n BKL: remove BKL from freevxfs\n BKL: remove BKL from qnx4\n autofs4: Only declare function when CONFIG_COMPAT is defined\n autofs: Only declare function when CONFIG_COMPAT is defined\n ncpfs: Lock socket in ncpfs while setting its callbacks\n fs/locks.c: prepare for BKL removal\n BKL: Remove BKL from ncpfs\n BKL: Remove BKL from OCFS2\n BKL: Remove BKL from squashfs\n BKL: Remove BKL from jffs2\n BKL: Remove BKL from ecryptfs\n BKL: Remove BKL from afs\n BKL: Remove BKL from USB gadgetfs\n BKL: Remove BKL from autofs4\n BKL: Remove BKL from isofs\n BKL: Remove BKL from fat\n BKL: Remove BKL from ext2 filesystem\n BKL: Remove BKL from do_new_mount()\n BKL: Remove BKL from cgroup\n BKL: Remove BKL from NTFS\n ...\n" }, { "commit": "cdff08e76612e53580139653403aedea979aa639", "tree": "1ebd74be361b7e34158974242a2303f7833c983d", "parents": [ "3f9bcca7820a6711307b6499952b13cfcfc31dd6" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Oct 21 22:46:14 2010 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Oct 21 22:46:14 2010 +0000" }, "message": "[CIFS] move close processing from cifs_close to cifsFileInfo_put\n\nNow that it\u0027s feasible for a cifsFileInfo to outlive the filp under\nwhich it was created, move the close processing into cifsFileInfo_put.\n\nThis means that the last user of the filehandle always does the actual\non the wire close call. This also allows us to get rid of the closePend\nflag from cifsFileInfo. If we have an active reference to the file\nthen it\u0027s never going to have a close pending.\n\ncifs_close is converted to simply put the filehandle.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "3f9bcca7820a6711307b6499952b13cfcfc31dd6", "tree": "6c380f5877562778335d6794e1e4a297f8970d77", "parents": [ "3e24e132878c83910b61eb7704511a6d96a0389f" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Mon Oct 18 23:29:37 2010 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Oct 21 13:14:27 2010 +0000" }, "message": "cifs: convert cifs_tcp_ses_lock from a rwlock to a spinlock\n\ncifs_tcp_ses_lock is a rwlock with protects the cifs_tcp_ses_list,\nserver-\u003esmb_ses_list and the ses-\u003etcon_list. It also protects a few\nref counters in server, ses and tcon. In most cases the critical section\ndoesn\u0027t seem to be large, in a few cases where it is slightly large, there\nseem to be really no benefit from concurrent access. I briefly considered RCU\nmechanism but it appears to me that there is no real need.\n\nReplace it with a spinlock and get rid of the last rwlock in the cifs code.\n\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "3e24e132878c83910b61eb7704511a6d96a0389f", "tree": "e67f001dd582ad202fc779df4b0a93867abc2ea2", "parents": [ "89f150f401c32b0a587dcb98d3bcfafe0b9c1c70" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Tue Oct 19 17:55:54 2010 +0200" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Oct 19 18:58:36 2010 +0000" }, "message": "cifs: cancel_delayed_work() + flush_scheduled_work() -\u003e cancel_delayed_work_sync()\n\nflush_scheduled_work() is going away.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "89f150f401c32b0a587dcb98d3bcfafe0b9c1c70", "tree": "cff93486560754e3c4652bf78ddd120daee5ba8c", "parents": [ "b33879aa834ebe03ced3dca4e3b822bd8894a474" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Tue Oct 19 11:47:52 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Oct 19 18:56:42 2010 +0000" }, "message": "Clean up two declarations of blob_len\n\n- Eliminate double declaration of variable blob_len\n- Modify function build_ntlmssp_auth_blob to return error code\n as well as length of the blob.\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@samba.org\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "b33879aa834ebe03ced3dca4e3b822bd8894a474", "tree": "9475bb683f03180a5fe7909d4182bb5273f5d7a6", "parents": [ "4477288a103631980750c86547d1fd54bfd2ba7d" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Oct 15 15:34:04 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Oct 18 01:32:05 2010 +0000" }, "message": "cifs: move cifsFileInfo_put to file.c\n\n...and make it non-inlined in preparation for the move of most of\ncifs_close to it.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "4477288a103631980750c86547d1fd54bfd2ba7d", "tree": "e0d56476a740f043f6687e063ff88d2982b4c444", "parents": [ "7a16f1961a5c61d1f60d9e0d3d171cf7793fb5cb" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Oct 15 15:34:03 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Oct 18 01:32:01 2010 +0000" }, "message": "cifs: convert GlobalSMBSeslock from a rwlock to regular spinlock\n\nConvert this lock to a regular spinlock\n\nA rwlock_t offers little value here. It\u0027s more expensive than a regular\nspinlock unless you have a fairly large section of code that runs under\nthe read lock and can benefit from the concurrency.\n\nAdditionally, we need to ensure that the refcounting for files isn\u0027t\nracy and to do that we need to lock areas that can increment it for\nwrite. That means that the areas that can actually use a read_lock are\nvery few and relatively infrequently used.\n\nWhile we\u0027re at it, change the name to something easier to type, and fix\na bug in find_writable_file. cifsFileInfo_put can sleep and shouldn\u0027t be\ncalled while holding the lock.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "7a16f1961a5c61d1f60d9e0d3d171cf7793fb5cb", "tree": "33a34dda237761765b2f2cbcea9de569a5dae726", "parents": [ "15ecb436c00fcb13b6dc32bdcbb9f75fc9b7613e" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Oct 18 01:09:48 2010 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Oct 18 01:09:48 2010 +0000" }, "message": "[CIFS] Fix minor checkpatch warning and update cifs version\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "15ecb436c00fcb13b6dc32bdcbb9f75fc9b7613e", "tree": "983bd08a195f701b3bffdb495fcf4e6f0e3b42cc", "parents": [ "2e396b83f6087b78dac5a18d6d0cf9f8426a00b3" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Oct 15 15:34:02 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Oct 18 01:07:31 2010 +0000" }, "message": "cifs: move cifs_new_fileinfo to file.c\n\nIt\u0027s currently in dir.c which makes little sense...\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nAcked-by: Dave Kleikamp \u003cshaggy@linux.vnet.ibm.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "2e396b83f6087b78dac5a18d6d0cf9f8426a00b3", "tree": "890c0af098920fe79e85f475a25fcc2f0d8c00bc", "parents": [ "7da4b49a0ec481239d9cd51f7053aa8e4e191fa7" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Oct 15 15:34:01 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Oct 18 01:07:20 2010 +0000" }, "message": "cifs: eliminate pfile pointer from cifsFileInfo\n\nAll the remaining users of cifsFileInfo-\u003epfile just use it to get\nat the f_flags/f_mode. Now that we store that separately in the\ncifsFileInfo, there\u0027s no need to consult the pfile at all from\na cifsFileInfo pointer.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nAcked-by: Dave Kleikamp \u003cshaggy@linux.vnet.ibm.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "7da4b49a0ec481239d9cd51f7053aa8e4e191fa7", "tree": "97f451d75ede084bb7f1c8d2bc843680763f2e16", "parents": [ "15886177e412db00aa86155fe72608c4ebf5a08f" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Oct 15 15:34:00 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Oct 18 01:04:23 2010 +0000" }, "message": "cifs: cifs_write argument change and cleanup\n\nHave cifs_write take a cifsFileInfo pointer instead of a filp. Since\ncifsFileInfo holds references on the dentry, and that holds one to\nthe inode, we can eliminate some unneeded NULL pointer checks.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nAcked-by: Dave Kleikamp \u003cshaggy@linux.vnet.ibm.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "15886177e412db00aa86155fe72608c4ebf5a08f", "tree": "eb4ac34ee40ae499ae2a68c44f71effe657b4504", "parents": [ "abfe1eedd682ea0f20e7035445982e6d371a2024" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Oct 15 15:33:59 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Oct 18 01:04:19 2010 +0000" }, "message": "cifs: clean up cifs_reopen_file\n\nAdd a f_flags field that holds the f_flags field from the filp. We\u0027ll\nneed this info in case the filp ever goes away before the cifsFileInfo\ndoes. Have cifs_reopen_file use that value instead of filp-\u003ef_flags\ntoo and have it take a cifsFileInfo arg instead of a filp.\n\nWhile we\u0027re at it, get rid of some bogus cargo-cult NULL pointer\nchecks in that function and reduce the level of indentation.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nAcked-by: Dave Kleikamp \u003cshaggy@linux.vnet.ibm.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "abfe1eedd682ea0f20e7035445982e6d371a2024", "tree": "70d5fe483cae0f782b8e78ea5c08ef2dbb45c2e7", "parents": [ "f6a53460e2a105904deeada737b3f878d78517b3" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Oct 15 15:33:58 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Oct 18 01:04:16 2010 +0000" }, "message": "cifs: eliminate the inode argument from cifs_new_fileinfo\n\nIt already takes a file pointer. The inode associated with that had damn\nwell better be the same one we\u0027re passing in anyway. Thus, there\u0027s no\nneed for a separate argument here.\n\nAlso, get rid of the bogus check for a null pCifsInode pointer. The\nCIFS_I macro uses container_of(), and that will virtually never return a\nNULL pointer anyway.\n\nFinally, move the setting of the canCache* flags outside of the lock.\nOther places in the code don\u0027t hold that lock when setting it, so I\nassume it\u0027s not really needed here either.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nAcked-by: Dave Kleikamp \u003cshaggy@linux.vnet.ibm.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "f6a53460e2a105904deeada737b3f878d78517b3", "tree": "f51122e6e3e2a5d3a864d17cad708d313b0ef12a", "parents": [ "608712fe8609492a8670638ea86b97fafe49ebba" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Oct 15 15:33:57 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Oct 18 00:34:35 2010 +0000" }, "message": "cifs: eliminate oflags option from cifs_new_fileinfo\n\nEliminate the poor, misunderstood \"oflags\" option from cifs_new_fileinfo.\nThe callers mostly pass in the filp-\u003ef_flags here.\n\nThat\u0027s not correct however since we\u0027re checking that value for\nthe presence of FMODE_READ. Luckily that only affects how the f_list is\nordered. What it really wants here is the file-\u003ef_mode. Just use that\nfield from the filp to determine it.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "608712fe8609492a8670638ea86b97fafe49ebba", "tree": "b175e2ac5e71ab8d6541a82d496d5dc6cc812acf", "parents": [ "2f4f26fcf393ef4a44abe10e79c1966e64e86055" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Oct 15 15:33:56 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Oct 18 00:34:29 2010 +0000" }, "message": "cifs: fix flags handling in cifs_posix_open\n\nThe way flags are passed and converted for cifs_posix_open is rather\nnon-sensical. Some callers call cifs_posix_convert_flags on the flags\nbefore they pass them to cifs_posix_open, whereas some don\u0027t. Two flag\nconversion steps is just confusing though.\n\nChange the function instead to clearly expect input in f_flags format,\nand fix the callers to pass that in. Then, have cifs_posix_open call\ncifs_convert_posix_flags to do the conversion. Move cifs_posix_open to\nfile.c as well so we can keep cifs_convert_posix_flags as a static\nfunction.\n\nFix it also to not ignore O_CREAT, O_EXCL and O_TRUNC, and instead have\ncifs_reopen_file mask those bits off before calling cifs_posix_open.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "2f4f26fcf393ef4a44abe10e79c1966e64e86055", "tree": "0a1f2430336cea006a47d4dc835a9e271ce90dc0", "parents": [ "6221ddd0f5e2ddc1d5d928119a2cde033d16f96d" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Oct 13 18:50:39 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Oct 15 18:22:21 2010 +0000" }, "message": "cifs: eliminate cifs_posix_open_inode_helper\n\ncifs: eliminate cifs_posix_open_inode_helper\n\nThis function is redundant. The only thing it does is set the canCache\nflags, but those get set in cifs_new_fileinfo anyway.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nAcked-by: Dave Kleikamp \u003cshaggy@linux.vnet.ibm.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6221ddd0f5e2ddc1d5d928119a2cde033d16f96d", "tree": "ded17afe18712966dcc26560d9036145a3f1f98c", "parents": [ "5d0d28824c76409f0d1a645bf0ae81318c8ffa42" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Fri Oct 01 21:23:33 2010 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Oct 15 15:19:55 2010 +0000" }, "message": "cifs: handle FindFirst failure gracefully\n\nFindFirst failure due to permission errors or any other errors are silently\nignored by cifs_readdir(). This could cause problem to applications that depend\non the error to do further processing.\n\nReproducer:\n - mount a cifs share\n - mkdir tdir;touch tdir/1 tdir/2 tdir/3\n - chmod -x tdir\n - ls tdir\n\nCurrently, we start calling filldir() for \u0027.\u0027 and \u0027..\u0027 before we know we\nwhether FindFirst could succeed or not. If FindFirst fails later, there is no\nway to notify VFS by setting buf.error and so VFS won\u0027t be able to catch this.\nFix this by moving the call to initiate_cifs_search() before we start doing\nfilldir().\n\nThis fixes https://bugzilla.samba.org/show_bug.cgi?id\u003d7535\n\nReported-by: Tom Dexter \u003cdigitalaudiorock@gmail.com\u003e\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "5d0d28824c76409f0d1a645bf0ae81318c8ffa42", "tree": "592838282fc891dc9a51424e0f57c0694ad31075", "parents": [ "d7c86ff8cd00abc730fe5d031f43dc9138b6324e" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Wed Oct 13 18:15:00 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Oct 14 18:05:19 2010 +0000" }, "message": "NTLM authentication and signing - Calculate auth response per smb session\n\nStart calculation auth response within a session. Move/Add pertinet\ndata structures like session key, server challenge and ntlmv2_hash in\na session structure. We should do the calculations within a session\nbefore copying session key and response over to server data\nstructures because a session setup can fail.\n\nOnly after a very first smb session succeeds, it copies/makes its\nsession key, session key of smb connection. This key stays with\nthe smb connection throughout its life.\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "d7c86ff8cd00abc730fe5d031f43dc9138b6324e", "tree": "a1f264ce2f33e3ac77a459291b9a5e45a63e3bff", "parents": [ "a5e18bc36e9e05ce0338d370a2ce4290910e43ea" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Oct 11 15:07:19 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Oct 12 18:08:01 2010 +0000" }, "message": "cifs: don\u0027t use vfsmount to pin superblock for oplock breaks\n\nFilesystems aren\u0027t really supposed to do anything with a vfsmount. It\u0027s\nconsidered a layering violation since vfsmounts are entirely managed at\nthe VFS layer.\n\nCIFS currently keeps an active reference to a vfsmount in order to\nprevent the superblock vanishing before an oplock break has completed.\nWhat we really want to do instead is to keep sb-\u003es_active high until the\noplock break has completed. This patch borrows the scheme that NFS uses\nfor handling sillyrenames.\n\nAn atomic_t is added to the cifs_sb_info. When it transitions from 0 to\n1, an extra reference to the superblock is taken (by bumping the\ns_active value). When it transitions from 1 to 0, that reference is\ndropped and a the superblock teardown may proceed if there are no more\nreferences to it.\n\nAlso, the vfsmount pointer is removed from cifsFileInfo and from\ncifs_new_fileinfo, and some bogus forward declarations are removed from\ncifsfs.h.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nAcked-by: Dave Kleikamp \u003cshaggy@linux.vnet.ibm.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "a5e18bc36e9e05ce0338d370a2ce4290910e43ea", "tree": "9d8b37c1dd20267c372ff8c98850eeb2f204449b", "parents": [ "1c456013e96f10915578dc61095d19b4906f64ac" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Oct 11 15:07:18 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Oct 12 18:06:42 2010 +0000" }, "message": "cifs: keep dentry reference in cifsFileInfo instead of inode reference\n\ncifsFileInfo is a bit problematic. It contains a reference back to the\nstruct file itself. This makes it difficult for a cifsFileInfo to exist\nwithout a corresponding struct file.\n\nIt would be better instead of the cifsFileInfo just held info pertaining\nto the open file on the server instead without any back refrences to the\nstruct file. This would allow it to exist after the filp to which it was\noriginally attached was closed.\n\nMuch of the use of the file pointer in this struct is to get at the\ndentry. Begin divorcing the cifsFileInfo from the struct file by\nkeeping a reference to the dentry. Since the dentry will have a\nreference to the inode, we can eliminate the \"pInode\" field too and\nconvert the igrab/iput to dget/dput.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nAcked-by: Dave Kleikamp \u003cshaggy@linux.vnet.ibm.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "1c456013e96f10915578dc61095d19b4906f64ac", "tree": "3343310e8c431227bdc5ff343585913ed2bfde0b", "parents": [ "9daa42e22030f0c5c357a5e1f8af658411beda6b" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Oct 12 11:32:42 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Oct 12 15:43:53 2010 +0000" }, "message": "cifs: on multiuser mount, set ownership to current_fsuid/current_fsgid (try #7)\n\ncommit 3aa1c8c2900065a51268430ab48a1b42fdfe5b45 made cifs_getattr set\nthe ownership of files to current_fsuid/current_fsgid when multiuser\nmounts were in use and when mnt_uid/mnt_gid were non-zero.\n\nIt should have instead based that decision on the\nCIFS_MOUNT_OVERR_UID/GID flags.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "9daa42e22030f0c5c357a5e1f8af658411beda6b", "tree": "35517e93c8dc79903ed08b86cba8c31b85fb2ca3", "parents": [ "0dd12c21951e085357046b68c3a108273062d2aa" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Sun Oct 10 13:21:05 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Oct 12 15:14:06 2010 +0000" }, "message": "CIFS ntlm authentication and signing - Build a proper av/ti pair blob for ntlmv2 without extended security authentication\n\nBuild an av pair blob as part of ntlmv2 (without extended security) auth\nrequest. Include netbios and dns names for domain and server and\na timestamp in the blob.\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@samba.org\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "0dd12c21951e085357046b68c3a108273062d2aa", "tree": "be512113fd0282dc82a9cdd7c7626dc2e7cb28d7", "parents": [ "6ea75952d7c671ea8b0d7b66f82afcafbb5d20c2" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Oct 08 12:20:12 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Oct 08 16:34:49 2010 +0000" }, "message": "cifs: initialize tlink_tree_lock and tlink_tree\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6ea75952d7c671ea8b0d7b66f82afcafbb5d20c2", "tree": "66cf5e73830d2730866f7e6bf6cf93f0684c5b99", "parents": [ "6b0cd00bc396daf5c2dcf17a8d82055335341f46", "d2445556137c38ae15d3191174bfd235630ed7cd" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Oct 08 03:42:03 2010 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Oct 08 03:42:03 2010 +0000" }, "message": "Merge branch \u0027for-next\u0027\n" }, { "commit": "d2445556137c38ae15d3191174bfd235630ed7cd", "tree": "b632efe8cf403ce6ca445d2dbce6d88a2232bace", "parents": [ "ccc46a7402200a3b28a8fa1605ea5405a9ef66f7" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Oct 08 03:38:46 2010 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Oct 08 03:38:46 2010 +0000" }, "message": "[CIFS] Remove build warning\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ccc46a7402200a3b28a8fa1605ea5405a9ef66f7", "tree": "5b80d775dcda91764711ab685bbd2c91c9eb955d", "parents": [ "2de970ff69bbcc5a4b7440df669a595b2b1acd73" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Oct 06 19:51:12 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Oct 08 03:33:08 2010 +0000" }, "message": "cifs: fix module refcount leak in find_domain_name\n\nfind_domain_name() uses load_nls_default which takes a module reference\non the appropriate NLS module, but doesn\u0027t put it.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nCc: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "2de970ff69bbcc5a4b7440df669a595b2b1acd73", "tree": "ec710893fcd1aa6313a3bbe2eafb3f463b364afc", "parents": [ "3aa1c8c2900065a51268430ab48a1b42fdfe5b45" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Oct 06 19:51:12 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Oct 08 03:31:21 2010 +0000" }, "message": "cifs: implement recurring workqueue job to prune old tcons\n\nCreate a workqueue job that cleans out unused tlinks. For now, it uses\na hardcoded expire time of 10 minutes. When it\u0027s done, the work rearms\nitself. On umount, the work is cancelled before tearing down the tlink\ntree.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "3aa1c8c2900065a51268430ab48a1b42fdfe5b45", "tree": "866119f148a97b306479a01b2e7180b2adb6eb61", "parents": [ "13cd4b7f7472eea7cbc1ab34e042842fbb902160" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Oct 07 14:46:28 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Oct 08 03:26:28 2010 +0000" }, "message": "cifs: on multiuser mount, set ownership to current_fsuid/current_fsgid (try #5)\n\n...when unix extensions aren\u0027t enabled. This makes everything on the\nmount appear to be owned by the current user.\n\nThis version of the patch differs from previous versions however in that\nthe admin can still force the ownership of all files to appear as a\nsingle user via the uid\u003d/gid\u003d options.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "13cd4b7f7472eea7cbc1ab34e042842fbb902160", "tree": "484c6d868b448d4386fbe16e01f1ca5e2d01a40c", "parents": [ "0eb8a132c449c755b7a3f18f33365b2040c47347" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Oct 07 18:46:32 2010 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Oct 07 18:46:32 2010 +0000" }, "message": "[CIFS] Various small checkpatch cleanups\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "0eb8a132c449c755b7a3f18f33365b2040c47347", "tree": "dcf3e0f4a69129827a8acb21b5884bd539cf6d27", "parents": [ "9d002df492b14c690425d9785530371b6c1ccbca" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Oct 06 19:51:12 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Oct 07 18:30:45 2010 +0000" }, "message": "cifs: add \"multiuser\" mount option\n\nThis allows someone to declare a mount as a multiuser mount.\n\nMultiuser mounts also imply \"noperm\" since we want to allow the server\nto handle permission checking. It also (for now) requires Kerberos\nauthentication. Eventually, we could expand this to other authtypes, but\nthat requires a scheme to allow per-user credential stashing in some\nform.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "9d002df492b14c690425d9785530371b6c1ccbca", "tree": "6ed1a52d0e348e985f7bd194d22ee6e7854fa9e8", "parents": [ "c9928f7040a6e5f39e028bea500e0fde910d4a96" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Oct 06 19:51:11 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Oct 07 18:18:00 2010 +0000" }, "message": "cifs: add routines to build sessions and tcons on the fly\n\nThis patch is rather large, but it\u0027s a bit difficult to do piecemeal...\n\nFor non-multiuser mounts, everything will basically work as it does\ntoday. A call to cifs_sb_tlink will return the \"master\" tcon link.\n\nTurn the tcon pointer in the cifs_sb into a radix tree that uses the\nfsuid of the process as a key. The value is a new \"tcon_link\" struct\nthat contains info about a tcon that\u0027s under construction.\n\nWhen a new process needs a tcon, it\u0027ll call cifs_sb_tcon. That will\nthen look up the tcon_link in the radix tree. If it exists and is\nvalid, it\u0027s returned.\n\nIf it doesn\u0027t exist, then we stuff a new tcon_link into the tree and\nmark it as pending and then go and try to build the session/tcon.\nIf that works, the tcon pointer in the tcon_link is updated and the\npending flag is cleared.\n\nIf the construction fails, then we set the tcon pointer to an ERR_PTR\nand clear the pending flag.\n\nIf the radix tree is searched and the tcon_link is marked pending\nthen we go to sleep and wait for the pending flag to be cleared.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "c9928f7040a6e5f39e028bea500e0fde910d4a96", "tree": "4a6b35fffc8b9b424e95073e8979e8ee406c9a4d", "parents": [ "29e07c82a9e8acebbb38ecc22b0b5005a0a5d839" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Mon Oct 04 19:56:13 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Oct 06 16:13:19 2010 +0000" }, "message": "ntlm authentication and signing - Correct response length for ntlmv2 authentication without extended security\n\nFix incorrect calculation of case sensitive response length in the\nntlmv2 (without extended security) response.\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "29e07c82a9e8acebbb38ecc22b0b5005a0a5d839", "tree": "6d97816e48753810f96cbb1af45ac5d13a358ca0", "parents": [ "6508d904e6fb66ce4c34617f72b38d6714c4b9f6" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Sep 29 19:51:12 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Oct 06 16:13:11 2010 +0000" }, "message": "cifs: fix cifs_show_options to show \"username\u003d\" or \"multiuser\"\n\n...based on CIFS_MOUNT_MULTIUSER flag.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6508d904e6fb66ce4c34617f72b38d6714c4b9f6", "tree": "a5d24f696c9a9045c3880671eeb5fbc6ca9f61ab", "parents": [ "13cfb7334eb6fd0fc06da5589aea1e947791f1d6" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Sep 29 19:51:11 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Oct 06 16:12:59 2010 +0000" }, "message": "cifs: have find_readable/writable_file filter by fsuid\n\nWhen we implement multiuser mounts, we\u0027ll need to filter filehandles\nby fsuid. Add a flag for multiuser mounts and code to filter by\nfsuid when it\u0027s set.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "13cfb7334eb6fd0fc06da5589aea1e947791f1d6", "tree": "56d884f6a5dbcf8b259247fdad55c9158bd4d865", "parents": [ "7ffec372458d163492e56e663a1b3a2d7be0a0a2" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Sep 29 19:51:11 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Oct 06 16:12:49 2010 +0000" }, "message": "cifs: have cifsFileInfo hold a reference to a tlink rather than tcon pointer\n\ncifsFileInfo needs a pointer to a tcon, but it doesn\u0027t currently hold a\nreference to it. Change it to keep a pointer to a tcon_link instead and\nhold a reference to it.\n\nThat will keep the tcon from being freed until the file is closed.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "7ffec372458d163492e56e663a1b3a2d7be0a0a2", "tree": "e404e3d1000ff41e9b27d0ecb4d6a47187e110d7", "parents": [ "f3983c2133e9bea9c8b4f690737d15e3e9b02491" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Sep 29 19:51:11 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Oct 06 16:12:44 2010 +0000" }, "message": "cifs: add refcounted and timestamped container for holding tcons\n\nEventually, we\u0027ll need to track the use of tcons on a per-sb basis, so that\nwe know when it\u0027s ok to tear them down. Begin this conversion by adding a\nnew \"tcon_link\" struct and accessors that get it. For now, the core data\nstructures are untouched -- cifs_sb still just points to a single tcon and\nthe pointers are just cast to deal with the accessor functions. A later\npatch will flesh this out.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "b89f432133851a01c0d28822f11cbdcc15781a75", "tree": "a3e9ba638a9b746985148f4525335d360ec7da56", "parents": [ "2e54eb96e2c801f33d95b5dade15212ac4d6c4a5" ], "author": { "name": "Arnd Bergmann", "email": "arnd@arndb.de", "time": "Sat Sep 18 15:09:31 2010 +0200" }, "committer": { "name": "Arnd Bergmann", "email": "arnd@arndb.de", "time": "Tue Oct 05 11:02:04 2010 +0200" }, "message": "fs/locks.c: prepare for BKL removal\n\nThis prepares the removal of the big kernel lock from the\nfile locking code. We still use the BKL as long as fs/lockd\nuses it and ceph might sleep, but we can flip the definition\nto a private spinlock as soon as that\u0027s done.\nAll users outside of fs/lockd get converted to use\nlock_flocks() instead of lock_kernel() where appropriate.\n\nBased on an earlier patch to use a spinlock from Matthew\nWilcox, who has attempted this a few times before, the\nearliest patch from over 10 years ago turned it into\na semaphore, which ended up being slower than the BKL\nand was subsequently reverted.\n\nSomeone should do some serious performance testing when\nthis becomes a spinlock, since this has caused problems\nbefore. Using a spinlock should be at least as good\nas the BKL in theory, but who knows...\n\nSigned-off-by: Arnd Bergmann \u003carnd@arndb.de\u003e\nAcked-by: Matthew Wilcox \u003cwilly@linux.intel.com\u003e\nCc: Christoph Hellwig \u003chch@lst.de\u003e\nCc: Trond Myklebust \u003ctrond.myklebust@fys.uio.no\u003e\nCc: \"J. Bruce Fields\" \u003cbfields@fieldses.org\u003e\nCc: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nCc: Miklos Szeredi \u003cmszeredi@suse.cz\u003e\nCc: Frederic Weisbecker \u003cfweisbec@gmail.com\u003e\nCc: Ingo Molnar \u003cmingo@redhat.com\u003e\nCc: John Kacur \u003cjkacur@redhat.com\u003e\nCc: Sage Weil \u003csage@newdream.net\u003e\nCc: linux-kernel@vger.kernel.org\nCc: linux-fsdevel@vger.kernel.org\n" }, { "commit": "b0991aa324b57dca8feef75ed75b24080ee4a9fc", "tree": "3d67c670002bc1efe0419ca2e927bc334c73be0c", "parents": [ "ba13d597a60a1a26614f18b76c1a2cad1a548e46" ], "author": { "name": "Jan Blunck", "email": "jblunck@infradead.org", "time": "Wed Feb 24 13:25:29 2010 +0100" }, "committer": { "name": "Arnd Bergmann", "email": "arnd@arndb.de", "time": "Mon Oct 04 21:10:36 2010 +0200" }, "message": "BKL: Remove BKL from CifsFS\n\nThe BKL is only used in put_super and fill_super that are both protected by\nthe superblocks s_umount rw_semaphore. Therefore it is safe to remove the\nBKL entirely.\n\nSigned-off-by: Jan Blunck \u003cjblunck@infradead.org\u003e\nCc: Steve French \u003csmfrench@gmail.com\u003e\nSigned-off-by: Arnd Bergmann \u003carnd@arndb.de\u003e\n" }, { "commit": "db71922217a214e5c9268448e537b54fc1f301ea", "tree": "9c9afbf29411547891f6968e5ade29ce59d66c07", "parents": [ "899611ee7d373e5eeda08e9a8632684e1ebbbf00" ], "author": { "name": "Jan Blunck", "email": "jblunck@infradead.org", "time": "Sun Aug 15 22:51:10 2010 +0200" }, "committer": { "name": "Arnd Bergmann", "email": "arnd@arndb.de", "time": "Mon Oct 04 21:10:10 2010 +0200" }, "message": "BKL: Explicitly add BKL around get_sb/fill_super\n\nThis patch is a preparation necessary to remove the BKL from do_new_mount().\nIt explicitly adds calls to lock_kernel()/unlock_kernel() around\nget_sb/fill_super operations for filesystems that still uses the BKL.\n\nI\u0027ve read through all the code formerly covered by the BKL inside\ndo_kern_mount() and have satisfied myself that it doesn\u0027t need the BKL\nany more.\n\ndo_kern_mount() is already called without the BKL when mounting the rootfs\nand in nfsctl. do_kern_mount() calls vfs_kern_mount(), which is called\nfrom various places without BKL: simple_pin_fs(), nfs_do_clone_mount()\nthrough nfs_follow_mountpoint(), afs_mntpt_do_automount() through\nafs_mntpt_follow_link(). Both later functions are actually the filesystems\nfollow_link inode operation. vfs_kern_mount() is calling the specified\nget_sb function and lets the filesystem do its job by calling the given\nfill_super function.\n\nTherefore I think it is safe to push down the BKL from the VFS to the\nlow-level filesystems get_sb/fill_super operation.\n\n[arnd: do not add the BKL to those file systems that already\n don\u0027t use it elsewhere]\n\nSigned-off-by: Jan Blunck \u003cjblunck@infradead.org\u003e\nSigned-off-by: Arnd Bergmann \u003carnd@arndb.de\u003e\nCc: Matthew Wilcox \u003cmatthew@wil.cx\u003e\nCc: Christoph Hellwig \u003chch@infradead.org\u003e\n" }, { "commit": "f569599ae70f0899035f8d5876a7939f629c5976", "tree": "4d7f7c32b9c89f15841bdbf372e75afe09bdef53", "parents": [ "522440ed55d2cc8855ea5f82bc067e0483b2e1be" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Sep 29 15:27:08 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Oct 01 17:50:08 2010 +0000" }, "message": "cifs: prevent infinite recursion in cifs_reconnect_tcon\n\ncifs_reconnect_tcon is called from smb_init. After a successful\nreconnect, cifs_reconnect_tcon will call reset_cifs_unix_caps. That\nfunction will, in turn call CIFSSMBQFSUnixInfo and CIFSSMBSetFSUnixInfo.\nThose functions also call smb_init.\n\nIt\u0027s possible for the session and tcon reconnect to succeed, and then\nfor another cifs_reconnect to occur before CIFSSMBQFSUnixInfo or\nCIFSSMBSetFSUnixInfo to be called. That\u0027ll cause those functions to call\nsmb_init and cifs_reconnect_tcon again, ad infinitum...\n\nBreak the infinite recursion by having those functions use a new\nsmb_init variant that doesn\u0027t attempt to perform a reconnect.\n\nReported-and-Tested-by: Michal Suchanek \u003chramrach@centrum.cz\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "522440ed55d2cc8855ea5f82bc067e0483b2e1be", "tree": "53d8bee5f3e0dd69affd167b4e29a7244d9fc2da", "parents": [ "899611ee7d373e5eeda08e9a8632684e1ebbbf00" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Sep 29 09:49:54 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Sep 29 19:23:23 2010 +0000" }, "message": "cifs: set backing_dev_info on new S_ISREG inodes\n\nTesting on very recent kernel (2.6.36-rc6) made this warning pop:\n\n WARNING: at fs/fs-writeback.c:87 inode_to_bdi+0x65/0x70()\n Hardware name:\n Dirtiable inode bdi default !\u003d sb bdi cifs\n\n...the following patch fixes it and seems to be the obviously correct\nthing to do for cifs.\n\nCc: stable@kernel.org\nAcked-by: Dave Kleikamp \u003cshaggy@linux.vnet.ibm.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "f3983c2133e9bea9c8b4f690737d15e3e9b02491", "tree": "89cc00b209368a41e5554ac1be103ef6e79d8fe1", "parents": [ "f7a40689fd1e963cb1006349e050c07584895db5" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Sep 22 16:17:40 2010 -0700" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Sep 29 19:04:33 2010 +0000" }, "message": "cifs: fix handling of signing with writepages (try #6)\n\nGet a reference to the file early so we can eventually base the decision\nabout signing on the correct tcon. If that doesn\u0027t work for some reason,\nthen fall back to generic_writepages. That\u0027s just as likely to fail, but\nit simplifies the error handling.\n\nIn truth, I\u0027m not sure how that could occur anyway, so maybe a NULL\nopen_file here ought to be a BUG()?\n\nAfter that, we drop the reference to the open_file and then we re-get\none prior to each WriteAndX call. This helps ensure that the filehandle\nisn\u0027t held open any longer than necessary and that open files are\nreclaimed prior to each write call.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" } ], "next": "f7a40689fd1e963cb1006349e050c07584895db5" }