)]}' { "log": [ { "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": "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": "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": "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": "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": "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": "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": "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": "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": "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": "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": "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": "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": "0d424ad0a4b8c08e45928bccfa5b4b240097b01b", "tree": "40f383b138d2e8121cda4b28f1f7c4b9642fe5ec", "parents": [ "f6acb9d0596889a774e142ed76cb05b90d9763d2" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Sep 20 16:01:35 2010 -0700" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Sep 29 19:04:33 2010 +0000" }, "message": "cifs: add cifs_sb_master_tcon and convert some callers to use it\n\nAt mount time, we\u0027ll always need to create a tcon that will serve as a\ntemplate for others that are associated with the mount. This tcon is\nknown as the \"master\" tcon.\n\nIn some cases, we\u0027ll need to use that tcon regardless of who\u0027s accessing\nthe mount. Add an accessor function for the master tcon and go ahead and\nswitch the appropriate places to use it.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "f6acb9d0596889a774e142ed76cb05b90d9763d2", "tree": "a6d74c288f9941bf011a68df4d1729048fd779af", "parents": [ "a6e8a8455c94565c53e1a1756d2ab9d9e3a902b8" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Sep 20 16:01:34 2010 -0700" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Sep 29 19:04:33 2010 +0000" }, "message": "cifs: temporarily rename cifs_sb-\u003etcon to ptcon to catch stragglers\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "a6e8a8455c94565c53e1a1756d2ab9d9e3a902b8", "tree": "544628293da6aa14a136c915d887d0abdc689a03", "parents": [ "ba00ba64cf0895e4c2ac507e56306363dc125a90" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Sep 20 16:01:33 2010 -0700" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Sep 29 19:04:32 2010 +0000" }, "message": "cifs: add function to get a tcon from cifs_sb\n\nWhen we convert cifs to do multiple sessions per mount, we\u0027ll need more\nthan one tcon per superblock. At that point \"cifs_sb-\u003etcon\" will make\nno sense. Add a new accessor function that gets a tcon given a cifs_sb.\nFor now, it just returns cifs_sb-\u003etcon. Later it\u0027ll do more.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "d3bf5221d3274b5015ad18a55060b074cca8d2f0", "tree": "5ac76fa5f694a83d800898b8bb0ab505f66a1597", "parents": [ "17edec6f563ba6b5630329fbe9473557bb475c3d" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Sep 22 19:15:36 2010 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Sep 29 19:04:32 2010 +0000" }, "message": "[CIFS] Fix ordering of cleanup on module init failure\n\nIf registering fs cache failed, we weren\u0027t cleaning up proc.\n\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nCC: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ab9db8b737210bec365593a04dd1c534220bb311", "tree": "09ea9395d7a54737c83f90e3558836d1ccbdd7c6", "parents": [ "5fe97cfddc426f3145e8673b68faab7e54462173" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Sep 21 08:14:46 2010 -0700" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Sep 29 19:04:31 2010 +0000" }, "message": "cifs: allow matching of tcp sessions in CifsNew state\n\nWith commit 7332f2a6217ee6925f83ef0e725013067ed316ba, cifsd will no\nlonger exit when the socket abends and the tcpStatus is CifsNew. With\nthat change, there\u0027s no reason to avoid matching an existing session in\nthis state.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "736a33205969c16f81d747db14ff4c0f133609a6", "tree": "39705109ad4aee778be5d2b3dfe2f3e7c9ad246c", "parents": [ "1b12b9c15b4371d83b729b8fc18c670e78a1479b" ], "author": { "name": "Stefan Metzmacher", "email": "metze@samba.org", "time": "Fri Jul 30 14:56:00 2010 +0200" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Sep 29 19:04:31 2010 +0000" }, "message": "cifs: add \"mfsymlinks\" mount option\n\nThis is the start for an implementation of \"Minshall+French Symlinks\"\n(see http://wiki.samba.org/index.php/UNIX_Extensions#Minshall.2BFrench_symlinks).\n\nSigned-off-by: Stefan Metzmacher \u003cmetze@samba.org\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "3eb9a8893a76cf1cda3b41c3212eb2cfe83eae0e", "tree": "fd4fb596113f27bcbe3bba1f2777c91f14521f48", "parents": [ "2b149f11978b44199954710d32c0eecf6c9efd9c" ], "author": { "name": "Ben Greear", "email": "greearb@candelatech.com", "time": "Wed Sep 01 17:06:02 2010 -0700" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Sep 29 19:04:29 2010 +0000" }, "message": "cifs: Allow binding to local IP address.\n\nWhen using multi-homed machines, it\u0027s nice to be able to specify\nthe local IP to use for outbound connections. This patch gives\ncifs the ability to bind to a particular IP address.\n\n Usage: mount -t cifs -o srcaddr\u003d192.168.1.50,user\u003dfoo, ...\n Usage: mount -t cifs -o srcaddr\u003d2002::100:1,user\u003dfoo, ...\n\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nAcked-by: Dr. David Holder \u003cdavid.holder@erion.co.uk\u003e\nSigned-off-by: Ben Greear \u003cgreearb@candelatech.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "2b149f11978b44199954710d32c0eecf6c9efd9c", "tree": "efd48d4b64fc07a244890d1fced9c767ae4f369b", "parents": [ "5f98ca9afb9c004f8948c0d40920503de447918a" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Sat Sep 18 22:02:18 2010 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Sep 29 19:04:29 2010 +0000" }, "message": "cifs NTLMv2/NTLMSSP ntlmv2 within ntlmssp autentication code\n\nAttribue Value (AV) pairs or Target Info (TI) pairs are part of\nntlmv2 authentication.\nStructure ntlmv2_resp had only definition for two av pairs.\nSo removed it, and now allocation of av pairs is dynamic.\nFor servers like Windows 7/2008, av pairs sent by server in\nchallege packet (type 2 in the ntlmssp exchange/negotiation) can\nvary.\n\nServer sends them during ntlmssp negotiation. So when ntlmssp is used\nas an authentication mechanism, type 2 challenge packet from server\nhas this information. Pluck it and use the entire blob for\nauthenticaiton purpose. If user has not specified, extract\n(netbios) domain name from the av pairs which is used to calculate\nntlmv2 hash. Servers like Windows 7 are particular about the AV pair\nblob.\n\nServers like Windows 2003, are not very strict about the contents\nof av pair blob used during ntlmv2 authentication.\nSo when security mechanism such as ntlmv2 is used (not ntlmv2 in ntlmssp),\nthere is no negotiation and so genereate a minimal blob that gets\nused in ntlmv2 authentication as well as gets sent.\n\nFields tilen and tilbob are session specific. AV pair values are defined.\n\nTo calculate ntlmv2 response we need ti/av pair blob.\n\nFor sec mech like ntlmssp, the blob is plucked from type 2 response from\nthe server. From this blob, netbios name of the domain is retrieved,\nif user has not already provided, to be included in the Target String\nas part of ntlmv2 hash calculations.\n\nFor sec mech like ntlmv2, create a minimal, two av pair blob.\n\nThe allocated blob is freed in case of error. In case there is no error,\nthis blob is used in calculating ntlmv2 response (in CalcNTLMv2_response)\nand is also copied on the response to the server, and then freed.\n\nThe type 3 ntlmssp response is prepared on a buffer,\n5 * sizeof of struct _AUTHENTICATE_MESSAGE, an empirical value large\nenough to hold _AUTHENTICATE_MESSAGE plus a blob with max possible\n10 values as part of ntlmv2 response and lmv2 keys and domain, user,\nworkstation names etc.\n\nAlso, kerberos gets selected as a default mechanism if server supports it,\nover the other security mechanisms.\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "460cf3411b858ad509d5255e0dfaf862a83c0299", "tree": "3ff261b74c1751b9b0edd904f8d2d7f09f7440f6", "parents": [ "bfa88ea7ee9e6b4fd673e45a8cc0a8e0b7ef4761" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Sep 14 11:38:24 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 14 23:21:03 2010 +0000" }, "message": "cifs: fix potential double put of TCP session reference\n\ncifs_get_smb_ses must be called on a server pointer on which it holds an\nactive reference. It first does a search for an existing SMB session. If\nit finds one, it\u0027ll put the server reference and then try to ensure that\nthe negprot is done, etc.\n\nIf it encounters an error at that point then it\u0027ll return an error.\nThere\u0027s a potential problem here though. When cifs_get_smb_ses returns\nan error, the caller will also put the TCP server reference leading to a\ndouble-put.\n\nFix this by having cifs_get_smb_ses only put the server reference if\nit found an existing session that it could use and isn\u0027t returning an\nerror.\n\nCc: stable@kernel.org\nReviewed-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": "32670396e7fc6e4f37451a69339968985461a374", "tree": "df01cb3461125f341d61bde1d527f575372803ef", "parents": [ "7332f2a6217ee6925f83ef0e725013067ed316ba" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Sep 03 12:00:50 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Sep 08 21:22:35 2010 +0000" }, "message": "cifs: prevent possible memory corruption in cifs_demultiplex_thread\n\ncifs_demultiplex_thread sets the addr.sockAddr.sin_port without any\nregard for the socket family. While it may be that the error in question\nhere never occurs on an IPv6 socket, it\u0027s probably best to be safe and\nset the port properly if it ever does.\n\nBreak the port setting code out of cifs_fill_sockaddr and into a new\nfunction, and call that from cifs_demultiplex_thread.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "7332f2a6217ee6925f83ef0e725013067ed316ba", "tree": "481c0774aa1620f9e3f20e2809307d9047dd7324", "parents": [ "522bbe65a2415fabce618186fc7777eb4c502989" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Sep 03 12:00:49 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Sep 08 21:22:33 2010 +0000" }, "message": "cifs: eliminate some more premature cifsd exits\n\nIf the tcpStatus is still CifsNew, the main cifs_demultiplex_loop can\nbreak out prematurely in some cases. This is wrong as we will almost\nalways have other structures with pointers to the TCP_Server_Info. If\nthe main loop breaks under any other condition other than tcpStatus \u003d\u003d\nCifsExiting, then it\u0027ll face a use-after-free situation.\n\nI don\u0027t see any reason to treat a CifsNew tcpStatus differently than\nCifsGood. I believe we\u0027ll still want to attempt to reconnect in either\ncase. What should happen in those situations is that the MIDs get marked\nas MID_RETRY_NEEDED. This will make CIFSSMBNegotiate return -EAGAIN, and\nthen the caller can retry the whole thing on a newly reconnected socket.\nIf that fails again in the same way, the caller of cifs_get_smb_ses\nshould tear down the TCP_Server_Info struct.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "522bbe65a2415fabce618186fc7777eb4c502989", "tree": "6eb66dac2892728d93e3220511c0ec7e35f40ce4", "parents": [ "4266d9118f85b050a341992f0cfab40d392ef32c" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Sep 03 12:00:49 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Sep 08 21:22:30 2010 +0000" }, "message": "cifs: prevent cifsd from exiting prematurely\n\nWhen cifs_demultiplex_thread exits, it does a number of cleanup tasks\nincluding freeing the TCP_Server_Info struct. Much of the existing code\nin cifs assumes that when there is a cisfSesInfo struct, that it holds a\nreference to a valid TCP_Server_Info struct.\n\nWe can never allow cifsd to exit when a cifsSesInfo struct is still\nholding a reference to the server. The server pointers will then point\nto freed memory.\n\nThis patch eliminates a couple of questionable conditions where it does\nthis. The idea here is to make an -EINTR return from kernel_recvmsg\nbehave the same way as -ERESTARTSYS or -EAGAIN. If the task was\nsignalled from cifs_put_tcp_session, then tcpStatus will be CifsExiting,\nand the kernel_recvmsg call will return quickly.\n\nThere\u0027s also another condition where this can occur too -- if the\ntcpStatus is still in CifsNew, then it will also exit if the server\ncloses the socket prematurely. I think we\u0027ll probably also need to fix\nthat situation, but that requires a bit more consideration.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "c8e56f1f4fb9f82f63e4ce6d73a14501d0432c76", "tree": "6d0988317a6eaf4b454c73a1dd95f89184f5b1ed", "parents": [ "745e507a9c79c6e1385d3414d5e56f3d4621a375" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Sep 08 21:10:58 2010 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Sep 08 21:10:58 2010 +0000" }, "message": "Revert \"[CIFS] Fix ntlmv2 auth with ntlmssp\"\n\nThis reverts commit 9fbc590860e75785bdaf8b83e48fabfe4d4f7d58.\n\nThe change to kernel crypto and fixes to ntlvm2 and ntlmssp\nseries, introduced a regression. Deferring this patch series\nto 2.6.37 after Shirish fixes it.\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nCC: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\n" }, { "commit": "24e6cf92fde1f140d8eb0bf7cd24c2c78149b6b2", "tree": "62ead9abea45977c5c9086284611bcb74f844aa2", "parents": [ "3ec6bbcdb4e85403f2c5958876ca9492afdf4031" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Aug 23 11:38:04 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Aug 23 17:38:31 2010 +0000" }, "message": "cifs: check for NULL session password\n\nIt\u0027s possible for a cifsSesInfo struct to have a NULL password, so we\nneed to check for that prior to running strncmp on it.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "9fbc590860e75785bdaf8b83e48fabfe4d4f7d58", "tree": "dccc154927cf1e12c702537b5bc028158b938e21", "parents": [ "bf4f12113812ac5be76c5590c6f50c8346f784a4" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Aug 20 20:42:26 2010 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Aug 20 20:42:26 2010 +0000" }, "message": "[CIFS] Fix ntlmv2 auth with ntlmssp\n\nMake ntlmv2 as an authentication mechanism within ntlmssp\ninstead of ntlmv1.\nParse type 2 response in ntlmssp negotiation to pluck\nAV pairs and use them to calculate ntlmv2 response token.\nAlso, assign domain name from the sever response in type 2\npacket of ntlmssp and use that (netbios) domain name in\ncalculation of response.\n\nEnable cifs/smb signing using rc4 and md5.\n\nChanged name of the structure mac_key to session_key to reflect\nthe type of key it holds.\n\nUse kernel crypto_shash_* APIs instead of the equivalent cifs functions.\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nAcked-by: Herbert Xu \u003cherbert@gondor.apana.org.au\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "fc87a40677bbe0937e2ff0642c7e83c9a4813f3d", "tree": "31912b8904ee316cee40d568db330c4e65e39325", "parents": [ "232341ba7fa15115d40f6aa0f8dd14e96e3ad375" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Aug 18 13:13:39 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Aug 18 17:26:25 2010 +0000" }, "message": "cifs: fix NULL pointer dereference in cifs_find_smb_ses\n\ncifs_find_smb_ses assumes that the vol-\u003epassword field is a valid\npointer, but that\u0027s only the case if a password was passed in via\nthe options string. It\u0027s possible that one won\u0027t be if there is\nno mount helper on the box.\n\nReported-by: diabel \u003cgacek-2004@wp.pl\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "67b7626a0512d12e34b38ff45e32c693cf9c79a1", "tree": "b62f4274a09d0e7ceee6325c077f487c2bceca3b", "parents": [ "f579903ef3e392251dc7e93cb521ddb622fbf8e0" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Thu Jul 22 18:33:01 2010 +0100" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Aug 05 17:17:50 2010 +0000" }, "message": "CIFS: Make cifs_convert_address() take a const src pointer and a length\n\nMake cifs_convert_address() take a const src pointer and a length so that all\nthe strlen() calls in their can be cut out and to make it unnecessary to modify\nthe src string.\n\nAlso return the data length from dns_resolve_server_name_to_ip() so that a\nstrlen() can be cut out of cifs_compose_mount_options() too.\n\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: David Howells \u003cdhowells@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "cb76d5e25008b76fb8e348c861d32659430ac3fa", "tree": "c3de6cdd116b1754c653b0fcb25d212b9dd69311", "parents": [ "f67909cf80051e8510194a51f88c4de323b92071" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Jul 26 14:25:08 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Aug 02 12:40:41 2010 +0000" }, "message": "cifs: fsc should not default to \"on\"\n\nI\u0027m not sure why this was merged with this flag hardcoded on, but it\nseems quite dangerous. Turn it off.\n\nAlso, mount.cifs hands unrecognized options off to the kernel so there\nshould be no need for changes there in order to support this.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "f636a34802e3913415410c6e595df2bf84851cff", "tree": "d5d54cff34c91603f2ba17075f22dc4b57267986", "parents": [ "3572d2857f61f720082740cc17e2d99b45e7af7f" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Jul 26 10:29:58 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Aug 02 12:40:40 2010 +0000" }, "message": "cifs: ignore the \"mand\", \"nomand\" and \"_netdev\" mount options\n\nThese are all handled by the userspace mount programs, but older versions\nof mount.cifs also handed them off to the kernel. Ignore them.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "9f841593ff65d2f801c7f80c4ed0955d30103f50", "tree": "01fcebaf639b556df3fc9b42149890ba8c12f9ce", "parents": [ "3e4b3e1f68c10510ec8d3076cffc5729b88f8de6" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jul 23 20:37:53 2010 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Aug 02 12:40:39 2010 +0000" }, "message": "[CIFS] relinquish fscache cookie before freeing CIFSTconInfo\n\nDoh, fix a use after free bug.\n\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nReviewed-and-Tested-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "3e4b3e1f68c10510ec8d3076cffc5729b88f8de6", "tree": "bee962570f8a54547cfb67550f76874d82981fea", "parents": [ "f55fdcca6bf1c17e86a270a8c0d81c6677c61222" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Jul 19 18:00:17 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Aug 02 12:40:39 2010 +0000" }, "message": "cifs: add separate cred_uid field to sesInfo\n\nRight now, there\u0027s no clear separation between the uid that owns the\ncredentials used to do the mount and the overriding owner of the files\non that mount.\n\nAdd a separate cred_uid field that is set to the real uid\nof the mount user. Unlike the linux_uid, the uid\u003d option does not\noverride this parameter. The parm is sent to cifs.upcall, which can then\npreferentially use the creduid\u003d parm instead of the uid\u003d parm for\nfinding credentials.\n\nThis is not the only way to solve this. We could try to do all of this\nin kernel instead by having a module parameter that affects what gets\npassed in the uid\u003d field of the upcall. That said, we have a lot more\nflexibility to change things in userspace so I think it probably makes\nsense to do it this way.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "fa1df75d4debde6d843e616df656f50a92958737", "tree": "51ed23f935dcd4536417613cf88c2dcfc369fd08", "parents": [ "56698236e1294848c63d4768673865ae5a9c69e0" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Mon Jul 05 18:13:36 2010 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Aug 02 12:40:37 2010 +0000" }, "message": "cifs: add mount option to enable local caching\n\nAdd a mount option \u0027fsc\u0027 to enable local caching on CIFS.\n\nI considered adding a separate debug bit for caching, but it appears that\ndebugging would be relatively easier with the normal CIFS_INFO level.\n\nAs the cifs-utils (userspace) changes are not done yet, this patch enables\n\u0027fsc\u0027 by default to enable testing.\n\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nAcked-by: David Howells \u003cdhowells@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "d03382ce9a89dbe27cba25130f0b90c0d631d5c5", "tree": "f2561981a606a788259ad10f0ec57f9486f85da4", "parents": [ "8913007e67106597fed4b9dd3787e8dca6915a83" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Mon Jul 05 18:12:27 2010 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Aug 02 12:40:36 2010 +0000" }, "message": "cifs: define superblock-level cache index objects and register them\n\nDefine superblock-level cache index objects (managed by cifsTconInfo structs).\nEach superblock object is created in a server-level index object and in itself\nan index into which inode-level objects are inserted.\n\nThe superblock object is keyed by sharename. The UniqueId/IndexNumber is used to\nvalidate that the exported share is the same since we accessed it last time.\n\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "4ff67b720c02c36e54d55b88c2931879b7db1cd2", "tree": "2a69a278edb4d80a9fb0760d188629fb6e8371fb", "parents": [ "daf5b0b6f3f6d7b15c2600426cc6c60a0e155218" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Jul 06 20:43:02 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Aug 02 12:40:35 2010 +0000" }, "message": "cifs: clean up cifs_find_smb_ses (try #2)\n\nThis patch replaces the earlier patch by the same name. The only\ndifference is that MAX_PASSWORD_SIZE has been increased to attempt to\nmatch the limits that windows enforces.\n\nDo a better job of matching sessions by authtype. Matching by username\nfor a Kerberos session is incorrect, and anonymous sessions need special\nhandling.\n\nAlso, in the case where we do match by username, we also need to match\nby password. That ensures that someone else doesn\u0027t \"borrow\" an existing\nsession without needing to know the password.\n\nFinally, passwords can be longer than 16 bytes. Bump MAX_PASSWORD_SIZE\nto 512 to match the size that the userspace mount helper allows.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "daf5b0b6f3f6d7b15c2600426cc6c60a0e155218", "tree": "9c6210671c489d493f02dd7ca1eb6799c21fd195", "parents": [ "4515148ef72bfda4ce3c8754149711d9972867ce" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Jul 06 20:43:02 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Aug 02 12:40:35 2010 +0000" }, "message": "cifs: match secType when searching for existing tcp session\n\nThe secType is a per-tcp session entity, but the current routine doesn\u0027t\nverify that it is acceptible when attempting to match an existing TCP\nsession.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "4515148ef72bfda4ce3c8754149711d9972867ce", "tree": "773960af3c94ee0bcda3d852aac7e510445c8e37", "parents": [ "50d971602a6c4bf1abe1f3873686f431d7539dfe" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Jul 06 20:43:02 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Aug 02 12:40:34 2010 +0000" }, "message": "cifs: move address comparison into separate function\n\nMove the address comparator out of cifs_find_tcp_session and into a\nseparate function for cleanliness. Also change the argument to\nthat function to a \"struct sockaddr\" pointer. Passing pointers to\nsockaddr_storage is a little odd since that struct is generally for\ndeclaring static storage.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "50d971602a6c4bf1abe1f3873686f431d7539dfe", "tree": "6c7b58ab4f26cfd4f98a271b4369fd43f9caae7a", "parents": [ "488f1d2d6cc9d665c9f09e4b54f77052732e3058" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Jul 06 20:43:01 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Aug 02 12:40:34 2010 +0000" }, "message": "cifs: set the port in sockaddr in a more clearly defined fashion\n\nThis patch should replace the patch I sent a couple of weeks ago to\nset the port in cifs_convert_address.\n\nCurrently we set this in cifs_find_tcp_session, but that\u0027s more of a\nside effect than anything. Add a new function called cifs_fill_sockaddr.\nHave it call cifs_convert_address and then set the port.\n\nThis also allows us to skip passing in the port as a separate parm to\ncifs_find_tcp_session.\n\nAlso, change cifs_convert_address take a struct sockaddr * rather than\nvoid * to make it clearer how this function should be called.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "488f1d2d6cc9d665c9f09e4b54f77052732e3058", "tree": "2e240da84cdb009b89508e0fa748bebe71e35766", "parents": [ "f579cf3cfd1e19ae5aab6929679d0c04bf1a6284" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Mon Jul 05 18:12:15 2010 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Aug 02 12:40:34 2010 +0000" }, "message": "cifs: define server-level cache index objects and register them\n\nDefine server-level cache index objects (as managed by TCP_ServerInfo structs)\nand register then with FS-Cache. Each server object is created in the CIFS\ntop-level index object and is itself an index into which superblock-level\nobjects are inserted.\n\nThe server objects are now keyed by {IPaddress,family,port} tuple.\n\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "198b5682781b97251afd9025dbf559a77969abdd", "tree": "bfe381f099e6986a833d93184b62bf9bcedfce69", "parents": [ "ebe6aa5ac456a13213ed563863e70dd441618a97" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sat Apr 24 07:57:48 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed May 05 23:18:27 2010 +0000" }, "message": "cifs: break negotiate protocol calls out of cifs_setup_session\n\nSo that we can reasonably set up the secType based on both the\nNegotiateProtocol response and the parsed mount options.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ebe6aa5ac456a13213ed563863e70dd441618a97", "tree": "7703cf9325497f97731fb100a6521821f7fc08f9", "parents": [ "d54ff73259a852d4b3886dc586587fdef5e9c8de" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sat Apr 24 07:57:47 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Apr 28 00:36:17 2010 +0000" }, "message": "cifs: eliminate \"first_time\" parm to CIFS_SessSetup\n\nWe can use the is_first_ses_reconnect() function to determine this.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ad6cca6d5d0f713e1987e20ed982cfa9eb16b27e", "tree": "996437a3ded92acbd27decc9695b6c895c99a7c0", "parents": [ "04912d6a20185473db025061b9b2c81fbdffc48b" ], "author": { "name": "Dan Carpenter", "email": "error27@gmail.com", "time": "Mon Apr 26 12:10:06 2010 +0200" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Apr 26 19:08:01 2010 +0000" }, "message": "cifs: change \u0026\u0026 to ||\n\nThis is a typo, if pvolume_info were NULL it would oops.\n\nThis function is used in clean up and error handling. The current code\nnever passes a NULL pvolume_info, but it could pass a NULL *pvolume_info\nif the kmalloc() failed.\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": "04912d6a20185473db025061b9b2c81fbdffc48b", "tree": "bcebae3f427f4ef3af20f407803bf0a3d31e6733", "parents": [ "d00c28de55a69d13cf2c6a99efc3e81a5d95af74" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sat Apr 24 07:57:45 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Apr 26 18:55:33 2010 +0000" }, "message": "cifs: rename \"extended_security\" to \"global_secflags\"\n\n...since that more accurately describes what that variable holds.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "d00c28de55a69d13cf2c6a99efc3e81a5d95af74", "tree": "a74daa468a4e9a4c2b6d2d8491beffbab489bbc5", "parents": [ "36988c76f007738cad5fe1c873a5fb0cda7eb2f6" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sat Apr 24 07:57:44 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Apr 26 18:55:31 2010 +0000" }, "message": "cifs: move tcon find/create into separate function\n\n...and out of cifs_mount.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "36988c76f007738cad5fe1c873a5fb0cda7eb2f6", "tree": "429a4dec9098546956e658b45df6f1b0a5d348a4", "parents": [ "a5fc4ce01867842f6a9cc317035df3081302bffc" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sat Apr 24 07:57:43 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Apr 26 18:55:28 2010 +0000" }, "message": "cifs: move SMB session creation code into separate function\n\n...it\u0027s mostly part of cifs_mount. Break it out into a separate\nfunction.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "a5fc4ce01867842f6a9cc317035df3081302bffc", "tree": "03e785a862a657a43b822f88704bb69d65e33e52", "parents": [ "fa588e0c57048b3d4bfcd772d80dc0615f83fd35" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sat Apr 24 07:57:42 2010 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Apr 26 18:54:54 2010 +0000" }, "message": "cifs: track local_nls in volume info\n\nAdd a local_nls field to the smb_vol struct and keep a pointer to the\nlocal_nls in it.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "b6b38f704a8193daba520493ebdaf7e819962fc8", "tree": "c38f4557b51b33573e36817888afa2accf3da9ca", "parents": [ "315e995c63a15cb4d4efdbfd70fe2db191917f7a" ], "author": { "name": "Joe Perches", "email": "joe@perches.com", "time": "Wed Apr 21 03:50:45 2010 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Apr 21 03:50:45 2010 +0000" }, "message": "[CIFS] Neaten cERROR and cFYI macros, reduce text space\n\nNeaten cERROR and cFYI macros, reduce text space\n~2.5K\n\nConvert \u0027__FILE__ \": \" fmt\u0027 to \u0027\"%s: \" fmt\u0027, __FILE__\u0027 to save text space\nSurround macros with do {} while\nAdd parentheses to macros\nMake statement expression macro from macro with assign\nRemove now unnecessary parentheses from cFYI and cERROR uses\n\ndefconfig with CIFS support old\n$ size fs/cifs/built-in.o\n text\t data\t bss\t dec\t hex\tfilename\n 156012\t 1760\t 148\t 157920\t 268e0\tfs/cifs/built-in.o\n\ndefconfig with CIFS support old\n$ size fs/cifs/built-in.o\n text\t data\t bss\t dec\t hex\tfilename\n 153508\t 1760\t 148\t 155416\t 25f18\tfs/cifs/built-in.o\n\nallyesconfig old:\n$ size fs/cifs/built-in.o\n text\t data\t bss\t dec\t hex\tfilename\n 309138\t 3864\t 74824\t 387826\t 5eaf2\tfs/cifs/built-in.o\n\nallyesconfig new\n$ size fs/cifs/built-in.o\n text\t data\t bss\t dec\t hex\tfilename\n 305655\t 3864\t 74824\t 384343\t 5dd57\tfs/cifs/built-in.o\n\nSigned-off-by: Joe Perches \u003cjoe@perches.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "5a0e3ad6af8660be21ca98a971cd00f331318c05", "tree": "5bfb7be11a03176a87296a43ac6647975c00a1d1", "parents": [ "ed391f4ebf8f701d3566423ce8f17e614cde9806" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Mar 24 17:04:11 2010 +0900" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Tue Mar 30 22:02:32 2010 +0900" }, "message": "include cleanup: Update gfp.h and slab.h includes to prepare for breaking implicit slab.h inclusion from percpu.h\n\npercpu.h is included by sched.h and module.h and thus ends up being\nincluded when building most .c files. percpu.h includes slab.h which\nin turn includes gfp.h making everything defined by the two files\nuniversally available and complicating inclusion dependencies.\n\npercpu.h -\u003e slab.h dependency is about to be removed. Prepare for\nthis change by updating users of gfp and slab facilities include those\nheaders directly instead of assuming availability. As this conversion\nneeds to touch large number of source files, the following script is\nused as the basis of conversion.\n\n http://userweb.kernel.org/~tj/misc/slabh-sweep.py\n\nThe script does the followings.\n\n* Scan files for gfp and slab usages and update includes such that\n only the necessary includes are there. ie. if only gfp is used,\n gfp.h, if slab is used, slab.h.\n\n* When the script inserts a new include, it looks at the include\n blocks and try to put the new include such that its order conforms\n to its surrounding. It\u0027s put in the include block which contains\n core kernel includes, in the same order that the rest are ordered -\n alphabetical, Christmas tree, rev-Xmas-tree or at the end if there\n doesn\u0027t seem to be any matching order.\n\n* If the script can\u0027t find a place to put a new include (mostly\n because the file doesn\u0027t have fitting include block), it prints out\n an error message indicating which .h file needs to be added to the\n file.\n\nThe conversion was done in the following steps.\n\n1. The initial automatic conversion of all .c files updated slightly\n over 4000 files, deleting around 700 includes and adding ~480 gfp.h\n and ~3000 slab.h inclusions. The script emitted errors for ~400\n files.\n\n2. Each error was manually checked. Some didn\u0027t need the inclusion,\n some needed manual addition while adding it to implementation .h or\n embedding .c file was more appropriate for others. This step added\n inclusions to around 150 files.\n\n3. The script was run again and the output was compared to the edits\n from #2 to make sure no file was left behind.\n\n4. Several build tests were done and a couple of problems were fixed.\n e.g. lib/decompress_*.c used malloc/free() wrappers around slab\n APIs requiring slab.h to be added manually.\n\n5. The script was run on all .h files but without automatically\n editing them as sprinkling gfp.h and slab.h inclusions around .h\n files could easily lead to inclusion dependency hell. Most gfp.h\n inclusion directives were ignored as stuff from gfp.h was usually\n wildly available and often used in preprocessor macros. Each\n slab.h inclusion directive was examined and added manually as\n necessary.\n\n6. percpu.h was updated not to include slab.h.\n\n7. Build test were done on the following configurations and failures\n were fixed. CONFIG_GCOV_KERNEL was turned off for all tests (as my\n distributed build env didn\u0027t work with gcov compiles) and a few\n more options had to be turned off depending on archs to make things\n build (like ipr on powerpc/64 which failed due to missing writeq).\n\n * x86 and x86_64 UP and SMP allmodconfig and a custom test config.\n * powerpc and powerpc64 SMP allmodconfig\n * sparc and sparc64 SMP allmodconfig\n * ia64 SMP allmodconfig\n * s390 SMP allmodconfig\n * alpha SMP allmodconfig\n * um on x86_64 SMP allmodconfig\n\n8. percpu.h modifications were reverted so that it could be applied as\n a separate patch and serve as bisection point.\n\nGiven the fact that I had only a couple of failures from tests on step\n6, I\u0027m fairly confident about the coverage of this conversion patch.\nIf there is a breakage, it\u0027s likely to be something in one of the arch\nheaders which should be easily discoverable easily on most builds of\nthe specific arch.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nGuess-its-ok-by: Christoph Lameter \u003ccl@linux-foundation.org\u003e\nCc: Ingo Molnar \u003cmingo@redhat.com\u003e\nCc: Lee Schermerhorn \u003cLee.Schermerhorn@hp.com\u003e\n" }, { "commit": "d7b619cf56218704ffce9d510aa497f0a0bcda0b", "tree": "ab382be7cce3ecd5152a339d1a839343aa7939f0", "parents": [ "122ca0076e5f84fa12943f22f6586ff285670783" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Feb 25 05:36:46 2010 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Feb 25 05:36:46 2010 +0000" }, "message": "[CIFS] pSesInfo-\u003esesSem is used as mutex. Rename it to session_mutex and\nconvert it to a real mutex.\n\nSigned-off-by: Thomas Gleixner \u003ctglx@linutronix.de\u003e\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6a5fa2362b628ee950080bef8895a6fb62f58ab4", "tree": "6c756adf9c571529a7ace4e9da807b85e7b1c985", "parents": [ "df9d1e8a4379119cbc47c474f168b5103e404e17" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jan 01 01:28:43 2010 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jan 01 01:28:43 2010 +0000" }, "message": "[CIFS] Add support for TCP_NODELAY\n\nmount option sockopt\u003dTCP_NODELAY helpful for faster networks\nboosting performance. Kernel bugzilla bug number 14032.\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "a2934c7b363ddcc001964f2444649f909e583bef", "tree": "c58eb5fd32591e1e54a37ed7b42f3fc4bb910d0e", "parents": [ "29e553631b2a0d4eebd23db630572e1027a9967a" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Dec 03 08:09:41 2009 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Dec 03 16:12:41 2009 +0000" }, "message": "cifs: NULL out tcon, pSesInfo, and srvTcp pointers when chasing DFS referrals\n\nThe scenario is this:\n\nThe kernel gets EREMOTE and starts chasing a DFS referral at mount time.\nThe tcon reference is put, which puts the session reference too, but\nneither pointer is zeroed out.\n\nThe mount gets retried (goto try_mount_again) with new mount info.\nSession setup fails fails and rc ends up being non-zero. The code then\nfalls through to the end and tries to put the previously freed tcon\npointer again. Oops at: cifs_put_smb_ses+0x14/0xd0\n\nFix this by moving the initialization of the rc variable and the tcon,\npSesInfo and srvTcp pointers below the try_mount_again label. Also, add\na FreeXid() before the goto to prevent xid \"leaks\".\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReported-by: Gustavo Carvalho Homem \u003cgustavo@angulosolido.pt\u003e\nCC: stable \u003cstable@kernel.org\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "f475f6775465283494346663f201ad04810d2e8a", "tree": "b5c054afa5509ee6957fb6a93d09999b9db975ab", "parents": [ "ec06aedd44541129840ed52e6165afa3796a27bf" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Nov 06 14:18:49 2009 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Nov 06 22:06:14 2009 +0000" }, "message": "cifs: don\u0027t use CIFSGetSrvInodeNumber in is_path_accessible\n\nBecause it\u0027s lighter weight, CIFS tries to use CIFSGetSrvInodeNumber to\nverify the accessibility of the root inode and then falls back to doing a\nfull QPathInfo if that fails with -EOPNOTSUPP. I have at least a report\nof a server that returns NT_STATUS_INTERNAL_ERROR rather than something\nthat translates to EOPNOTSUPP.\n\nRather than trying to be clever with that call, just have\nis_path_accessible do a normal QPathInfo. That call is widely\nsupported and it shouldn\u0027t increase the overhead significantly.\n\nCc: Stable \u003cstable@kernel.org\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "8347a5cdd1422eea0470ed586274c7f29e274b47", "tree": "cc83dbc0307c9d2393078e27d01e9f451eccd578", "parents": [ "0d9df2515dbceb67d343c0f10fd3ff218380d524" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Oct 06 18:31:29 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Oct 06 18:31:29 2009 +0000" }, "message": "[CIFS] Fixing to avoid invalid kfree() in cifs_get_tcp_session()\n\ntrivial bug in fs/cifs/connect.c .\nThe bug is caused by fail of extract_hostname()\nwhen mounting cifs file system.\n\nThis is the situation when I noticed this bug.\n\n% sudo mount -t cifs //192.168.10.208 mountpoint -o options...\n\nThen my kernel says,\n\n[ 1461.807776] ------------[ cut here ]------------\n[ 1461.807781] kernel BUG at mm/slab.c:521!\n[ 1461.807784] invalid opcode: 0000 [#2] PREEMPT SMP\n[ 1461.807790] last sysfs file:\n/sys/devices/pci0000:00/0000:00:1e.0/0000:09:02.0/resource\n[ 1461.807793] CPU 0\n[ 1461.807796] Modules linked in: nls_iso8859_1 usbhid sbp2 uhci_hcd\nehci_hcd i2c_i801 ohci1394 ieee1394 psmouse serio_raw pcspkr sky2 usbcore\nevdev\n[ 1461.807816] Pid: 3446, comm: mount Tainted: G D 2.6.32-rc2-vanilla\n[ 1461.807820] RIP: 0010:[\u003cffffffff810b888e\u003e] [\u003cffffffff810b888e\u003e]\nkfree+0x63/0x156\n[ 1461.807829] RSP: 0018:ffff8800b4f7fbb8 EFLAGS: 00010046\n[ 1461.807832] RAX: ffffea00033fff98 RBX: ffff8800afbae7e2 RCX:\n0000000000000000\n[ 1461.807836] RDX: ffffea0000000000 RSI: 000000000000005c RDI:\nffffffffffffffea\n[ 1461.807839] RBP: ffff8800b4f7fbf8 R08: 0000000000000001 R09:\n0000000000000000\n[ 1461.807842] R10: 0000000000000000 R11: ffff8800b4f7fbf8 R12:\n00000000ffffffea\n[ 1461.807845] R13: ffff8800afb23000 R14: ffff8800b4f87bc0 R15:\nffffffffffffffea\n[ 1461.807849] FS: 00007f52b6f187c0(0000) GS:ffff880007600000(0000)\nknlGS:0000000000000000\n[ 1461.807852] CS: 0010 DS: 0000 ES: 0000 CR0: 000000008005003b\n[ 1461.807855] CR2: 0000000000613000 CR3: 00000000af8f9000 CR4:\n00000000000006f0\n[ 1461.807858] DR0: 0000000000000000 DR1: 0000000000000000 DR2:\n0000000000000000\n[ 1461.807861] DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7:\n0000000000000400\n[ 1461.807865] Process mount (pid: 3446, threadinfo ffff8800b4f7e000, task\nffff8800950e4380)\n[ 1461.807867] Stack:\n[ 1461.807869] 0000000000000202 0000000000000282 ffff8800b4f7fbf8\nffff8800afbae7e2\n[ 1461.807876] \u003c0\u003e 00000000ffffffea ffff8800afb23000 ffff8800b4f87bc0\nffff8800b4f7fc28\n[ 1461.807884] \u003c0\u003e ffff8800b4f7fcd8 ffffffff81159f6d ffffffff81147bc2\nffffffff816bfb48\n[ 1461.807892] Call Trace:\n[ 1461.807899] [\u003cffffffff81159f6d\u003e] cifs_get_tcp_session+0x440/0x44b\n[ 1461.807904] [\u003cffffffff81147bc2\u003e] ? find_nls+0x1c/0xe9\n[ 1461.807909] [\u003cffffffff8115b889\u003e] cifs_mount+0x16bc/0x2167\n[ 1461.807917] [\u003cffffffff814455bd\u003e] ? _spin_unlock+0x30/0x4b\n[ 1461.807923] [\u003cffffffff81150da9\u003e] cifs_get_sb+0xa5/0x1a8\n[ 1461.807928] [\u003cffffffff810c1b94\u003e] vfs_kern_mount+0x56/0xc9\n[ 1461.807933] [\u003cffffffff810c1c64\u003e] do_kern_mount+0x47/0xe7\n[ 1461.807938] [\u003cffffffff810d8632\u003e] do_mount+0x712/0x775\n[ 1461.807943] [\u003cffffffff810d671f\u003e] ? copy_mount_options+0xcf/0x132\n[ 1461.807948] [\u003cffffffff810d8714\u003e] sys_mount+0x7f/0xbf\n[ 1461.807953] [\u003cffffffff8144509a\u003e] ? lockdep_sys_exit_thunk+0x35/0x67\n[ 1461.807960] [\u003cffffffff81011cc2\u003e] system_call_fastpath+0x16/0x1b\n[ 1461.807963] Code: 00 00 00 00 ea ff ff 48 c1 e8 0c 48 6b c0 68 48 01 d0\n66 83 38 00 79 04 48 8b 40 10 66 83 38 00 79 04 48 8b 40 10 80 38 00 78 04\n\u003c0f\u003e 0b eb fe 4c 8b 70 58 4c 89 ff 41 8b 76 4c e8 b8 49 fb ff e8\n[ 1461.808022] RIP [\u003cffffffff810b888e\u003e] kfree+0x63/0x156\n[ 1461.808027] RSP \u003cffff8800b4f7fbb8\u003e\n[ 1461.808031] ---[ end trace ffe26fcdc72c0ce4 ]---\n\nThe reason of this bug is that the error handling code of\ncifs_get_tcp_session()\ncalls kfree() when corresponding kmalloc() failed.\n(The kmalloc() is called by extract_hostname().)\n\nSigned-off-by: Hitoshi Mitake \u003cmitake@dcl.info.waseda.ac.jp\u003e\nCC: Stable \u003cstable@kernel.org\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "3bc303c254335dbd7c7012cc1760b12f1d5514d3", "tree": "7da17fbfd697216d9ed0ccd64ea9c03aaf3d52c1", "parents": [ "48541bd3dd4739b4d574b44ea47660c88d833677" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Sep 21 06:47:50 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Sep 24 18:33:18 2009 +0000" }, "message": "cifs: convert oplock breaks to use slow_work facility (try #4)\n\nThis is the fourth respin of the patch to convert oplock breaks to\nuse the slow_work facility.\n\nA customer of ours was testing a backport of one of the earlier\npatchsets, and hit a \"Busy inodes after umount...\" problem. An oplock\nbreak job had raced with a umount, and the superblock got torn down and\nits memory reused. When the oplock break job tried to dereference the\ninode-\u003ei_sb, the kernel oopsed.\n\nThis patchset has the oplock break job hold an inode and vfsmount\nreference until the oplock break completes. With this, there should be\nno need to take a tcon reference (the vfsmount implicitly holds one\nalready).\n\nCurrently, when an oplock break comes in there\u0027s a chance that the\noplock break job won\u0027t occur if the allocation of the oplock_q_entry\nfails. There are also some rather nasty races in the allocation and\nhandling these structs.\n\nRather than allocating oplock queue entries when an oplock break comes\nin, add a few extra fields to the cifsFileInfo struct. Get rid of the\ndedicated cifs_oplock_thread as well and queue the oplock break job to\nthe slow_work thread pool.\n\nThis approach also has the advantage that the oplock break jobs can\npotentially run in parallel rather than be serialized like they are\ntoday.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ca43e3beee38623a3f80691b28623d6ecf214742", "tree": "779d99d0f4a29d3fc6541f48423da9852eb6bbbe", "parents": [ "bdb97adcdf0993adbd2eef44b4533620d43792de" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 01 17:20:50 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 01 17:20:50 2009 +0000" }, "message": "[CIFS] Fix checkpatch warnings\n\nAlso update version number to 1.61\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "bdb97adcdf0993adbd2eef44b4533620d43792de", "tree": "5d2a120233c93d6a372d26d5ce3fc6cb6525fa6e", "parents": [ "1b3859bc9e20d764316346665fc93ecea2d2b176" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Thu Aug 20 13:03:34 2009 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 01 17:08:48 2009 +0000" }, "message": "PATCH] cifs: fix broken mounts when a SSH tunnel is used (try #4)\n\nOne more try..\n\nIt seems there is a regression that got introduced while Jeff fixed\nall the mount/umount races. While attempting to find whether a tcp\nsession is already existing, we were not checking whether the \"port\"\nused are the same. When a second mount is attempted with a different\n\"port\u003d\" option, it is being ignored. Because of this the cifs mounts\nthat uses a SSH tunnel appears to be broken.\n\nSteps to reproduce:\n\n1. create 2 shares\n# SSH Tunnel a SMB session\n2. ssh -f -L 6111:127.0.0.1:445 root@localhost \"sleep 86400\"\n3. ssh -f -L 6222:127.0.0.1:445 root@localhost \"sleep 86400\"\n4. tcpdump -i lo 6111 \u0026\n5. mkdir -p /mnt/mnt1\n6. mkdir -p /mnt/mnt2\n7. mount.cifs //localhost/a /mnt/mnt1 -o username\u003dguest,ip\u003d127.0.0.1,port\u003d6111\n#(shows tcpdump activity on port 6111)\n8. mount.cifs //localhost/b /mnt/mnt2 -o username\u003dguest,ip\u003d127.0.0.1,port\u003d6222\n#(shows tcpdump activity only on port 6111 and not on 6222\n\nFix by adding a check to compare the port _only_ if the user tries to\noverride the tcp port with \"port\u003d\" option, before deciding that an\nexisting tcp session is found. Also, clean up a bit by replacing\nif-else if by a switch statment while at it as suggested by Jeff.\n\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "9b9d6b2434fe942895c341b9a982f158529788ec", "tree": "402b7f233fae53f45d5ad3a445360200b6fa2cc7", "parents": [ "5bd9052d79daa4c8beb45436c408b6de672adb82" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Jul 31 06:56:09 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sun Aug 02 03:47:25 2009 +0000" }, "message": "cifs: reinstate original behavior when uid\u003d/gid\u003d options are specified\n\nThis patch fixes the regression reported here:\n\nhttp://bugzilla.kernel.org/show_bug.cgi?id\u003d13861\n\ncommit 4ae1507f6d266d0cc3dd36e474d83aad70fec9e4 changed the default\nbehavior when the uid\u003d or gid\u003d option was specified for a mount. The\nexisting behavior was to always clobber the ownership information\nprovided by the server when these options were specified. The above\ncommit changed this behavior so that these options simply provided\ndefaults when the server did not provide this information (unless\n\"forceuid\" or \"forcegid\" were specified)\n\nThis patch reverts this change so that the default behavior is restored.\nIt also adds \"noforceuid\" and \"noforcegid\" options to make it so that\nownership information from the server is preserved, even when the mount\nhas uid\u003d or gid\u003d options specified.\n\nIt also adds a couple of printk notices that pop up when forceuid or\nforcegid options are specified without a uid\u003d or gid\u003d option.\n\nReported-by: Tom Chiverton \u003cbugzilla.kernel.org@falkensweb.com\u003e\nReviewed-by: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "7b91e2661addd8e2419cb45f6a322aa5dab9bcee", "tree": "cf17b0ca2bffa52c7e450cd75d5277c35cc9eda4", "parents": [ "fc013a58859b7cf85e53a05804a74952fe0a4117" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jul 23 15:22:30 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Jul 28 00:51:59 2009 +0000" }, "message": "cifs: fix error handling in mount-time DFS referral chasing code\n\nIf the referral is malformed or the hostname can\u0027t be resolved, then\nthe current code generates an oops. Fix it to handle these errors\ngracefully.\n\nReported-by: Sandro Mathys \u003csm@sandro-mathys.ch\u003e\nAcked-by: Igor Mammedov \u003cniallain@gmail.com\u003e\nCC: Stable \u003cstable@kernel.org\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "03aa3a49ad3592a9e4e1ab19c6da3e852288caf1", "tree": "eaaa507b6debb862f551d9d3c750b4889e3131ed", "parents": [ "ce6e7fcd43aab1f77e56aa36936dd7d2d05a1ffa" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Jul 21 19:42:03 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Jul 22 21:08:00 2009 +0000" }, "message": "cifs: fix sb-\u003es_maxbytes so that it casts properly to a signed value\n\nThis off-by-one bug causes sendfile() to not work properly. When a task\ncalls sendfile() on a file on a CIFS filesystem, the syscall returns -1\nand sets errno to EOVERFLOW.\n\ndo_sendfile uses s_maxbytes to verify the returned offset of the file.\nThe problem there is that this value is cast to a signed value (loff_t).\nWhen this is done on the s_maxbytes value that cifs uses, it becomes\nnegative and the comparisons against it fail.\n\nEven though s_maxbytes is an unsigned value, it seems that it\u0027s not OK\nto set it in such a way that it\u0027ll end up negative when it\u0027s cast to a\nsigned value. These casts happen in other codepaths besides sendfile\ntoo, but the VFS is a little hard to follow in this area and I can\u0027t\nbe sure if there are other bugs that this will fix.\n\nIt\u0027s not clear to me why s_maxbytes isn\u0027t just declared as loff_t in the\nfirst place, but either way we still need to fix these values to make\nsendfile work properly. This is also an opportunity to replace the magic\nbit-shift values here with the standard #defines for this.\n\nThis fixes the reproducer program I have that does a sendfile and\nwill probably also fix the situation where apache is serving from a\nCIFS share.\n\nAcked-by: Johannes Weiner \u003channes@cmpxchg.org\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "90a98b2f3f3647fb17667768a348b2b219f2a9f7", "tree": "784f9cf1728176d7f604fede2ce845cfd6bcf31c", "parents": [ "287638b2c533165c3c03dfa15196c2ba583cd287" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Jul 20 13:40:52 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Jul 20 18:24:37 2009 +0000" }, "message": "cifs: free nativeFileSystem field before allocating a new one\n\n...otherwise, we\u0027ll leak this memory if we have to reconnect (e.g. after\nnetwork failure).\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nCC: Stable \u003cstable@kernel.org\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "71a394faaad07090af5de5c075ec2f5bca0fbb35", "tree": "9e4f67deea3ecfc6bb571a930c16c6d6e5b10f3f", "parents": [ "ad8034f19792736db5c259103c2eaaf72887bbb4" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jun 26 04:07:18 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jun 26 04:07:18 2009 +0000" }, "message": "[CIFS] remove unknown mount option warning message\n\nJeff\u0027s previous patch which removed the unneeded rw/ro\nparsing can cause a minor warning in dmesg (about the\nunknown rw or ro mount option) at mount time. This\npatch makes cifs ignore them in kernel to remove the warning\n(they are already handled in the mount helper and VFS).\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6debdbc0ba6253ac519cd5a3d22e30f1f9f1dd12", "tree": "523ef732a835d25a30d997bbff6fcf8321360857", "parents": [ "6459340cfcc6f6d165b27c3dd955aeb55a1b73d3" ], "author": { "name": "Simo Leone", "email": "simo@archlinux.org", "time": "Thu Jun 25 02:44:43 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jun 25 02:44:43 2009 +0000" }, "message": "[CIFS] Copy struct *after* setting the port, instead of before.\n\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Simo Leone \u003csimo@archlinux.org\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6459340cfcc6f6d165b27c3dd955aeb55a1b73d3", "tree": "1dd835d4931f2f02c62b7ac6f4520858ef0f5580", "parents": [ "b48a485884b5afb3e33b1871bcbd246b67491923" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jun 25 00:56:55 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jun 25 02:33:01 2009 +0000" }, "message": "cifs: remove rw/ro options\n\ncifs: remove rw/ro options\n\nThese options are handled at the VFS layer. They only ever set the\noption in the smb_vol struct. Nothing was ever done with them afterward\nanyway.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "681bf72e4893a187cf6b6b62c08fc193f81c8c2f", "tree": "5c580474a21edf38140407bd42f28853e9354e25", "parents": [ "268875b9d1dd1bf0b523c59e736da9bc20c8ce1f" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jun 11 10:27:31 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jun 25 01:14:36 2009 +0000" }, "message": "cifs: have cifs parse scope_id out of IPv6 addresses and use it\n\nThis patch has CIFS look for a \u0027%\u0027 in an IPv6 address. If one is\npresent then it will try to treat that value as a numeric interface\nindex suitable for stuffing into the sin6_scope_id field.\n\nThis should allow people to mount servers on IPv6 link-local addresses.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nAcked-by: David Holder \u003cdavid@erion.co.uk\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "1e68b2b2756fc3488ecbade5ad5f13302b3aaafc", "tree": "3ee4b36a1a00d449126050ce3bb1758ba88d7c2f", "parents": [ "340481a36498bf3fe404bcecb2e2d6188e950bff" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jun 11 10:27:30 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat Jun 13 08:17:30 2009 +0000" }, "message": "cifs: add new routine for converting AF_INET and AF_INET6 addrs\n\n...to consolidate some logic used in more than one place.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "61b6bc525a34931bb73e4c95bfe009cd9572a288", "tree": "4c60f5233b72c7ab1a4a4e6e123d49ba2de3c13c", "parents": [ "58f7f68f228c3aba2ba4468d92e2cec35724ba2e" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Jun 10 10:04:58 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Jun 10 18:34:35 2009 +0000" }, "message": "cifs: remove never-used in6_addr option\n\nThis option was never used to my knowledge. Remove it before someone\ndoes...\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "58f7f68f228c3aba2ba4468d92e2cec35724ba2e", "tree": "3afd0bfcfa97ebd76154f10cfc6dae4d839589d4", "parents": [ "f0472d0ec89bef2ea4432828c3daa1b26ef569aa" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Jun 10 09:57:55 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Jun 10 15:39:14 2009 +0000" }, "message": "cifs: add addr\u003d mount option alias for ip\u003d\n\nWhen you look in /proc/mounts, the address of the server gets displayed\nas \"addr\u003d\". That\u0027s really a better option to use anyway since it\u0027s more\ngeneric. What if we eventually want to support non-IP transports? It\nalso makes CIFS option consistent with the NFS option of the same name.\n\nBegin the migration to that option name by adding an alias for ip\u003d\ncalled addr\u003d.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "4ae1507f6d266d0cc3dd36e474d83aad70fec9e4", "tree": "ae41fd204f772ecc8f9adc2e727a9eb01999784e", "parents": [ "50b64e3b77d569c217a48e078cd565dbd6462ad0" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sun May 24 18:45:15 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat Jun 06 21:03:27 2009 +0000" }, "message": "cifs: make overriding of ownership conditional on new mount options\n\nWe have a bit of a problem with the uid\u003d option. The basic issue is that\nit means too many things and has too many side-effects.\n\nIt\u0027s possible to allow an unprivileged user to mount a filesystem if the\nuser owns the mountpoint, /bin/mount is setuid root, and the mount is\nset up in /etc/fstab with the \"user\" option.\n\nWhen doing this though, /bin/mount automatically adds the \"uid\u003d\" and\n\"gid\u003d\" options to the share. This is fortunate since the correct uid\u003d\noption is needed in order to tell the upcall what user\u0027s credcache to\nuse when generating the SPNEGO blob.\n\nOn a mount without unix extensions this is fine -- you generally will\nwant the files to be owned by the \"owner\" of the mount. The problem\ncomes in on a mount with unix extensions. With those enabled, the\nuid/gid options cause the ownership of files to be overriden even though\nthe server is sending along the ownership info.\n\nThis means that it\u0027s not possible to have a mount by an unprivileged\nuser that shows the server\u0027s file ownership info. The result is also\ninode permissions that have no reflection at all on the server. You\nsimply cannot separate ownership from the mode in this fashion.\n\nThis behavior also makes MultiuserMount option less usable. Once you\npass in the uid\u003d option for a mount, then you can\u0027t use unix ownership\ninfo and allow someone to share the mount.\n\nWhile I\u0027m not thrilled with it, the only solution I can see is to stop\nmaking uid\u003d/gid\u003d force the overriding of ownership on mounts, and to add\nnew mount options that turn this behavior on.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "50b64e3b77d569c217a48e078cd565dbd6462ad0", "tree": "2001150788641aad1f4824046d336f0b5d4dd32c", "parents": [ "b96d31a62f714566fa6420851b3bb3615c796322" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Jun 02 06:55:20 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Jun 02 15:45:40 2009 +0000" }, "message": "cifs: fix IPv6 address length check\n\nFor IPv6 the userspace mount helper sends an address in the \"ip\u003d\"\noption. This check fails if the length is \u003e 35 characters. I have no\nidea where the magic 35 character limit came from, but it\u0027s clearly not\nenough for IPv6. Fix it by making it use the INET6_ADDRSTRLEN #define.\n\nWhile we\u0027re at it, use the same #define for the address length in SPNEGO\nupcalls.\n\nReported-by: Charles R. Anderson \u003ccra@wpi.edu\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "a0c9217f64ee3cd1e534966da8c5f05768e1ab09", "tree": "c38584b9ca0376028b3273ce9515e84a3ddb8aec", "parents": [ "bd433d4cf4d8593a5f1764776b91f1794fce5a77" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed May 27 15:40:47 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 28 15:04:17 2009 +0000" }, "message": "cifs: make serverino the default when mounting\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "f55ed1a83d099f275c9560ad7d4c4700d1e54bdd", "tree": "863038ab400c725cd7be24ff8782a313ed6a20a0", "parents": [ "46a7574caf5bc533c24b315800ed323c187614f5" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue May 26 16:28:11 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue May 26 21:10:55 2009 +0000" }, "message": "cifs: tighten up default file_mode/dir_mode\n\nThe current default file mode is 02767 and dir mode is 0777. This is\nextremely \"loose\". Given that CIFS is a single-user protocol, these\npermissions allow anyone to use the mount -- in effect, giving anyone on\nthe machine access to the credentials used to mount the share.\n\nChange this by making the default permissions restrict write access to\nthe default owner of the mount. Give read and execute permissions to\neveryone else. These are the same permissions that VFAT mounts get by\ndefault so there is some precedent here.\n\nNote that this patch also removes the mandatory locking flags from the\ndefault file_mode. After having looked at how these flags are used by\nthe kernel, I don\u0027t think that keeping them as the default offers any\nreal benefit. That flag combination makes it so that the kernel enforces\nmandatory locking.\n\nSince the server is going to do that for us anyway, I don\u0027t think we\nwant the client to enforce this by default on applications that just\nwant advisory locks. Anyone that does want this behavior can always\nenable it by setting the file_mode appropriately.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ac68392460ffefed13020967bae04edc4d3add06", "tree": "9b5e43db83752e8927ee5e449dfafefe12ae1f7d", "parents": [ "844823cb822932d2c599abf38692e3d6a5b5a320" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed May 06 04:16:04 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed May 06 04:16:04 2009 +0000" }, "message": "[CIFS] Allow raw ntlmssp code to be enabled with sec\u003dntlmssp\n\nOn mount, \"sec\u003dntlmssp\" can now be specified to allow\n\"rawntlmssp\" security to be enabled during\nCIFS session establishment/authentication (ntlmssp used to\nrequire specifying krb5 which was counterintuitive).\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "2edd6c5b0517b9131ede9e74cb121898ccd73042", "tree": "0c811f33cd5dfd20d48a0585fb744e9f937823f2", "parents": [ "051a2a0d3242b448281376bb63cfa9385e0b6c68" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat May 02 04:55:39 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat May 02 04:55:39 2009 +0000" }, "message": "[CIFS] NTLMSSP support moving into new file, old dead code removed\n\nRemove dead NTLMSSP support from connect.c prior to addition of\nthe new code to replace it.\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "0e0d2cf32743c660aab20e40aeb2155c06a256db", "tree": "94dfb57ad091f283172628a3bba754e1562ded7d", "parents": [ "e14b2fe1e64d3e4bd2f328ff9d1969f318f55954" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 01 05:27:32 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 01 05:27:32 2009 +0000" }, "message": "[CIFS] Remove sparse endian warnings\n\nRemoves two sparse CHECK_ENDIAN warnings from Jeffs earlier patch,\nand removes the dead readlink code (after noting where in\nfindfirst we will need to add something like that in the future\nto handle the newly discovered unexpected error on FindFirst of NTFS symlinks.\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "e14b2fe1e64d3e4bd2f328ff9d1969f318f55954", "tree": "d1157ece05e36215b09699048c17bfbfeb2ec001", "parents": [ "cf398e3a117b2b63da724c2365d53ce31bd7240a" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 01 04:37:43 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 01 04:37:43 2009 +0000" }, "message": "[CIFS] Add remaining ntlmssp flags and standardize field names\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "9e39b0ae8af46c83b85dae7ff5251911a80fce5a", "tree": "3e6efa6cb324e77e82a930cb8afba768e78d574f", "parents": [ "d185cda7712fd1d9e349174639d76eadc66679be" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Apr 30 21:31:15 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Apr 30 21:31:15 2009 +0000" }, "message": "[CIFS] Remove unneeded QuerySymlink call and fix mapping for unmapped status\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "d185cda7712fd1d9e349174639d76eadc66679be", "tree": "6f72704ac0d107b0d53f7ee0be7ddb4d2b34e552", "parents": [ "5c2503a8e339fbc82f49d5706c5a4ad650dd9711" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Apr 30 17:45:10 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Apr 30 17:45:10 2009 +0000" }, "message": "[CIFS] rename cifs_strndup to cifs_strndup_from_ucs\n\nIn most cases, cifs_strndup is converting from Unicode (UCS2 / UTF-32) to\nthe configured local code page for the Linux mount (usually UTF8), so\nJeff suggested that to make it more clear that cifs_strndup is doing\na conversion not just memory allocation and copy, rename the function\nto including \"from_ucs\" (ie Unicode)\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "5c2503a8e339fbc82f49d5706c5a4ad650dd9711", "tree": "3b56e1e41b35ef743cc469b91f0aff184ec06898", "parents": [ "1af28ceb923d04357733642a3dbc4497da4db1c2" ], "author": { "name": "Igor Mammedov", "email": "niallain@gmail.com", "time": "Tue Apr 21 19:31:05 2009 +0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Apr 30 17:24:09 2009 +0000" }, "message": "Added loop check when mounting DFS tree.\n\nAdded loop check when mounting DFS tree. mount will fail with\nELOOP if referral walks exceed MAX_NESTED_LINK count.\n\nSigned-off-by: Igor Mammedov \u003cniallain@gmail.com\u003e\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "20418acd6874792359b42c12d159f42f17593f34", "tree": "67ebbc84ccfaab785e3118dd39292a358b5e1e44", "parents": [ "f58841666bc22e827ca0dcef7b71c7bc2758ce82" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Apr 30 16:13:32 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Apr 30 16:13:32 2009 +0000" }, "message": "[CIFS] Remove older session setup implementation\n\nTwo years ago, when the session setup code in cifs was rewritten and moved\nto fs/cifs/sess.c, we were asked to keep the old code for a release or so\n(which could be reenabled at runtime) since it was such a large change and\nbecause the asn (SPNEGO) and NTLMSSP code was not rewritten and needed to\nbe. This was useful to avoid regressions, but is long overdue to be removed.\nNow that the Kerberos (asn/spnego) code is working in fs/cifs/sess.c,\nand the NTLMSSP code moved (NTLMSSP blob setup be rewritten with the\nnext patch in this series) quite a bit of dead code from fs/cifs/connect.c\nnow can be removed.\n\nThis old code should have been removed last year, but the earlier krb5\npatches did not move/remove the NTLMSSP code which we had asked to\nbe done first. Since no one else volunteered, I am doing it now.\n\nIt is extremely important that we continue to examine the documentation\nfor this area, to make sure our code continues to be uptodate with\nchanges since Windows 2003.\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "cc20c031bb067eb3280a1c4b5c42295093e24863", "tree": "278fbc1a5c3943a1e85552748c40ca63d94e2c84", "parents": [ "066ce6899484d9026acd6ba3a8dbbedb33d7ae1b" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Apr 30 07:16:21 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Apr 30 15:45:00 2009 +0000" }, "message": "cifs: convert CIFSTCon to use new unicode helper functions\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nAcked-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "22c9d52bc03b880045ab1081890a38f11b272ae7", "tree": "7614c19810a85be56f4c454dbe2aeda82eef3843", "parents": [ "313fecfa69bbad0a10d3313a50a89d3064f47ce1" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Apr 16 13:48:49 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Apr 17 01:26:50 2009 +0000" }, "message": "cifs: remove unneeded bcc_ptr update in CIFSTCon\n\nThis pointer isn\u0027t used again after this point. It\u0027s also not updated in\nthe ascii case, so there\u0027s no need to update it here.\n\nPointed-out-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": "313fecfa69bbad0a10d3313a50a89d3064f47ce1", "tree": "bf75ff66a0e629af493b1a6f2d1b6148569b6b4a", "parents": [ "f083def68f84b04fe3f97312498911afce79609e" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Apr 16 11:21:53 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Apr 17 01:26:50 2009 +0000" }, "message": "cifs: add cFYI messages with some of the saved strings from ssetup/tcon\n\n...to make it easier to find problems in this area in the future.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "f083def68f84b04fe3f97312498911afce79609e", "tree": "f7550b49ad4b364f542dd1afe1bd2736fd34485f", "parents": [ "27b87fe52baba0a55e9723030e76fce94fabcea4" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Apr 16 11:21:52 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Apr 17 01:26:50 2009 +0000" }, "message": "cifs: fix buffer size for tcon-\u003enativeFileSystem field\n\nThe buffer for this was resized recently to fix a bug. It\u0027s still\npossible however that a malicious server could overflow this field\nby sending characters in it that are \u003e2 bytes in the local charset.\nDouble the size of the buffer to account for this possibility.\n\nAlso get rid of some really strange and seemingly pointless NULL\ntermination. It\u0027s NULL terminating the string in the source buffer,\nbut by the time that happens, we\u0027ve already copied the string.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "2d6d589d8009b37ae03244059c93e0e8cf46910e", "tree": "654ed166f84bd36ee612dc7e654dab230456f09c", "parents": [ "fbec9ab952d4810960e620035c8e95f0fbbae4be" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Apr 09 00:36:44 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Apr 17 01:26:49 2009 +0000" }, "message": "[CIFS] remove some build warnings\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "d036f50fc202e1a851a25dc5edc215ebd0086201", "tree": "e88defe256ea00a9cd87ae948710cf5b22404a80", "parents": [ "1bfe73c258addc388b90fe8c2c6bbc0f0c9c10dd" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Apr 03 03:12:08 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Apr 17 01:26:48 2009 +0000" }, "message": "[CIFS] Fix build break from recent DFS patch when DFS support not enabled\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "1bfe73c258addc388b90fe8c2c6bbc0f0c9c10dd", "tree": "5561ad05bda869ce83525888ed8961f580d74844", "parents": [ "85a6dac54a7e28112488b02523202985edc7e639" ], "author": { "name": "Igor Mammedov", "email": "niallain@gmail.com", "time": "Wed Apr 01 17:54:42 2009 +0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Apr 17 01:26:48 2009 +0000" }, "message": "Remote DFS root support.\n\nAllows to mount share on a server that returns -EREMOTE\n at the tree connect stage or at the check on a full path\n accessibility.\n\nSigned-off-by: Igor Mammedov \u003cniallain@gmail.com\u003e\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "74496d365ad171d11f21da4a8be71c945f6ec825", "tree": "8e7296257188a6eb0fcb18811fbade47c9cd0100", "parents": [ "0f4d634c59a4e062bef81c00d9e63333f2a83b46" ], "author": { "name": "Wei Yongjun", "email": "yjwei@cn.fujitsu.com", "time": "Tue Mar 31 16:28:36 2009 +0800" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Apr 17 01:26:48 2009 +0000" }, "message": "cifs: remove some pointless conditionals before kfree()\n\nRemove some pointless conditionals before kfree().\n\nSigned-off-by: Wei Yongjun \u003cyjwei@cn.fujitsu.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "b363b3304bcf68c4541683b2eff70b29f0446a5b", "tree": "da2a563b452cc14f900394f3ad56aa47701c5ea5", "parents": [ "c6c00919ab16717f228aac20ee72dc83c4430537" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Mar 18 05:57:22 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Mar 18 05:57:22 2009 +0000" }, "message": "[CIFS] Fix memory overwrite when saving nativeFileSystem field during mount\n\nCIFS can allocate a few bytes to little for the nativeFileSystem field\nduring tree connect response processing during mount. This can result\nin a \"Redzone overwritten\" message to be logged.\n\nSigned-off-by: Sridhar Vinay \u003cvinaysridhar@in.ibm.com\u003e\nAcked-by: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\nCC: Stable \u003cstable@kernel.org\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "4717bed6806dab0270e5bfbc45e9f999e63ededd", "tree": "d0a59d62c3fdd24329e970036f052a5138d6579a", "parents": [ "7fc8f4e95bf9564045985bb206af8e28a5e4e28f" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Feb 24 14:44:19 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Mar 12 01:36:20 2009 +0000" }, "message": "[CIFS] fix build error\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "be652445fdccb8e5d4391928c3b45324ea37f9e1", "tree": "9912b7022b06d3ad08ff73d71505f018fe96922b", "parents": [ "10e70afa75c90702b2326abaaa757d6b7835636f" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Feb 23 15:21:59 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Mar 12 01:36:20 2009 +0000" }, "message": "[CIFS] Add new nostrictsync cifs mount option to avoid slow SMB flush\n\nIf this mount option is set, when an application does an\nfsync call then the cifs client does not send an SMB Flush\nto the server (to force the server to write all dirty data\nfor this file immediately to disk), although cifs still sends\nall dirty (cached) file data to the server and waits for the\nserver to respond to the write write. Since SMB Flush can be\nvery slow, and some servers may be reliable enough (to risk\ndelaying slightly flushing the data to disk on the server),\nturning on this option may be useful to improve performance for\napplications that fsync too much, at a small risk of server\ncrash. If this mount option is not set, by default cifs will\nsend an SMB flush request (and wait for a response) on every\nfsync call.\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "e4cce94c9c8797b08faf6a79396df4d175e377fa", "tree": "6eb306603652bf08ca10fef4f899332f30d71e09", "parents": [ "e1f81c8a417be466e85a38b61679aa6860ec7331" ], "author": { "name": "Igor Mammedov", "email": "niallain@gmail.com", "time": "Tue Feb 10 14:10:26 2009 +0300" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat Feb 21 03:36:21 2009 +0000" }, "message": "[CIFS] Prevent OOPs when mounting with remote prefixpath.\n\nFixes OOPs with message \u0027kernel BUG at fs/cifs/cifs_dfs_ref.c:274!\u0027.\nChecks if the prefixpath in an accesible while we are still in cifs_mount\nand fails with reporting a error if we can\u0027t access the prefixpath\n\nShould fix Samba bugs 6086 and 5861 and kernel bug 12192\n\nSigned-off-by: Igor Mammedov \u003cniallain@gmail.com\u003e\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" } ], "next": "0e2bedaa394f74fa9f75ee937488c33d90039b5a" }